Add warning for unknown properties.

Author: jimfb

src/addons/transitions/ReactTransitionGroup.js

@@ -222,9 +222,21 @@ var ReactTransitionGroup = React.createClass({
         ));
       }
     }
+
+    // Do not forward ReactTransitionGroup props
+    var props = Object.assign({}, this.props);
+    delete props.transitionLeave;
+    delete props.transitionName;
+    delete props.transitionAppear;
+    delete props.transitionEnter;
+    delete props.childFactory;
+    delete props.transitionLeaveTimeout;
+    delete props.transitionEnterTimeout;
+    delete props.component;
+
     return React.createElement(
       this.props.component,
-      this.props,
+      typeof this.props.component === 'string' ? props : this.props,
       childrenToRender
     );
   },

src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js

@@ -148,6 +148,19 @@ describe('ReactDOMComponent', function() {
       expect(console.error.argsForCall.length).toBe(2);
     });
 
+    it('should warn for unknown prop', function() {
+      spyOn(console, 'error');
+      var container = document.createElement('div');
+      ReactDOM.render(<div foo="bar" />, container);
+      expect(console.error.argsForCall.length).toBe(1);
+      expect(console.error.argsForCall[0][0]).toContain(
+        'Warning: Unknown prop foo.  See https://fb.me/react-unknown-prop'
+      );
+      expect(console.error.argsForCall[0][0]).toContain(
+        'ReactDOMComponent-test.js'
+      );
+    });
+
     it('should warn about styles with numeric string values for non-unitless properties', function() {
       spyOn(console, 'error');
 

src/renderers/dom/shared/devtools/ReactDOMUnknownPropertyDevtool.js

@@ -22,6 +22,15 @@ if (__DEV__) {
     dangerouslySetInnerHTML: true,
     key: true,
     ref: true,
+
+    defaultValue: true,
+    valueLink: true,
+    defaultChecked: true,
+    checkedLink: true,
+    innerHTML: true,
+    suppressContentEditableWarning: true,
+    onFocusIn: true,
+    onFocusOut: true,
   };
   var warnedProperties = {};
 
@@ -48,16 +57,6 @@ if (__DEV__) {
         null
     );
 
-    // For now, only warn when we have a suggested correction. This prevents
-    // logging too much when using transferPropsTo.
-    warning(
-      standardName == null,
-      'Unknown DOM property %s. Did you mean %s? %s',
-      name,
-      standardName,
-      formatSource(source)
-    );
-
     var registrationName = (
       EventPluginRegistry.possibleRegistrationNames.hasOwnProperty(
         lowerCasedName
@@ -66,13 +65,33 @@ if (__DEV__) {
       null
     );
 
-    warning(
-      registrationName == null,
-      'Unknown event handler property %s. Did you mean `%s`? %s',
-      name,
-      registrationName,
-      formatSource(source)
-    );
+    if (standardName != null) {
+      warning(
+        standardName == null,
+        'Unknown DOM property %s. Did you mean %s? %s',
+        name,
+        standardName,
+        formatSource(source)
+      );
+    } else if (registrationName != null) {
+      warning(
+        registrationName == null,
+        'Unknown event handler property %s. Did you mean `%s`? %s',
+        name,
+        registrationName,
+        formatSource(source)
+      );
+    } else {
+      // We were unable to guess which prop the user intended.
+      // It is likely that the user was just blindly spreading/forwarding props
+      // Components should be careful to only render valid props/attributes.
+      warning(
+        false,
+        'Unknown prop %s.  See https://fb.me/react-unknown-prop %s',
+        name,
+        formatSource(source)
+      );
+    }
   };
 
   var formatSource = function(source) {
@@ -85,6 +104,9 @@ function handleElement(element) {
   if (element == null || typeof element.type !== 'string') {
     return;
   }
+  if (element.type.indexOf('-') >= 0 || element.props.is != null) {
+    return;
+  }
   for (var key in element.props) {
     warnUnknownProperty(key, element._source);
   }