@samitouri / QOS-React-2 / commits / c46dadea55

Remove deprecated methods since 0.12 shipped

Paul O’Shannessy committed Dec 3, 2014 at 14:29 UTC c46dadea55728a5c782861c15ed4989221c0674e
4 files changed +2 -88
src/browser/ui/React.js
+1 -32
@@ -35,7 +35,6 @@ var ReactRef = require('ReactRef');
35 var ReactServerRendering = require('ReactServerRendering');
36
37 var assign = require('Object.assign');
38 -var deprecated = require('deprecated');
38 var onlyChild = require('onlyChild');
39
40 ReactDefaultInjection.inject();
@@ -78,37 +77,7 @@ var React = {
77 withContext: ReactContext.withContext,
78
79 // Hook for JSX spread, don't use this for anything else.
81 - __spread: assign,
82 -
83 - // Deprecations (remove for 0.13)
84 - renderComponent: deprecated(
85 - 'React',
86 - 'renderComponent',
87 - 'render',
88 - this,
89 - render
90 - ),
91 - renderComponentToString: deprecated(
92 - 'React',
93 - 'renderComponentToString',
94 - 'renderToString',
95 - this,
96 - ReactServerRendering.renderToString
97 - ),
98 - renderComponentToStaticMarkup: deprecated(
99 - 'React',
100 - 'renderComponentToStaticMarkup',
101 - 'renderToStaticMarkup',
102 - this,
103 - ReactServerRendering.renderToStaticMarkup
104 - ),
105 - isValidComponent: deprecated(
106 - 'React',
107 - 'isValidComponent',
108 - 'isValidElement',
109 - this,
110 - ReactElement.isValidElement
111 - )
80 + __spread: assign
81 };
82
83 // Inject the runtime into a devtools global hook regardless of browser.
src/browser/ui/ReactMount.js
-10
@@ -24,7 +24,6 @@ var ReactUpdates = require('ReactUpdates');
24
25 var emptyObject = require('emptyObject');
26 var containsNode = require('containsNode');
27 -var deprecated = require('deprecated');
27 var getReactRootElementInContainer = require('getReactRootElementInContainer');
28 var instantiateReactComponent = require('instantiateReactComponent');
29 var invariant = require('invariant');
@@ -816,13 +815,4 @@ ReactPerf.measureMethods(ReactMount, 'ReactMount', {
815 _mountImageIntoNode: '_mountImageIntoNode'
816 });
817
819 -// Deprecations (remove for 0.13)
820 -ReactMount.renderComponent = deprecated(
821 - 'ReactMount',
822 - 'renderComponent',
823 - 'render',
824 - this,
825 - ReactMount.render
826 -);
827 -
818 module.exports = ReactMount;
src/classic/propTypes/ReactPropTypes.js
+1 -17
@@ -14,7 +14,6 @@
14 var ReactElement = require('ReactElement');
15 var ReactPropTypeLocationNames = require('ReactPropTypeLocationNames');
16
17 -var deprecated = require('deprecated');
17 var emptyFunction = require('emptyFunction');
18
19 /**
@@ -85,22 +84,7 @@ var ReactPropTypes = {
84 objectOf: createObjectOfTypeChecker,
85 oneOf: createEnumTypeChecker,
86 oneOfType: createUnionTypeChecker,
88 - shape: createShapeTypeChecker,
89 -
90 - component: deprecated(
91 - 'React.PropTypes',
92 - 'component',
93 - 'element',
94 - this,
95 - elementTypeChecker
96 - ),
97 - renderable: deprecated(
98 - 'React.PropTypes',
99 - 'renderable',
100 - 'node',
101 - this,
102 - nodeTypeChecker
103 - )
87 + shape: createShapeTypeChecker
88 };
89
90 function createChainableTypeChecker(validate) {
src/classic/propTypes/__tests__/ReactPropTypes-test.js
-29
@@ -405,35 +405,6 @@ describe('ReactPropTypes', function() {
405 it('should accept empty array for required props', function() {
406 typeCheckPass(PropTypes.node.isRequired, []);
407 });
408 -
409 - it('should still work for deprecated typechecks', function() {
410 - // We can't use typeCheckPass here because the warning module may do
411 - // something different in some environments. Luckily they should be fine
412 - // if they detect that console.warn is spied upon.
413 - spyOn(console, 'warn');
414 -
415 - // typeCheckPass(PropTypes.renderable, []);
416 - var error = PropTypes.renderable(
417 - {testProp: []},
418 - 'testProp',
419 - 'testComponent',
420 - ReactPropTypeLocations.prop
421 - );
422 -
423 - expect(error).toBe(undefined);
424 - expect(console.warn.calls.length).toBe(1);
425 -
426 - // typeCheckPass(PropTypes.renderable.isRequired, []);
427 - error = PropTypes.renderable.isRequired(
428 - {testProp: []},
429 - 'testProp',
430 - 'testComponent',
431 - ReactPropTypeLocations.prop
432 - );
433 -
434 - expect(error).toBe(undefined);
435 - expect(console.warn.calls.length).toBe(1);
436 - });
408 });
409
410 describe('ObjectOf Type', function() {