:Gijs (he/him) Reporter
	Description • 8 years ago

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.

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 2 • 8 years ago

Comment on attachment 8753411 [details] [diff] [review]
1265392_1.patch

woops, now it can never be unset.

	David Lawrence [:dkl]
	Comment 4 • 8 years ago

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);
     }
 }

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 5 • 8 years ago

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? :(

	David Lawrence [:dkl]
	Comment 6 • 8 years ago

(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

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 7 • 8 years ago

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