From 5a0a3372003b3a61d65ae61d56d81df3d6415a23 Mon Sep 17 00:00:00 2001
From: Nishant0928 <Nishant.Singh@contractors.onbase.com>
Date: Tue, 16 Apr 2024 22:55:26 +0530
Subject: [PATCH] ELEMENTS-1732: update permission popup shows empty permission
 on re-editing

---
 .../nuxeo-popup-permission.js                         | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/ui/nuxeo-document-permissions/nuxeo-popup-permission.js b/ui/nuxeo-document-permissions/nuxeo-popup-permission.js
index 8c032844f..57b777de7 100644
--- a/ui/nuxeo-document-permissions/nuxeo-popup-permission.js
+++ b/ui/nuxeo-document-permissions/nuxeo-popup-permission.js
@@ -194,9 +194,7 @@ import '../nuxeo-button-styles.js';
           </paper-dialog-scrollable>
 
           <div class="buttons">
-            <paper-button dialog-dismiss class="secondary" on-click="doCancel"
-              >[[i18n('popupPermission.cancel')]]</paper-button
-            >
+            <paper-button dialog-dismiss class="secondary">[[i18n('popupPermission.cancel')]]</paper-button>
             <dom-if if="{{!updatingACE}}">
               <template>
                 <paper-button noink class="primary small" on-click="doCreateAndAdd" id="createAndAddPermissionButton">
@@ -273,6 +271,7 @@ import '../nuxeo-button-styles.js';
     }
 
     togglePopup() {
+      this.set('params.invalid', false);
       this.$.popupRight.toggle();
     }
 
@@ -288,11 +287,6 @@ import '../nuxeo-button-styles.js';
       this._doSend(true);
     }
 
-    doCancel() {
-      this.params = this._getResetParams();
-      this.set('params.invalid', false);
-    }
-
     _computeTitle() {
       if (this.updatingACE) {
         return this.i18n('popupPermission.updatePermission');
@@ -353,6 +347,7 @@ import '../nuxeo-button-styles.js';
       if (this.updatingACE) {
         this.$.replaceOp.execute();
       } else {
+        this.set('params.invalid', false);
         this.$.createOp.execute();
       }