fix: Edit identity (primary key) field (marmelab#332 issue on GitHub)

src/javascripts/ng-admin/Crud/form/FormController.js

@@ -25,6 +25,9 @@ define(function () {
         this.view = view;
         this.entity = this.view.getEntity();
 
+        // in case of entity identifier being modified
+        this.origEntityId = this.$scope.entry.values[this.entity.identifier().name()]
+
         $scope.$on('$destroy', this.destroy.bind(this));
     };
 
@@ -33,7 +36,7 @@ define(function () {
             form = this.form,
             entry = this.$scope.entry,
             fields = this.view.getFields(),
-            identifierField = this.view.getEntity().identifier(),
+            identifierField = this.entity.identifier(),
             mappedObject,
             field,
             i,
@@ -99,7 +102,7 @@ define(function () {
             notification = this.notification;
         progression.start();
         this.UpdateQueries
-            .updateOne(this.view, entry)
+            .updateOne(this.view, entry, this.origEntityId)
             .then(function () {
                 progression.done();
                 notification.log('Changes successfully saved.', {addnCls: 'humane-flatty-success'});

src/javascripts/ng-admin/Crud/repository/UpdateQueries.js

@@ -18,13 +18,14 @@ define(function (require) {
      * Update an entity
      * Put the data to the API to create the new object
      *
-     * @param {View}   view      the formView related to the entity
-     * @param {Object} rawEntity the entity's object
+     * @param {View}   view             the formView related to the entity
+     * @param {Object} rawEntity        the entity's object
+     * @param {String} origEntityId     if entity identifier is modified
      *
      * @returns {promise} the updated object
      */
-    UpdateQueries.prototype.updateOne = function (view, rawEntity) {
-        var entityId = rawEntity[view.getEntity().identifier().name()];
+    UpdateQueries.prototype.updateOne = function (view, rawEntity, origEntityId) {
+        var entityId = origEntityId || rawEntity[view.getEntity().identifier().name()];
 
         // Get element data
         return this.Restangular