openlp-core team mailing list archive

[Tim Bentley <tim.bentley@xxxxxxxxx>]

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

Requested reviews:
  OpenLP Core (openlp-core)
Related bugs:
  Bug #1309886 in OpenLP: "Exception when clicking "Load new media" button"
  https://bugs.launchpad.net/openlp/+bug/1309886

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

Fix call path
Fix PEP8 errors missed by bug in ci command
-- 
https://code.launchpad.net/~trb143/openlp/bug-1309886/+merge/218198
Your team OpenLP Core is requested to review the proposed merge of lp:~trb143/openlp/bug-1309886 into lp:openlp.

=== modified file 'openlp/plugins/media/lib/mediaitem.py'
--- openlp/plugins/media/lib/mediaitem.py	2014-05-01 17:49:43 +0000
+++ openlp/plugins/media/lib/mediaitem.py	2014-05-03 20:01:24 +0000
@@ -229,7 +229,7 @@
         self.service_path = os.path.join(AppLocation.get_section_data_path(self.settings_section), 'thumbnails')
         check_directory_exists(self.service_path)
         self.load_list(Settings().value(self.settings_section + '/media files'))
-        self.populate_display_types()
+        self.rebuild_players()
 
     def rebuild_players(self):
         """

=== modified file 'tests/functional/openlp_core_lib/test_image_manager.py'
--- tests/functional/openlp_core_lib/test_image_manager.py	2014-04-15 20:30:20 +0000
+++ tests/functional/openlp_core_lib/test_image_manager.py	2014-05-03 20:01:24 +0000
@@ -171,4 +171,4 @@
         self.lock.release()
         # The sleep time is adjusted in the test case.
         time.sleep(self.sleep_time)
-        return ''
\ No newline at end of file
+        return ''

=== modified file 'tests/functional/openlp_core_ui/test_media.py'
--- tests/functional/openlp_core_ui/test_media.py	2014-04-20 22:23:26 +0000
+++ tests/functional/openlp_core_ui/test_media.py	2014-05-03 20:01:24 +0000
@@ -125,4 +125,4 @@
 
             # THEN: the used_players should be an empty list, and the overridden player should be an empty string
             self.assertEqual(['vlc', 'webkit', 'phonon'], used_players, 'Used players should be correct')
-            self.assertEqual('vlc,webkit,phonon', overridden_player, 'Overridden player should be a string of players')
\ No newline at end of file
+            self.assertEqual('vlc,webkit,phonon', overridden_player, 'Overridden player should be a string of players')

=== modified file 'tests/interfaces/openlp_core_common/test_historycombobox.py'
--- tests/interfaces/openlp_core_common/test_historycombobox.py	2014-05-01 16:57:45 +0000
+++ tests/interfaces/openlp_core_common/test_historycombobox.py	2014-05-03 20:01:24 +0000
@@ -62,4 +62,4 @@
         self.combo.addItem('test2')
 
         # THEN: The list of items should contain both strings.
-        self.assertEqual(self.combo.getItems(), ['test1', 'test2'])
\ No newline at end of file
+        self.assertEqual(self.combo.getItems(), ['test1', 'test2'])

=== modified file 'tests/interfaces/openlp_core_ui/test_shortcutlistform.py'
--- tests/interfaces/openlp_core_ui/test_shortcutlistform.py	2014-04-20 14:13:09 +0000
+++ tests/interfaces/openlp_core_ui/test_shortcutlistform.py	2014-05-03 20:01:24 +0000
@@ -75,4 +75,4 @@
             # THEN: The button should be changed.
             self.assertEqual(button.text(), text, "The text should match.")
             mocked_check_method.assert_called_once_with(True)
-            self.assertEqual(button.isEnabled(), enabled, "The button should be disabled.")
\ No newline at end of file
+            self.assertEqual(button.isEnabled(), enabled, "The button should be disabled.")