diff options
author | David Lawrence <dkl@mozilla.com> | 2014-04-16 20:27:09 +0200 |
---|---|---|
committer | David Lawrence <dkl@mozilla.com> | 2014-04-16 20:27:09 +0200 |
commit | e93a383f01ba32b3949e688b9f261ca57606d802 (patch) | |
tree | 94f312e7dc9aac2d1de7392bd28a8f399a5d9a72 /template/en | |
parent | 65a0dd9e19a7465217ef596709a3cb76235dd92b (diff) | |
download | bugzilla-e93a383f01ba32b3949e688b9f261ca57606d802.tar.gz bugzilla-e93a383f01ba32b3949e688b9f261ca57606d802.tar.xz |
Bug 994619 - show_bug product list is sometimes missing the actual product a bug is in
r=gerv,a=justdave
Diffstat (limited to 'template/en')
-rw-r--r-- | template/en/default/global/product-select.html.tmpl | 30 | ||||
-rw-r--r-- | template/en/default/reports/duplicates.html.tmpl | 1 |
2 files changed, 21 insertions, 10 deletions
diff --git a/template/en/default/global/product-select.html.tmpl b/template/en/default/global/product-select.html.tmpl index 8e43c9d0e..ba5ce73b4 100644 --- a/template/en/default/global/product-select.html.tmpl +++ b/template/en/default/global/product-select.html.tmpl @@ -47,29 +47,39 @@ [% END %] [% IF Param('useclassification') %] - [% product_ids = {} %] + [% classifications = {} %] [% IF products %] [% FOREACH p = products %] - [% key = p.id -%] - [% product_ids.$key = p.id -%] + [% IF NOT classifications.${p.classification.name}.defined %] + [% classifications.${p.classification.name} = [] %] + [% END %] + [% classifications.${p.classification.name}.push(p) %] [% END %] - [% END %] - [% FOREACH c = user.get_selectable_classifications %] - <optgroup label="[% c.name FILTER html %]"> + [% ELSE %] + [% FOREACH c = user.get_selectable_classifications %] + [% classifications.${c.name} = [] %] [% FOREACH p = user.get_selectable_products(c.id) %] - [% NEXT IF (products && ! product_ids.exists(p.id)) %] + [% classifications.${c.name}.push(p) %] + [% END %] + [% END %] + [% END %] + [% FOREACH c = classifications.keys.sort %] + <optgroup label="[% c FILTER html %]"> + [% FOREACH p = classifications.$c %] <option value="[% p.$valueattribute FILTER html %]" - [% " selected" IF (cgi.param(name) == p.name) || (value == p.name) %]> + [% " selected" IF (cgi.param(name) == p.name) || (value.contains(p.name)) %]> [% p.name FILTER html %] </option> [% END %] </optgroup> [% END %] [% ELSE %] - [% products = user.get_selectable_products UNLESS products %] + [% IF NOT products.defined %] + [% products = user.get_selectable_products %] + [% END %] [% FOREACH p = products %] <option value="[% p.$valueattribute FILTER html %]" - [% " selected" IF (cgi.param(name) == p.name) || (value == p.name) %]> + [% " selected" IF (cgi.param(name) == p.name) || (value.contains(p.name)) %]> [% p.name FILTER html %] </option> [% END %] diff --git a/template/en/default/reports/duplicates.html.tmpl b/template/en/default/reports/duplicates.html.tmpl index 0d47aae75..35c141265 100644 --- a/template/en/default/reports/duplicates.html.tmpl +++ b/template/en/default/reports/duplicates.html.tmpl @@ -55,6 +55,7 @@ id => "product" name => "product" multiple => 5 + value => product %] </td> </tr> |