Merge pull request clauderic#66 from krisl/changable-collection

Claudéric Demers · web-flow · commit 15ff005887c5 · 2016-10-17T21:07:59.000-04:00
Support changing an item's collection prop on the fly
diff --git a/src/SortableElement/index.js b/src/SortableElement/index.js
@@ -41,6 +41,10 @@ export default function SortableElement (WrappedComponent, config = {withRef: fa
                     this.setDraggable(collection, index);
                 }
             }
+            else if (this.props.collection !== nextProps.collection) {
+                this.removeDraggable(this.props.collection);
+                this.setDraggable(nextProps.collection, nextProps.index);
+            }
         }
         componentWillUnmount() {
             let {collection, disabled} = this.props;

Original file line number	Diff line number	Diff line change
`@@ -41,6 +41,10 @@ export default function SortableElement (WrappedComponent, config = {withRef: fa`
`41`	`41`	`this.setDraggable(collection, index);`
`42`	`42`	`}`
`43`	`43`	`}`
	`44`	`+ else if (this.props.collection !== nextProps.collection) {`
	`45`	`+ this.removeDraggable(this.props.collection);`
	`46`	`+ this.setDraggable(nextProps.collection, nextProps.index);`
	`47`	`+ }`
`44`	`48`	`}`
`45`	`49`	`componentWillUnmount() {`
`46`	`50`	`let {collection, disabled} = this.props;`