ELEMENTS-1732: update permission popup shows empty permission on re-e…

…diting

ui/nuxeo-document-permissions/nuxeo-popup-permission.js

@@ -194,7 +194,7 @@ import '../nuxeo-button-styles.js';
           </paper-dialog-scrollable>
 
           <div class="buttons">
-            <paper-button dialog-dismiss class="secondary" on-click="doCancel"
+            <paper-button dialog-dismiss class="secondary"
               >[[i18n('popupPermission.cancel')]]</paper-button
             >
             <dom-if if="{{!updatingACE}}">
@@ -273,6 +273,8 @@ import '../nuxeo-button-styles.js';
     }
 
     togglePopup() {
+      this.set('params.invalid', false);
+
       this.$.popupRight.toggle();
     }
 
@@ -288,10 +290,6 @@ import '../nuxeo-button-styles.js';
       this._doSend(true);
     }
 
-    doCancel() {
-      this.params = this._getResetParams();
-      this.set('params.invalid', false);
-    }
 
     _computeTitle() {
       if (this.updatingACE) {
@@ -322,6 +320,7 @@ import '../nuxeo-button-styles.js';
       }
       if (!this.shareWithExternal && !this.params.username && (!this.params.users || this.params.users.length === 0)) {
         this.set('params.invalid', true);
+
         return;
       }
 
@@ -353,6 +352,7 @@ import '../nuxeo-button-styles.js';
       if (this.updatingACE) {
         this.$.replaceOp.execute();
       } else {
+        this.set('params.invalid', false);
         this.$.createOp.execute();
       }
 
@@ -445,4 +445,4 @@ import '../nuxeo-button-styles.js';
 
   customElements.define(PopupPermission.is, PopupPermission);
   Nuxeo.PopupPermission = PopupPermission;
-}
+}