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)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.16
People
(Reporter: nn4l, Assigned: bbaetz)
References
Details
(Keywords: regression)
Attachments
(1 file)
1.33 KB,
patch
|
jouni
:
review+
jouni
:
review+
|
Details | Diff | Splinter Review |
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).
Assignee | ||
Comment 1•22 years ago
|
||
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
Assignee | ||
Comment 2•22 years ago
|
||
Remove the appropriate entry....
Assignee | ||
Comment 3•22 years ago
|
||
-> me
Assignee | ||
Comment 4•22 years ago
|
||
Oh, and ignore the last hunk of the patch - I forgot to save my edited patch before attaching it.
Status: NEW → ASSIGNED
Comment 5•22 years ago
|
||
Comment on attachment 88109 [details] [diff] [review] v1 Without the last hunk, 2xr=jouni
Attachment #88109 -
Flags: review+
Assignee | ||
Comment 6•22 years ago
|
||
Checked in, trunk + branch
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 7•22 years ago
|
||
Yep, sorry. I think I caught the added query case, but not the deleted one. <sigh> Gerv
Assignee | ||
Comment 8•22 years ago
|
||
*** Bug 153915 has been marked as a duplicate of this bug. ***
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•