openlp-core team mailing list archive

[Tim Bentley <tim.bentley@xxxxxxxxx>]

Tim Bentley has proposed merging lp:~trb143/openlp/bug-820534 into lp:openlp.

Requested reviews:
  OpenLP Core (openlp-core)
Related bugs:
  Bug #820534 in OpenLP: "multiple custom slide delete causes traceback"
  https://bugs.launchpad.net/openlp/+bug/820534

For more details, see:
https://code.launchpad.net/~trb143/openlp/bug-820534/+merge/72948

Stop the traceback by re-searching for the list after a deletion.  This is the same as songs.
-- 
https://code.launchpad.net/~trb143/openlp/bug-820534/+merge/72948
Your team OpenLP Core is requested to review the proposed merge of lp:~trb143/openlp/bug-820534 into lp:openlp.

=== modified file 'openlp/plugins/custom/lib/mediaitem.py'
--- openlp/plugins/custom/lib/mediaitem.py	2011-06-29 06:53:15 +0000
+++ openlp/plugins/custom/lib/mediaitem.py	2011-08-25 20:00:27 +0000
@@ -217,8 +217,7 @@
                 for item in self.listView.selectedIndexes()]
             for id in id_list:
                 self.plugin.manager.delete_object(CustomSlide, id)
-            for row in row_list:
-                self.listView.takeItem(row)
+            self.onSearchTextButtonClick()
 
     def onFocus(self):
         self.searchTextEdit.setFocus()