ascribe/onion
1
0
Fork 0
mirror of https://github.com/ascribe/onion.git synced 2025-01-20 17:51:25 +01:00
Code Issues Releases Wiki Activity

Use object destructuring instead of omitFromObject

Browse Source
This commit is contained in:
Brett Sun 2016-02-26 10:20:56 +01:00
parent dcdae7b7b1
commit 916d06bc74
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
js/components/app_route_wrapper.js
View File

@ -2,8 +2,6 @@
import React from 'react'; import React from 'react';
import { omitFromObject } from '../utils/general_utils';
const AppRouteWrapper = React.createClass({ const AppRouteWrapper = React.createClass({
propTypes: { propTypes: {
children: React.PropTypes.oneOfType([ children: React.PropTypes.oneOfType([
@ -13,19 +11,18 @@ const AppRouteWrapper = React.createClass({
}, },
render() { render() {
const propsToPropagate = omitFromObject(this.props, ['children']); let { children, ...propsToPropagate } = this.props;
let childrenWithProps = this.props.children;
// If there are more props given, propagate them into the child routes by cloning the routes // If there are more props given, propagate them into the child routes by cloning the routes
if (Object.keys(propsToPropagate).length) { if (Object.keys(propsToPropagate).length) {
childrenWithProps = React.Children.map(this.props.children, (child) => { children = React.Children.map(children, (child) => {
return React.cloneElement(child, propsToPropagate); return React.cloneElement(child, propsToPropagate);
}); });
} }
return ( return (
<div className="container ascribe-body"> <div className="container ascribe-body">
{childrenWithProps} {children}
</div> </div>
); );
} }