After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 518727 - Possible data loss due to UI problem ("All" topic being rename-able)
Possible data loss due to UI problem ("All" topic being rename-able)
Status: RESOLVED OBSOLETE
Product: epiphany
Classification: Core
Component: Bookmarks
git master
Other Linux
: High critical
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-02-25 23:06 UTC by Diego Escalante Urrelo (not reading bugmail)
Modified: 2014-11-22 10:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix new bookmark topic addition when search entry is not empty. (994 bytes, patch)
2010-05-26 18:34 UTC, Vincas Dargis
none Details | Review
ephy-bookmarks-editor: fix new topic selection bug (1.22 KB, patch)
2011-01-03 20:17 UTC, Diego Escalante Urrelo (not reading bugmail)
committed Details | Review

Description Diego Escalante Urrelo (not reading bugmail) 2008-02-25 23:06:09 UTC
Turns out you can trick the bookmarks editor to rename the All topic.
This of course is not good, it's even worse when I lost my bookmarks because I renamed the topic lots of time without noticing what was happening.

To reproduce:
1. open bookmark editor, 
2. type something in search entry, 
3. click file > new topic, 
4. type your name and drop your jaw as you see All topic renamed

I'm unable to reproduce the data loss, but it happened and it sucked :). Some warnings were printed after it on the console, but I was too desperate to take note of them.
Comment 1 Christian Persch 2009-04-10 18:23:01 UTC
Reality check.
Comment 2 Vincas Dargis 2010-05-26 18:34:47 UTC
Created attachment 162039 [details] [review]
Fix new bookmark topic addition when search entry is not empty.

Possible patch.

I'm not sure is this a walk-around or a solution.

It looks like gtk_tree_model_sort_convert_child_iter_to_iter() (in ephy_node_view_select_node()) returns different iterator then expected if search entry is not empty.
Comment 3 Diego Escalante Urrelo (not reading bugmail) 2011-01-03 20:17:01 UTC
Created attachment 177431 [details] [review]
ephy-bookmarks-editor: fix new topic selection bug

When the search entry is not empty and "New topic" is activated from the menu,
the "All" topic will set for renaming instead of the new "Type a topic" node.
This can cause serious data loss.

Bug #518727

Signed-off-by: Diego Escalante Urrelo <descalante@igalia.com>
Comment 4 Diego Escalante Urrelo (not reading bugmail) 2011-01-11 01:27:44 UTC
Comment on attachment 177431 [details] [review]
ephy-bookmarks-editor: fix new topic selection bug

Leaving open for a proper fix someday.

Attachment 177431 [details] pushed as 876c61d - ephy-bookmarks-editor: fix new topic selection bug
Comment 5 Kat 2013-07-05 22:22:18 UTC
This bug does not appear to exist in 3.8.2: I cannot duplicate it.
Comment 6 Kat 2014-11-22 10:36:19 UTC
Still can't reproduce in 3.14.2 so closing as obsolete.