fluxible-router: remove decorator support (#657)

Co-authored-by: Pablo Palacios <[email protected]>

packages/fluxible-router/docs/api/handleHistory.md

@@ -54,33 +54,6 @@ AppComponent = provideContext(AppComponent);
 module.exports = AppComponent;
 ```
 
-### Decorator Usage
-
-***Decorators are an evolving proposal and should be used with caution
-as the API may change at any point. Decorator support in
-fluxible-addons-react was built against Babel 5's implementation of
-decorators. As of Babel 6, support for decorators has been removed although
-third party transforms have been attempted with limited success.
-
-Decorators are also only proposed for classes and properties and therefore
-will not work with stateless functional components. See
-[decorator pattern](https://github.com/wycats/javascript-decorators) for
-more information on the proposal.***
-
-```js
-// components/App.jsx
-import {provideContext} from 'fluxible-addons-react';
-import {handleHistory} from 'fluxible-router';
-
-@provideContext
-@handleHistory
-class AppComponent extends React.Component {
-    //...
-}
-
-export default AppComponent;
-```
-
 ## onbeforeunload Support
 
 The `History` API does not allow `popstate` events to be cancelled, which results in `window.onbeforeunload()` methods not being triggered.  This is problematic for users, since application state could be lost when they navigate to a certain page without knowing the consequences.

packages/fluxible-router/docs/api/handleRoute.md

@@ -41,29 +41,3 @@ MyComponent = handleRoute(MyComponent);
 
 module.exports = MyComponent;
 ```
-
-### Decorator Usage
-
-***Decorators are an evolving proposal and should be used with caution
-as the API may change at any point. Decorator support in
-fluxible-addons-react was built against Babel 5's implementation of
-decorators. As of Babel 6, support for decorators has been removed although
-third party transforms have been attempted with limited success.
-
-Decorators are also only proposed for classes and properties and therefore
-will not work with stateless functional components. See
-[decorator pattern](https://github.com/wycats/javascript-decorators) for
-more information on the proposal.***
-
-```js
-// components/App.jsx
-import {provideContext} from 'fluxible-addons-react';
-import {handleHistory} from 'fluxible-router';
-
-@handleRoute
-class MyComponent extends React.Component {
-    //...
-}
-
-export default MyComponent;
-```

packages/fluxible-router/src/lib/handleHistory.js

@@ -319,13 +319,5 @@ function createComponent(Component, opts) {
  * @returns {React.Component}
  */
 module.exports = function handleHistory(Component, opts) {
-    // support decorator pattern
-    if (arguments.length === 0 || typeof arguments[0] !== 'function') {
-        opts = arguments[0];
-        return function handleHistoryDecorator(componentToDecorate) {
-            return createComponent(componentToDecorate, opts);
-        };
-    }
-
     return createComponent.apply(null, arguments);
 };

packages/fluxible-router/src/lib/handleRoute.js

@@ -60,12 +60,5 @@ function createComponent(Component) {
 }
 
 module.exports = function handleRoute(Component) {
-    // support decorator pattern
-    if (arguments.length === 0) {
-        return function handleRouteDecorator(componentToDecorate) {
-            return createComponent(componentToDecorate);
-        };
-    }
-
     return createComponent.apply(null, arguments);
 };

packages/fluxible-router/tests/mocks/MockAppComponent.js

@@ -38,30 +38,3 @@ module.exports = provideContext(handleHistory(MockAppComponent, {
 module.exports.createWrappedMockAppComponent = function createWrappedMockAppComponent(opts) {
     return provideContext(handleHistory(MockAppComponent, opts), customContextTypes);
 };
-
-module.exports.createDecoratedMockAppComponent = function createDecoratedMockAppComponent(opts) {
-    @provideContext
-    @handleHistory(opts)
-    class DecoratedMockAppComponent extends React.Component {
-        static contextTypes = {
-            getStore: PropTypes.func.isRequired
-        };
-        static propTypes = {
-            children: PropTypes.object,
-            currentRoute: PropTypes.object
-        };
-        constructor(props, context) {
-            super(props, context);
-        }
-        render() {
-            if (!this.props.children) {
-                return null;
-            }
-            return React.cloneElement(this.props.children, {
-                currentRoute: this.props.currentRoute
-            });
-        }
-    }
-
-    return DecoratedMockAppComponent;
-};