From b4b0c1e0424f3cf8730a9bd3f88ddd6b61ff5f95 Mon Sep 17 00:00:00 2001
From: Arun Prakash Jana <engineerarun@gmail.com>
Date: Sun, 20 Mar 2016 11:09:07 +0530
Subject: [PATCH] Compact the DB when removing a bookmark.

The record at the last index is moved to the removed index.

Signed-off-by: Arun Prakash Jana <engineerarun@gmail.com>
---
 buku | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/buku b/buku
index c60526e..542686c 100755
--- a/buku
+++ b/buku
@@ -341,6 +341,23 @@ def searchdb(cur, keywords):
 
 
 
+# Move last row to empty position to compact DB
+def compactDB(conn, cur, index):
+    cur.execute('SELECT MAX(id) from bookmarks')
+    results = cur.fetchall()
+    for row in results:
+        if row[0] > index:
+            cur.execute('SELECT id, URL, metadata, tags FROM bookmarks WHERE id = ?', (row[0],))
+            results = cur.fetchall()
+            for row in results:
+                cur.execute('DELETE FROM bookmarks WHERE id = ?', (row[0],))
+                conn.commit()
+                cur.execute('INSERT INTO bookmarks(id, URL, metadata, tags) VALUES (?, ?, ?, ?)', (index, row[1], row[2], row[3],))
+                conn.commit()
+                print("Index %d moved to %d" % (row[0], index))
+
+
+
 # Delete a single record or remove the table
 def cleardb(conn, cur, index):
     if index == None: # Remove the table
@@ -354,10 +371,11 @@ def cleardb(conn, cur, index):
         print("All bookmarks deleted")
     else: # Remove a single entry
         try:
-            cur.execute("DELETE FROM bookmarks WHERE id = ?", (int(index),))
+            cur.execute('DELETE FROM bookmarks WHERE id = ?', (int(index),))
             conn.commit()
             if cur.rowcount == 1:
                 print("Removed index %d" % int(index))
+                compactDB(conn, cur, int(index))
             else:
                 print("No matching index")
         except IndexError: