From 8ede2d466677e4bf5b32a26707aac9bca8090cdb Mon Sep 17 00:00:00 2001
From: Charles Haley <cbhaley@i.wont.say.com>
Date: Wed, 26 Feb 2014 12:36:46 +0100
Subject: [PATCH] Make the set of books to refresh into a local variable buy
 using it as an object instance instead of by assignment.

---
 src/calibre/gui2/device.py | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/src/calibre/gui2/device.py b/src/calibre/gui2/device.py
index f53e9f1d65..3f3f8daf9c 100644
--- a/src/calibre/gui2/device.py
+++ b/src/calibre/gui2/device.py
@@ -1751,9 +1751,7 @@ class DeviceMixin(object):  # {{{
             self.db_book_title_cache = db_book_title_cache
             self.db_book_uuid_cache = db_book_uuid_cache
 
-        # This is really a local variable, but it must be made into a class
-        # attribute so that the updateq function (below) can change it.
-        self.sbil_book_ids_to_refresh = set()
+        book_ids_to_refresh = set()
 
         def update_book(id_, book) :
             if not update_metadata:
@@ -1771,7 +1769,7 @@ class DeviceMixin(object):  # {{{
                 if self.device_manager.device is not None:
                     set_of_ids = self.device_manager.device.synchronize_with_db(db, id_, book)
                     if set_of_ids is not None:
-                        self.sbil_book_ids_to_refresh |= set_of_ids
+                        book_ids_to_refresh.update(set_of_ids)
                         return True
 
                 return (db.metadata_last_modified(id_, index_is_id=True) !=
@@ -1884,11 +1882,11 @@ class DeviceMixin(object):  # {{{
                                 FunctionDispatcher(self.metadata_synced), booklists,
                                 plugboards, add_as_step_to_job)
 
-            if self.sbil_book_ids_to_refresh:
+            if book_ids_to_refresh:
                 try:
                     prints('DeviceJob: set_books_in_library refreshing GUI for ',
-                           len(self.sbil_book_ids_to_refresh), 'books')
-                    self.library_view.model().refresh_ids(self.sbil_book_ids_to_refresh,
+                           len(book_ids_to_refresh), 'books')
+                    self.library_view.model().refresh_ids(book_ids_to_refresh,
                                       current_row=self.library_view.currentIndex().row())
                 except:
                     # This shouldn't ever happen, but just in case ...