Closed
Bug 419243
Opened 17 years ago
Closed 17 years ago
get_new_status_and_resolution() no longer exists and bug moving fails
Categories
(Bugzilla :: Creating/Changing Bugs, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.2
People
(Reporter: guy.pyrzak, Assigned: LpSolit)
References
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
5.53 KB,
patch
|
mkanat
:
review+
|
Details | Diff | Splinter Review |
Try to move a bug to another install using the landfill. You will run into this bug.
In process bug, line 438: my ($status, $resolution) = $bug->get_new_status_and_resolution('move');
Flags: blocking3.2?
![]() |
Assignee | |
Updated•17 years ago
|
Flags: blocking3.2? → blocking3.2+
Keywords: regression
OS: Mac OS X → All
Hardware: PC → All
Target Milestone: --- → Bugzilla 3.2
![]() |
Assignee | |
Updated•17 years ago
|
Assignee: create-and-change → LpSolit
![]() |
Assignee | |
Comment 2•17 years ago
|
||
Attachment #305544 -
Flags: review?(mkanat)
![]() |
Assignee | |
Updated•17 years ago
|
Status: NEW → ASSIGNED
Summary: get_new_status_and_resolution is missing from Bug.pm → get_new_status_and_resolution() no longer exists and bug moving fails
Comment 3•17 years ago
|
||
Comment on attachment 305544 [details] [diff] [review]
patch, v1
I'm not totally happy with adding $moving, but I suppose I see why it's there.
Now that we have so many parameters to set_status and set_resolution, they need to be named parameters after the first one. For example, set_status needs to be:
set_status($status, { resolution => $resolution, dupe_of => $dupe_of, moving => 1 });
Attachment #305544 -
Flags: review?(mkanat) → review-
![]() |
Assignee | |
Comment 4•17 years ago
|
||
Attachment #305544 -
Attachment is obsolete: true
Attachment #305619 -
Flags: review?(mkanat)
Comment 5•17 years ago
|
||
Comment on attachment 305619 [details] [diff] [review]
patch, v2
Okay, looks good to me.
Attachment #305619 -
Flags: review?(mkanat) → review+
Updated•17 years ago
|
Flags: approval+
![]() |
Assignee | |
Comment 6•17 years ago
|
||
Checking in process_bug.cgi;
/cvsroot/mozilla/webtools/bugzilla/process_bug.cgi,v <-- process_bug.cgi
new revision: 1.407; previous revision: 1.406
done
Checking in Bugzilla/Bug.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Bug.pm,v <-- Bug.pm
new revision: 1.235; previous revision: 1.234
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•