Closed Bug 123679 Opened 19 years ago Closed 12 years ago

Deleted bookmarks appear in search result window


(SeaMonkey :: Bookmarks & History, defect)

Not set


(Not tracked)



(Reporter: kasumi, Assigned: neil)




(1 file, 1 obsolete file)

tested 2002-02-04-11-trunk
OS = Win 98 JA 4.10.1998

1. Launch Navigator
2. Access to
3. Add this page to bookmark from toolbar or menubar
4. Delete this page from book mark
5. To search this page, select Manage bookmarks from toolbar or menubar
6. Select Edit/Find bookmarks or click Find in Sidebar/Bookmarks tab
7. Select "name" to first listbox, select "contains" to second listbox
   then type nfl to input box, click Find button
8. You will have deleted in the search result window

Expected: deleted pages should not be searched.
Related to bug 93189 and/or bug 96789?
93189 seems like rename problem.
96789 is mentioning about imported pages.
This bug is regardless of rename nor import. 
According to bug 93189 comment #1, not all the bookmark data is removed when 
the bookmark is deleted.  Maybe this data is what Find picks up?

Bug 96789 is a problem identical to this one, dealing specifically to Imported 
IE Favorites.  Maybe bug 96789 is a specific example of a more general problem?
>not all the bookmark data is removed when 
>the bookmark is deleted.  Maybe this data is what Find picks up?
I think so.

96789 is not good example because this is imported data.
We need to doubt about importing process.
Priority: -- → P3
Target Milestone: --- → mozilla1.2
removing self from cc list
Flags: blocking1.5a?
Flags: blocking1.4.x?
Anybody still alive?. This bug is OLD, SERIOUS and should be "obvious".
While this would be nice to fix, I don't think we'd hold either the 1.4.x
release or the 1.5a release for it.

Adding dependency on bug 93189.
Depends on: 93189
Flags: blocking1.5a?
Flags: blocking1.5a-
Flags: blocking1.4.x?
Flags: blocking1.4.x-
Mass reassign of my non-Firefox bugs to
Assignee: bugs → ben_seamonkey
Product: Browser → Seamonkey
Blocks: 275249
Assignee: ben_seamonkey → p_ch
QA Contact: claudius → bookmarks
Target Milestone: mozilla1.2alpha → ---
Reassigning as per Bug #32644
Assignee: p_ch → nobody
this may have been fixed (in ff) by vlad's fix for #255255
Priority: P3 → --
Duplicate of this bug: 371742
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.2a1pre) Gecko/20090715 SeaMonkey/2.1a1pre] (home, optim default) (W2Ksp4)
 + + bug 503724 patch)
Keywords: 4xp
Blocks: 96789
Depends on: 255255
Flags: wanted-seamonkey2?
Whiteboard: [Port bug 255255!!?]
I ported bug 255255 and bug 301048 and have a patch locally. The good news is that the core of this bug is already fixed in what I have so far, the even better news is that in the end it will hopefully enable/fix delete/cut while in search mode as well. :-) It just needs a little more work. Stay tuned.
Assignee: nobody → jh
Attached patch proposed patch (obsolete) — Splinter Review
For better comparison, this is the full diff of the patch from bug 255255:

This fixes/changes (Bookmark Manager and Bookmarks sidebar):
- bookmarks deleted in normal mode are also removed from Search mode view
- enables Delete and Cut (and their shortcuts) in Search mode

Test cases (Bookmark Manager and Bookmarks sidebar):
- create bookmark, verify it's visible in Search mode, delete it in normal mode, verify it's also gone in normal mode and Search mode
- create bookmark, verify it's visible in Search mode, delete it in Search mode, verify it's gone in Search mode and normal mode
- use cut instead of delete, then paste in normal or Search mode
- use Undo after delete or cut
Attachment #389572 - Flags: review?(neil)
Duplicate of this bug: 506208
Comment on attachment 389572 [details] [diff] [review]
proposed patch

Sorry, I don't think this is the right way to fix this; strangely enough the hint came from the code that makes cut work in the Search view (which is a separate bug worth fixing) - the local search service needs to check that the search results have parents.
Attachment #389572 - Flags: review?(neil) → review-
Attached patch Proposed patchSplinter Review
Assignee: jh → neil
Attachment #389572 - Attachment is obsolete: true
Attachment #392713 - Flags: review?(iann_bugzilla)
Attachment #392713 - Flags: review?(iann_bugzilla) → review+
Pushed changeset 29283536a639 to comm-central.
Closed: 12 years ago
Resolution: --- → FIXED
No longer depends on: 255255
Hardware: x86 → All
Whiteboard: [Port bug 255255!!?]
I'll continue with the rest of what my patch fixed in bug 81893 (Delete/Cut) and bug 509029 (newline added when copying bookmark).
Depends on: 255255
Hardware: All → x86
Flags: wanted-seamonkey2?
You need to log in before you can comment on or make changes to this bug.