diff --git a/ChangeLog b/ChangeLog
index a02aee76b..6c893df68 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2008-02-03 Gary Burton <gary.burton@zen.co.uk>
+	* src/gen/db/base.py: Functions using the pop function expect a return
+	value. Bug #1632
+
 2008-02-02 Raphael Ackermann <raphael.ackermann@gmail.com>
 	* src/ExportOptions.py: fix mnemonics for filter
 
diff --git a/src/gen/db/base.py b/src/gen/db/base.py
index 562c3a60b..95572776b 100644
--- a/src/gen/db/base.py
+++ b/src/gen/db/base.py
@@ -109,7 +109,7 @@ class GrampsDbBookmarks:
         self.bookmarks.remove(item)
 
     def pop(self, item):
-        self.bookmarks.pop(item)
+        return self.bookmarks.pop(item)
 
     def insert(self, pos, item):
         self.bookmarks.insert(pos, item)