Makes it possible to save user data only with changes

Change-Id: I2bde131eb83b9f73d68b9375b28ed0054320665c

ui/app/admin/user_diff.ts

@@ -41,22 +41,21 @@ export class UserDiff {
   };
 
   ngOnInit() {
+    let versionsCount = this.versions.length - 1;
+
     this.currentVersion = this.versions[this.index];
 
-    if (!this.index) return;
+    if (this.index === versionsCount) return;
 
     let currentData = this.currentVersion.data;
-    let previousData = this.versions[this.index - 1].data;
+    let previousData = this.versions[this.index + 1].data;
 
     _.forEach(this.fields, (name, field) => {
-      let [oldValue, newValue] = [currentData[field], previousData[field]];
+      let [newValue, oldValue] = [currentData[field], previousData[field]];
 
-      console.log(field, oldValue, newValue);
       if (!_.isEqual(oldValue, newValue)) {
         this.diff.push({name, oldValue, newValue});
       }
     });
-
-    console.log(this.diff);
   }
 }

ui/app/admin/users.ts

@@ -43,6 +43,7 @@ export class UsersComponent {
   shownUserId: string = null;
   pagedData: pagedResult = {} as pagedResult;
   canSeeRoles: boolean = false;
+  fields = ['login', 'full_name', 'email', 'role_id'];
 
   constructor(
     private auth: AuthService,
@@ -91,6 +92,12 @@ export class UsersComponent {
     return role ? role.data.name : '-';
   }
 
+  hasChanges(user: User) {
+    return _.some(this.fields, field => {
+      return _.get(this.newUser.data, field) !== _.get(user.data, field);
+    });
+  }
+
   editUser(user: User = null) {
     this.newUser = _.isNull(user) ? new User({}) : user.clone();
     this.shownUserId = null;

ui/app/templates/users.html

@@ -146,7 +146,7 @@
                   type="button"
                   class="btn btn-primary pull-right"
                   (click)="saveUser()"
-                  [disabled]="!editUserForm.form.valid"
+                  [disabled]="!editUserForm.form.valid || !hasChanges(user)"
                   [shownFor]="'edit_user'"
                 >
                   Save