Closed Bug 152541 Opened 22 years ago Closed 22 years ago

After deleting remembered query it is still in page footer

Categories

(Bugzilla :: Query/Bug List, defect)

2.16
x86
Windows NT
defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 2.16

People

(Reporter: nn4l, Assigned: bbaetz)

References

Details

(Keywords: regression)

Attachments

(1 file)

Delete a remembered query. The "Query is gone" page shows the page footer with
the deleted query name still in place. All following page footers are ok though.

This happens with 2.16rc2 and and 2.17 (bugzilla.mozilla.org).
Thats because GetUserInfo has already pushed the stored queries into the
appropriate template var.

userprefs.cgi calls GetUserInfo again, and adding a thing to the footer manually
pushes the var.

We need to remove this particular entry from the $::vars->{'user'}->{'queries'}
array.
Keywords: regression
Target Milestone: --- → Bugzilla 2.16
Attached patch v1Splinter Review
Remove the appropriate entry....
-> me
Assignee: endico → bbaetz
Keywords: patch, review
Oh, and ignore the last hunk of the patch - I forgot to save my edited patch
before attaching it.
Status: NEW → ASSIGNED
Comment on attachment 88109 [details] [diff] [review]
v1

Without the last hunk, 2xr=jouni
Attachment #88109 - Flags: review+
Checked in, trunk + branch
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Yep, sorry. I think I caught the added query case, but not the deleted one. <sigh>

Gerv
*** Bug 153915 has been marked as a duplicate of this bug. ***
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

Created:
Updated:
Size: