Presentation bug fixes

openlp/plugins/presentations/lib/impresscontroller.py

@@ -363,7 +363,7 @@ class ImpressDocument(PresentationDocument):
 
     def get_slide_preview_file(self, slide_no):
         """
-        Returns an image path containing a preview for theimpresscontroller.py
+        Returns an image path containing a preview for the
         requested slide
 
         ``slide_no``

openlp/plugins/presentations/lib/mediaitem.py

@@ -143,10 +143,9 @@ class PresentationMediaItem(MediaManagerItem):
             #not sure of this has errors
             #John please can you look at .
             for cidx in self.controllers:
-                if self.controllers[cidx].enabled:
-                    self.controllers[cidx].remove_doc(filepath)
-                    self.controllers[cidx].presentation_deleted()
-                    self.controllers[cidx].close_presentation()
+                doc = self.controllers[cidx].add_doc(filepath)
+                doc.presentation_deleted()
+                doc.close_presentation()
 
     def generateSlideData(self, service_item, item=None):
         items = self.ListView.selectedIndexes()

openlp/plugins/presentations/lib/presentationcontroller.py

@@ -115,7 +115,6 @@ class PresentationController(object):
         """
         return False
 
-
     def start_process(self):
         """
         Loads a running version of the presentation application in the background.
@@ -148,6 +147,9 @@ class PresentationController(object):
         if doc in self.docs:
            self.docs.remove(doc)
 
+    def close_presentation(self):
+        pass
+
 class PresentationDocument(object):
     """
     Base class for presentation documents to inherit from.
@@ -258,7 +260,7 @@ class PresentationDocument(object):
         Close presentation and clean up objects
         Triggered by new object being added to SlideController
         """
-        self.controller.delete_doc(self)
+        self.controller.close_presentation()
 
     def is_active(self):
         """