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)

2.20
defect
Not set
trivial

Tracking

()

RESOLVED FIXED
Bugzilla 2.20

People

(Reporter: Wurblzap, Assigned: Wurblzap)

Details

Attachments

(1 file, 1 obsolete file)

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.
Target Milestone: --- → Bugzilla 2.20
Attached patch Patch (obsolete) β€” β€” Splinter Review
Attachment #189531 - Flags: review?(wicked)
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-
Attached patch Patch 1.1 β€” β€” Splinter Review
Comments addressed.

I added mailrecipients to the PROCESS call, too.
Attachment #189531 - Attachment is obsolete: true
Attachment #190380 - Flags: review?(wicked)
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+
Flags: approval?
Flags: approval2.20?
Flags: approval?
Flags: approval2.20?
Flags: approval2.20+
Flags: approval+
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.

Attachment

General

Created:
Updated:
Size: