Closed
Bug 92515
Opened 23 years ago
Closed 19 years ago
Should be able to change resolution without re-opening...
Categories
(Bugzilla :: Creating/Changing Bugs, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.0
People
(Reporter: mozilla-work, Assigned: LpSolit)
References
Details
(Keywords: selenium, Whiteboard: [flow:status,resolution])
Attachments
(2 files, 1 obsolete file)
7.41 KB,
patch
|
wicked
:
review+
|
Details | Diff | Splinter Review |
756 bytes,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
Quite often, users will use the wrong resolution when resolving a bug. For
example, using (FIXED instead of INVALID or WORKSFORME). In order to change the
resolution, I need to re-open and and re-resolve. This generates two e-mails
for nitpicking so I don't usually do it.
It should be possible to change the resolution without having to reopen.
Possibly there could be a permission which controlled who had this privilege.
Comment 1•23 years ago
|
||
I've often wished for this myself. :-)
Priority: -- → P3
Target Milestone: --- → Bugzilla 2.16
Comment 2•23 years ago
|
||
If we do this, then I'd say that anyone who can do it in two steps should be
able to do it in one.
Reporter | ||
Comment 3•23 years ago
|
||
OK. Fair enough.
Comment 4•23 years ago
|
||
I'm not sure about this request, but it may just be inertia - I can't think of
any reason not to do it.
Currently there's a field called "Priority, status, severity, and milestone
changes" on e-mail prefs. If we did this we'd need to change this to include
resolutions.
Comment 5•23 years ago
|
||
I'd just change the wording on the existing option. For purposes of
notifications a resolution change should be considered a status change.
Comment 6•23 years ago
|
||
See also bug 87840, which is about changing the duplication target, so it's a
special case of this bug.
Comment 7•23 years ago
|
||
-> Bugzilla product, Changing Bugs component, reassigning.
Assignee: justdave → myk
Component: Bugzilla → Creating/Changing Bugs
Product: Webtools → Bugzilla
Version: Bugzilla 2.12 → 2.12
Updated•23 years ago
|
Whiteboard: [flow:status,resolution]
Comment 8•23 years ago
|
||
We are currently trying to wrap up Bugzilla 2.16. We are now close enough to
release time that anything that wasn't already ranked at P1 isn't going to make
the cut. Thus this is being retargetted at 2.18. If you strongly disagree with
this retargetting, please comment, however, be aware that we only have about 2
weeks left to review and test anything at this point, and we intend to devote
this time to the remaining bugs that were designated as release blockers.
Target Milestone: Bugzilla 2.16 → Bugzilla 2.18
Comment 9•23 years ago
|
||
*** Bug 136631 has been marked as a duplicate of this bug. ***
Comment 10•22 years ago
|
||
*** Bug 179104 has been marked as a duplicate of this bug. ***
Comment 11•22 years ago
|
||
*** Bug 187640 has been marked as a duplicate of this bug. ***
Updated•21 years ago
|
OS: Solaris → All
Hardware: Sun → All
Comment 12•21 years ago
|
||
Enhancements which don't currently have patches on them which are targetted at
2.18 are being retargetted to 2.20 because we're about to freeze for 2.18.
Consideration will be taken for moving items back to 2.18 on a case-by-case
basis (but is unlikely for enhancements)
Target Milestone: Bugzilla 2.18 → Bugzilla 2.20
Comment 13•20 years ago
|
||
Bugzilla 2.20 feature set is now frozen as of 15 Sept 2004. Anything flagged
enhancement that hasn't already landed is being pushed out. If this bug is
otherwise ready to land, we'll handle it on a case-by-case basis, please set the
blocking2.20 flag to '?' if you think it qualifies.
Target Milestone: Bugzilla 2.20 → Bugzilla 2.22
Comment 14•19 years ago
|
||
*** Bug 295724 has been marked as a duplicate of this bug. ***
Comment 15•19 years ago
|
||
*** Bug 285540 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 16•19 years ago
|
||
I'll take it as soon as we unfreeze if nobody else wants to. I want it in 2.24.
Target Milestone: Bugzilla 2.22 → Bugzilla 2.24
Assignee | ||
Updated•19 years ago
|
Assignee: myk → LpSolit
Priority: P3 → --
Assignee | ||
Comment 17•19 years ago
|
||
Attachment #214790 -
Flags: review?(wicked)
Comment 18•19 years ago
|
||
Comment on attachment 214790 [details] [diff] [review]
patch, v1
This block from CheckCanChangeField() should be dealt with too:
# A resolution change is always accompanied by a status change. So, we
# always OK resolution changes; if they really can't do this, we will
# notice it when status is checked.
if ($field eq "resolution") {
return 1;
}
>Index: process_bug.cgi
>===================================================================
>- /^resolve$/ && CheckonComment( "resolve" ) && do {
>+ /^(resolve|change_resolution)$/ && CheckonComment( "resolve" ) && do {
Shouldn't use commentonresolve to mandate a comment when change_resolution is done because we are not actually resolving the bug anymore. Maybe add a commentonchangeresolution parameter?
>- # Check here, because its the only place we require the resolution
> check_field('resolution', scalar $cgi->param('resolution'),
> \@::settable_resolution);
Nit: Why remove the comment? Isn't it still valid?
>- # Make sure the bug is not already marked as a dupe
>- # (may appear in race condition)
>- my $dupe_of =
>- $dbh->selectrow_array("SELECT dupe_of FROM duplicates
>- WHERE dupe = ?",
>- undef, $cgi->param('id'));
>- if ($dupe_of) {
>- ThrowUserError("dupe_entry_found", { dupe_of => $dupe_of });
>- }
>+ # If the bug was already marked as a duplicate, remove
>+ # the existing entry.
>+ $dbh->do('DELETE FROM duplicates WHERE dupe = ?',
>+ undef, $cgi->param('id'));
This change is alarming. Bug state shouldn't be changed at this stage because a) it won't be logged and b) the script could still end with an error.
Attachment #214790 -
Flags: review?(wicked) → review-
Assignee | ||
Comment 19•19 years ago
|
||
Attachment #214790 -
Attachment is obsolete: true
Attachment #215053 -
Flags: review?(wicked)
Comment 20•19 years ago
|
||
Comment on attachment 215053 [details] [diff] [review]
patch, v2
>Index: process_bug.cgi
>===================================================================
>- /^resolve$/ && CheckonComment( "resolve" ) && do {
>+ /^(resolve|change_resolution)$/ && CheckonComment( "resolve" ) && do {
Nit: I still think a separate param would be better.
Looks like dupe_of changes are not logged in the activity log nor bugmailed about. This seems to happen even before this patch so I won't hold review for that. Although, one would have seen the change to REOPEN and then to RESOLVED DUPLICATE again previously. Resolution changes themself are logged and bugmailed about correctly.
There will be multiple duplicate notations but that's a different problem too. We should really stop doing these comments and simply show duplication states in the bug info.
Attachment #215053 -
Flags: review?(wicked+bz) → review+
Updated•19 years ago
|
Flags: approval?
Updated•19 years ago
|
Flags: approval? → approval+
Assignee | ||
Comment 21•19 years ago
|
||
Checking in process_bug.cgi;
/cvsroot/mozilla/webtools/bugzilla/process_bug.cgi,v <-- process_bug.cgi
new revision: 1.310; previous revision: 1.309
done
Checking in template/en/default/bug/knob.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/knob.html.tmpl,v <-- knob.html.tmpl
new revision: 1.20; previous revision: 1.19
done
Checking in template/en/default/global/user-error.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/global/user-error.html.tmpl,v <-- user-error.html.tmpl
new revision: 1.162; previous revision: 1.161
done
Comment 22•18 years ago
|
||
Added to relnotes in bug 349423. Please let me know if I missed any critical information about this bug in the release notes.
Keywords: relnote
Assignee | ||
Updated•18 years ago
|
Flags: testcase?
Assignee | ||
Comment 23•14 years ago
|
||
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/qa/4.0/
modified t/test_bug_edit.t
Committed revision 175.
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/qa/3.6/
modified t/test_bug_edit.t
Committed revision 146.
Attachment #510051 -
Flags: review+
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•