After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 71286 - should be able to go straight to "RESOLVED" from "NEEDINFO"
should be able to go straight to "RESOLVED" from "NEEDINFO"
Status: RESOLVED FIXED
Product: bugzilla.gnome.org
Classification: Infrastructure
Component: general
unspecified
Other other
: Normal enhancement
: ---
Assigned To: Bugzilla Maintainers
Bugzilla Maintainers
: 91210 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2002-02-12 16:17 UTC by Darin Adler
Modified: 2005-02-27 00:14 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch v1 (1.27 KB, patch)
2005-02-26 22:25 UTC, Olav Vitters
none Details | Review
Patch v2 (2.08 KB, patch)
2005-02-26 23:22 UTC, Olav Vitters
none Details | Review
Patch v3 (4.06 KB, patch)
2005-02-26 23:50 UTC, Olav Vitters
none Details | Review
Additional patch to fix small issue (1.00 KB, patch)
2005-02-27 00:10 UTC, Olav Vitters
none Details | Review

Description Darin Adler 2002-02-12 16:17:46 UTC
It's a real pain to have to re-open a bug just to change it to "RESOLVED".

In fact, I'd also like to be able to go to "ASSIGNED" directly from
"NEEDINFO".
Comment 1 Gregory Leblanc 2002-02-12 20:55:26 UTC
I tried to make a patch for this at one point, but it turns out to be
rather more complex than I thought.  The bugzilla developers upstream
say this functionality will be available in the stock 2.18, but that
seems quite some long time away.  Perhaps I'll take another look at
the code, and see how hard it really is.
Comment 2 Luis Villa 2004-03-25 17:31:36 UTC
Greg, any idea on this one?
Comment 3 Olav Vitters 2005-02-26 22:25:08 UTC
Created attachment 37984 [details] [review]
Patch v1

Patch adds NEEDINFO to the OpenStates. Without that, Bugzilla will not always
do the right thing (example: NEEDINFO -> ASSIGNED won't work). After this
NEEDINFO has the same options as a bug in UNCONFIRMED/NEW/REOPENED/ASSIGNED
state.

Patch also adds the closed option back to when the bug is in NEEDINFO, so the
bug can still be closed like now.

Note: Haven't fully tested all options/states, doing that now... seems to work
though :)
Comment 4 Elijah Newren 2005-02-26 22:42:55 UTC
Actually, I think I'd rather force a bug to go to one of the resolved states
than straight to closed.  I think Andrew said that going straight from needinfo
to closed caused some sanity checks to fail; granted we do ignore those now, but
I think it'd be nicer to always give a reason for closing a needinfo bug and
thus just use the resolved state instead.

Thoughts, opinions?
Comment 5 Olav Vitters 2005-02-26 23:22:09 UTC
Created attachment 37987 [details] [review]
Patch v2

Unconfirmed->needinfo didn't work. Also forgot to add an reopen for needinfo.
Removed the needinfo->closed option.

Passes the following checks:
Mark unconfirmed bug needinfo
Mark new bug needinfo
Reopen needinfo everconfirmed=0 bug (state goes to unconfirmed)
Reopen needinfo everconfirmed=1 bug (state goes to reopened)
Reopen needinfo everconfirmed=1 bug + mark new (state goes to new)
Accept needinfo bug (state goes to assigned)
Marking needinfo duplicate (state goes to resolved duplicate)
Marking needinfo fixed (state goes to resolved fixed)

Following are possible errors:
Reassign needinfo everconfirmed=1 bug (state goes to new)
--> maybe not what we want?
Also goes to new when reassigning to owner of selected component
Comment 6 Olav Vitters 2005-02-26 23:50:01 UTC
Created attachment 37989 [details] [review]
Patch v3

Reassigning now keeps the needinfo state. When a bug was never confirmed and
the status is needinfo, I now hide the 'reassign and mark NEW' options. These
make no sense after above change.

This patch passes all my tests :)
Comment 7 Olav Vitters 2005-02-27 00:10:31 UTC
Created attachment 37990 [details] [review]
Additional patch to fix small issue

Fixes small issue when bug had everconfirmed=1 and was in the needinfo state
(the javascript selected the wrong 'knob'). Eg when you would fill in a
duplicate number, actually the change resolution to fixed would be selected.

Patch is to be applied after v3
Comment 8 Olav Vitters 2005-02-27 00:12:54 UTC
Marking FIXED!! :-)
Comment 9 Olav Vitters 2005-02-27 00:14:57 UTC
*** Bug 91210 has been marked as a duplicate of this bug. ***