Do removal from bookmarkbar via the database

diff --git a/midori/midori-browser.c b/midori/midori-browser.c
index 41a4eec83a0fa8b0c39ee37b4567fdfe3ae503a3..4566d744ce4a4e83ca80da25f0c11b46021304ba 100644 (file)
--- a/midori/midori-browser.c
+++ b/midori/midori-browser.c
@@ -3981,33 +3981,27 @@ midori_browser_bookmark_edit_activate_cb (GtkWidget*     menuitem,
                                           MidoriBrowser* browser)
 {
     KatzeItem* item;
-    const gchar* uri;
 
     item = (KatzeItem*)g_object_get_data (G_OBJECT (menuitem), "KatzeItem");
-    uri = katze_item_get_uri (item);
 
-    if (!uri)
-        midori_browser_edit_bookmark_dialog_new (browser, item, FALSE, TRUE);
-    else
+    if (KATZE_ITEM_IS_BOOKMARK (item))
         midori_browser_edit_bookmark_dialog_new (browser, item, FALSE, FALSE);
+    else
+        midori_browser_edit_bookmark_dialog_new (browser, item, FALSE, TRUE);
 }
 
 static void
 midori_browser_bookmark_delete_activate_cb (GtkWidget*     menuitem,
                                             MidoriBrowser* browser)
 {
+    sqlite3* db;
     KatzeItem* item;
-    const gchar* uri;
-    KatzeItem* parent;
 
+    db = g_object_get_data (G_OBJECT (browser->bookmarks), "db");
     item = (KatzeItem*)g_object_get_data (G_OBJECT (menuitem), "KatzeItem");
-    uri = katze_item_get_uri (item);
-
-    /* FIXME: Even toplevel items should technically have a parent */
-    g_return_if_fail (katze_item_get_parent (item));
 
-    parent = katze_item_get_parent (item);
-    katze_array_remove_item (KATZE_ARRAY (parent), item);
+    midori_bookmarks_remove_item_from_db (db, item);
+    g_object_unref (item);
 }
 
 static void