diff --git a/src/core/ReactCompositeComponent.js b/src/core/ReactCompositeComponent.js index 42a1dcb45b..5c1e065665 100644 --- a/src/core/ReactCompositeComponent.js +++ b/src/core/ReactCompositeComponent.js @@ -556,22 +556,14 @@ function mixStaticSpecIntoComponent(Constructor, statics) { } var isInherited = name in Constructor; - var result = property; - if (isInherited) { - var existingProperty = Constructor[name]; - var existingType = typeof existingProperty; - var propertyType = typeof property; - invariant( - existingType === 'function' && propertyType === 'function', - 'ReactCompositeComponent: You are attempting to define ' + - '`%s` on your component more than once, but that is only supported ' + - 'for functions, which are chained together. This conflict may be ' + - 'due to a mixin.', - name - ); - result = createChainedFunction(existingProperty, property); - } - Constructor[name] = result; + invariant( + !isInherited, + 'ReactCompositeComponent: You are attempting to define ' + + '`%s` on your component more than once. This conflict may be ' + + 'due to a mixin.', + name + ); + Constructor[name] = property; } } diff --git a/src/core/__tests__/ReactCompositeComponent-test.js b/src/core/__tests__/ReactCompositeComponent-test.js index abfedc5dee..2deb31e912 100644 --- a/src/core/__tests__/ReactCompositeComponent-test.js +++ b/src/core/__tests__/ReactCompositeComponent-test.js @@ -1317,9 +1317,33 @@ describe('ReactCompositeComponent', function() { }); }).toThrow( 'Invariant Violation: ReactCompositeComponent: You are attempting to ' + - 'define `abc` on your component more than once, but that is only ' + - 'supported for functions, which are chained together. This conflict ' + - 'may be due to a mixin.' + 'define `abc` on your component more than once. This conflict may be ' + + 'due to a mixin.' + ); + }); + + it("should throw if mixins override functions in statics", function() { + expect(function() { + var Mixin = { + statics: { + abc: function() { console.log('foo'); } + } + }; + React.createClass({ + mixins: [Mixin], + + statics: { + abc: function() { console.log('bar'); } + }, + + render: function() { + return ; + } + }); + }).toThrow( + 'Invariant Violation: ReactCompositeComponent: You are attempting to ' + + 'define `abc` on your component more than once. This conflict may be ' + + 'due to a mixin.' ); });