"Delete Groups" warning for groups on products is misleading

RESOLVED FIXED in Bugzilla 4.4

Status

()

--
minor
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: gerv, Assigned: gerv)

Tracking

unspecified
Bugzilla 4.4
Bug Flags:
approval +
approval4.4 +

Details

Attachments

(1 attachment)

If you go to delete a group, it warns you about possible problems you may have. It lists all products for which that group is enabled, and sometimes it says:

"WARNING: This product is currently hidden. Deleting this group will make this product publicly visible."

However, this message does not match the conditions which display it.

The code in delete.html.tmpl goes like this:

        [% SET hidden = 0 %]
        [% IF data.controls.othercontrol == constants.CONTROLMAPMANDATORY
              AND data.controls.membercontrol == constants.CONTROLMAPMANDATORY
              AND data.controls.entry 
        %]
          [% SET hidden = 1 %]
        [% END %]

        ...

          [% IF hidden %]
            <strong>WARNING: This product is currently hidden.
            Deleting this group will make this product publicly visible.
            </strong>
          [% END %]

 So the message is displayed if the group to be deleted is MANDATORY/MANDATORY/Entry. Deleting it will certainly make the product _more_ visible (if you are using AND groups), but it will not necessarily make it public - that depends on what other groups are set on the product.

We should either update the logic to be more complex, or change the error message, or simply remove it. And we need to cope with the OR groups case.

Gerv

Comment 1

5 years ago
(In reply to Gervase Markham [:gerv] from comment #0)

> "WARNING: This product is currently hidden. Deleting this group will make
> this product publicly visible."

To not make the code too complex, we should simply reword it to something like s/will/could/ and appending "if no other group applies" or something similar.
Created attachment 8371533 [details] [diff] [review]
Patch v.1

As suggested by LpSolit.

Gerv
Assignee: administration → gerv
Status: NEW → ASSIGNED
Attachment #8371533 - Flags: review?(LpSolit)

Comment 3

5 years ago
Comment on attachment 8371533 [details] [diff] [review]
Patch v.1

r=LpSolit
Attachment #8371533 - Flags: review?(LpSolit) → review+

Updated

5 years ago
Severity: normal → minor
Flags: approval?
Flags: approval4.4?
Target Milestone: --- → Bugzilla 4.4
Flags: approval?
Flags: approval4.4?
Flags: approval4.4+
Flags: approval+
Committing  to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/                                                                                                                                                          
modified template/en/default/admin/groups/delete.html.tmpl
Committed revision 8909.     

Gerv

Comment 5

5 years ago
Please also commit to the 4.4 branch and close the bug.
Committing  to: bzr+ssh://gerv@mozilla.org@bzr.mozilla.org/bugzilla/4.4/                   
modified template/en/default/admin/groups/delete.html.tmpl
Committed revision 8655.

Gerv
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.