From ea3f0d615728dbf896c0b6163740906e1df1a673 Mon Sep 17 00:00:00 2001
From: Mads Rasmussen <mads@umbraco.com>
Date: Fri, 25 Aug 2017 09:31:13 +0200
Subject: [PATCH] fixing edge case model change

---
 .../src/views/content/content.rights.controller.js    | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/Umbraco.Web.UI.Client/src/views/content/content.rights.controller.js b/src/Umbraco.Web.UI.Client/src/views/content/content.rights.controller.js
index f8eea10c0b..6217cb2d2a 100644
--- a/src/Umbraco.Web.UI.Client/src/views/content/content.rights.controller.js
+++ b/src/Umbraco.Web.UI.Client/src/views/content/content.rights.controller.js
@@ -1,7 +1,7 @@
 (function () {
     "use strict";
 
-    function ContentRightsController($scope, contentResource, localizationService, angularHelper) {
+    function ContentRightsController($scope, $timeout, contentResource, localizationService, angularHelper) {
 
         var vm = this;
         var currentForm;
@@ -146,9 +146,12 @@
                 initData(userGroups);
 
                 // clear dirty state on the form so we don't see the discard changes notification
-                if(currentForm) {
-                  currentForm.$dirty = false;
-                }
+                // we use a timeout here because in some cases the initData reformats the userGroups model and triggers a change after the form state was changed
+                $timeout(function() {
+                  if(currentForm) {
+                    currentForm.$dirty = false;
+                  }
+                });
 
                 vm.saveState = "success";
                 vm.saveSuccces = true;