Patch for bug 274392: Preserve search name when clicking Edit from Saves Searches...

diff --git a/buglist.cgi b/buglist.cgi
index 7858f9590185450693bc02f3059fcd8688fb2f05..1ae44467fcbc5d51b3dbc42d02d54ccfeb952a04 100755 (executable)
--- a/buglist.cgi
+++ b/buglist.cgi
@@ -921,6 +921,10 @@ if ($dotweak) {
     }
 }
 
+# If we're editing a stored query, use the existing query name as default for
+# the "Remember search as" field.
+$vars->{'defaultsavename'} = $cgi->param('query_based_on');
+
 
 ################################################################################
 # HTTP Header Generation

diff --git a/query.cgi b/query.cgi
index bd60e5b6703f20b0daa0461139526e79157cb7b4..4c3fa97d35cd699be2447c619252a26eb7ee3431 100755 (executable)
--- a/query.cgi
+++ b/query.cgi
@@ -395,6 +395,9 @@ if (($::FORM{'query_format'} || $::FORM{'format'} || "") eq "create-series") {
     $vars->{'category'} = Bugzilla::Chart::getVisibleSeries();
 }
 
+$vars->{'known_name'} = $cgi->param('known_name');
+
+
 # Add in the defaults.
 $vars->{'default'} = \%default;
 

diff --git a/template/en/default/account/prefs/saved-searches.html.tmpl b/template/en/default/account/prefs/saved-searches.html.tmpl
index 5055565e3c5e8318cf6055e835b553aae736f6f2..a99b9ed8514858728de33fd957641529b4fb0490 100644 (file)
--- a/template/en/default/account/prefs/saved-searches.html.tmpl
+++ b/template/en/default/account/prefs/saved-searches.html.tmpl
@@ -67,7 +67,7 @@
           <a href="buglist.cgi?[% q.query FILTER html %]">Run</a>
         </td>
         <td>
-          <a href="query.cgi?[% q.query FILTER html %]">Edit</a>
+          <a href="query.cgi?[% q.query FILTER html %]&known_name=[% q.name FILTER url_quote %]">Edit</a>
         </td>
         <td>
           <a href="buglist.cgi?cmdtype=dorem&amp;remaction=forget&amp;namedcmd=

diff --git a/template/en/default/list/list.html.tmpl b/template/en/default/list/list.html.tmpl
index c0291024cc0b8f386fe5b96893ee5ec11a51b205..5a3926c5e587e3e18ab439f6894b7c10c124f16d 100644 (file)
--- a/template/en/default/list/list.html.tmpl
+++ b/template/en/default/list/list.html.tmpl
@@ -22,6 +22,7 @@
 [%# INTERFACE:
   # searchtype: string. Type of search - either "series", "saved" or undef.
   # ...
+  # defaultsavename: string. The default name for saving the query.
   #%]
 
 [%############################################################################%]
@@ -159,7 +160,11 @@
     [% END %]
     
     <td valign="middle">
-      <a href="query.cgi?[% urlquerypart FILTER html %]">Edit&nbsp;Search</a>
+        [% editqueryname = searchname OR defaultsavename OR '' %]
+        <a href="query.cgi?[% urlquerypart FILTER html %]
+        [% IF editqueryname != '' %]&amp;known_name=
+            [% editqueryname FILTER url_quote %]
+        [% END %]">Edit&nbsp;Search</a>
     </td>
       
     [% IF searchtype == "saved" %]
@@ -179,7 +184,8 @@
                  value="[% urlquerypart FILTER html %]">
           <input type="hidden" name="cmdtype" value="doit">
           <input type="hidden" name="remtype" value="asnamed">
-          <input type="text" name="newqueryname" size="20"> 
+          <input type="text" name="newqueryname" size="20"
+                 value="[% defaultsavename FILTER html %]"> 
         </form> 
       </td>
     [% END %]  

diff --git a/template/en/default/search/knob.html.tmpl b/template/en/default/search/knob.html.tmpl
index c92ad92cc12fb5ad86a704dd99b32debd538246d..6c87cc4b9cfbb66ff344d9c8c7c876f49ff7d28a 100644 (file)
--- a/template/en/default/search/knob.html.tmpl
+++ b/template/en/default/search/knob.html.tmpl
@@ -21,6 +21,15 @@
   #                 Jouni Heikniemi <jouni@heikniemi.net>
   #%]
 
+[%# INTERFACE:
+  # (incomplete!)
+  # ...
+  # known_name: string. Possibly known stored name for the query being
+  #                     edited. This value is just passed through in a
+  #                     hidden field.
+  #%]
+
+
 [% PROCESS global/variables.none.tmpl %]
 
 [%# This is not necessary for English templates, but useful for localisers. %]
@@ -46,6 +55,10 @@
 
 <p>  
   <input type="submit" value="[% button_name FILTER html %]">
+  [% IF known_name %]
+    <input type="hidden" name="query_based_on" 
+           value="[% known_name FILTER html %]">
+  [% END %]
 </p>
 
 <p>

diff --git a/template/en/default/search/search-specific.html.tmpl b/template/en/default/search/search-specific.html.tmpl
index b503075523a55682f41d8ffa33b319fef517c0c5..e57566bd0555b5d45631aef1de901a25b19e2edd 100644 (file)
--- a/template/en/default/search/search-specific.html.tmpl
+++ b/template/en/default/search/search-specific.html.tmpl
@@ -100,6 +100,10 @@ for "crash secure SSL flash".
     <td></td>
     <td>
       <input type="submit" value="Search">
+      [% IF known_name %]
+        <input type="hidden" name="query_based_on" 
+               value="[% known_name FILTER html %]">
+      [% END %]
     </td>
   </tr>
 </table>