Drop bookmark type field and save meta boolean properly

diff --git a/midori/main.c b/midori/main.c
index 0b40b412899566b83ad1c611a132eaefca9e0ed4..9c38e3b4a294598f2a9983e5b730d353637b7b10 100644 (file)
--- a/midori/main.c
+++ b/midori/main.c
@@ -447,7 +447,7 @@ midori_bookmarks_initialize (KatzeArray*  array,
 
     if (sqlite3_exec (db,
                       "CREATE TABLE IF NOT EXISTS "
-                      "bookmarks (uri text, title text, folder text, type integer, "
+                      "bookmarks (uri text, title text, folder text, "
                       "desc text, app integer, toolbar integer);",
                       NULL, NULL, errmsg) != SQLITE_OK)
         return NULL;

diff --git a/midori/midori-browser.c b/midori/midori-browser.c
index 99488a9583cc05e1e1b605598dfa6ef281b9f39a..8113567b676a5eafef6cb364002e7b06258e92ba 100644 (file)
--- a/midori/midori-browser.c
+++ b/midori/midori-browser.c
@@ -842,7 +842,7 @@ midori_browser_edit_bookmark_dialog_new (MidoriBrowser* browser,
         #if HAVE_SQLITE
         i = 0;
         n = 1;
-        sqlcmd = "SELECT title from bookmarks where type=0";
+        sqlcmd = "SELECT title from bookmarks where uri=''";
         result = sqlite3_prepare_v2 (db, sqlcmd, -1, &statement, NULL);
         while ((result = sqlite3_step (statement)) == SQLITE_ROW)
         {
@@ -6540,7 +6540,7 @@ midori_bookmarkbar_populate (MidoriBrowser* browser)
 
             subsqlcmd = g_strdup_printf ("SELECT uri, title, app FROM bookmarks WHERE "
                                          " folder = '%s'", katze_item_get_name (item));
-            subfolder = katze_array_from_sqlite (db, sqlcmd);
+            subfolder = katze_array_from_sqlite (db, subsqlcmd);
             katze_item_set_name (KATZE_ITEM (subfolder), katze_item_get_name (item));
             midori_bookmarkbar_insert_item (browser->bookmarkbar, KATZE_ITEM (subfolder));
             g_free (subsqlcmd);

diff --git a/panels/midori-bookmarks.c b/panels/midori-bookmarks.c
index 791f786ed3cbe3c448952ef20fc126eaab07cf32..3bd2ef4d63ea17d86fe98c78ab32c0c315cda58f 100644 (file)
--- a/panels/midori-bookmarks.c
+++ b/panels/midori-bookmarks.c
@@ -177,11 +177,13 @@ midori_bookmarks_insert_item_db (sqlite3*   db,
 {
     gchar* sqlcmd;
     char* errmsg = NULL;
-    int type = 0;
     gchar* parent;
+    gchar* uri;
 
     if (KATZE_ITEM_IS_BOOKMARK (item))
-        type = 1;
+        uri = g_strdup (katze_item_get_uri (item));
+    else
+        uri = g_strdup ("");
 
     if (folder)
         parent = g_strdup (folder);
@@ -194,14 +196,13 @@ midori_bookmarks_insert_item_db (sqlite3*   db,
        handling of katze_item_get_meta_integer(). */
     /* FIXME: Need proper single quote escaping. */
     sqlcmd = g_strdup_printf (
-            "INSERT into bookmarks (uri, title, folder, type, toolbar, app) values"
-            " ('%s', '%s', '%s', %d, %d, %d)",
-            katze_item_get_uri (item),
+            "INSERT into bookmarks (uri, title, folder, toolbar, app) values"
+            " ('%s', '%s', '%s', %d, %d)",
+            uri,
             katze_item_get_name (item),
             parent,
-            type,
-            (int)katze_item_get_meta_integer (item, "toolbar"),
-            (int)katze_item_get_meta_integer (item, "app"));
+            katze_item_get_meta_boolean (item, "toolbar"),
+            katze_item_get_meta_boolean (item, "app"));
 
     if (sqlite3_exec (db, sqlcmd, NULL, NULL, &errmsg) != SQLITE_OK)
     {
@@ -209,6 +210,7 @@ midori_bookmarks_insert_item_db (sqlite3*   db,
         sqlite3_free (errmsg);
     }
 
+    g_free (uri);
     g_free (parent);
     g_free (sqlcmd);
 }
@@ -226,8 +228,7 @@ midori_bookmarks_remove_item_from_db (sqlite3*   db,
             katze_item_get_uri (item));
     else
        sqlcmd = sqlite3_mprintf (
-            "DELETE FROM bookmarks WHERE folder = '%q' OR title = '%q'",
-            katze_item_get_name (item),
+            "DELETE FROM bookmarks WHERE title = '%q'",
             katze_item_get_name (item));
 
     if (sqlite3_exec (db, sqlcmd, NULL, NULL, &errmsg) != SQLITE_OK)