Fix bug with item deletion in repeat

controls/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gutenblock-controls",
-  "version": "0.3.4",
+  "version": "0.3.5",
   "description": "Useful inspector controls for gutenberg",
   "main": "dist/index.js",
   "scripts": {

controls/src/repeat/index.js

@@ -21,36 +21,27 @@ export default class Repeat extends React.Component {
     return null;
   }
 
-  update(name, value, tabId, parentChange, customUpdate) {
-    let { attribute, attributes, setAttributes } = this.props;
+  update(name, value, tabId, onChange) {
+    let { attribute, attributes } = this.props;
 
-    //if nested repeats, pass state up to top most one
-    if (parentChange) {
-      let currentAttributes = attributes[attribute] || [];
+    let currentAttributes = attributes[attribute] || [];
 
-      let foundAttribute = currentAttributes.find(
-        (attr, index) => index === tabId
+    let foundAttribute = currentAttributes.find(
+      (attr, index) => index === tabId
+    );
+    let newAttributes;
+    if (!foundAttribute)
+      newAttributes = [...currentAttributes, { [name]: value }];
+    else
+      newAttributes = currentAttributes.map(
+        (attr, index) => (index === tabId ? { ...attr, [name]: value } : attr)
       );
-      let newAttributes;
-      if (!foundAttribute)
-        newAttributes = [...currentAttributes, { [name]: value }];
-      else
-        newAttributes = currentAttributes.map(
-          (attr, index) => (index === tabId ? { ...attr, [name]: value } : attr)
-        );
-
-      return parentChange(attribute, newAttributes);
-    }
 
-    let topLevel = [...attributes[attribute]];
-
-    topLevel[tabId] = { ...topLevel[tabId], [name]: value };
-
-    setAttributes({ [attribute]: topLevel });
+    return onChange(attribute, newAttributes);
   }
 
-  delete(childAttributes, childAttribute, tabId, onDelete, customDelete) {
-    let { attribute, attributes, setAttributes } = this.props;
+  delete(childAttributes, childAttribute, tabId, onChange) {
+    let { attribute, attributes } = this.props;
 
     let newAttributes;
     if (!childAttributes)
@@ -68,9 +59,7 @@ export default class Repeat extends React.Component {
       );
     }
 
-    if (onDelete) return onDelete(newAttributes, attribute);
-
-    setAttributes({ [attribute]: newAttributes });
+    return onChange(attribute, newAttributes);
   }
 
   renderChildren(index) {
@@ -80,7 +69,6 @@ export default class Repeat extends React.Component {
       setAttributes,
       attributes,
       onChange,
-      onDelete,
     } = this.props;
 
     return React.Children.map(children, child =>
@@ -92,16 +80,9 @@ export default class Repeat extends React.Component {
           attributes && attributes[attribute] && attributes[attribute][index]
             ? attributes[attribute][index]
             : {},
-        onChange: (name, value) =>
-          this.update(name, value, index, onChange, child.onUpdate),
+        onChange: (name, value) => this.update(name, value, index, onChange),
         onDelete: (childAttributes, childAttribute) =>
-          this.delete(
-            childAttributes,
-            childAttribute,
-            index,
-            onDelete,
-            child.onDelete
-          ),
+          this.delete(childAttributes, childAttribute, index, onChange),
         style: {
           marginLeft: '10px',
           marginTop: '15px',

package.json

@@ -1,6 +1,6 @@
 {
   "name": "gutenblock",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "description": "",
   "main": "index.js",
   "scripts": {

plugin/package.json

@@ -14,7 +14,7 @@
   "license": "ISC",
   "dependencies": {
     "@babel/polyfill": "^7.0.0-beta.46",
-    "gutenblock-controls": "^0.3.4",
+    "gutenblock-controls": "^0.3.5",
     "react": "^16.3.2",
     "react-dom": "^16.3.2",
     "react-hot-loader": "^4.0.1"