diff options
author | Byron Jones <glob@mozilla.com> | 2015-07-13 10:51:24 +0200 |
---|---|---|
committer | Byron Jones <glob@mozilla.com> | 2015-07-13 10:51:24 +0200 |
commit | dd1071f93f5dc4dbbe33763b99f74c3e744ab668 (patch) | |
tree | 108809f0220654e3712a5599cd9bd1e9fe939b81 | |
parent | 377acf2d61b8618f7b1e85f2a81c367b33d83a78 (diff) | |
download | bugzilla-dd1071f93f5dc4dbbe33763b99f74c3e744ab668.tar.gz bugzilla-dd1071f93f5dc4dbbe33763b99f74c3e744ab668.tar.xz |
Bug 1182500: Clearing group flag doesn't work when also moving bug to a different component
-rw-r--r-- | extensions/BugModal/web/bug_modal.js | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/extensions/BugModal/web/bug_modal.js b/extensions/BugModal/web/bug_modal.js index 1a7f7ea81..462e7cff6 100644 --- a/extensions/BugModal/web/bug_modal.js +++ b/extensions/BugModal/web/bug_modal.js @@ -952,9 +952,20 @@ $(function() { }); // update groups + var dirtyGroups = []; + $('#module-security').find('input[name=groups]').each(function() { + var that = $(this); + var defaultChecked = !!that.attr('checked'); + if (defaultChecked !== that.is(':checked')) { + dirtyGroups.push({ name: that.val(), value: that.is(':checked') }); + } + }); $('#module-security .module-content') .html(data.groups) .addClass('attention'); + $.each(dirtyGroups, function() { + $('#module-security').find('input[value=' + this.name + ']').prop('checked', this.value); + }); }, function() { $('#product-throbber').hide(); |