Fix #8971 (books can be inadvertently deleted)

Q-Qaysaneah · Feb 14, 2011 · 1677268 · 1677268
1 parent 75c4b89
commit 1677268
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/calibre/gui2/actions/delete.py b/src/calibre/gui2/actions/delete.py
@@ -271,18 +271,18 @@ def delete_books(self, *args):
                         partial(self.library_ids_deleted, current_row=row))
         # Device view is visible.
         else:
-            if not confirm('<p>'+_('The selected books will be '
-                                   '<b>permanently deleted</b> '
-                                   'from your device. Are you sure?')
-                                +'</p>', 'device_delete_books', self.gui):
-                return
             if self.gui.stack.currentIndex() == 1:
                 view = self.gui.memory_view
             elif self.gui.stack.currentIndex() == 2:
                 view = self.gui.card_a_view
             else:
                 view = self.gui.card_b_view
             paths = view.model().paths(rows)
+            if not confirm('<p>'+_('The selected books will be '
+                                   '<b>permanently deleted</b> '
+                                   'from your device. Are you sure?')
+                                +'</p>', 'device_delete_books', self.gui):
+                return
             job = self.gui.remove_paths(paths)
             self.delete_memory[job] = (paths, view.model())
             view.model().mark_for_deletion(job, rows)