Closed
Bug 301018
Opened 19 years ago
Closed 19 years ago
Add INTERFACE section to bug/create/created.html.tmpl
Categories
(Bugzilla :: Creating/Changing Bugs, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: Wurblzap, Assigned: Wurblzap)
Details
Attachments
(1 file, 1 obsolete file)
|
1.43 KB,
patch
|
wicked
:
review+
|
Details | Diff | Splinter Review |
Bug 289382 introduced an INTERFACE section to bug/create/created.html.tmpl, but only on the 2.18 branch because it didn't affect the trunk. The plan is to copy this interface to the 2.20 branch and the trunk.
| Assignee | ||
Updated•19 years ago
|
Target Milestone: --- → Bugzilla 2.20
Comment 2•19 years ago
|
||
Comment on attachment 189531 [details] [diff] [review] Patch >+[%# INTERFACE: >+ # sentmail: array of hash; bugs for which BugMail should be sent, contains: ... >+ # mailrecipients: hash; contains the BugMail recipients, for details on >+ # this contents, see template bug/process/bugmail.html.tmpl On tip, mailrecipients is not in sentmail hash but rather is an individual variable. Otherwise it's same (except it contains only changer element for created bugs as documented in the bugmail template). >+ # bug_list: array of integers; storted bug list (used in template Nit: typo, sorted not storted (looks like this was on my original patch too).
Attachment #189531 -
Flags: review?(wicked) → review-
| Assignee | ||
Comment 3•19 years ago
|
||
Comments addressed. I added mailrecipients to the PROCESS call, too.
Attachment #189531 -
Attachment is obsolete: true
Attachment #190380 -
Flags: review?(wicked)
Comment 4•19 years ago
|
||
Comment on attachment 190380 [details] [diff] [review] Patch 1.1 >Index: template/en/default/bug/create/created.html.tmpl >@@ -32,6 +46,7 @@ ... >+ mailrecipients = mailrecipients Nit: This is actually not needed because PROCESS allows called template to see all the variables of the calling template. I tested this change and it seems it doesn't break things either while making things more explicit. Otherwise, this looks good to me.
Attachment #190380 -
Flags: review?(wicked) → review+
Updated•19 years ago
|
Flags: approval?
Flags: approval2.20?
Updated•19 years ago
|
Flags: approval?
Flags: approval2.20?
Flags: approval2.20+
Flags: approval+
Comment 5•19 years ago
|
||
tip: Checking in template/en/default/bug/create/created.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/create/created.html.tmpl,v <-- created.html.tmpl new revision: 1.9; previous revision: 1.8 done 2.20rc1: Checking in template/en/default/bug/create/created.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/create/created.html.tmpl,v <-- created.html.tmpl new revision: 1.8.10.1; previous revision: 1.8 done
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•