Closed Bug 419243 Opened 16 years ago Closed 16 years ago

get_new_status_and_resolution() no longer exists and bug moving fails

Categories

(Bugzilla :: Creating/Changing Bugs, defect)

3.1.2
defect
Not set
blocker

Tracking

()

RESOLVED FIXED
Bugzilla 3.2

People

(Reporter: guy.pyrzak, Assigned: LpSolit)

References

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

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?
Flags: blocking3.2? → blocking3.2+
Keywords: regression
OS: Mac OS X → All
Hardware: PC → All
Target Milestone: --- → Bugzilla 3.2
Regression due to bug 402791.
Depends on: 402791
Assignee: create-and-change → LpSolit
Attached patch patch, v1 (obsolete) — Splinter Review
Attachment #305544 - Flags: review?(mkanat)
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 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-
Attached patch patch, v2Splinter Review
Attachment #305544 - Attachment is obsolete: true
Attachment #305619 - Flags: review?(mkanat)
Comment on attachment 305619 [details] [diff] [review]
patch, v2

Okay, looks good to me.
Attachment #305619 - Flags: review?(mkanat) → review+
Flags: approval+
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: 16 years ago
Resolution: --- → FIXED
Blocks: 414236
Blocks: 431730
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: