Merge pull request #7 from costimize/1

Reinitialize sortable on componentDidUpdate

Author: cheton

src/index.jsx

@@ -105,9 +105,8 @@ const SortableMixin = (options = defaultOptions) => (Component) => class extends
                 }, 0);
             };
         });
-
-        const domNode = ReactDOM.findDOMNode(sortableComponent.refs[this.sortableOptions.ref] || sortableComponent);
-        this.sortableInstance = Sortable.create(domNode, copyOptions);
+        this.populatedOptions = copyOptions
+        this.initSortable(sortableComponent);
     }
     componentWillReceiveProps(nextProps) {
         const sortableComponent = this.refs[refName];
@@ -120,7 +119,23 @@ const SortableMixin = (options = defaultOptions) => (Component) => class extends
             sortableComponent.setState(newState);
         }
     }
+    componentDidUpdate(prevProps) {
+        const model = this.sortableOptions.model;
+        const prevItems = prevProps[model];
+        const currItems = this.props[model];
+        if(prevItems !== currItems) {
+            this.initSortable(this.refs[refName]);
+        }
+    }
     componentWillUnmount() {
+        this.destroySortable();
+    }
+    initSortable(sortableComponent) {
+        this.destroySortable();
+        const domNode = ReactDOM.findDOMNode(sortableComponent.refs[this.sortableOptions.ref] || sortableComponent);
+        this.sortableInstance = Sortable.create(domNode, this.populatedOptions);
+    }
+    destroySortable() {
         if (this.sortableInstance) {
             this.sortableInstance.destroy();
             this.sortableInstance = null;