Fixed issue with double calls on add user

internet2 · Mar 21, 2022 · e6857fb · e6857fb
1 parent 707895b
commit e6857fb
Showing 1 changed file with 16 additions and 7 deletions.
diff --git a/View/Elements/Components/subscriberList.ctp b/View/Elements/Components/subscriberList.ctp
@@ -74,19 +74,26 @@
             var $this = $(this);
             group = $this.data('id');
             modal = $('#subscribers');
-            modal.modal('toggle');
-
-            modal.find('#add-user-form').submit(function(ev) {
-                ev.preventDefault();
-                var field = $(ev.target).find('#addUser');
-                var user = field.val();
-                onAddUser(user, group, field);
+            modal.modal('toggle').on('hide.bs.modal', function() {
+                modal.find('#add-user-form').off('submit', onAddUserSubmit);
             });
 
+            modal.find('#add-user-form').on('submit', onAddUserSubmit);
+
             load = loadModalData(group);
             load();
         });
 
+        function onAddUserSubmit(ev) {
+            ev.preventDefault();
+            ev.stopPropagation();
+            var field = $(ev.target).find('#addUser');
+            var user = field.val();
+            onAddUser(user, group, field);
+
+            console.log('onAddUserSubmit', ev);
+        }
+
         function loadModalData(id) {
             return function() {
                 modal.addClass('loading');
@@ -137,6 +144,7 @@
             $('#subscribers .error').hide();
             $('#add-user-form #addUser').val('');
             $('.member-del-btn').off('click', onRemoveUser);
+            // $('#add-user-form').off('submit', onAddUserSubmit);
         }
 
         function onRemoveUser(ev) {
@@ -157,6 +165,7 @@
         }
 
         function onAddUser(user, group, field) {
+            console.log('onAddUser', user, group.field);
             $.ajax({
                 method: 'POST',
                 url: addUrl + '?group=' + group + '&userId=' + user,