partkeepr

commit 34959ee33cd4e44c9f1b7f5345a8dd97cb877cbe
parent 9efaedf8a56285993cd42deb9b64defd3fbaa013
Author: Timo A. Hummel <felicitus@felicitus.org>
Date:   Tue,  8 Mar 2016 16:58:31 +0100

Merge pull request #607 from partkeepr/PartKeepr-582

Added bugfix where duplicated data gets moved to a new entity instead…
Diffstat:
M
src/PartKeepr/FrontendBundle/Resources/public/js/Data/HydraModel.js
 | 
10
+++++++++-

1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/src/PartKeepr/FrontendBundle/Resources/public/js/Data/HydraModel.js b/src/PartKeepr/FrontendBundle/Resources/public/js/Data/HydraModel.js
@@ -73,7 +73,7 @@ Ext.define("PartKeepr.data.HydraModel", {
      */
     setAssociationData: function (data)
     {
-        var setterName, getterName, roleName, store;
+        var setterName, getterName, roleName, store, idProperty;
 
         for (roleName in data) {
             if (this.associations[roleName]) {
@@ -81,6 +81,14 @@ Ext.define("PartKeepr.data.HydraModel", {
                 if (this.associations[roleName].isMany === true) {
                     getterName = this.associations[roleName].getterName;
                     store = this[getterName]();
+
+                    for (var i=0;i<data[roleName].length;i++) {
+                        // Delete existing IDs for duplicated data
+                        if (data[roleName][i].isEntity) {
+                            idProperty = data[roleName][i].idProperty;
+                            delete data[roleName][i].data[idProperty];
+                        }
+                    }
                     store.add(data[roleName]);
                 } else {
                     setterName = this.associations[roleName].setterName;