Commit cae81151 authored by timeless%mozdev.org's avatar timeless%mozdev.org

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

r=kiko
parent 12a601d9
...@@ -329,12 +329,17 @@ elsif ($::FORM{'cmdtype'} eq "doit" && $::FORM{'remember'}) { ...@@ -329,12 +329,17 @@ elsif ($::FORM{'cmdtype'} eq "doit" && $::FORM{'remember'}) {
} }
my $new_in_footer = $tofooter; 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. # Don't add it to the list if they are reusing an existing query name.
foreach my $query (@{$vars->{'user'}{'queries'}}) { foreach my $query (@{$vars->{'user'}{'queries'}}) {
if ($query->{'name'} eq $name && $query->{'linkinfooter'} == 1) { if ($query->{'name'} eq $name) {
$vars->{'message'} = "buglist_updated_named_query";
if ($query->{'linkinfooter'} == 1) {
$new_in_footer = 0; $new_in_footer = 0;
} }
break;
}
} }
if ($new_in_footer) { if ($new_in_footer) {
...@@ -344,7 +349,6 @@ elsif ($::FORM{'cmdtype'} eq "doit" && $::FORM{'remember'}) { ...@@ -344,7 +349,6 @@ elsif ($::FORM{'cmdtype'} eq "doit" && $::FORM{'remember'}) {
push(@{$vars->{'user'}{'queries'}}, \%query); push(@{$vars->{'user'}{'queries'}}, \%query);
} }
$vars->{'message'} = "buglist_new_named_query";
$vars->{'queryname'} = $name; $vars->{'queryname'} = $name;
} }
} }
......
...@@ -36,10 +36,15 @@ ...@@ -36,10 +36,15 @@
[% title = "Adding field to query page..." %] [% title = "Adding field to query page..." %]
[% link = "Click here if the page does not redisplay automatically." %] [% link = "Click here if the page does not redisplay automatically." %]
[% # --- %] [% # --- %]
[% ELSIF message_tag == "buglist_load_named_query" %] [% ELSIF message_tag == "buglist_load_named_query" %]
[% title = BLOCK %]Loading your query named [% namedcmd %][% END %] [% title = BLOCK %]Loading your query named [% namedcmd %][% END %]
[% link = "Click here if the page does not redisplay automatically." %] [% 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" %] [% ELSIF message_tag == "buglist_new_default_query" %]
OK, you now have a new default query. You may OK, you now have a new default query. You may
also bookmark the result of any individual query. also bookmark the result of any individual query.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment