From 0ffcbbbc49cb763d15af08113eee7fc58ee19428 Mon Sep 17 00:00:00 2001
From: "lpsolit%gmail.com"
 <lpsolit%gmail.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Wed, 19 Aug 2009 14:48:43 +0000
Subject: [PATCH] =?UTF-8?q?Bug=20505015:=20Selections=20are=20not=20persis?=
 =?UTF-8?q?tent=20across=20submissions=20-=20Patch=20by=20Fr=C3=83=C2=A9d?=
 =?UTF-8?q?=C3=83=C2=A9ric=20Buclin=20<LpSolit@gmail.com>=20r=3Dghendricks?=
 =?UTF-8?q?=20a=3DLpSolit?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

git-svn-id: svn://10.0.0.236/trunk@258169 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../en/default/reports/create-chart.html.tmpl | 30 +++++--------------
 .../default/reports/series-common.html.tmpl   | 10 ++++---
 2 files changed, 14 insertions(+), 26 deletions(-)

diff --git a/mozilla/webtools/bugzilla/template/en/default/reports/create-chart.html.tmpl b/mozilla/webtools/bugzilla/template/en/default/reports/create-chart.html.tmpl
index 4762c62c0c9..0466b11c187 100644
--- a/mozilla/webtools/bugzilla/template/en/default/reports/create-chart.html.tmpl
+++ b/mozilla/webtools/bugzilla/template/en/default/reports/create-chart.html.tmpl
@@ -50,6 +50,7 @@ function subcatSelected() {
     i++;
   }
 
+  namewidget.disabled = false;
   namewidget.options[0].selected = true;
   
   checkNewState();
@@ -98,23 +99,16 @@ function subcatSelected() {
         <td>
           <noscript>
             <input type="submit" name="action-assemble" value="Update --&gt;"
-                   id="action-assemble">
+                   id="action-assemble2">
           </noscript>
         </td>
         
-        <td align="left">
-          <select name="name" id="name" style="width: 15em"
-                  size="5" multiple="multiple"
-                  [%+ "disabled=\"disabled\"" UNLESS name.keys.size %]>
-            [% FOREACH x = name.keys.sort %]
-              <option value="[% name.$x FILTER html %]">
-                [% x FILTER html %]</option>
-            [% END %]
-            [% UNLESS name.keys.size %]
-              <option value="" disabled="disabled"></option>
-            [% END %]
-          </select>
-        </td>
+        [% PROCESS series_select sel = { name => 'name',
+                                         size => 5,
+                                         multiple => 1,
+                                         # We want to use the series ID as value,
+                                         # not its name.
+                                         value_in_hash => 1 } %]
 
         <td align="center" valign="middle"> 
           <input type="submit" name="action-add" value="Add To List"
@@ -124,14 +118,6 @@ function subcatSelected() {
     [% END %]
   </table>
 
-  <script type="text/javascript">
-    document.chartform.category[0].selected = true;
-    document.chartform.subcategory.disabled = '';
-    document.chartform.name.disabled = '';
-    catSelected();
-    subcatSelected();
-  </script>
-
   <h3>List Of Data Sets To Plot</h3>
 
   [% IF chart.lines.size > 0 %]
diff --git a/mozilla/webtools/bugzilla/template/en/default/reports/series-common.html.tmpl b/mozilla/webtools/bugzilla/template/en/default/reports/series-common.html.tmpl
index 35586cb4785..cecf288ec12 100644
--- a/mozilla/webtools/bugzilla/template/en/default/reports/series-common.html.tmpl
+++ b/mozilla/webtools/bugzilla/template/en/default/reports/series-common.html.tmpl
@@ -48,7 +48,6 @@ var series = {
 [% END %] 
 };
 
-[%# Should attempt to preserve selection across invocations @@@ %]
 [%# This function takes necessary action on selection of a category %]
 function catSelected() {
   var cat = document.chartform.category.value;
@@ -67,7 +66,8 @@ function catSelected() {
   [% IF newtext %]
     subcatwidget.options[i] = new Option("[% newtext FILTER js %]", "");
   [% END %]  
-  
+
+  subcatwidget.disabled = false;
   subcatwidget.options[0].selected = true;
   
   if (document.chartform.action[1]) {
@@ -100,11 +100,13 @@ function checkNewState() {
   <td align="left">
     <select name="[% sel.name %]" id="[% sel.name %]"
             size="[% sel.size %]" style="width: 15em"
+            [%+ 'multiple="multiple"' IF sel.multiple %]
             [%+ "disabled=\"disabled\"" UNLESS ${sel.name}.keys.size || newtext %]
             [%+ "onchange=\"$sel.onchange\"" IF sel.onchange %]>
       [% FOREACH x = ${sel.name}.keys.sort %]
-        <option value="[% x FILTER html %]"
-          [% " selected" IF default.${sel.name} == x %]>
+        [% value = sel.value_in_hash ? ${sel.name}.$x : x %]
+        <option value="[% value FILTER html %]"
+          [% " selected" IF default.${sel.name} == value %]>
           [% x FILTER html %]</option>
       [% END %]
       [% IF newtext %]