Closed Bug 119671 Opened 23 years ago Closed 23 years ago

Enter Bug with *ONE* product fails when not logged in

Categories

(Bugzilla :: Creating/Changing Bugs, defect, P1)

2.15
defect

Tracking

()

RESOLVED DUPLICATE of bug 103953

People

(Reporter: burnus, Assigned: gerv)

References

Details

Attachments

(1 file)

This is with the current CVS version. a) Products > 1 If you click on 'new' enter_bug.cgi asks you "First, you must pick a product on which to enter a bug." if you aren't logged in, you now are asked to loggin (in enter_bug.cgi?product=FOO) b) Products == 1 enter_bug.cgi doesn't ask you which Product you want and thus you have two posibilities: i) you are logged in and go to form and you can enter your bug. ii) you are not yet logged in and it fails in enter_bug.cgi with: "Software error: Can't use an undefined value as an ARRAY reference at CGI.pl line 1036." In this line $i is "product" CONCLUSTION: + Bugzilla shouldn't fail if there is only one product and not-logged-in
Component: Reporting/Charting → Creating/Changing Bugs
Mixed up component.
Suggest Target=2.16
Target Milestone: --- → Bugzilla 2.16
This sets also MFORM to the $product which is used by CGI.pl. This only effects MFORM if only one product is used which is not yet set.
Keywords: patch, review
Priority: -- → P1
OS: Linux → All
Hardware: PC → All
I believe this is fixed in my templatisation patch over in bug 103953. Please wait for that to land and check if this problem still occurs. Gerv
Depends on: 103953
gerv: This isn't fixed in your patch - I noticed this bug while testing that patch out. Also, I think that attachment 65426 [details] [diff] [review] from bug 120543 is better, because it makes confirm_login more robust. Whoever reviews, either mark one as a dupe of the other, or maybe we want these both in?
ACtually, after talking to just dfave, we should have both patches, for both robustness (mine), and avoiding unexpected things (this patch). This one will need to wait for templatisation, though.
Keywords: patch, review
justdave suggested tieing $FORM{'foo'} to $MFORM{'foo'}[0]. Not sure if we want that for 2.16, though.
This is fixed in the latest patch on bug 103953. Gerv
*** This bug has been marked as a duplicate of 103953 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
clearing target milestone on resolved non-FIXED bugs so they'll get re-triaged if they get reopened.
Target Milestone: Bugzilla 2.16 → ---
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: