From: timeless%mozdev.org <>
Date: Mon, 26 May 2003 05:26:00 +0000 (+0000)
Subject: Bug 207085 Updating a stored query shouldn't return the same message as creating one
X-Git-Tag: bugzilla-2.17.5~142
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cae811513ebcca61c83fc18499cc987f58a5fdc1;p=thirdparty%2Fbugzilla.git

Bug 207085 Updating a stored query shouldn't return the same message as creating one
r=kiko
---

diff --git a/buglist.cgi b/buglist.cgi
index 7a2c894bfd..d0469cacec 100755
--- a/buglist.cgi
+++ b/buglist.cgi
@@ -329,11 +329,16 @@ elsif ($::FORM{'cmdtype'} eq "doit" && $::FORM{'remember'}) {
         }
         
         my $new_in_footer = $tofooter;
+        $vars->{'message'}   = "buglist_new_named_query";
         
         # Don't add it to the list if they are reusing an existing query name.
         foreach my $query (@{$vars->{'user'}{'queries'}}) {
-            if ($query->{'name'} eq $name && $query->{'linkinfooter'} == 1) {
-                $new_in_footer = 0;
+            if ($query->{'name'} eq $name) {
+                $vars->{'message'} = "buglist_updated_named_query";
+                if ($query->{'linkinfooter'} == 1) {
+                    $new_in_footer = 0;
+                }
+                break;
             }
         }        
         
@@ -344,7 +349,6 @@ elsif ($::FORM{'cmdtype'} eq "doit" && $::FORM{'remember'}) {
             push(@{$vars->{'user'}{'queries'}}, \%query);
         }
         
-        $vars->{'message'}   = "buglist_new_named_query";
         $vars->{'queryname'} = $name;
     }
 }
diff --git a/template/en/default/global/messages.html.tmpl b/template/en/default/global/messages.html.tmpl
index 482f7e35f6..13136d6cfb 100644
--- a/template/en/default/global/messages.html.tmpl
+++ b/template/en/default/global/messages.html.tmpl
@@ -36,10 +36,15 @@
     [% title = "Adding field to query page..." %]
     [% link  = "Click here if the page does not redisplay automatically." %]
     [% # --- %] 
+
   [% ELSIF message_tag == "buglist_load_named_query" %]
     [% title = BLOCK %]Loading your query named [% namedcmd %][% END %]
     [% link  = "Click here if the page does not redisplay automatically." %]
     [% # --- %] 
+
+  [% ELSIF message_tag == "buglist_updated_named_query" %]
+    OK, your query named <code>[% queryname FILTER html %]</code> is updated.
+
   [% ELSIF message_tag == "buglist_new_default_query" %]
     OK, you now have a new default query.  You may
     also bookmark the result of any individual query.