Make Phonon optional

Fixes: https://launchpad.net/bugs/1224404

openlp/core/ui/maindisplay.py

@@ -33,7 +33,13 @@ import cgi
 import logging
 
 from PyQt4 import QtCore, QtGui, QtWebKit, QtOpenGL
-from PyQt4.phonon import Phonon
+
+PHONON_AVAILABLE = True
+try:
+    from PyQt4.phonon import Phonon
+except ImportError:
+    PHONON_AVAILABLE = False
+PHONON_AVAILABLE = False
 
 from openlp.core.common import Registry, RegistryProperties, OpenLPMixin, Settings, translate, is_macosx
 from openlp.core.lib import ServiceItem, ImageSource, ScreenList, build_html, expand_tags, image_to_byte
@@ -139,7 +145,7 @@ class MainDisplay(OpenLPMixin, Display, RegistryProperties):
         self.override = {}
         self.retranslateUi()
         self.media_object = None
-        if self.is_live:
+        if self.is_live and PHONON_AVAILABLE:
             self.audio_player = AudioPlayer(self)
         else:
             self.audio_player = None

openlp/core/ui/slidecontroller.py

@@ -580,6 +580,7 @@ class SlideController(DisplayController, RegistryProperties):
         self.display.setup()
         if self.is_live:
             self.__add_actions_to_widget(self.display)
+        if self.display.audio_player:
             self.display.audio_player.connectSlot(QtCore.SIGNAL('tick(qint64)'), self.on_audio_time_remaining)
         # The SlidePreview's ratio.
         try:
@@ -834,26 +835,27 @@ class SlideController(DisplayController, RegistryProperties):
         self.slide_list = {}
         if self.is_live:
             self.song_menu.menu().clear()
-            self.display.audio_player.reset()
+            if self.display.audio_player:
-            self.set_audio_items_visibility(False)
+                self.display.audio_player.reset()
-            self.audio_pause_item.setChecked(False)
+                self.set_audio_items_visibility(False)
-            # If the current item has background audio
+                self.audio_pause_item.setChecked(False)
-            if self.service_item.is_capable(ItemCapabilities.HasBackgroundAudio):
+                # If the current item has background audio
-                self.log_debug('Starting to play...')
+                if self.service_item.is_capable(ItemCapabilities.HasBackgroundAudio):
-                self.display.audio_player.add_to_playlist(self.service_item.background_audio)
+                    self.log_debug('Starting to play...')
-                self.track_menu.clear()
+                    self.display.audio_player.add_to_playlist(self.service_item.background_audio)
-                for counter in range(len(self.service_item.background_audio)):
+                    self.track_menu.clear()
-                    action = self.track_menu.addAction(os.path.basename(self.service_item.background_audio[counter]))
+                    for counter in range(len(self.service_item.background_audio)):
-                    action.setData(counter)
+                        action = self.track_menu.addAction(os.path.basename(self.service_item.background_audio[counter]))
-                    action.triggered.connect(self.on_track_triggered)
+                        action.setData(counter)
-                self.display.audio_player.repeat = \
+                        action.triggered.connect(self.on_track_triggered)
-                    Settings().value(self.main_window.general_settings_section + '/audio repeat list')
+                    self.display.audio_player.repeat = \
-                if Settings().value(self.main_window.general_settings_section + '/audio start paused'):
+                        Settings().value(self.main_window.general_settings_section + '/audio repeat list')
-                    self.audio_pause_item.setChecked(True)
+                    if Settings().value(self.main_window.general_settings_section + '/audio start paused'):
-                    self.display.audio_player.pause()
+                        self.audio_pause_item.setChecked(True)
-                else:
+                        self.display.audio_player.pause()
-                    self.display.audio_player.play()
+                    else:
-                self.set_audio_items_visibility(True)
+                        self.display.audio_player.play()
+                    self.set_audio_items_visibility(True)
         row = 0
         width = self.main_window.control_splitter.sizes()[self.split]
         for frame_number, frame in enumerate(self.service_item.get_frames()):

tests/functional/openlp_core_ui/test_slidecontroller.py

@@ -479,6 +479,34 @@ class TestSlideController(TestCase):
         mocked_preview_widget.current_slide_number.assert_called_with()
         mocked_process_item.assert_called_once_with(mocked_item, 7)
 
+    def on_slide_blank_test(self):
+        """
+        Test on_slide_blank
+        """
+        # GIVEN: An instance of SlideController and a mocked on_blank_display
+        slide_controller = SlideController(None)
+        slide_controller.on_blank_display = MagicMock()
+
+        # WHEN: Calling on_slide_blank
+        slide_controller.on_slide_blank()
+
+        # THEN: on_blank_display should have been called with True
+        slide_controller.on_blank_display.assert_called_once_with(True)
+
+    def on_slide_unblank_test(self):
+        """
+        Test on_slide_unblank
+        """
+        # GIVEN: An instance of SlideController and a mocked on_blank_display
+        slide_controller = SlideController(None)
+        slide_controller.on_blank_display = MagicMock()
+
+        # WHEN: Calling on_slide_unblank
+        slide_controller.on_slide_unblank()
+
+        # THEN: on_blank_display should have been called with False
+        slide_controller.on_blank_display.assert_called_once_with(False)
+
     def on_slide_selected_index_no_service_item_test(self):
         """
         Test that when there is no service item, the on_slide_selected_index() method returns immediately