Move group insertion out of post_bug.cgi and into Bugzilla::Bug

RESOLVED FIXED in Bugzilla 3.0

Status

()

--
enhancement
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: mkanat, Assigned: mkanat)

Tracking

2.23
Bugzilla 3.0
Dependency tree / graph
Bug Flags:
approval +

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

12 years ago
Okay, now we want to move the actual code that inserts groups into the DB out of post_bug.cgi and into Bugzilla::Bug.
(Assignee)

Updated

12 years ago
Depends on: 323239
(Assignee)

Updated

12 years ago
Depends on: 351332
No longer depends on: 321332
(Assignee)

Comment 1

12 years ago
Created attachment 236740 [details] [diff] [review]
v1

Okay, here's the patch. It requires both of the blocking patches.
Assignee: create-and-change → mkanat
Status: NEW → ASSIGNED
Attachment #236740 - Flags: review?(bugzilla-mozilla)
(Assignee)

Updated

12 years ago
Blocks: 351345
(Assignee)

Comment 2

12 years ago
Created attachment 236838 [details] [diff] [review]
v1.1

Fixed bitrot.
Attachment #236740 - Attachment is obsolete: true
Attachment #236838 - Flags: review?(bugzilla-mozilla)
Attachment #236740 - Flags: review?(bugzilla-mozilla)
Version: 2.99 → 2.23
(Assignee)

Updated

12 years ago
No longer blocks: 351345
(Assignee)

Comment 3

12 years ago
Created attachment 237542 [details] [diff] [review]
v1.2

Fixed bitrot.
Attachment #236838 - Attachment is obsolete: true
Attachment #237542 - Flags: review?(LpSolit)
Attachment #236838 - Flags: review?(bugzilla-mozilla)

Comment 4

12 years ago
Comment on attachment 237542 [details] [diff] [review]
v1.2

looks good. r=LpSolit
Attachment #237542 - Flags: review?(LpSolit) → review+

Updated

12 years ago
Flags: approval?
Flags: approval? → approval+
(Assignee)

Comment 5

12 years ago
Checking in post_bug.cgi;
/cvsroot/mozilla/webtools/bugzilla/post_bug.cgi,v  <--  post_bug.cgi
new revision: 1.176; previous revision: 1.175
done
Checking in Bugzilla/Bug.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Bug.pm,v  <--  Bug.pm
new revision: 1.151; previous revision: 1.150
done
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.