Last Comment Bug 551918 - Deleting a cert while filtered results in a random item being displayed
: Deleting a cert while filtered results in a random item being displayed
Product: Core Graveyard
Classification: Graveyard
Component: Security: UI (show other bugs)
: Trunk
: All All
: -- normal (vote)
: ---
Assigned To: timeless
Depends on:
  Show dependency treegraph
Reported: 2010-03-12 02:06 PST by timeless
Modified: 2016-09-27 13:03 PDT (History)
1 user (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---

update after mutation (2.34 KB, patch)
2010-03-15 19:01 PDT, timeless
kaie: review-
Details | Diff | Splinter Review

Description timeless 2010-03-12 02:06:44 PST
in bug 469555 comment 10, kaie notes:

You have a bug when deleting a cert. I used the "people" tab. I typed a search
string, until the list showed one single cert, only. I deleted that cert.

I now expected the shown list to be empty, but the list showed some other
non-matching cert.
Comment 1 timeless 2010-03-15 19:01:30 PDT
Created attachment 432718 [details] [diff] [review]
update after mutation

this also avoids resetting the string needlessly
Comment 2 Kai Engert (:kaie) (on vacation) 2010-03-25 10:46:39 PDT
The patch works for me, but please make the minor changes we discussed in IRC.

Please rename temp variable hint to newHint.
Please add the comment in front of the statement with the bitwise XOR operator.
Comment 3 Kai Engert (:kaie) (on vacation) 2010-03-25 10:57:20 PDT
Comment on attachment 432718 [details] [diff] [review]
update after mutation

r-, please add the comments.

why do you need the "value:" ?

r+ if you address my requests and explain why value: prefix is necessary.

(Patch worked for me when testing)
Comment 4 timeless 2011-04-12 03:21:49 PDT
murdering useless bug

Note You need to log in before you can comment on or make changes to this bug.