163457 - bugs not registered as "new"

Status: RESOLVED FIXED

(Bugzilla :: Bugzilla-General, defect)

Description

[Bjarne D Mathiesen]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC; en-US; rv:1.1) Gecko/20020814 MultiZilla/v1.1.20
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC; en-US; rv:1.1) Gecko/20020814 MultiZilla/v1.1.20

I've been given the authorization to register bugs as "new"
when using the helper form they get registered as "unconfirmed"
in the old version my bugs started as "new"

If using the enter bug form, my bugs can start as "new"

Reproducible: Always

Steps to Reproduce:

Comment 1

[Bradley Baetz (:bbaetz)]

-> bugzilla helper, but ISTR that gerv did that on purpose

Comment 2

[Bjarne D Mathiesen]

seems to be a problem only when I report bugs for the browser
this bug did start as "new"

Comment 3

[Gervase Markham [:gerv]]

I didn't do this on purpose, and he is absolutely right. I wonder why this is.
I'll try and work it out.

Gerv

Comment 4

[Gervase Markham [:gerv]]

Attachment: Patch v.1

This problem is due to some convoluted logic in post_bug, which gets it wrong
when bug_status is absent completely. Rather than add bug_status to the new
template, I've fixed the logic in post_bug instead.

Gerv

Comment 5

[Gervase Markham [:gerv]]

-> me, Bugzilla product.

Comment 6

[timeless]

Comment on attachment 95891 [details] [diff] [review]
Patch v.1

yeah this bit me once, someone decided to triage my &format=simple filed bug.
however i like being able to file unconfirmed bugs, although i'm not certain
i'd ever use that option in simple mode.  Ok, conclusion: if i want to file an
unconfirmed bug, i'll use advanced mode.

for reference, the old logic was:
if there's a status, and if it's unconfirmed or the user can't edit/confirm
then remove the flag. when there's no flag, it's unconfirmed.

the new logic is, if the user can edit[/confirm] then use the provided status
or mark it as new.

r=timeless

Comment 7

[Bradley Baetz (:bbaetz)]

Comment on attachment 95891 [details] [diff] [review]
Patch v.1

Use ||= for the $::FORM{'bug_status'}. I don't think that any of the  carefully
aranged logic is broke by this, but you probably want to keep some of the
comment.

r=bbaetz

Comment 8

[Gervase Markham [:gerv]]

Fixed.

Checking in post_bug.cgi;
/cvsroot/mozilla/webtools/bugzilla/post_bug.cgi,v  <--  post_bug.cgi
new revision: 1.61; previous revision: 1.60
done

Gerv

Comment 9

[Gervase Markham [:gerv]]

*** Bug 164460 has been marked as a duplicate of this bug. ***