Closed Bug 556167 Opened 14 years ago Closed 14 years ago

Move the setting of dependson/blocked into Bugzilla::Bug::set_all

Categories

(Bugzilla :: Creating/Changing Bugs, enhancement)

Product:
Bugzilla
Component:
Creating/Changing Bugs
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Bugzilla 4.0

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

Attachments

(1 file, 1 obsolete file)

v2
3.50 KB, patch
dkl
: review+
Details | Diff | Splinter Review 
      No description provided.
Attached patch v1 (obsolete) — — Splinter Review 
For this one, I went with an add/remove/set API, which is probably what all the other fields will end up being.
Assignee: create-and-change → mkanat
Status: NEW → ASSIGNED
Attachment #436097 - Flags: review?(dkl)
Attached patch v2 — — Splinter Review 
Oops, v1 also contained the keywords patch.
Attachment #436097 - Attachment is obsolete: true
Attachment #436099 - Flags: review?(dkl)
Attachment #436097 - Flags: review?(dkl)
  By the way, if, while you're testing edit-multiple, you just keep getting blank pages, you need to comment out the mail = SendBugMail call in template/en/default/bug/process/bugmail.html.tmpl, just for the duration of the testing. It's a known bug in TT.
Blocks: 556373
Comment on attachment 436099 [details] [diff] [review]
v2

Everything looks great and works as expected. r=dkl
Attachment #436099 - Flags: review?(dkl) → review+
Flags: approval?
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified process_bug.cgi
modified Bugzilla/Bug.pm
Committed revision 7175.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: approval? → approval+
Resolution: --- → FIXED
Blocks: 790909
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: