See the change at: https://bugzilla.mozilla.org/show_activity.cgi?id=959893 at 2016-04-18 14:56:53 BST Clicking the "stop following" button should *only* remove me from the CC list and do NOTHING else.

Comment on attachment 8753411 [details] [diff] [review] 1265392_1.patch woops, now it can never be unset.

Comment on attachment 8753419 [details] [diff] [review] 1265392_2.patch Review of attachment 8753419 [details] [diff] [review]: ----------------------------------------------------------------- I would not do the i_am_webservice check as we want to be able to set the value via API even though it is not documented currently. This change works for me just as well to fix the issue. diff --git a/extensions/RestrictComments/Extension.pm b/extensions/RestrictComments/Extension.pm index 001332a..35780fb 100644 --- a/extensions/RestrictComments/Extension.pm +++ b/extensions/RestrictComments/Extension.pm @@ -47,8 +47,8 @@ sub _can_restrict_comments { sub object_end_of_set_all { my ($self, $args) = @_; my $object = $args->{object}; - if ($self->_can_restrict_comments($object)) { - my $input = Bugzilla->input_params; + my $input = Bugzilla->input_params; + if (exists $input->{restrict_comments} && $self->_can_restrict_comments($object)) { $object->set('restrict_comments', $input->{restrict_comments} ? 1 : undef); } }

I think my patch allows it to be updated by a webservice or by the frontend. checkboxes are never sent over the wire when unchecked... am I wrong? :(

(In reply to Dylan William Hardison [:dylan] from comment #5) > I think my patch allows it to be updated by a webservice or by the frontend. > checkboxes are never sent over the wire when unchecked... am I wrong? :( Ok at first read it was confusing what was going on. So technically it would work via API but I now see that the check is just redundant. You only need to check for the existence of the 'restrict_comments' key in either case. As far as the API works, if a attribute does not exist in the PUT data, it is simply ignored and the DB value is not changed. But if it does exist, then it can be either true or false and will change the value if needed. The UI is slightly different in that it used to treat non-existent keys as empty values and actually change the DB value but that has gotten better. dkl

To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git 73fffe5..d7db0c1 master -> master