Closed Bug 207085 Opened 21 years ago Closed 21 years ago

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

Categories

(Bugzilla :: Query/Bug List, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: timeless, Assigned: timeless)

Details

Attachments

(1 file)

 
Attached patch add responseSplinter Review
Attachment #124184 - Flags: review?(kiko)
Comment on attachment 124184 [details] [diff] [review]
add response

>         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;
>             }
>         }        

You missed a close-bracket there and the indent is wrong. I'd also suggest
adding a break to that if clause as a micro-optim-correctness thing.

r=kiko with those changes, this is nice.
Attachment #124184 - Flags: review?(kiko) → review-
you mean last :) oh well. fixed.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Next time please get approval for your bug *before* you check it in.  Thanks.
Flags: approval+
Target Milestone: --- → Bugzilla 2.18
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: