Closed Bug 330754 Opened 18 years ago Closed 18 years ago

Remove AnyEntryGroups()

Categories

(Bugzilla :: Bugzilla-General, enhancement)

2.23
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 3.0

People

(Reporter: LpSolit, Assigned: LpSolit)

References

Details

Attachments

(1 file)

This function is only used in enter_bug.cgi to determine at which moment we request the user to log in. IMO, this is useless and confusing. If the installation has no product restricted to a group, you will first have to select a product, and then to authenticate. If there is a product restricted to a group, you first have to authenticate, then to choose a product. The final result is that you have to authenticate in both cases, so let's do it *before* choosing a product, in all cases.
Attached patch patch, v1Splinter Review
Attachment #215347 - Flags: review?(mkanat)
Comment on attachment 215347 [details] [diff] [review]
patch, v1

r=mkanat by inspection. This is obviously correct.
Attachment #215347 - Flags: review?(mkanat) → review+
Severity: trivial → enhancement
Flags: approval?
This makes a person without cookies have to log in one more time before filing a bug (if all products are public).  Oh well.
Flags: approval? → approval+
Checking in enter_bug.cgi;
/cvsroot/mozilla/webtools/bugzilla/enter_bug.cgi,v  <--  enter_bug.cgi
new revision: 1.133; previous revision: 1.132
done
Checking in globals.pl;
/cvsroot/mozilla/webtools/bugzilla/globals.pl,v  <--  globals.pl
new revision: 1.362; previous revision: 1.361
done
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: