From e19161e0399465627a27d068cdcd91606c61d81d Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Thu, 7 Mar 2013 16:05:17 +0100
Subject: [PATCH 01/39] use loop video attribute; fixed regression prevent
 media adding

---
 openlp/core/ui/media/mediacontroller.py |   2 +-
 openlp/core/ui/media/webkitplayer.py    | 130 +++++++-----------------
 2 files changed, 35 insertions(+), 97 deletions(-)

diff --git a/openlp/core/ui/media/mediacontroller.py b/openlp/core/ui/media/mediacontroller.py
index cc39cb5f8..c6df532e4 100644
--- a/openlp/core/ui/media/mediacontroller.py
+++ b/openlp/core/ui/media/mediacontroller.py
@@ -440,7 +440,7 @@ class MediaController(object):
         controller.media_info = MediaInfo()
         controller.media_info.volume = 0
         controller.media_info.file_info = QtCore.QFileInfo(service_item.get_frame_path())
-        display = controller._display
+        display = controller.preview_display
         if not self._check_file_type(controller, display, service_item):
             # Media could not be loaded correctly
             critical_error_message_box(translate('MediaPlugin.MediaItem', 'Unsupported File'),
diff --git a/openlp/core/ui/media/webkitplayer.py b/openlp/core/ui/media/webkitplayer.py
index 7262622bb..031e3bbd0 100644
--- a/openlp/core/ui/media/webkitplayer.py
+++ b/openlp/core/ui/media/webkitplayer.py
@@ -44,113 +44,57 @@ VIDEO_CSS = u"""
     z-index:3;
     background-color: %(bgcolor)s;
 }
-#video1 {
-    background-color: %(bgcolor)s;
-    z-index:4;
-}
-#video2 {
+#video {
     background-color: %(bgcolor)s;
     z-index:4;
 }
 """
 
 VIDEO_JS = u"""
-    var video_timer = null;
-    var current_video = '1';
+    function show_video(state, path, volume, loop, variable_value){
+        // Sometimes  video.currentTime stops slightly short of video.duration and video.ended is intermittent!
 
-    function show_video(state, path, volume, loop, varVal){
-        // Note, the preferred method for looping would be to use the
-        // video tag loop attribute.
-        // But QtWebKit doesn't support this. Neither does it support the
-        // onended event, hence the setInterval()
-        // In addition, setting the currentTime attribute to zero to restart
-        // the video raises an INDEX_SIZE_ERROR: DOM Exception 1
-        // To complicate it further, sometimes vid.currentTime stops
-        // slightly short of vid.duration and vid.ended is intermittent!
-        //
-        // Note, currently the background may go black between loops. Not
-        // desirable. Need to investigate using two <video>'s, and hiding/
-        // preloading one, and toggle between the two when looping.
-
-        if(current_video=='1'){
-            var vid = document.getElementById('video1');
-            var vid2 = document.getElementById('video2');
-        } else {
-            var vid = document.getElementById('video2');
-            var vid2 = document.getElementById('video1');
-        }
+        var video = document.getElementById('video');
         if(volume != null){
-            vid.volume = volume;
-            vid2.volume = volume;
+            video.volume = volume;
         }
         switch(state){
-            case 'init':
-                vid.src = 'file:///' + path;
-                vid2.src = 'file:///' + path;
-                if(loop == null) loop = false;
-                vid.looping = loop;
-                vid2.looping = loop;
-                vid.load();
-                break;
             case 'load':
-                vid2.style.visibility = 'hidden';
-                vid2.load();
+                video.src = 'file:///' + path;
+                if(loop == true) {
+                    video.loop = true;
+                }
+                video.load();
                 break;
             case 'play':
-                vid.play();
-                if(vid.looping){
-                    video_timer = setInterval(
-                        function() {
-                            show_video('poll');
-                        }, 200);
-                }
+                video.play();
                 break;
             case 'pause':
-                if(video_timer!=null){
-                    clearInterval(video_timer);
-                    video_timer = null;
-                }
-                vid.pause();
+                video.pause();
                 break;
             case 'stop':
                 show_video('pause');
-                vid.currentTime = 0;
-                break;
-            case 'poll':
-                if(vid.ended||vid.currentTime+0.2>vid.duration)
-                    show_video('swap');
-                break;
-            case 'swap':
-                show_video('pause');
-                if(current_video=='1')
-                    current_video = '2';
-                else
-                    current_video = '1';
-                show_video('load');
-                show_video('play');
-                show_video('setVisible',null,null,null,'visible');
+                video.currentTime = 0;
                 break;
             case 'close':
                 show_video('stop');
-                vid.src = '';
-                vid2.src = '';
+                video.src = '';
                 break;
             case 'length':
-                return vid.duration;
+                return video.duration;
             case 'currentTime':
-                return vid.currentTime;
+                return video.currentTime;
             case 'seek':
-                // doesnt work currently
-                vid.currentTime = varVal;
+                video.currentTime = variable_value;
                 break;
             case 'isEnded':
-                return vid.ended;
+                return video.ended;
             case 'setVisible':
-                vid.style.visibility = varVal;
+                video.style.visibility = variable_value;
                 break;
             case 'setBackBoard':
                 var back = document.getElementById('videobackboard');
-                back.style.visibility = varVal;
+                back.style.visibility = variable_value;
                 break;
        }
     }
@@ -158,10 +102,7 @@ VIDEO_JS = u"""
 
 VIDEO_HTML = u"""
 <div id="videobackboard" class="size" style="visibility:hidden"></div>
-<video id="video1" class="size" style="visibility:hidden" autobuffer preload>
-</video>
-<video id="video2" class="size" style="visibility:hidden" autobuffer preload>
-</video>
+<video id="video" class="size" style="visibility:hidden" autobuffer preload></video>
 """
 
 FLASH_CSS = u"""
@@ -173,25 +114,21 @@ FLASH_CSS = u"""
 FLASH_JS = u"""
     function getFlashMovieObject(movieName)
     {
-        if (window.document[movieName])
-        {
+        if (window.document[movieName]){
             return window.document[movieName];
         }
-        if (document.embeds && document.embeds[movieName])
+        if (document.embeds && document.embeds[movieName]){
             return document.embeds[movieName];
+        }
     }
 
-    function show_flash(state, path, volume, varVal){
+    function show_flash(state, path, volume, variable_value){
         var text = document.getElementById('flash');
         var flashMovie = getFlashMovieObject("OpenLPFlashMovie");
         var src = "src = 'file:///" + path + "'";
-        var view_parm = " wmode='opaque'" +
-            " width='100%%'" +
-            " height='100%%'";
-        var swf_parm = " name='OpenLPFlashMovie'" +
-            " autostart='true' loop='false' play='true'" +
-            " hidden='false' swliveconnect='true' allowscriptaccess='always'" +
-            " volume='" + volume + "'";
+        var view_parm = " wmode='opaque'" + " width='100%%'" + " height='100%%'";
+        var swf_parm = " name='OpenLPFlashMovie'" + " autostart='true' loop='false' play='true'" +
+            " hidden='false' swliveconnect='true' allowscriptaccess='always'" + " volume='" + volume + "'";
 
         switch(state){
             case 'load':
@@ -220,12 +157,13 @@ FLASH_JS = u"""
             case 'currentTime':
                 return flashMovie.CurrentFrame();
             case 'seek':
-//                flashMovie.GotoFrame(varVal);
+//                flashMovie.GotoFrame(variable_value);
                 break;
             case 'isEnded':
-                return false;//TODO check flash end
+                //TODO check flash end
+                return false;
             case 'setVisible':
-                text.style.visibility = varVal;
+                text.style.visibility = variable_value;
                 break;
         }
     }
@@ -338,7 +276,7 @@ class WebkitPlayer(MediaPlayer):
             controller.media_info.is_flash = True
             js = u'show_flash("load","%s");' % (path.replace(u'\\', u'\\\\'))
         else:
-            js = u'show_video("init", "%s", %s, %s);' % (path.replace(u'\\', u'\\\\'), str(vol), loop)
+            js = u'show_video("load", "%s", %s, %s);' % (path.replace(u'\\', u'\\\\'), str(vol), loop)
         display.frame.evaluateJavaScript(js)
         return True
 
@@ -450,7 +388,7 @@ class WebkitPlayer(MediaPlayer):
             currentTime = display.frame.evaluateJavaScript(u'show_flash("currentTime");')
             length = display.frame.evaluateJavaScript(u'show_flash("length");')
         else:
-            if display.frame.evaluateJavaScript(u'show_video("isEnded");') == 'true':
+            if display.frame.evaluateJavaScript(u'show_video("isEnded");'):
                 self.stop(display)
             currentTime = display.frame.evaluateJavaScript(u'show_video("currentTime");')
             # check if conversion was ok and value is not 'NaN'

From 5acac7976ce0abdfb1a2e04b5cc3ac94ceec1545 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Sat, 9 Mar 2013 15:46:57 +0100
Subject: [PATCH 02/39] reduced overlap time; renamed variable

---
 openlp/core/lib/htmlbuilder.py       |  4 ++--
 openlp/core/ui/media/webkitplayer.py | 16 ++++++++--------
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/openlp/core/lib/htmlbuilder.py b/openlp/core/lib/htmlbuilder.py
index 35b7bd9e2..6496aeef1 100644
--- a/openlp/core/lib/htmlbuilder.py
+++ b/openlp/core/lib/htmlbuilder.py
@@ -164,8 +164,8 @@ sup {
             text.innerHTML = new_text;
             return;
         }
-        // Fade text out. 0.2 to minimize the time "nothing" is shown on the screen.
-        text.style.opacity = '0.2';
+        // Fade text out. 0.1 to minimize the time "nothing" is shown on the screen.
+        text.style.opacity = '0.1';
         // Fade new text in after the old text has finished fading out.
         timer = window.setTimeout(function(){_show_text(text, new_text)}, 400);
     }
diff --git a/openlp/core/ui/media/webkitplayer.py b/openlp/core/ui/media/webkitplayer.py
index 031e3bbd0..f4be20450 100644
--- a/openlp/core/ui/media/webkitplayer.py
+++ b/openlp/core/ui/media/webkitplayer.py
@@ -82,7 +82,7 @@ VIDEO_JS = u"""
                 break;
             case 'length':
                 return video.duration;
-            case 'currentTime':
+            case 'current_time':
                 return video.currentTime;
             case 'seek':
                 video.currentTime = variable_value;
@@ -154,7 +154,7 @@ FLASH_JS = u"""
                 break;
             case 'length':
                 return flashMovie.TotalFrames();
-            case 'currentTime':
+            case 'current_time':
                 return flashMovie.CurrentFrame();
             case 'seek':
 //                flashMovie.GotoFrame(variable_value);
@@ -385,25 +385,25 @@ class WebkitPlayer(MediaPlayer):
         """
         controller = display.controller
         if controller.media_info.is_flash:
-            currentTime = display.frame.evaluateJavaScript(u'show_flash("currentTime");')
+            current_time = display.frame.evaluateJavaScript(u'show_flash("current_time");')
             length = display.frame.evaluateJavaScript(u'show_flash("length");')
         else:
             if display.frame.evaluateJavaScript(u'show_video("isEnded");'):
                 self.stop(display)
-            currentTime = display.frame.evaluateJavaScript(u'show_video("currentTime");')
+            current_time = display.frame.evaluateJavaScript(u'show_video("current_time");')
             # check if conversion was ok and value is not 'NaN'
-            if currentTime and currentTime != float('inf'):
-                currentTime = int(currentTime * 1000)
+            if current_time and current_time != float('inf'):
+                current_time = int(current_time * 1000)
             length = display.frame.evaluateJavaScript(u'show_video("length");')
             # check if conversion was ok and value is not 'NaN'
             if length and length != float('inf'):
                 length = int(length * 1000)
-        if currentTime > 0:
+        if current_time > 0:
             controller.media_info.length = length
             controller.seekSlider.setMaximum(length)
             if not controller.seekSlider.isSliderDown():
                 controller.seekSlider.blockSignals(True)
-                controller.seekSlider.setSliderPosition(currentTime)
+                controller.seekSlider.setSliderPosition(current_time)
                 controller.seekSlider.blockSignals(False)
 
     def get_info(self):

From b71d3e9e5378025126870ce8afd60c0a75fb7c0c Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Thu, 18 Apr 2013 19:50:39 +0200
Subject: [PATCH 03/39] fixed bug-1170435

Fixes: https://launchpad.net/bugs/1170435
---
 openlp/plugins/songs/forms/editsongform.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openlp/plugins/songs/forms/editsongform.py b/openlp/plugins/songs/forms/editsongform.py
index 49a20762a..beddee475 100644
--- a/openlp/plugins/songs/forms/editsongform.py
+++ b/openlp/plugins/songs/forms/editsongform.py
@@ -697,7 +697,7 @@ class EditSongForm(QtGui.QDialog, Ui_EditSongDialog):
         for verse in verses:
             if not verse in order:
                 verses_not_used.append(verse)
-        self.warning_label.setVisible(len(verses_not_used) > 0)
+        self.warning_label.setVisible(len(verses_not_used) > 0 and bool(text))
 
     def on_copyright_insert_button_triggered(self):
         text = self.copyright_edit.text()

From e7f553afb1fbe9c64b8dd71e34f9043e1c9ed4fa Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Thu, 18 Apr 2013 20:27:22 +0200
Subject: [PATCH 04/39] added test

---
 openlp/plugins/songs/forms/editsongform.py    |  1 +
 .../songs/forms/test_editsongform.py          | 59 ++++++++++++++++++-
 2 files changed, 57 insertions(+), 3 deletions(-)

diff --git a/openlp/plugins/songs/forms/editsongform.py b/openlp/plugins/songs/forms/editsongform.py
index beddee475..818ed1b87 100644
--- a/openlp/plugins/songs/forms/editsongform.py
+++ b/openlp/plugins/songs/forms/editsongform.py
@@ -697,6 +697,7 @@ class EditSongForm(QtGui.QDialog, Ui_EditSongDialog):
         for verse in verses:
             if not verse in order:
                 verses_not_used.append(verse)
+        print len(verses_not_used)
         self.warning_label.setVisible(len(verses_not_used) > 0 and bool(text))
 
     def on_copyright_insert_button_triggered(self):
diff --git a/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py b/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
index 2c5de6535..9c74c38c0 100644
--- a/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
+++ b/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
@@ -34,7 +34,7 @@ class TestEditSongForm(TestCase):
         del self.main_window
         del self.app
 
-    def ui_defaults_test(self):
+    def ui_defaults_(self):
         """
         Test that the EditSongForm defaults are correct
         """
@@ -43,5 +43,58 @@ class TestEditSongForm(TestCase):
         self.assertFalse(self.form.author_remove_button.isEnabled(), u'The author remove button should not be enabled')
         self.assertFalse(self.form.topic_remove_button.isEnabled(), u'The topic remove button should not be enabled')
 
-    def is_verse_edit_form_executed_test(self):
-        pass
\ No newline at end of file
+    def is_verse_edit_form_executed_t(self):
+        pass
+
+    def verse_order_warning_hidden_(self):
+        """
+        Test if the verse order warning lable is visible, when a verse order is specified
+        """
+        # GIVEN: Mocked methods.
+        mocked_row_count = MagicMock()
+        mocked_row_count.return_value = 1
+        self.form.verse_list_widget.rowCount = mocked_row_count
+        # Mock out the verse.
+        mocked_verse = MagicMock()
+        self.form.verse_list_widget.item = mocked_verse
+        mocked_verse_data_method = MagicMock()
+        mocked_verse_data_method.return_value = u'V1'
+        mocked_verse.data = mocked_verse_data_method
+        mocked_item_method = MagicMock()
+        mocked_item_method.return_value = mocked_verse
+        mocked_extract_verse_order_method = MagicMock()
+        mocked_extract_verse_order_method.return_value = [u'V1']
+        self.form._extract_verse_order = mocked_extract_verse_order_method
+
+        # WHEN: Call the method.
+        self.form.on_verse_order_text_changed(u'V1')
+
+        # THEN: The warning lable should be hidden.
+        assert not self.form.warning_label.isVisible(), u'The warning lable should be hidden.'
+
+    def bug_1170435_no_text_test(self):
+        """
+        Regression test for bug 1170435 (test if lable hidden, when no verse order is specified)
+        """
+        # GIVEN: Mocked methods.
+        mocked_row_count = MagicMock()
+        mocked_row_count.return_value = 0
+        self.form.verse_list_widget.rowCount = mocked_row_count
+        # Mock out the verse. (We want a verse type to be returned).
+        mocked_verse = MagicMock()
+        self.form.verse_list_widget.item = mocked_verse
+        mocked_verse_data_method = MagicMock()
+        mocked_verse_data_method.return_value = u'V1'
+        mocked_verse.data = mocked_verse_data_method
+        mocked_item_method = MagicMock()
+        mocked_item_method.return_value = mocked_verse
+        mocked_extract_verse_order_method = MagicMock()
+        mocked_extract_verse_order_method.return_value = []
+        self.form._extract_verse_order = mocked_extract_verse_order_method
+
+        # WHEN: Call the method.
+        self.form.on_verse_order_text_changed(u'')
+
+        # THEN: The warning lable should be hidden.
+        assert  not self.form.warning_label.isVisible(), \
+            u'The lable should be visible because the verse order was left empty.'

From 31aaafa254c5c14f14adcbe035423a0cd6d75ae6 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Thu, 18 Apr 2013 20:27:54 +0200
Subject: [PATCH 05/39] clean ups

---
 openlp/plugins/songs/forms/editsongform.py                      | 1 -
 .../interfaces/openlp_plugins/songs/forms/test_editsongform.py  | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/openlp/plugins/songs/forms/editsongform.py b/openlp/plugins/songs/forms/editsongform.py
index 818ed1b87..beddee475 100644
--- a/openlp/plugins/songs/forms/editsongform.py
+++ b/openlp/plugins/songs/forms/editsongform.py
@@ -697,7 +697,6 @@ class EditSongForm(QtGui.QDialog, Ui_EditSongDialog):
         for verse in verses:
             if not verse in order:
                 verses_not_used.append(verse)
-        print len(verses_not_used)
         self.warning_label.setVisible(len(verses_not_used) > 0 and bool(text))
 
     def on_copyright_insert_button_triggered(self):
diff --git a/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py b/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
index 9c74c38c0..829bc1fbc 100644
--- a/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
+++ b/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
@@ -46,7 +46,7 @@ class TestEditSongForm(TestCase):
     def is_verse_edit_form_executed_t(self):
         pass
 
-    def verse_order_warning_hidden_(self):
+    def verse_order_warning_hidden_test(self):
         """
         Test if the verse order warning lable is visible, when a verse order is specified
         """

From c0d67ab45f7c14bbfffe2318dc4e19b11254d56d Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Thu, 18 Apr 2013 20:28:11 +0200
Subject: [PATCH 06/39] fixed method name

---
 .../interfaces/openlp_plugins/songs/forms/test_editsongform.py  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py b/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
index 829bc1fbc..6c90fa0c0 100644
--- a/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
+++ b/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
@@ -72,7 +72,7 @@ class TestEditSongForm(TestCase):
         # THEN: The warning lable should be hidden.
         assert not self.form.warning_label.isVisible(), u'The warning lable should be hidden.'
 
-    def bug_1170435_no_text_test(self):
+    def bug_1170435_test(self):
         """
         Regression test for bug 1170435 (test if lable hidden, when no verse order is specified)
         """

From 6f0ea093279d283b4da0b5aef05fb490988d0af1 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Fri, 14 Jun 2013 11:56:53 +0200
Subject: [PATCH 07/39] increased possible min/max line spacing

Fixes: https://launchpad.net/bugs/645452
---
 openlp/core/ui/themewizard.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/openlp/core/ui/themewizard.py b/openlp/core/ui/themewizard.py
index 7fc78a59e..7c9f7df15 100644
--- a/openlp/core/ui/themewizard.py
+++ b/openlp/core/ui/themewizard.py
@@ -178,8 +178,8 @@ class Ui_ThemeWizard(object):
         self.lineSpacingLabel = QtGui.QLabel(self.mainAreaPage)
         self.lineSpacingLabel.setObjectName(u'LineSpacingLabel')
         self.lineSpacingSpinBox = QtGui.QSpinBox(self.mainAreaPage)
-        self.lineSpacingSpinBox.setMinimum(-50)
-        self.lineSpacingSpinBox.setMaximum(50)
+        self.lineSpacingSpinBox.setMinimum(-250)
+        self.lineSpacingSpinBox.setMaximum(250)
         self.lineSpacingSpinBox.setObjectName(u'LineSpacingSpinBox')
         self.mainAreaLayout.addRow(self.lineSpacingLabel, self.lineSpacingSpinBox)
         self.outlineCheckBox = QtGui.QCheckBox(self.mainAreaPage)

From d253ef5852215873e717ec48ab49821c4f711e2a Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Sat, 15 Jun 2013 21:51:11 +0100
Subject: [PATCH 08/39] Fix bug and add new test or bibles

---
 openlp/core/ui/slidecontroller.py             | 18 +++++-----
 .../openlp_plugins/bibles/__init__.py         |  1 +
 .../openlp_plugins/bibles/test_lib_http.py    | 33 +++++++++++++++++++
 3 files changed, 42 insertions(+), 10 deletions(-)
 create mode 100644 tests/functional/openlp_plugins/bibles/__init__.py
 create mode 100644 tests/functional/openlp_plugins/bibles/test_lib_http.py

diff --git a/openlp/core/ui/slidecontroller.py b/openlp/core/ui/slidecontroller.py
index 5cec2659c..2266026d6 100644
--- a/openlp/core/ui/slidecontroller.py
+++ b/openlp/core/ui/slidecontroller.py
@@ -591,7 +591,8 @@ class SlideController(DisplayController):
         """
         request = self.sender().text()
         slide_no = self.slideList[request]
-        self.__updatePreviewSelection(slide_no)
+        width = self.main_window.controlSplitter.sizes()[self.split]
+        self.preview_widget.replace_service_item(self.service_item, width, slide_no)
         self.slideSelected()
 
     def receive_spin_delay(self):
@@ -757,10 +758,8 @@ class SlideController(DisplayController):
                     self.display.audio_player.play()
                 self.setAudioItemsVisibility(True)
         row = 0
-        text = []
         width = self.main_window.controlSplitter.sizes()[self.split]
         for framenumber, frame in enumerate(self.service_item.get_frames()):
-            slideHeight = 0
             if self.service_item.is_text():
                 if frame[u'verseTag']:
                     # These tags are already translated.
@@ -776,7 +775,6 @@ class SlideController(DisplayController):
                     row += 1
                     self.slideList[unicode(row)] = row - 1
             else:
-                slideHeight = width * (1 / self.ratio)
                 row += 1
                 self.slideList[unicode(row)] = row - 1
                 # If current slide set background to image
@@ -867,7 +865,7 @@ class SlideController(DisplayController):
             Settings().remove(self.main_window.general_settings_section + u'/screen blank')
         self.blankPlugin()
         self.updatePreview()
-        self.onToggleLoop()
+        self.on_toggle_loop()
 
     def onThemeDisplay(self, checked=None):
         """
@@ -886,7 +884,7 @@ class SlideController(DisplayController):
             Settings().remove(self.main_window.general_settings_section + u'/screen blank')
         self.blankPlugin()
         self.updatePreview()
-        self.onToggleLoop()
+        self.on_toggle_loop()
 
     def onHideDisplay(self, checked=None):
         """
@@ -905,7 +903,7 @@ class SlideController(DisplayController):
             Settings().remove(self.main_window.general_settings_section + u'/screen blank')
         self.hidePlugin(checked)
         self.updatePreview()
-        self.onToggleLoop()
+        self.on_toggle_loop()
 
     def blankPlugin(self):
         """
@@ -1073,7 +1071,7 @@ class SlideController(DisplayController):
             self.preview_widget.change_slide(row)
             self.slideSelected()
 
-    def onToggleLoop(self):
+    def on_toggle_loop(self):
         """
         Toggles the loop state.
         """
@@ -1117,7 +1115,7 @@ class SlideController(DisplayController):
         else:
             self.play_slides_loop.setIcon(build_icon(u':/media/media_time.png'))
             self.play_slides_loop.setText(UiStrings().PlaySlidesInLoop)
-        self.onToggleLoop()
+        self.on_toggle_loop()
 
     def onPlaySlidesOnce(self, checked=None):
         """
@@ -1138,7 +1136,7 @@ class SlideController(DisplayController):
         else:
             self.play_slides_once.setIcon(build_icon(u':/media/media_time'))
             self.play_slides_once.setText(UiStrings().PlaySlidesToEnd)
-        self.onToggleLoop()
+        self.on_toggle_loop()
 
     def setAudioItemsVisibility(self, visible):
         """
diff --git a/tests/functional/openlp_plugins/bibles/__init__.py b/tests/functional/openlp_plugins/bibles/__init__.py
new file mode 100644
index 000000000..f87606f07
--- /dev/null
+++ b/tests/functional/openlp_plugins/bibles/__init__.py
@@ -0,0 +1 @@
+__author__ = 'tim'
diff --git a/tests/functional/openlp_plugins/bibles/test_lib_http.py b/tests/functional/openlp_plugins/bibles/test_lib_http.py
new file mode 100644
index 000000000..b9982b42e
--- /dev/null
+++ b/tests/functional/openlp_plugins/bibles/test_lib_http.py
@@ -0,0 +1,33 @@
+"""
+    Package to test the openlp.plugin.bible.lib.https package.
+"""
+
+from unittest import TestCase
+from mock import MagicMock, patch
+
+from openlp.core.lib import Registry
+from openlp.plugins.bibles.lib.http import BGExtract
+
+
+class TestBibleHTTP(TestCase):
+
+    def setUp(self):
+        """
+        Set up the Registry
+        """
+        Registry.create()
+        Registry().register(u'service_list', MagicMock())
+        Registry().register(u'application', MagicMock())
+
+    def bible_gateway_extract_test(self):
+        """
+        Test the Bible Gateway retrieval of book list for NIV
+        """
+        # GIVEN: A new Bible Gateway extraction class
+        handler = BGExtract()
+
+        # WHEN: The Books list is called
+        books = handler.get_books_from_http(u'NIV')
+
+        # THEN: We should get back a valid service item
+        assert len(books) == 66, u'The bible should not have had its lenght changed'

From e8d5ad773628b24c0ba9a76c35b7ba9b91de9b4e Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Sun, 16 Jun 2013 08:54:16 +0100
Subject: [PATCH 09/39] Clean up slidecontroller

---
 openlp/core/ui/maindisplay.py                 |   2 +-
 openlp/core/ui/mainwindow.py                  |   2 +-
 openlp/core/ui/servicemanager.py              |  12 +-
 openlp/core/ui/slidecontroller.py             | 250 +++++++++---------
 .../openlp_plugins/bibles/test_lib_http.py    |  49 +++-
 5 files changed, 177 insertions(+), 138 deletions(-)

diff --git a/openlp/core/ui/maindisplay.py b/openlp/core/ui/maindisplay.py
index 02ae469d6..7069cb9b7 100644
--- a/openlp/core/ui/maindisplay.py
+++ b/openlp/core/ui/maindisplay.py
@@ -288,7 +288,7 @@ class MainDisplay(Display):
         self.image(path)
         # Update the preview frame.
         if self.is_live:
-            self.live_controller.updatePreview()
+            self.live_controller.update_preview()
         return True
 
     def image(self, path):
diff --git a/openlp/core/ui/mainwindow.py b/openlp/core/ui/mainwindow.py
index 6b86bfe7a..4607c441f 100644
--- a/openlp/core/ui/mainwindow.py
+++ b/openlp/core/ui/mainwindow.py
@@ -669,7 +669,7 @@ class MainWindow(QtGui.QMainWindow, Ui_MainWindow):
         Check and display message if screen blank on setup.
         """
         settings = Settings()
-        self.live_controller.mainDisplaySetBackground()
+        self.live_controller.main_display_set_background()
         if settings.value(u'%s/screen blank' % self.general_settings_section):
             if settings.value(u'%s/blank warning' % self.general_settings_section):
                 QtGui.QMessageBox.question(self, translate('OpenLP.MainWindow', 'OpenLP Main Display Blanked'),
diff --git a/openlp/core/ui/servicemanager.py b/openlp/core/ui/servicemanager.py
index c16e52c23..444edc814 100644
--- a/openlp/core/ui/servicemanager.py
+++ b/openlp/core/ui/servicemanager.py
@@ -1267,7 +1267,7 @@ class ServiceManager(QtGui.QWidget, ServiceManagerDialog):
                 newItem.merge(item[u'service_item'])
                 item[u'service_item'] = newItem
                 self.repaint_service_list(item_count + 1, 0)
-                self.live_controller.replaceServiceManagerItem(newItem)
+                self.live_controller.replace_service_manager_item(newItem)
                 self.set_modified()
 
     def add_service_item(self, item, rebuild=False, expand=None, replace=False, repaint=True, selected=False):
@@ -1289,7 +1289,7 @@ class ServiceManager(QtGui.QWidget, ServiceManagerDialog):
             item.merge(self.service_items[sitem][u'service_item'])
             self.service_items[sitem][u'service_item'] = item
             self.repaint_service_list(sitem, child)
-            self.live_controller.replaceServiceManagerItem(item)
+            self.live_controller.replace_service_manager_item(item)
         else:
             item.render()
             # nothing selected for dnd
@@ -1312,7 +1312,7 @@ class ServiceManager(QtGui.QWidget, ServiceManagerDialog):
                 self.repaint_service_list(self.drop_position, -1)
             # if rebuilding list make sure live is fixed.
             if rebuild:
-                self.live_controller.replaceServiceManagerItem(item)
+                self.live_controller.replace_service_manager_item(item)
         self.drop_position = 0
         self.set_modified()
 
@@ -1323,7 +1323,7 @@ class ServiceManager(QtGui.QWidget, ServiceManagerDialog):
         self.application.set_busy_cursor()
         item, child = self.find_service_item()
         if self.service_items[item][u'service_item'].is_valid:
-            self.preview_controller.addServiceManagerItem(self.service_items[item][u'service_item'], child)
+            self.preview_controller.add_service_manager_item(self.service_items[item][u'service_item'], child)
         else:
             critical_error_message_box(translate('OpenLP.ServiceManager', 'Missing Display Handler'),
                 translate('OpenLP.ServiceManager',
@@ -1361,12 +1361,12 @@ class ServiceManager(QtGui.QWidget, ServiceManagerDialog):
             child = row
         self.application.set_busy_cursor()
         if self.service_items[item][u'service_item'].is_valid:
-            self.live_controller.addServiceManagerItem(self.service_items[item][u'service_item'], child)
+            self.live_controller.add_service_manager_item(self.service_items[item][u'service_item'], child)
             if Settings().value(self.main_window.general_settings_section + u'/auto preview'):
                 item += 1
                 if self.service_items and item < len(self.service_items) and \
                         self.service_items[item][u'service_item'].is_capable(ItemCapabilities.CanPreview):
-                    self.preview_controller.addServiceManagerItem(self.service_items[item][u'service_item'], 0)
+                    self.preview_controller.add_service_manager_item(self.service_items[item][u'service_item'], 0)
                     next_item = self.service_manager_list.topLevelItem(item)
                     self.service_manager_list.setCurrentItem(next_item)
                     self.live_controller.preview_widget.setFocus()
diff --git a/openlp/core/ui/slidecontroller.py b/openlp/core/ui/slidecontroller.py
index 2266026d6..c9bb8b02c 100644
--- a/openlp/core/ui/slidecontroller.py
+++ b/openlp/core/ui/slidecontroller.py
@@ -194,15 +194,15 @@ class SlideController(DisplayController):
             self.toolbar.add_toolbar_widget(self.hide_menu)
             self.blank_screen = create_action(self, u'blankScreen',
                 text=translate('OpenLP.SlideController', 'Blank Screen'), icon=u':/slides/slide_blank.png',
-                checked=False, can_shortcuts=True, category=self.category, triggers=self.onBlankDisplay)
+                checked=False, can_shortcuts=True, category=self.category, triggers=self.on_blank_display)
             self.theme_screen = create_action(self, u'themeScreen',
                 text=translate('OpenLP.SlideController', 'Blank to Theme'), icon=u':/slides/slide_theme.png',
                 checked=False, can_shortcuts=True, category=self.category,
-                triggers=self.onThemeDisplay)
+                triggers=self.on_theme_display)
             self.desktop_screen = create_action(self, u'desktopScreen',
                 text=translate('OpenLP.SlideController', 'Show Desktop'), icon=u':/slides/slide_desktop.png',
                 checked=False, can_shortcuts=True, category=self.category,
-                triggers=self.onHideDisplay)
+                triggers=self.on_hide_display)
             self.hide_menu.setDefaultAction(self.blank_screen)
             self.hide_menu.menu().addAction(self.blank_screen)
             self.hide_menu.menu().addAction(self.theme_screen)
@@ -230,10 +230,10 @@ class SlideController(DisplayController):
             self.toolbar.add_toolbar_widget(self.play_slides_menu)
             self.play_slides_loop = create_action(self, u'playSlidesLoop', text=UiStrings().PlaySlidesInLoop,
                 icon=u':/media/media_time.png', checked=False, can_shortcuts=True,
-                category=self.category, triggers=self.onPlaySlidesLoop)
+                category=self.category, triggers=self.on_play_slides_loop)
             self.play_slides_once = create_action(self, u'playSlidesOnce', text=UiStrings().PlaySlidesToEnd,
                 icon=u':/media/media_time.png', checked=False, can_shortcuts=True,
-                category=self.category, triggers=self.onPlaySlidesOnce)
+                category=self.category, triggers=self.on_play_slides_once)
             if Settings().value(self.main_window.advanced_settings_section + u'/slide limits') == SlideLimits.Wrap:
                 self.play_slides_menu.setDefaultAction(self.play_slides_loop)
             else:
@@ -249,12 +249,12 @@ class SlideController(DisplayController):
             self.toolbar.add_toolbar_widget(self.delay_spin_box)
         else:
             self.toolbar.add_toolbar_action(u'goLive', icon=u':/general/general_live.png',
-                tooltip=translate('OpenLP.SlideController', 'Move to live.'), triggers=self.onGoLive)
+                tooltip=translate('OpenLP.SlideController', 'Move to live.'), triggers=self.on_go_live)
             self.toolbar.add_toolbar_action(u'addToService', icon=u':/general/general_add.png',
-                tooltip=translate('OpenLP.SlideController', 'Add to Service.'), triggers=self.onPreviewAddToService)
+                tooltip=translate('OpenLP.SlideController', 'Add to Service.'), triggers=self.on_preview_add_to_service)
             self.toolbar.addSeparator()
             self.toolbar.add_toolbar_action(u'editSong', icon=u':/general/general_edit.png',
-                tooltip=translate('OpenLP.SlideController', 'Edit and reload song preview.'), triggers=self.onEditSong)
+                tooltip=translate('OpenLP.SlideController', 'Edit and reload song preview.'), triggers=self.on_edit_song)
         self.controller_layout.addWidget(self.toolbar)
         # Build the Media Toolbar
         self.media_controller.register_controller(self)
@@ -272,7 +272,7 @@ class SlideController(DisplayController):
                 icon=u':/slides/media_playback_pause.png', text=translate('OpenLP.SlideController', 'Pause Audio'),
                 tooltip=translate('OpenLP.SlideController', 'Pause audio.'),
                 checked=False, visible=False, category=self.category, context=QtCore.Qt.WindowShortcut,
-                can_shortcuts=True, triggers=self.onAudioPauseClicked)
+                can_shortcuts=True, triggers=self.set_audio_pause_clicked)
             self.audio_menu = QtGui.QMenu(translate('OpenLP.SlideController', 'Background Audio'), self.toolbar)
             self.audio_pause_item.setMenu(self.audio_menu)
             self.audio_pause_item.setParent(self.toolbar)
@@ -281,7 +281,7 @@ class SlideController(DisplayController):
             self.nextTrackItem = create_action(self, u'nextTrackItem', text=UiStrings().NextTrack,
                 icon=u':/slides/media_playback_next.png',
                 tooltip=translate('OpenLP.SlideController', 'Go to next audio track.'),
-                category=self.category, can_shortcuts=True, triggers=self.onNextTrackClicked)
+                category=self.category, can_shortcuts=True, triggers=self.on_next_track_clicked)
             self.audio_menu.addAction(self.nextTrackItem)
             self.trackMenu = self.audio_menu.addMenu(translate('OpenLP.SlideController', 'Tracks'))
             self.audio_time_label = QtGui.QLabel(u' 00:00 ', self.toolbar)
@@ -348,10 +348,10 @@ class SlideController(DisplayController):
                 can_shortcuts=True,
                 context=QtCore.Qt.WidgetWithChildrenShortcut,
                 category=self.category if s.get(u'configurable') else None,
-                triggers=self._slideShortcutActivated) for s in shortcuts])
-            self.shortcutTimer.timeout.connect(self._slideShortcutActivated)
+                triggers=self._slide_shortcut_activated) for s in shortcuts])
+            self.shortcutTimer.timeout.connect(self._slide_shortcut_activated)
         # Signals
-        self.preview_widget.clicked.connect(self.onSlideSelected)
+        self.preview_widget.clicked.connect(self.on_slide_selected)
         if self.is_live:
             # Need to use event as called across threads and UI is updated
             QtCore.QObject.connect(self, QtCore.SIGNAL(u'slidecontroller_toggle_display'), self.toggle_display)
@@ -359,11 +359,11 @@ class SlideController(DisplayController):
             self.toolbar.set_widget_visible(self.loop_list, False)
             self.toolbar.set_widget_visible(self.wide_menu, False)
         else:
-            self.preview_widget.doubleClicked.connect(self.onGoLiveClick)
+            self.preview_widget.doubleClicked.connect(self.on_preview_add_to_service)
             self.toolbar.set_widget_visible([u'editSong'], False)
         if self.is_live:
-            self.setLiveHotkeys(self)
-            self.__addActionsToWidget(self.controller)
+            self.set_live_hotkeys(self)
+            self.__add_actions_to_widget(self.controller)
         else:
             self.controller.addActions([self.nextItem, self.previous_item])
         Registry().register_function(u'slidecontroller_%s_stop_loop' % self.type_prefix, self.on_stop_loop)
@@ -378,7 +378,7 @@ class SlideController(DisplayController):
         QtCore.QObject.connect(self, QtCore.SIGNAL(u'slidecontroller_%s_previous' % self.type_prefix),
             self.on_slide_selected_previous)
 
-    def _slideShortcutActivated(self):
+    def _slide_shortcut_activated(self):
         """
         Called, when a shortcut has been activated to jump to a chorus, verse,
         etc.
@@ -425,7 +425,7 @@ class SlideController(DisplayController):
             self.shortcutTimer.stop()
             self.current_shortcut = u''
             self.preview_widget.change_slide(self.slideList[matches[0]])
-            self.slideSelected()
+            self.slide_selected()
         elif sender_name != u'shortcutTimer':
             # Start the time as we did not have any match.
             self.shortcutTimer.start(350)
@@ -435,28 +435,28 @@ class SlideController(DisplayController):
                 # We had more than one match for example "V1" and "V10", but
                 # "V1" was the slide we wanted to go.
                 self.preview_widget.change_slide(self.slideList[self.current_shortcut])
-                self.slideSelected()
+                self.slide_selected()
            # Reset the shortcut.
             self.current_shortcut = u''
 
-    def setLiveHotkeys(self, parent=None):
+    def set_live_hotkeys(self, parent=None):
         """
         Set the live hotkeys
         """
         self.previousService = create_action(parent, u'previousService',
             text=translate('OpenLP.SlideController', 'Previous Service'),
             can_shortcuts=True, context=QtCore.Qt.WidgetWithChildrenShortcut, category=self.category,
-            triggers=self.servicePrevious)
+            triggers=self.service_previous)
         self.nextService = create_action(parent, 'nextService',
             text=translate('OpenLP.SlideController', 'Next Service'),
             can_shortcuts=True, context=QtCore.Qt.WidgetWithChildrenShortcut, category=self.category,
-            triggers=self.serviceNext)
+            triggers=self.service_next)
         self.escapeItem = create_action(parent, 'escapeItem',
             text=translate('OpenLP.SlideController', 'Escape Item'),
             can_shortcuts=True, context=QtCore.Qt.WidgetWithChildrenShortcut, category=self.category,
-            triggers=self.liveEscape)
+            triggers=self.live_escape)
 
-    def liveEscape(self):
+    def live_escape(self):
         """
         If you press ESC on the live screen it should close the display temporarily.
         """
@@ -468,24 +468,24 @@ class SlideController(DisplayController):
         Toggle the display settings triggered from remote messages.
         """
         if action == u'blank' or action == u'hide':
-            self.onBlankDisplay(True)
+            self.on_blank_display(True)
         elif action == u'theme':
-            self.onThemeDisplay(True)
+            self.on_theme_display(True)
         elif action == u'desktop':
-            self.onHideDisplay(True)
+            self.on_hide_display(True)
         elif action == u'show':
-            self.onBlankDisplay(False)
-            self.onThemeDisplay(False)
-            self.onHideDisplay(False)
+            self.on_blank_display(False)
+            self.on_theme_display(False)
+            self.on_hide_display(False)
 
-    def servicePrevious(self):
+    def service_previous(self):
         """
         Live event to select the previous service item from the service manager.
         """
         self.keypress_queue.append(ServiceItemAction.Previous)
         self._process_queue()
 
-    def serviceNext(self):
+    def service_next(self):
         """
         Live event to select the next service item from the service manager.
         """
@@ -520,7 +520,7 @@ class SlideController(DisplayController):
         self.display = MainDisplay(self, self.is_live, self)
         self.display.setup()
         if self.is_live:
-            self.__addActionsToWidget(self.display)
+            self.__add_actions_to_widget(self.display)
             self.display.audio_player.connectSlot(QtCore.SIGNAL(u'tick(qint64)'), self.on_audio_time_remaining)
         # The SlidePreview's ratio.
         try:
@@ -538,7 +538,7 @@ class SlideController(DisplayController):
         if self.service_item:
             self.refresh_service_item()
 
-    def __addActionsToWidget(self, widget):
+    def __add_actions_to_widget(self, widget):
         """
         Add actions to the widget specified by `widget`
         """
@@ -567,9 +567,9 @@ class SlideController(DisplayController):
             self.preview_display.setFixedSize(QtCore.QSize(max_width, max_width / self.ratio))
             self.preview_display.screen = {
                 u'size': self.preview_display.geometry()}
-        self.onControllerSizeChanged(self.controller.width())
+        self.on_controller_size_changed(self.controller.width())
 
-    def onControllerSizeChanged(self, width):
+    def on_controller_size_changed(self, width):
         """
         Change layout of display control buttons on controller size change
         """
@@ -585,7 +585,7 @@ class SlideController(DisplayController):
                 self.toolbar.set_widget_visible(self.wide_menu, False)
                 self.toolbar.set_widget_visible(self.narrow_menu)
 
-    def onSongBarHandler(self):
+    def on_song_bar_handler(self):
         """
         Some song handler
         """
@@ -593,7 +593,7 @@ class SlideController(DisplayController):
         slide_no = self.slideList[request]
         width = self.main_window.controlSplitter.sizes()[self.split]
         self.preview_widget.replace_service_item(self.service_item, width, slide_no)
-        self.slideSelected()
+        self.slide_selected()
 
     def receive_spin_delay(self):
         """
@@ -681,49 +681,49 @@ class SlideController(DisplayController):
         """
         log.debug(u'add_service_item live = %s' % self.is_live)
         item.render()
-        slideno = 0
+        slide_no = 0
         if self.song_edit:
-            slideno = self.selected_row
+            slide_no = self.selected_row
         self.song_edit = False
-        self._process_item(item, slideno)
+        self._process_item(item, slide_no)
 
-    def replaceServiceManagerItem(self, item):
+    def replace_service_manager_item(self, item):
         """
         Replacement item following a remote edit
         """
         if item == self.service_item:
             self._process_item(item, self.preview_widget.current_slide_number())
 
-    def addServiceManagerItem(self, item, slideno):
+    def add_service_manager_item(self, item, slide_no):
         """
         Method to install the service item into the controller and
         request the correct toolbar for the plugin.
         Called by ServiceManager
         """
-        log.debug(u'addServiceManagerItem live = %s' % self.is_live)
-        # If no valid slide number is specified we take the first one, but we
-        # remember the initial value to see if we should reload the song or not
-        slidenum = slideno
-        if slideno == -1:
+        log.debug(u'add_service_manager_item live = %s' % self.is_live)
+        # If no valid slide number is specified we take the first one, but we remember the initial value to see if we
+        # should reload the song or not
+        slidenum = slide_no
+        if slide_no == -1:
             slidenum = 0
         # If service item is the same as the current one, only change slide
-        if slideno >= 0 and item == self.service_item:
+        if slide_no >= 0 and item == self.service_item:
             self.preview_widget.change_slide(slidenum)
-            self.slideSelected()
+            self.slide_selected()
         else:
             self._process_item(item, slidenum)
             if self.is_live and item.auto_play_slides_loop and item.timed_slide_interval > 0:
                 self.play_slides_loop.setChecked(item.auto_play_slides_loop)
                 self.delay_spin_box.setValue(int(item.timed_slide_interval))
-                self.onPlaySlidesLoop()
+                self.on_play_slides_loop()
             elif self.is_live and item.auto_play_slides_once and item.timed_slide_interval > 0:
                 self.play_slides_once.setChecked(item.auto_play_slides_once)
                 self.delay_spin_box.setValue(int(item.timed_slide_interval))
-                self.onPlaySlidesOnce()
+                self.on_play_slides_once()
 
     def _process_item(self, service_item, slideno):
         """
-        Loads a ServiceItem into the system from ServiceManager
+        Loads a ServiceItem into the system from ServiceManager 
         Display the slide number passed
         """
         log.debug(u'processManagerItem live = %s' % self.is_live)
@@ -732,13 +732,13 @@ class SlideController(DisplayController):
         # take a copy not a link to the servicemanager copy.
         self.service_item = copy.copy(service_item)
         if old_item and self.is_live and old_item.is_capable(ItemCapabilities.ProvidesOwnDisplay):
-            self._resetBlank()
+            self._reset_blank()
         Registry().execute(u'%s_start' % service_item.name.lower(), [service_item, self.is_live, self.hide_mode(), slideno])
         self.slideList = {}
         if self.is_live:
             self.song_menu.menu().clear()
             self.display.audio_player.reset()
-            self.setAudioItemsVisibility(False)
+            self.set_audio_items_visibility(False)
             self.audio_pause_item.setChecked(False)
             # If the current item has background audio
             if self.service_item.is_capable(ItemCapabilities.HasBackgroundAudio):
@@ -748,7 +748,7 @@ class SlideController(DisplayController):
                 for counter in range(len(self.service_item.background_audio)):
                     action = self.trackMenu.addAction(os.path.basename(self.service_item.background_audio[counter]))
                     action.setData(counter)
-                    action.triggered.connect(self.onTrackTriggered)
+                    action.triggered.connect(self.on_track_triggered)
                 self.display.audio_player.repeat = Settings().value(
                     self.main_window.general_settings_section + u'/audio repeat list')
                 if Settings().value(self.main_window.general_settings_section + u'/audio start paused'):
@@ -756,7 +756,7 @@ class SlideController(DisplayController):
                     self.display.audio_player.pause()
                 else:
                     self.display.audio_player.play()
-                self.setAudioItemsVisibility(True)
+                self.set_audio_items_visibility(True)
         row = 0
         width = self.main_window.controlSplitter.sizes()[self.split]
         for framenumber, frame in enumerate(self.service_item.get_frames()):
@@ -770,7 +770,7 @@ class SlideController(DisplayController):
                     if verse_def not in self.slideList:
                         self.slideList[verse_def] = framenumber
                         if self.is_live:
-                            self.song_menu.menu().addAction(verse_def, self.onSongBarHandler)
+                            self.song_menu.menu().addAction(verse_def, self.on_song_bar_handler)
                 else:
                     row += 1
                     self.slideList[unicode(row)] = row - 1
@@ -789,8 +789,8 @@ class SlideController(DisplayController):
         if not self.service_item.is_image():
             self.display.build_html(self.service_item)
         if service_item.is_media():
-            self.onMediaStart(service_item)
-        self.slideSelected(True)
+            self.on_media_start(service_item)
+        self.slide_selected(True)
         if old_item:
             # Close the old item after the new one is opened
             # This avoids the service theme/desktop flashing on screen
@@ -799,7 +799,7 @@ class SlideController(DisplayController):
             if old_item.is_command() and not service_item.is_command():
                 Registry().execute(u'%s_stop' % old_item.name.lower(), [old_item, self.is_live])
             if old_item.is_media() and not service_item.is_media():
-                self.onMediaClose()
+                self.on_media_close()
         Registry().execute(u'slidecontroller_%s_started' % self.type_prefix, [service_item])
 
     # Screen event methods
@@ -812,49 +812,49 @@ class SlideController(DisplayController):
             return
         if self.service_item.is_command():
             Registry().execute(u'%s_slide' % self.service_item.name.lower(), [self.service_item, self.is_live, index])
-            self.updatePreview()
+            self.update_preview()
         else:
             self.preview_widget.change_slide(index)
-            self.slideSelected()
+            self.slide_selected()
 
-    def mainDisplaySetBackground(self):
+    def main_display_set_background(self):
         """
         Allow the main display to blank the main display at startup time
         """
-        log.debug(u'mainDisplaySetBackground live = %s' % self.is_live)
+        log.debug(u'main_display_set_background live = %s' % self.is_live)
         display_type = Settings().value(self.main_window.general_settings_section + u'/screen blank')
         if self.screens.which_screen(self.window()) != self.screens.which_screen(self.display):
             # Order done to handle initial conversion
             if display_type == u'themed':
-                self.onThemeDisplay(True)
+                self.on_theme_display(True)
             elif display_type == u'hidden':
-                self.onHideDisplay(True)
+                self.on_hide_display(True)
             elif display_type == u'blanked':
-                self.onBlankDisplay(True)
+                self.on_blank_display(True)
             else:
                 Registry().execute(u'live_display_show')
         else:
-            self.liveEscape()
+            self.live_escape()
 
     def on_slide_blank(self):
         """
         Handle the slidecontroller blank event
         """
-        self.onBlankDisplay(True)
+        self.on_blank_display(True)
 
     def on_slide_unblank(self):
         """
         Handle the slidecontroller unblank event
         """
-        self.onBlankDisplay(False)
+        self.on_blank_display(False)
 
-    def onBlankDisplay(self, checked=None):
+    def on_blank_display(self, checked=None):
         """
         Handle the blank screen button actions
         """
         if checked is None:
             checked = self.blank_screen.isChecked()
-        log.debug(u'onBlankDisplay %s' % checked)
+        log.debug(u'on_blank_display %s' % checked)
         self.hide_menu.setDefaultAction(self.blank_screen)
         self.blank_screen.setChecked(checked)
         self.theme_screen.setChecked(False)
@@ -863,17 +863,17 @@ class SlideController(DisplayController):
             Settings().setValue(self.main_window.general_settings_section + u'/screen blank', u'blanked')
         else:
             Settings().remove(self.main_window.general_settings_section + u'/screen blank')
-        self.blankPlugin()
-        self.updatePreview()
+        self.blank_plugin()
+        self.update_preview()
         self.on_toggle_loop()
 
-    def onThemeDisplay(self, checked=None):
+    def on_theme_display(self, checked=None):
         """
         Handle the Theme screen button
         """
         if checked is None:
             checked = self.theme_screen.isChecked()
-        log.debug(u'onThemeDisplay %s' % checked)
+        log.debug(u'on_theme_display %s' % checked)
         self.hide_menu.setDefaultAction(self.theme_screen)
         self.blank_screen.setChecked(False)
         self.theme_screen.setChecked(checked)
@@ -882,17 +882,17 @@ class SlideController(DisplayController):
             Settings().setValue(self.main_window.general_settings_section + u'/screen blank', u'themed')
         else:
             Settings().remove(self.main_window.general_settings_section + u'/screen blank')
-        self.blankPlugin()
-        self.updatePreview()
+        self.blank_plugin()
+        self.update_preview()
         self.on_toggle_loop()
 
-    def onHideDisplay(self, checked=None):
+    def on_hide_display(self, checked=None):
         """
         Handle the Hide screen button
         """
         if checked is None:
             checked = self.desktop_screen.isChecked()
-        log.debug(u'onHideDisplay %s' % checked)
+        log.debug(u'on_hide_display %s' % checked)
         self.hide_menu.setDefaultAction(self.desktop_screen)
         self.blank_screen.setChecked(False)
         self.theme_screen.setChecked(False)
@@ -901,16 +901,16 @@ class SlideController(DisplayController):
             Settings().setValue(self.main_window.general_settings_section + u'/screen blank', u'hidden')
         else:
             Settings().remove(self.main_window.general_settings_section + u'/screen blank')
-        self.hidePlugin(checked)
-        self.updatePreview()
+        self.hide_plugin(checked)
+        self.update_preview()
         self.on_toggle_loop()
 
-    def blankPlugin(self):
+    def blank_plugin(self):
         """
         Blank/Hide the display screen within a plugin if required.
         """
         hide_mode = self.hide_mode()
-        log.debug(u'blankPlugin %s ', hide_mode)
+        log.debug(u'blank_plugin %s ', hide_mode)
         if self.service_item is not None:
             if hide_mode:
                 if not self.service_item.is_command():
@@ -927,11 +927,11 @@ class SlideController(DisplayController):
             else:
                 Registry().execute(u'live_display_show')
 
-    def hidePlugin(self, hide):
+    def hide_plugin(self, hide):
         """
         Tell the plugin to hide the display screen.
         """
-        log.debug(u'hidePlugin %s ', hide)
+        log.debug(u'hide_plugin %s ', hide)
         if self.service_item is not None:
             if hide:
                 Registry().execute(u'live_display_hide', HideMode.Screen)
@@ -946,13 +946,13 @@ class SlideController(DisplayController):
             else:
                 Registry().execute(u'live_display_show')
 
-    def onSlideSelected(self):
+    def on_slide_selected(self):
         """
         Slide selected in controller
         """
-        self.slideSelected()
+        self.slide_selected()
 
-    def slideSelected(self, start=False):
+    def slide_selected(self, start=False):
         """
         Generate the preview when you click on a slide.
         if this is the Live Controller also display on the screen
@@ -975,7 +975,7 @@ class SlideController(DisplayController):
                         self.display.image(to_display)
                     # reset the store used to display first image
                     self.service_item.bg_image_bytes = None
-            self.updatePreview()
+            self.update_preview()
             self.selected_row = row
             self.preview_widget.change_slide(row)
         Registry().execute(u'slidecontroller_%s_changed' % self.type_prefix, row)
@@ -986,14 +986,14 @@ class SlideController(DisplayController):
         The slide has been changed. Update the slidecontroller accordingly
         """
         self.preview_widget.change_slide(row)
-        self.updatePreview()
+        self.update_preview()
         Registry().execute(u'slidecontroller_%s_changed' % self.type_prefix, row)
 
-    def updatePreview(self):
+    def update_preview(self):
         """
         This updates the preview frame, for example after changing a slide or using *Blank to Theme*.
         """
-        log.debug(u'updatePreview %s ' % self.screens.current[u'primary'])
+        log.debug(u'update_preview %s ' % self.screens.current[u'primary'])
         if not self.screens.current[u'primary'] and self.service_item and \
                 self.service_item.is_capable(ItemCapabilities.ProvidesOwnDisplay):
             # Grab now, but try again in a couple of seconds if slide change is slow
@@ -1029,7 +1029,7 @@ class SlideController(DisplayController):
             return
         Registry().execute(u'%s_next' % self.service_item.name.lower(), [self.service_item, self.is_live])
         if self.service_item.is_command() and self.is_live:
-            self.updatePreview()
+            self.update_preview()
         else:
             row = self.preview_widget.current_slide_number() + 1
             if row == self.preview_widget.slide_count():
@@ -1037,7 +1037,7 @@ class SlideController(DisplayController):
                     if self.slide_limits == SlideLimits.Wrap:
                         row = 0
                     elif self.is_live and self.slide_limits == SlideLimits.Next:
-                        self.serviceNext()
+                        self.service_next()
                         return
                     else:
                         row = self.preview_widget.slide_count() - 1
@@ -1046,7 +1046,7 @@ class SlideController(DisplayController):
                 else:
                     row = self.preview_widget.slide_count() - 1
             self.preview_widget.change_slide(row)
-            self.slideSelected()
+            self.slide_selected()
 
     def on_slide_selected_previous(self):
         """
@@ -1056,7 +1056,7 @@ class SlideController(DisplayController):
             return
         Registry().execute(u'%s_previous' % self.service_item.name.lower(), [self.service_item, self.is_live])
         if self.service_item.is_command() and self.is_live:
-            self.updatePreview()
+            self.update_preview()
         else:
             row = self.preview_widget.current_slide_number() - 1
             if row == -1:
@@ -1069,7 +1069,7 @@ class SlideController(DisplayController):
                 else:
                     row = 0
             self.preview_widget.change_slide(row)
-            self.slideSelected()
+            self.slide_selected()
 
     def on_toggle_loop(self):
         """
@@ -1077,11 +1077,11 @@ class SlideController(DisplayController):
         """
         hide_mode = self.hide_mode()
         if hide_mode is None and (self.play_slides_loop.isChecked() or self.play_slides_once.isChecked()):
-            self.onStartLoop()
+            self.on_start_loop()
         else:
             self.on_stop_loop()
 
-    def onStartLoop(self):
+    def on_start_loop(self):
         """
         Start the timer loop running and store the timer id
         """
@@ -1096,7 +1096,7 @@ class SlideController(DisplayController):
             self.killTimer(self.timer_id)
             self.timer_id = 0
 
-    def onPlaySlidesLoop(self, checked=None):
+    def on_play_slides_loop(self, checked=None):
         """
         Start or stop 'Play Slides in Loop'
         """
@@ -1104,7 +1104,7 @@ class SlideController(DisplayController):
             checked = self.play_slides_loop.isChecked()
         else:
             self.play_slides_loop.setChecked(checked)
-        log.debug(u'onPlaySlidesLoop %s' % checked)
+        log.debug(u'on_play_slides_loop %s' % checked)
         if checked:
             self.play_slides_loop.setIcon(build_icon(u':/media/media_stop.png'))
             self.play_slides_loop.setText(UiStrings().StopPlaySlidesInLoop)
@@ -1117,7 +1117,7 @@ class SlideController(DisplayController):
             self.play_slides_loop.setText(UiStrings().PlaySlidesInLoop)
         self.on_toggle_loop()
 
-    def onPlaySlidesOnce(self, checked=None):
+    def on_play_slides_once(self, checked=None):
         """
         Start or stop 'Play Slides to End'
         """
@@ -1125,7 +1125,7 @@ class SlideController(DisplayController):
             checked = self.play_slides_once.isChecked()
         else:
             self.play_slides_once.setChecked(checked)
-        log.debug(u'onPlaySlidesOnce %s' % checked)
+        log.debug(u'on_play_slides_once %s' % checked)
         if checked:
             self.play_slides_once.setIcon(build_icon(u':/media/media_stop.png'))
             self.play_slides_once.setText(UiStrings().StopPlaySlidesToEnd)
@@ -1138,13 +1138,13 @@ class SlideController(DisplayController):
             self.play_slides_once.setText(UiStrings().PlaySlidesToEnd)
         self.on_toggle_loop()
 
-    def setAudioItemsVisibility(self, visible):
+    def set_audio_items_visibility(self, visible):
         """
         Set the visibility of the audio stuff
         """
         self.toolbar.set_widget_visible(self.audio_list, visible)
 
-    def onAudioPauseClicked(self, checked):
+    def set_audio_pause_clicked(self, checked):
         """
         Pause the audio player
         """
@@ -1162,7 +1162,7 @@ class SlideController(DisplayController):
         if event.timerId() == self.timer_id:
             self.on_slide_selected_next(self.play_slides_loop.isChecked())
 
-    def onEditSong(self):
+    def on_edit_song(self):
         """
         From the preview display requires the service Item to be editied
         """
@@ -1171,14 +1171,14 @@ class SlideController(DisplayController):
         if new_item:
             self.add_service_item(new_item)
 
-    def onPreviewAddToService(self):
+    def on_preview_add_to_service(self):
         """
         From the preview display request the Item to be added to service
         """
         if self.service_item:
             self.service_manager.add_service_item(self.service_item)
 
-    def onGoLiveClick(self):
+    def on_go_live_click(self):
         """
         triggered by clicking the Preview slide items
         """
@@ -1188,10 +1188,10 @@ class SlideController(DisplayController):
             if self.service_item.is_command():
                 Registry().execute(u'%s_stop' % self.service_item.name.lower(), [self.service_item, self.is_live])
             if self.service_item.is_media():
-                self.onMediaClose()
-            self.onGoLive()
+                self.on_media_close()
+            self.on_go_live()
 
-    def onGoLive(self):
+    def on_go_live(self):
         """
         If preview copy slide item to live controller from Preview Controller
         """
@@ -1200,41 +1200,41 @@ class SlideController(DisplayController):
             if self.service_item.from_service:
                 self.service_manager.preview_live(self.service_item.unique_identifier, row)
             else:
-                self.live_controller.addServiceManagerItem(self.service_item, row)
+                self.live_controller.add_service_manager_item(self.service_item, row)
 
-    def onMediaStart(self, item):
+    def on_media_start(self, item):
         """
         Respond to the arrival of a media service item
         """
-        log.debug(u'SlideController onMediaStart')
+        log.debug(u'SlideController on_media_start')
         self.media_controller.video(self.controller_type, item, self.hide_mode())
         if not self.is_live:
             self.preview_display.show()
             self.slide_preview.hide()
 
-    def onMediaClose(self):
+    def on_media_close(self):
         """
         Respond to a request to close the Video
         """
-        log.debug(u'SlideController onMediaClose')
+        log.debug(u'SlideController on_media_close')
         self.media_controller.media_reset(self)
         self.preview_display.hide()
         self.slide_preview.show()
 
-    def _resetBlank(self):
+    def _reset_blank(self):
         """
         Used by command items which provide their own displays to reset the
         screen hide attributes
         """
         hide_mode = self.hide_mode()
         if hide_mode == HideMode.Blank:
-            self.onBlankDisplay(True)
+            self.on_blank_display(True)
         elif hide_mode == HideMode.Theme:
-            self.onThemeDisplay(True)
+            self.on_theme_display(True)
         elif hide_mode == HideMode.Screen:
-            self.onHideDisplay(True)
+            self.on_hide_display(True)
         else:
-            self.hidePlugin(False)
+            self.hide_plugin(False)
 
     def hide_mode(self):
         """
@@ -1251,7 +1251,7 @@ class SlideController(DisplayController):
         else:
             return None
 
-    def onNextTrackClicked(self):
+    def on_next_track_clicked(self):
         """
         Go to the next track when next is clicked
         """
@@ -1266,7 +1266,7 @@ class SlideController(DisplayController):
         seconds %= 60
         self.audio_time_label.setText(u' %02d:%02d ' % (minutes, seconds))
 
-    def onTrackTriggered(self):
+    def on_track_triggered(self):
         """
         Start playing a track
         """
diff --git a/tests/functional/openlp_plugins/bibles/test_lib_http.py b/tests/functional/openlp_plugins/bibles/test_lib_http.py
index b9982b42e..3d2e849de 100644
--- a/tests/functional/openlp_plugins/bibles/test_lib_http.py
+++ b/tests/functional/openlp_plugins/bibles/test_lib_http.py
@@ -3,10 +3,10 @@
 """
 
 from unittest import TestCase
-from mock import MagicMock, patch
+from mock import MagicMock
 
 from openlp.core.lib import Registry
-from openlp.plugins.bibles.lib.http import BGExtract
+from openlp.plugins.bibles.lib.http import BGExtract, CWExtract
 
 
 class TestBibleHTTP(TestCase):
@@ -19,9 +19,9 @@ class TestBibleHTTP(TestCase):
         Registry().register(u'service_list', MagicMock())
         Registry().register(u'application', MagicMock())
 
-    def bible_gateway_extract_test(self):
+    def bible_gateway_extract_books_test(self):
         """
-        Test the Bible Gateway retrieval of book list for NIV
+        Test the Bible Gateway retrieval of book list for NIV bible
         """
         # GIVEN: A new Bible Gateway extraction class
         handler = BGExtract()
@@ -30,4 +30,43 @@ class TestBibleHTTP(TestCase):
         books = handler.get_books_from_http(u'NIV')
 
         # THEN: We should get back a valid service item
-        assert len(books) == 66, u'The bible should not have had its lenght changed'
+        assert len(books) == 66, u'The bible should not have had any books added or removed'
+
+    def bible_gateway_extract_verse_test(self):
+        """
+        Test the Bible Gateway retrieval of verse list for NIV bible John 3
+        """
+        # GIVEN: A new Bible Gateway extraction class
+        handler = BGExtract()
+
+        # WHEN: The Books list is called
+        results = handler.get_bible_chapter(u'NIV', u'John', 3)
+
+        # THEN: We should get back a valid service item
+        assert len(results.verselist) == 36, u'The bible should not have had any books added or removed'
+
+    def crosswalk_extract_books_test(self):
+        """
+        Test Crosswalk retrieval of book list for NIV bible
+        """
+        # GIVEN: A new Bible Gateway extraction class
+        handler = CWExtract()
+
+        # WHEN: The Books list is called
+        books = handler.get_books_from_http(u'niv')
+
+        # THEN: We should get back a valid service item
+        assert len(books) == 66, u'The bible should not have had any books added or removed'
+
+    def crosswalk_extract_verse_test(self):
+        """
+        Test the Crosswalk retrieval of verse list for NIV bible John 3
+        """
+        # GIVEN: A new Bible Gateway extraction class
+        handler = CWExtract()
+
+        # WHEN: The Books list is called
+        results = handler.get_bible_chapter(u'niv', u'john', 3)
+
+        # THEN: We should get back a valid service item
+        assert len(results.verselist) == 36, u'The bible should not have had any books added or removed'
\ No newline at end of file

From 7bf713a8303138dce7bc09321241b88fa00aa180 Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Sun, 16 Jun 2013 09:28:38 +0100
Subject: [PATCH 10/39] minor updates

---
 tests/functional/openlp_plugins/bibles/test_lib_http.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tests/functional/openlp_plugins/bibles/test_lib_http.py b/tests/functional/openlp_plugins/bibles/test_lib_http.py
index 3d2e849de..a97c1a188 100644
--- a/tests/functional/openlp_plugins/bibles/test_lib_http.py
+++ b/tests/functional/openlp_plugins/bibles/test_lib_http.py
@@ -43,7 +43,7 @@ class TestBibleHTTP(TestCase):
         results = handler.get_bible_chapter(u'NIV', u'John', 3)
 
         # THEN: We should get back a valid service item
-        assert len(results.verselist) == 36, u'The bible should not have had any books added or removed'
+        assert len(results.verselist) == 36, u'The book of John should not have had any verses added or removed'
 
     def crosswalk_extract_books_test(self):
         """
@@ -69,4 +69,5 @@ class TestBibleHTTP(TestCase):
         results = handler.get_bible_chapter(u'niv', u'john', 3)
 
         # THEN: We should get back a valid service item
-        assert len(results.verselist) == 36, u'The bible should not have had any books added or removed'
\ No newline at end of file
+        assert len(results.verselist) == 36, u'The book of John should not have had any verses added or removed'
+

From bb36a67ac2067fe2c5b490d4d167bdbb6abbbd6e Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Sun, 16 Jun 2013 14:36:09 +0100
Subject: [PATCH 11/39] Move tests

---
 .../openlp_plugins/bibles/__init__.py                           | 0
 .../openlp_plugins/bibles/test_lib_http.py                      | 2 +-
 2 files changed, 1 insertion(+), 1 deletion(-)
 rename tests/{functional => interfaces}/openlp_plugins/bibles/__init__.py (100%)
 rename tests/{functional => interfaces}/openlp_plugins/bibles/test_lib_http.py (97%)

diff --git a/tests/functional/openlp_plugins/bibles/__init__.py b/tests/interfaces/openlp_plugins/bibles/__init__.py
similarity index 100%
rename from tests/functional/openlp_plugins/bibles/__init__.py
rename to tests/interfaces/openlp_plugins/bibles/__init__.py
diff --git a/tests/functional/openlp_plugins/bibles/test_lib_http.py b/tests/interfaces/openlp_plugins/bibles/test_lib_http.py
similarity index 97%
rename from tests/functional/openlp_plugins/bibles/test_lib_http.py
rename to tests/interfaces/openlp_plugins/bibles/test_lib_http.py
index a97c1a188..c19f267ed 100644
--- a/tests/functional/openlp_plugins/bibles/test_lib_http.py
+++ b/tests/interfaces/openlp_plugins/bibles/test_lib_http.py
@@ -60,7 +60,7 @@ class TestBibleHTTP(TestCase):
 
     def crosswalk_extract_verse_test(self):
         """
-        Test the Crosswalk retrieval of verse list for NIV bible John 3
+        Test Crosswalk retrieval of verse list for NIV bible John 3
         """
         # GIVEN: A new Bible Gateway extraction class
         handler = CWExtract()

From 6b881e312fa0097b0a03af1c78976095c73cbd10 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Sun, 16 Jun 2013 19:28:44 +0200
Subject: [PATCH 12/39] added second error message

---
 openlp/plugins/songs/forms/editsongdialog.py | 7 ++++---
 openlp/plugins/songs/forms/editsongform.py   | 9 ++++++++-
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/openlp/plugins/songs/forms/editsongdialog.py b/openlp/plugins/songs/forms/editsongdialog.py
index 4bf2c91d9..19dad3eae 100644
--- a/openlp/plugins/songs/forms/editsongdialog.py
+++ b/openlp/plugins/songs/forms/editsongdialog.py
@@ -275,7 +275,6 @@ class Ui_EditSongDialog(object):
         self.bottom_layout.setObjectName(u'bottom_layout')
         self.warning_label = QtGui.QLabel(edit_song_dialog)
         self.warning_label.setObjectName(u'warning_label')
-        self.warning_label.setVisible(False)
         self.bottom_layout.addWidget(self.warning_label)
         self.button_box = create_button_box(edit_song_dialog, u'button_box', [u'cancel', u'save'])
         self.bottom_layout.addWidget(self.button_box)
@@ -323,8 +322,10 @@ class Ui_EditSongDialog(object):
         self.from_media_button.setText(translate('SongsPlugin.EditSongForm', 'Add &Media'))
         self.audio_remove_button.setText(translate('SongsPlugin.EditSongForm', '&Remove'))
         self.audio_remove_all_button.setText(translate('SongsPlugin.EditSongForm', 'Remove &All'))
-        self.warning_label.setText(
-            translate('SongsPlugin.EditSongForm', '<strong>Warning:</strong> Not all of the verses are in use.'))
+        self.not_all_verses_used_warning = \
+            translate('SongsPlugin.EditSongForm', '<strong>Warning:</strong> Not all of the verses are in use.')
+        self.no_verse_order_entered_warning =  \
+            translate('SongsPlugin.EditSongForm', '<strong>Warning:</strong> You have not entered a verse order.')
 
 
 def create_combo_box(parent, name):
diff --git a/openlp/plugins/songs/forms/editsongform.py b/openlp/plugins/songs/forms/editsongform.py
index 81cfce2fd..79d6bdd76 100644
--- a/openlp/plugins/songs/forms/editsongform.py
+++ b/openlp/plugins/songs/forms/editsongform.py
@@ -456,6 +456,8 @@ class EditSongForm(QtGui.QDialog, Ui_EditSongDialog):
         self.title_edit.setFocus()
         # Hide or show the preview button.
         self.preview_button.setVisible(preview)
+        # Check if all verse tags are used.
+        self.on_verse_order_text_changed(self.verse_order_edit.text())
 
     def tag_rows(self):
         """
@@ -697,7 +699,12 @@ class EditSongForm(QtGui.QDialog, Ui_EditSongDialog):
         for verse in verses:
             if not verse in order:
                 verses_not_used.append(verse)
-        self.warning_label.setVisible(len(verses_not_used) > 0 and bool(text))
+        label_text = u''
+        if not self.verse_order_edit.text():
+            label_text = self.no_verse_order_entered_warning
+        elif verses_not_used:
+            label_text = self.not_all_verses_used_warning
+        self.warning_label.setText(label_text)
 
     def on_copyright_insert_button_triggered(self):
         text = self.copyright_edit.text()

From bb7bbdf10c8e3caf8f9a2aa1b363ede278406976 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Sun, 16 Jun 2013 21:42:50 +0200
Subject: [PATCH 13/39] use isinstance instead of type

---
 openlp/plugins/images/lib/mediaitem.py | 2 +-
 scripts/check_dependencies.py          | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/openlp/plugins/images/lib/mediaitem.py b/openlp/plugins/images/lib/mediaitem.py
index deed14594..3475570fc 100644
--- a/openlp/plugins/images/lib/mediaitem.py
+++ b/openlp/plugins/images/lib/mediaitem.py
@@ -473,7 +473,7 @@ class ImageMediaItem(MediaManagerItem):
             This boolean is set to True when the list in the interface should be reloaded after saving the new images
         """
         for filename in images_list:
-            if type(filename) is not str and type(filename) is not unicode:
+            if not isinstance(filename, basestring):
                 continue
             log.debug(u'Adding new image: %s', filename)
             imageFile = ImageFilenames()
diff --git a/scripts/check_dependencies.py b/scripts/check_dependencies.py
index 16a2c10e7..6c1d528ac 100755
--- a/scripts/check_dependencies.py
+++ b/scripts/check_dependencies.py
@@ -98,9 +98,9 @@ OPTIONAL_MODULES = [
 w = sys.stdout.write
 
 def check_vers(version, required, text):
-    if type(version) is not str:
+    if not isinstance(version, str):
         version = '.'.join(map(str, version))
-    if type(required) is not str:
+    if not isinstance(required, str):
         required = '.'.join(map(str, required))
     w('  %s >= %s ...    ' % (text, required))
     if LooseVersion(version) >= LooseVersion(required):

From 62c5908d8555caa213ea15046aae007837db7be4 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Tue, 18 Jun 2013 11:09:54 +0200
Subject: [PATCH 14/39] python3 division in slidecontroller

---
 openlp/core/ui/pluginform.py      | 2 +-
 openlp/core/ui/slidecontroller.py | 9 +++++----
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/openlp/core/ui/pluginform.py b/openlp/core/ui/pluginform.py
index 954203d26..dd497bd68 100644
--- a/openlp/core/ui/pluginform.py
+++ b/openlp/core/ui/pluginform.py
@@ -31,7 +31,7 @@ The actual plugin view form
 """
 import logging
 
-from PyQt4 import QtCore, QtGui
+from PyQt4 import QtGui
 
 from openlp.core.lib import PluginStatus, Registry, translate
 from plugindialog import Ui_PluginViewDialog
diff --git a/openlp/core/ui/slidecontroller.py b/openlp/core/ui/slidecontroller.py
index c9bb8b02c..427bb5cf1 100644
--- a/openlp/core/ui/slidecontroller.py
+++ b/openlp/core/ui/slidecontroller.py
@@ -29,6 +29,7 @@
 """
 The :mod:`slidecontroller` module contains the most important part of OpenLP - the slide controller
 """
+from __future__ import division
 import os
 import logging
 import copy
@@ -93,7 +94,7 @@ class SlideController(DisplayController):
         Registry().register_function(u'bootstrap_post_set_up', self.screen_size_changed)
         self.screens = ScreenList()
         try:
-            self.ratio = float(self.screens.current[u'size'].width()) / float(self.screens.current[u'size'].height())
+            self.ratio = self.screens.current[u'size'].width() / self.screens.current[u'size'].height()
         except ZeroDivisionError:
             self.ratio = 1
         self.loop_list = [
@@ -524,7 +525,7 @@ class SlideController(DisplayController):
             self.display.audio_player.connectSlot(QtCore.SIGNAL(u'tick(qint64)'), self.on_audio_time_remaining)
         # The SlidePreview's ratio.
         try:
-            self.ratio = float(self.screens.current[u'size'].width()) / float(self.screens.current[u'size'].height())
+            self.ratio = self.screens.current[u'size'].width() / self.screens.current[u'size'].height()
         except ZeroDivisionError:
             self.ratio = 1
         self.media_controller.setup_display(self.display, False)
@@ -553,7 +554,7 @@ class SlideController(DisplayController):
         splitters is moved or when the screen size is changed. Note, that this
         method is (also) called frequently from the mainwindow *paintEvent*.
         """
-        if self.ratio < float(self.preview_frame.width()) / float(self.preview_frame.height()):
+        if self.ratio < self.preview_frame.width() / self.preview_frame.height():
             # We have to take the height as limit.
             max_height = self.preview_frame.height() - self.grid.margin() * 2
             self.slide_preview.setFixedSize(QtCore.QSize(max_height * self.ratio, max_height))
@@ -723,7 +724,7 @@ class SlideController(DisplayController):
 
     def _process_item(self, service_item, slideno):
         """
-        Loads a ServiceItem into the system from ServiceManager 
+        Loads a ServiceItem into the system from ServiceManager
         Display the slide number passed
         """
         log.debug(u'processManagerItem live = %s' % self.is_live)

From 1e11f4d821430c78a37da112bd4b5b5b17afbe84 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Tue, 18 Jun 2013 11:12:49 +0200
Subject: [PATCH 15/39] true division

---
 openlp/core/ui/listpreviewwidget.py | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/openlp/core/ui/listpreviewwidget.py b/openlp/core/ui/listpreviewwidget.py
index ca04c9688..e15b83fbd 100644
--- a/openlp/core/ui/listpreviewwidget.py
+++ b/openlp/core/ui/listpreviewwidget.py
@@ -76,7 +76,7 @@ class ListPreviewWidget(QtGui.QTableWidget):
             else:
                 # Sort out image heights.
                 for framenumber in range(len(self.service_item.get_frames())):
-                    height = self.viewport().width() / self.screen_ratio
+                    height = self.viewport().width() // self.screen_ratio
                     self.setRowHeight(framenumber, height)
 
     def screen_size_changed(self, screen_ratio):
@@ -101,7 +101,7 @@ class ListPreviewWidget(QtGui.QTableWidget):
         for framenumber, frame in enumerate(self.service_item.get_frames()):
             self.setRowCount(self.slide_count() + 1)
             item = QtGui.QTableWidgetItem()
-            slideHeight = 0
+            slide_height = 0
             if self.service_item.is_text():
                 if frame[u'verseTag']:
                     # These tags are already translated.
@@ -125,12 +125,12 @@ class ListPreviewWidget(QtGui.QTableWidget):
                     image = self.image_manager.get_image(frame[u'path'], ImageSource.ImagePlugin)
                     label.setPixmap(QtGui.QPixmap.fromImage(image))
                 self.setCellWidget(framenumber, 0, label)
-                slideHeight = width / self.screen_ratio
+                slide_height = width // self.screen_ratio
                 row += 1
             text.append(unicode(row))
             self.setItem(framenumber, 0, item)
-            if slideHeight:
-                self.setRowHeight(framenumber, slideHeight)
+            if slide_height:
+                self.setRowHeight(framenumber, slide_height)
         self.setVerticalHeaderLabels(text)
         if self.service_item.is_text():
             self.resizeRowsToContents()

From 658b6f89dae34f024a865b7ab94e3f6928d1c0f0 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Tue, 18 Jun 2013 21:32:04 +0200
Subject: [PATCH 16/39] import

---
 openlp/core/ui/listpreviewwidget.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openlp/core/ui/listpreviewwidget.py b/openlp/core/ui/listpreviewwidget.py
index e15b83fbd..ae6d0bed8 100644
--- a/openlp/core/ui/listpreviewwidget.py
+++ b/openlp/core/ui/listpreviewwidget.py
@@ -30,7 +30,7 @@
 The :mod:`listpreviewwidget` is a widget that lists the slides in the slide controller.
 It is based on a QTableWidget but represents its contents in list form.
 """
-
+from __future__ import division
 from PyQt4 import QtCore, QtGui
 
 from openlp.core.lib import ImageSource, Registry, ServiceItem

From aeecc041ec65b780eff4e0af9204038679484a0a Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Thu, 20 Jun 2013 20:04:21 +0200
Subject: [PATCH 17/39] removed spaces

---
 openlp/core/utils/__init__.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/openlp/core/utils/__init__.py b/openlp/core/utils/__init__.py
index 0a594b5cf..bfd0b0740 100644
--- a/openlp/core/utils/__init__.py
+++ b/openlp/core/utils/__init__.py
@@ -346,9 +346,9 @@ def get_uno_instance(resolver):
     """
     log.debug(u'get UNO Desktop Openoffice - resolve')
     if UNO_CONNECTION_TYPE == u'pipe':
-        return resolver.resolve(u'uno:pipe,name=openlp_pipe; urp;StarOffice.ComponentContext')
+        return resolver.resolve(u'uno:pipe,name=openlp_pipe;urp;StarOffice.ComponentContext')
     else:
-        return resolver.resolve(u'uno:socket,host=localhost,port=2002; urp;StarOffice.ComponentContext')
+        return resolver.resolve(u'uno:socket,host=localhost,port=2002;urp;StarOffice.ComponentContext')
 
 
 def format_time(text, local_time):

From 0f0097c1a4510e9ae07cc4bdc4be1cdefdd33ea1 Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Thu, 20 Jun 2013 20:12:55 +0100
Subject: [PATCH 18/39] Fix presentations

---
 openlp/plugins/presentations/lib/mediaitem.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/openlp/plugins/presentations/lib/mediaitem.py b/openlp/plugins/presentations/lib/mediaitem.py
index fcfc495ed..cef30a498 100644
--- a/openlp/plugins/presentations/lib/mediaitem.py
+++ b/openlp/plugins/presentations/lib/mediaitem.py
@@ -80,15 +80,15 @@ class PresentationMediaItem(MediaManagerItem):
         """
         Build the list of file extensions to be used in the Open file dialog.
         """
-        file_type = u''
+        file_type_list = u''
         for controller in self.controllers:
             if self.controllers[controller].enabled():
                 file_types = self.controllers[controller].supports + self.controllers[controller].also_supports
                 for file_type in file_types:
                     if file_type.find(file_type) == -1:
-                        file_type += u'*.%s ' % file_type
+                        file_type_list += u'*.%s ' % file_type
                         self.service_manager.supported_suffixes(file_type)
-        self.on_new_file_masks = translate('PresentationPlugin.MediaItem', 'Presentations (%s)') % file_type
+        self.on_new_file_masks = translate('PresentationPlugin.MediaItem', 'Presentations (%s)') % file_type_list
 
     def required_icons(self):
         """

From 4d1061f72309d91092b4efb11e96f3922c60afa3 Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Fri, 21 Jun 2013 06:16:35 +0100
Subject: [PATCH 19/39] Clean up registry for application

---
 openlp/core/lib/mediamanageritem.py           | 12 +++--
 openlp/core/lib/registry.py                   |  3 --
 openlp/core/ui/firsttimeform.py               | 12 +++--
 openlp/core/ui/maindisplay.py                 | 12 +++--
 openlp/core/ui/mainwindow.py                  | 15 ++++--
 openlp/core/ui/media/mediaplayer.py           | 12 +++--
 openlp/core/ui/pluginform.py                  | 12 +++--
 openlp/core/ui/servicemanager.py              | 12 +++--
 openlp/core/ui/thememanager.py                | 12 +++--
 openlp/core/ui/wizard.py                      | 12 +++--
 openlp/plugins/bibles/forms/editbibleform.py  | 12 +++--
 openlp/plugins/bibles/lib/db.py               | 12 +++--
 openlp/plugins/bibles/lib/http.py             | 48 ++++++++++++-------
 .../songs/forms/duplicatesongremovalform.py   | 12 +++--
 .../songs/forms/songmaintenanceform.py        | 12 +++--
 openlp/plugins/songs/lib/openlyricsexport.py  | 12 +++--
 16 files changed, 147 insertions(+), 75 deletions(-)

diff --git a/openlp/core/lib/mediamanageritem.py b/openlp/core/lib/mediamanageritem.py
index c3e1fa366..11565607e 100644
--- a/openlp/core/lib/mediamanageritem.py
+++ b/openlp/core/lib/mediamanageritem.py
@@ -728,10 +728,14 @@ class MediaManagerItem(QtGui.QWidget):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
diff --git a/openlp/core/lib/registry.py b/openlp/core/lib/registry.py
index 7e880f1f9..cee9e0329 100644
--- a/openlp/core/lib/registry.py
+++ b/openlp/core/lib/registry.py
@@ -103,9 +103,6 @@ class Registry(object):
         ``key``
             The service to be deleted.
         """
-        if self.running_under_test is False:
-            log.error(u'Invalid Method call for key %s' % key)
-            raise KeyError(u'Invalid Method call for key %s' % key)
         if key in self.service_list:
             del self.service_list[key]
 
diff --git a/openlp/core/ui/firsttimeform.py b/openlp/core/ui/firsttimeform.py
index 0f3f3cc18..d6bb59daf 100644
--- a/openlp/core/ui/firsttimeform.py
+++ b/openlp/core/ui/firsttimeform.py
@@ -484,10 +484,14 @@ class FirstTimeForm(QtGui.QWizard, Ui_FirstTimeWizard):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
diff --git a/openlp/core/ui/maindisplay.py b/openlp/core/ui/maindisplay.py
index 7069cb9b7..74cea4f1f 100644
--- a/openlp/core/ui/maindisplay.py
+++ b/openlp/core/ui/maindisplay.py
@@ -494,11 +494,15 @@ class MainDisplay(Display):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
 
diff --git a/openlp/core/ui/mainwindow.py b/openlp/core/ui/mainwindow.py
index 4607c441f..6a285de4c 100644
--- a/openlp/core/ui/mainwindow.py
+++ b/openlp/core/ui/mainwindow.py
@@ -1064,6 +1064,9 @@ class MainWindow(QtGui.QMainWindow, Ui_MainWindow):
         if self.live_controller.display:
             self.live_controller.display.close()
             self.live_controller.display = None
+        if os.name == u'nt':
+            # Needed for Windows to stop crashes on exit
+            Registry.remove(u'application')
 
     def service_changed(self, reset=False, serviceName=None):
         """
@@ -1374,10 +1377,14 @@ class MainWindow(QtGui.QMainWindow, Ui_MainWindow):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
diff --git a/openlp/core/ui/media/mediaplayer.py b/openlp/core/ui/media/mediaplayer.py
index c1f060f60..d76bbef58 100644
--- a/openlp/core/ui/media/mediaplayer.py
+++ b/openlp/core/ui/media/mediaplayer.py
@@ -153,10 +153,14 @@ class MediaPlayer(object):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
\ No newline at end of file
diff --git a/openlp/core/ui/pluginform.py b/openlp/core/ui/pluginform.py
index dd497bd68..d54bb8b6d 100644
--- a/openlp/core/ui/pluginform.py
+++ b/openlp/core/ui/pluginform.py
@@ -166,10 +166,14 @@ class PluginForm(QtGui.QDialog, Ui_PluginViewDialog):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
diff --git a/openlp/core/ui/servicemanager.py b/openlp/core/ui/servicemanager.py
index 444edc814..718a66ec8 100644
--- a/openlp/core/ui/servicemanager.py
+++ b/openlp/core/ui/servicemanager.py
@@ -1588,10 +1588,14 @@ class ServiceManager(QtGui.QWidget, ServiceManagerDialog):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
diff --git a/openlp/core/ui/thememanager.py b/openlp/core/ui/thememanager.py
index be0e3bfa1..07595b4a5 100644
--- a/openlp/core/ui/thememanager.py
+++ b/openlp/core/ui/thememanager.py
@@ -836,10 +836,14 @@ class ThemeManager(QtGui.QWidget):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
diff --git a/openlp/core/ui/wizard.py b/openlp/core/ui/wizard.py
index 0b142b459..0101b9d0e 100644
--- a/openlp/core/ui/wizard.py
+++ b/openlp/core/ui/wizard.py
@@ -320,10 +320,14 @@ class OpenLPWizard(QtGui.QWizard):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
diff --git a/openlp/plugins/bibles/forms/editbibleform.py b/openlp/plugins/bibles/forms/editbibleform.py
index b05fb68fb..36e41eb51 100644
--- a/openlp/plugins/bibles/forms/editbibleform.py
+++ b/openlp/plugins/bibles/forms/editbibleform.py
@@ -191,10 +191,14 @@ class EditBibleForm(QtGui.QDialog, Ui_EditBibleDialog):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
diff --git a/openlp/plugins/bibles/lib/db.py b/openlp/plugins/bibles/lib/db.py
index 837dbec38..b8e85afaf 100644
--- a/openlp/plugins/bibles/lib/db.py
+++ b/openlp/plugins/bibles/lib/db.py
@@ -544,11 +544,15 @@ class BibleDB(QtCore.QObject, Manager):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
 
diff --git a/openlp/plugins/bibles/lib/http.py b/openlp/plugins/bibles/lib/http.py
index efab9106b..332194049 100644
--- a/openlp/plugins/bibles/lib/http.py
+++ b/openlp/plugins/bibles/lib/http.py
@@ -301,11 +301,15 @@ class BGExtract(object):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
 
@@ -377,11 +381,15 @@ class BSExtract(object):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
 
@@ -477,11 +485,15 @@ class CWExtract(object):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
 
@@ -667,11 +679,15 @@ class HTTPBible(BibleDB):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
 
diff --git a/openlp/plugins/songs/forms/duplicatesongremovalform.py b/openlp/plugins/songs/forms/duplicatesongremovalform.py
index d612a5627..0b2b2457a 100644
--- a/openlp/plugins/songs/forms/duplicatesongremovalform.py
+++ b/openlp/plugins/songs/forms/duplicatesongremovalform.py
@@ -349,10 +349,14 @@ class DuplicateSongRemovalForm(OpenLPWizard):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
\ No newline at end of file
diff --git a/openlp/plugins/songs/forms/songmaintenanceform.py b/openlp/plugins/songs/forms/songmaintenanceform.py
index 9f003b2a0..9c2d49a5d 100644
--- a/openlp/plugins/songs/forms/songmaintenanceform.py
+++ b/openlp/plugins/songs/forms/songmaintenanceform.py
@@ -525,10 +525,14 @@ class SongMaintenanceForm(QtGui.QDialog, Ui_SongMaintenanceDialog):
 
     def _get_application(self):
         """
-        Adds the application to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
diff --git a/openlp/plugins/songs/lib/openlyricsexport.py b/openlp/plugins/songs/lib/openlyricsexport.py
index f9ac6ebb3..be68a55c6 100644
--- a/openlp/plugins/songs/lib/openlyricsexport.py
+++ b/openlp/plugins/songs/lib/openlyricsexport.py
@@ -84,10 +84,14 @@ class OpenLyricsExport(object):
 
     def _get_application(self):
         """
-        Adds the openlp to the class dynamically
+        Adds the openlp to the class dynamically.
+        Windows needs to access the application in a dynamic manner.
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)

From 6265ce93de7ab59643c7f7f20cacce0b43e0a25d Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Fri, 21 Jun 2013 12:16:50 +0200
Subject: [PATCH 20/39] fixed tests

---
 openlp/plugins/songs/forms/editsongform.py    | 28 ++++---
 .../songs/forms/test_editsongform.py          | 83 +++++++++++--------
 2 files changed, 65 insertions(+), 46 deletions(-)

diff --git a/openlp/plugins/songs/forms/editsongform.py b/openlp/plugins/songs/forms/editsongform.py
index 79d6bdd76..9ad8809b7 100644
--- a/openlp/plugins/songs/forms/editsongform.py
+++ b/openlp/plugins/songs/forms/editsongform.py
@@ -685,23 +685,31 @@ class EditSongForm(QtGui.QDialog, Ui_EditSongDialog):
             self.verse_edit_button.setEnabled(False)
             self.verse_delete_button.setEnabled(False)
 
+
     def on_verse_order_text_changed(self, text):
-        verses = []
-        verse_names = []
-        order = self._extract_verse_order(text)
+        """
+        Checks if the verse order is complete or missing. Shows a error message according to the state of the verse
+        order.
+
+        ``text``
+            The text of the verse order edit (ignored).
+        """
+        # Extract all verses which were used in the order.
+        verses_in_order = self._extract_verse_order(self.verse_order_edit.text())
+        # Find the verses which were not used in the order.
+        verses_not_used = []
         for index in range(self.verse_list_widget.rowCount()):
             verse = self.verse_list_widget.item(index, 0)
             verse = verse.data(QtCore.Qt.UserRole)
-            if verse not in verse_names:
-                verses.append(verse)
-                verse_names.append(u'%s%s' % (VerseType.translated_tag(verse[0]), verse[1:]))
-        verses_not_used = []
-        for verse in verses:
-            if not verse in order:
+            print(verse)
+            if verse not in verses_in_order:
                 verses_not_used.append(verse)
+        # Set the label text.
         label_text = u''
-        if not self.verse_order_edit.text():
+        # No verse order was entered.
+        if not verses_in_order:
             label_text = self.no_verse_order_entered_warning
+        # The verse order does not contain all verses.
         elif verses_not_used:
             label_text = self.not_all_verses_used_warning
         self.warning_label.setText(label_text)
diff --git a/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py b/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
index 6c90fa0c0..9e560c5e6 100644
--- a/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
+++ b/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
@@ -34,7 +34,7 @@ class TestEditSongForm(TestCase):
         del self.main_window
         del self.app
 
-    def ui_defaults_(self):
+    def ui_defaults_test(self):
         """
         Test that the EditSongForm defaults are correct
         """
@@ -43,58 +43,69 @@ class TestEditSongForm(TestCase):
         self.assertFalse(self.form.author_remove_button.isEnabled(), u'The author remove button should not be enabled')
         self.assertFalse(self.form.topic_remove_button.isEnabled(), u'The topic remove button should not be enabled')
 
-    def is_verse_edit_form_executed_t(self):
+    def is_verse_edit_form_executed_test(self):
         pass
 
-    def verse_order_warning_hidden_test(self):
+    def verse_order_no_warning_test(self):
         """
-        Test if the verse order warning lable is visible, when a verse order is specified
+        Test if the verse order warning is not shown
         """
         # GIVEN: Mocked methods.
-        mocked_row_count = MagicMock()
-        mocked_row_count.return_value = 1
-        self.form.verse_list_widget.rowCount = mocked_row_count
+        given_verse_order = u'V1 V2'
+        self.form.verse_list_widget.rowCount = MagicMock(return_value=2)
         # Mock out the verse.
-        mocked_verse = MagicMock()
-        self.form.verse_list_widget.item = mocked_verse
-        mocked_verse_data_method = MagicMock()
-        mocked_verse_data_method.return_value = u'V1'
-        mocked_verse.data = mocked_verse_data_method
-        mocked_item_method = MagicMock()
-        mocked_item_method.return_value = mocked_verse
-        mocked_extract_verse_order_method = MagicMock()
-        mocked_extract_verse_order_method.return_value = [u'V1']
-        self.form._extract_verse_order = mocked_extract_verse_order_method
+        first_verse = MagicMock()
+        first_verse.data = MagicMock(return_value=u'V1')
+        second_verse = MagicMock()
+        second_verse.data = MagicMock(return_value= u'V2')
+        self.form.verse_list_widget.item = MagicMock(side_effect=[first_verse, second_verse])
+        self.form._extract_verse_order = MagicMock(return_value=given_verse_order.split())
 
         # WHEN: Call the method.
-        self.form.on_verse_order_text_changed(u'V1')
+        self.form.on_verse_order_text_changed(given_verse_order)
 
         # THEN: The warning lable should be hidden.
-        assert not self.form.warning_label.isVisible(), u'The warning lable should be hidden.'
+        print  self.form.warning_label.text()
+        assert self.form.warning_label.text() == u'', u'There should be no warning.'
+
+    def verse_order_incomplete_warning_test(self):
+        """
+        Test if the verse-order-incomple warning is shown
+        """
+        # GIVEN: Mocked methods.
+        given_verse_order = u'V1'
+        self.form.verse_list_widget.rowCount = MagicMock(return_value=2)
+        # Mock out the verse.
+        first_verse = MagicMock()
+        first_verse.data = MagicMock(return_value=u'V1')
+        second_verse = MagicMock()
+        second_verse.data = MagicMock(return_value= u'V2')
+        self.form.verse_list_widget.item = MagicMock(side_effect=[first_verse, second_verse])
+        self.form._extract_verse_order = MagicMock(return_value=[given_verse_order])
+
+        # WHEN: Call the method.
+        self.form.on_verse_order_text_changed(given_verse_order)
+
+        # THEN: The warning lable should be hidden.
+        assert self.form.warning_label.text() == u'<strong>Warning:</strong> Not all of the verses are in use.', \
+            u'The verse-order-incomplete warning should be shown.'
 
     def bug_1170435_test(self):
         """
-        Regression test for bug 1170435 (test if lable hidden, when no verse order is specified)
+        Regression test for bug 1170435 (test if "no verse order" message is shown)
         """
         # GIVEN: Mocked methods.
-        mocked_row_count = MagicMock()
-        mocked_row_count.return_value = 0
-        self.form.verse_list_widget.rowCount = mocked_row_count
+        given_verse_order = u''
+        self.form.verse_list_widget.rowCount = MagicMock(return_value=1)
         # Mock out the verse. (We want a verse type to be returned).
         mocked_verse = MagicMock()
-        self.form.verse_list_widget.item = mocked_verse
-        mocked_verse_data_method = MagicMock()
-        mocked_verse_data_method.return_value = u'V1'
-        mocked_verse.data = mocked_verse_data_method
-        mocked_item_method = MagicMock()
-        mocked_item_method.return_value = mocked_verse
-        mocked_extract_verse_order_method = MagicMock()
-        mocked_extract_verse_order_method.return_value = []
-        self.form._extract_verse_order = mocked_extract_verse_order_method
-
+        mocked_verse.data = MagicMock(return_value=u'V1')
+        self.form.verse_list_widget.item = MagicMock(return_value=mocked_verse)
+        self.form._extract_verse_order = MagicMock(return_value=[given_verse_order])
+        self.form.verse_order_edit.text = MagicMock(return_value=given_verse_order)
         # WHEN: Call the method.
-        self.form.on_verse_order_text_changed(u'')
+        self.form.on_verse_order_text_changed(given_verse_order)
 
         # THEN: The warning lable should be hidden.
-        assert  not self.form.warning_label.isVisible(), \
-            u'The lable should be visible because the verse order was left empty.'
+        assert self.form.warning_label.text() == u'<strong>Warning:</strong> You have not entered a verse order.',  \
+            u'The no-verse-order message should be shown.'

From 4f8d93638ba44678ebcff8306acb40be1abf942c Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Fri, 21 Jun 2013 20:13:59 +0200
Subject: [PATCH 21/39] fixes

---
 openlp/plugins/songs/forms/editsongform.py          |  1 -
 .../openlp_plugins/songs/forms/test_editsongform.py | 13 ++++++-------
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/openlp/plugins/songs/forms/editsongform.py b/openlp/plugins/songs/forms/editsongform.py
index 9ad8809b7..24d0d3024 100644
--- a/openlp/plugins/songs/forms/editsongform.py
+++ b/openlp/plugins/songs/forms/editsongform.py
@@ -701,7 +701,6 @@ class EditSongForm(QtGui.QDialog, Ui_EditSongDialog):
         for index in range(self.verse_list_widget.rowCount()):
             verse = self.verse_list_widget.item(index, 0)
             verse = verse.data(QtCore.Qt.UserRole)
-            print(verse)
             if verse not in verses_in_order:
                 verses_not_used.append(verse)
         # Set the label text.
diff --git a/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py b/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
index 9e560c5e6..013e91bc1 100644
--- a/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
+++ b/tests/interfaces/openlp_plugins/songs/forms/test_editsongform.py
@@ -64,8 +64,7 @@ class TestEditSongForm(TestCase):
         # WHEN: Call the method.
         self.form.on_verse_order_text_changed(given_verse_order)
 
-        # THEN: The warning lable should be hidden.
-        print  self.form.warning_label.text()
+        # THEN: No text should be shown.
         assert self.form.warning_label.text() == u'', u'There should be no warning.'
 
     def verse_order_incomplete_warning_test(self):
@@ -86,8 +85,8 @@ class TestEditSongForm(TestCase):
         # WHEN: Call the method.
         self.form.on_verse_order_text_changed(given_verse_order)
 
-        # THEN: The warning lable should be hidden.
-        assert self.form.warning_label.text() == u'<strong>Warning:</strong> Not all of the verses are in use.', \
+        # THEN: The verse-order-incomplete text should be shown.
+        assert self.form.warning_label.text() == self.form.not_all_verses_used_warning, \
             u'The verse-order-incomplete warning should be shown.'
 
     def bug_1170435_test(self):
@@ -101,11 +100,11 @@ class TestEditSongForm(TestCase):
         mocked_verse = MagicMock()
         mocked_verse.data = MagicMock(return_value=u'V1')
         self.form.verse_list_widget.item = MagicMock(return_value=mocked_verse)
-        self.form._extract_verse_order = MagicMock(return_value=[given_verse_order])
+        self.form._extract_verse_order = MagicMock(return_value=[])
         self.form.verse_order_edit.text = MagicMock(return_value=given_verse_order)
         # WHEN: Call the method.
         self.form.on_verse_order_text_changed(given_verse_order)
 
-        # THEN: The warning lable should be hidden.
-        assert self.form.warning_label.text() == u'<strong>Warning:</strong> You have not entered a verse order.',  \
+        # THEN: The no-verse-order message should be shown.
+        assert self.form.warning_label.text() == self.form.no_verse_order_entered_warning,  \
             u'The no-verse-order message should be shown.'

From c30748873c0c74f4a303d09178740ab7926f1b2d Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Sun, 23 Jun 2013 19:22:44 +0200
Subject: [PATCH 22/39] python3 fix

---
 openlp/core/ui/media/webkitplayer.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openlp/core/ui/media/webkitplayer.py b/openlp/core/ui/media/webkitplayer.py
index 2df28de36..e534a974b 100644
--- a/openlp/core/ui/media/webkitplayer.py
+++ b/openlp/core/ui/media/webkitplayer.py
@@ -398,7 +398,7 @@ class WebkitPlayer(MediaPlayer):
             # check if conversion was ok and value is not 'NaN'
             if length and length != float('inf'):
                 length = int(length * 1000)
-        if current_time > 0:
+        if current_time:
             controller.media_info.length = length
             controller.seek_slider.setMaximum(length)
             if not controller.seek_slider.isSliderDown():

From d32ece04a5304120b8afa014aea92b2a7eb03bc6 Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Sun, 23 Jun 2013 20:51:17 +0100
Subject: [PATCH 23/39] add missing imports

---
 openlp/core/lib/plugin.py                         | 10 +++++++---
 openlp/core/ui/maindisplay.py                     |  1 +
 openlp/core/ui/media/mediaplayer.py               |  2 ++
 openlp/core/ui/pluginform.py                      |  1 +
 openlp/plugins/bibles/forms/editbibleform.py      |  1 +
 openlp/plugins/bibles/lib/http.py                 |  2 ++
 openlp/plugins/songs/forms/songmaintenanceform.py |  1 +
 7 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/openlp/core/lib/plugin.py b/openlp/core/lib/plugin.py
index b4f851b24..4a82783e3 100644
--- a/openlp/core/lib/plugin.py
+++ b/openlp/core/lib/plugin.py
@@ -30,6 +30,7 @@
 Provide the generic plugin functionality for OpenLP plugins.
 """
 import logging
+import os
 
 from PyQt4 import QtCore
 
@@ -424,8 +425,11 @@ class Plugin(QtCore.QObject):
         """
         Adds the openlp to the class dynamically
         """
-        if not hasattr(self, u'_application'):
-            self._application = Registry().get(u'application')
-        return self._application
+        if os.name == u'nt':
+            Registry().get(u'application')
+        else:
+            if not hasattr(self, u'_application'):
+                self._application = Registry().get(u'application')
+            return self._application
 
     application = property(_get_application)
diff --git a/openlp/core/ui/maindisplay.py b/openlp/core/ui/maindisplay.py
index 74cea4f1f..ee3ddd6b5 100644
--- a/openlp/core/ui/maindisplay.py
+++ b/openlp/core/ui/maindisplay.py
@@ -38,6 +38,7 @@ Some of the code for this form is based on the examples at:
 from __future__ import division
 import cgi
 import logging
+import os
 import sys
 
 from PyQt4 import QtCore, QtGui, QtWebKit, QtOpenGL
diff --git a/openlp/core/ui/media/mediaplayer.py b/openlp/core/ui/media/mediaplayer.py
index d76bbef58..bfa3bb5da 100644
--- a/openlp/core/ui/media/mediaplayer.py
+++ b/openlp/core/ui/media/mediaplayer.py
@@ -29,6 +29,8 @@
 """
 The :mod:`~openlp.core.ui.media.mediaplayer` module contains the MediaPlayer class.
 """
+import os
+
 from openlp.core.lib import Registry
 from openlp.core.ui.media import MediaState
 
diff --git a/openlp/core/ui/pluginform.py b/openlp/core/ui/pluginform.py
index d54bb8b6d..4e766c15b 100644
--- a/openlp/core/ui/pluginform.py
+++ b/openlp/core/ui/pluginform.py
@@ -30,6 +30,7 @@
 The actual plugin view form
 """
 import logging
+import os
 
 from PyQt4 import QtGui
 
diff --git a/openlp/plugins/bibles/forms/editbibleform.py b/openlp/plugins/bibles/forms/editbibleform.py
index 36e41eb51..53ab37e0f 100644
--- a/openlp/plugins/bibles/forms/editbibleform.py
+++ b/openlp/plugins/bibles/forms/editbibleform.py
@@ -28,6 +28,7 @@
 ###############################################################################
 
 import logging
+import os
 import re
 
 from PyQt4 import QtGui
diff --git a/openlp/plugins/bibles/lib/http.py b/openlp/plugins/bibles/lib/http.py
index 332194049..7e441becc 100644
--- a/openlp/plugins/bibles/lib/http.py
+++ b/openlp/plugins/bibles/lib/http.py
@@ -29,6 +29,7 @@
 """
 The :mod:`http` module enables OpenLP to retrieve scripture from bible websites.
 """
+import os
 import logging
 import re
 import socket
@@ -691,6 +692,7 @@ class HTTPBible(BibleDB):
 
     application = property(_get_application)
 
+
 def get_soup_for_bible_ref(reference_url, header=None, pre_parse_regex=None, pre_parse_substitute=None):
     """
     Gets a webpage and returns a parsed and optionally cleaned soup or None.
diff --git a/openlp/plugins/songs/forms/songmaintenanceform.py b/openlp/plugins/songs/forms/songmaintenanceform.py
index 9c2d49a5d..ff9e2d562 100644
--- a/openlp/plugins/songs/forms/songmaintenanceform.py
+++ b/openlp/plugins/songs/forms/songmaintenanceform.py
@@ -27,6 +27,7 @@
 # Temple Place, Suite 330, Boston, MA 02111-1307 USA                          #
 ###############################################################################
 import logging
+import os
 
 from PyQt4 import QtGui, QtCore
 from sqlalchemy.sql import and_

From 32c7663a45b97aedb48b81f4ee00be80c57b2431 Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Mon, 24 Jun 2013 17:21:46 +0100
Subject: [PATCH 24/39] Convert formatting tags to json

---
 openlp/core/lib/formattingtags.py                      | 10 ++++++++--
 openlp/core/lib/settings.py                            |  1 +
 .../functional/openlp_core_lib/test_formattingtags.py  |  8 ++++----
 3 files changed, 13 insertions(+), 6 deletions(-)

diff --git a/openlp/core/lib/formattingtags.py b/openlp/core/lib/formattingtags.py
index b2d8f6ea7..c136f00aa 100644
--- a/openlp/core/lib/formattingtags.py
+++ b/openlp/core/lib/formattingtags.py
@@ -30,6 +30,7 @@
 Provide HTML Tag management and Formatting Tag access class
 """
 import cPickle
+import json
 
 from openlp.core.lib import Settings, translate
 
@@ -66,7 +67,8 @@ class FormattingTags(object):
                     if isinstance(tag[element], unicode):
                         tag[element] = tag[element].encode('utf8')
         # Formatting Tags were also known as display tags.
-        Settings().setValue(u'displayTags/html_tags', cPickle.dumps(tags) if tags else u'')
+        Settings().setValue(u'displayTags/html_tags', json.dumps(tags) if tags else u'')
+        Settings().setValue(u'displayTags/html_tags_json', True)
 
     @staticmethod
     def load_tags():
@@ -159,10 +161,14 @@ class FormattingTags(object):
 
         # Formatting Tags were also known as display tags.
         user_expands = Settings().value(u'displayTags/html_tags')
+        user_format_json = Settings().value(u'displayTags/html_tags_json')
         # cPickle only accepts str not unicode strings
         user_expands_string = str(user_expands)
         if user_expands_string:
-            user_tags = cPickle.loads(user_expands_string)
+            if user_format_json:
+                user_tags = json.loads(user_expands_string)
+            else:
+                user_tags = cPickle.loads(user_expands_string)
             for tag in user_tags:
                 for element in tag:
                     if isinstance(tag[element], str):
diff --git a/openlp/core/lib/settings.py b/openlp/core/lib/settings.py
index 5e73ffc1d..427c24c58 100644
--- a/openlp/core/lib/settings.py
+++ b/openlp/core/lib/settings.py
@@ -116,6 +116,7 @@ class Settings(QtCore.QSettings):
         u'advanced/x11 bypass wm': X11_BYPASS_DEFAULT,
         u'crashreport/last directory': u'',
         u'displayTags/html_tags': u'',
+        u'displayTags/html_tags_json': False,
         u'core/audio repeat list': False,
         u'core/auto open': False,
         u'core/auto preview': False,
diff --git a/tests/functional/openlp_core_lib/test_formattingtags.py b/tests/functional/openlp_core_lib/test_formattingtags.py
index 26d87f466..326dde5c8 100644
--- a/tests/functional/openlp_core_lib/test_formattingtags.py
+++ b/tests/functional/openlp_core_lib/test_formattingtags.py
@@ -33,11 +33,11 @@ class TestFormattingTags(TestCase):
         """
         with patch(u'openlp.core.lib.translate') as mocked_translate, \
                 patch(u'openlp.core.lib.settings') as mocked_settings, \
-                patch(u'openlp.core.lib.formattingtags.cPickle') as mocked_cPickle:
+                patch(u'openlp.core.lib.formattingtags.json') as mocked_json:
             # GIVEN: Our mocked modules and functions.
             mocked_translate.side_effect = lambda module, string_to_translate, comment: string_to_translate
             mocked_settings.value.return_value = u''
-            mocked_cPickle.load.return_value = []
+            mocked_json.load.return_value = []
 
             # WHEN: Get the display tags.
             FormattingTags.load_tags()
@@ -54,11 +54,11 @@ class TestFormattingTags(TestCase):
         """
         with patch(u'openlp.core.lib.translate') as mocked_translate, \
                 patch(u'openlp.core.lib.settings') as mocked_settings, \
-                patch(u'openlp.core.lib.formattingtags.cPickle') as mocked_cPickle:
+                patch(u'openlp.core.lib.formattingtags.json') as mocked_json:
             # GIVEN: Our mocked modules and functions.
             mocked_translate.side_effect = lambda module, string_to_translate: string_to_translate
             mocked_settings.value.return_value = u''
-            mocked_cPickle.loads.side_effect = [[], [TAG]]
+            mocked_json.loads.side_effect = [[], [TAG]]
 
             # WHEN: Get the display tags.
             FormattingTags.load_tags()

From 26e80be5b76016c9ae2384168753e12af21f9020 Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Mon, 24 Jun 2013 17:54:23 +0100
Subject: [PATCH 25/39] Who forgot the return

---
 openlp/core/lib/mediamanageritem.py                    | 2 +-
 openlp/core/lib/plugin.py                              | 2 +-
 openlp/core/ui/firsttimeform.py                        | 2 +-
 openlp/core/ui/maindisplay.py                          | 2 +-
 openlp/core/ui/mainwindow.py                           | 2 +-
 openlp/core/ui/media/mediaplayer.py                    | 2 +-
 openlp/core/ui/pluginform.py                           | 2 +-
 openlp/core/ui/servicemanager.py                       | 2 +-
 openlp/core/ui/thememanager.py                         | 2 +-
 openlp/core/ui/wizard.py                               | 2 +-
 openlp/plugins/bibles/forms/editbibleform.py           | 2 +-
 openlp/plugins/bibles/lib/db.py                        | 2 +-
 openlp/plugins/bibles/lib/http.py                      | 8 ++++----
 openlp/plugins/songs/forms/duplicatesongremovalform.py | 2 +-
 openlp/plugins/songs/forms/songmaintenanceform.py      | 2 +-
 openlp/plugins/songs/lib/openlyricsexport.py           | 2 +-
 16 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/openlp/core/lib/mediamanageritem.py b/openlp/core/lib/mediamanageritem.py
index 11565607e..973d457bb 100644
--- a/openlp/core/lib/mediamanageritem.py
+++ b/openlp/core/lib/mediamanageritem.py
@@ -732,7 +732,7 @@ class MediaManagerItem(QtGui.QWidget):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/core/lib/plugin.py b/openlp/core/lib/plugin.py
index 4a82783e3..a79ba850a 100644
--- a/openlp/core/lib/plugin.py
+++ b/openlp/core/lib/plugin.py
@@ -426,7 +426,7 @@ class Plugin(QtCore.QObject):
         Adds the openlp to the class dynamically
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/core/ui/firsttimeform.py b/openlp/core/ui/firsttimeform.py
index d6bb59daf..f06b02a9a 100644
--- a/openlp/core/ui/firsttimeform.py
+++ b/openlp/core/ui/firsttimeform.py
@@ -488,7 +488,7 @@ class FirstTimeForm(QtGui.QWizard, Ui_FirstTimeWizard):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/core/ui/maindisplay.py b/openlp/core/ui/maindisplay.py
index ee3ddd6b5..eab89cb7d 100644
--- a/openlp/core/ui/maindisplay.py
+++ b/openlp/core/ui/maindisplay.py
@@ -499,7 +499,7 @@ class MainDisplay(Display):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/core/ui/mainwindow.py b/openlp/core/ui/mainwindow.py
index 6a285de4c..ffc0ebf16 100644
--- a/openlp/core/ui/mainwindow.py
+++ b/openlp/core/ui/mainwindow.py
@@ -1381,7 +1381,7 @@ class MainWindow(QtGui.QMainWindow, Ui_MainWindow):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/core/ui/media/mediaplayer.py b/openlp/core/ui/media/mediaplayer.py
index bfa3bb5da..5d9e3663f 100644
--- a/openlp/core/ui/media/mediaplayer.py
+++ b/openlp/core/ui/media/mediaplayer.py
@@ -159,7 +159,7 @@ class MediaPlayer(object):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/core/ui/pluginform.py b/openlp/core/ui/pluginform.py
index 4e766c15b..4f4824fb7 100644
--- a/openlp/core/ui/pluginform.py
+++ b/openlp/core/ui/pluginform.py
@@ -171,7 +171,7 @@ class PluginForm(QtGui.QDialog, Ui_PluginViewDialog):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/core/ui/servicemanager.py b/openlp/core/ui/servicemanager.py
index 718a66ec8..9ef1269ba 100644
--- a/openlp/core/ui/servicemanager.py
+++ b/openlp/core/ui/servicemanager.py
@@ -1592,7 +1592,7 @@ class ServiceManager(QtGui.QWidget, ServiceManagerDialog):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/core/ui/thememanager.py b/openlp/core/ui/thememanager.py
index 07595b4a5..58f29cab6 100644
--- a/openlp/core/ui/thememanager.py
+++ b/openlp/core/ui/thememanager.py
@@ -840,7 +840,7 @@ class ThemeManager(QtGui.QWidget):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/core/ui/wizard.py b/openlp/core/ui/wizard.py
index 0101b9d0e..ca89808ff 100644
--- a/openlp/core/ui/wizard.py
+++ b/openlp/core/ui/wizard.py
@@ -324,7 +324,7 @@ class OpenLPWizard(QtGui.QWizard):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/plugins/bibles/forms/editbibleform.py b/openlp/plugins/bibles/forms/editbibleform.py
index 53ab37e0f..4e0a5a7c0 100644
--- a/openlp/plugins/bibles/forms/editbibleform.py
+++ b/openlp/plugins/bibles/forms/editbibleform.py
@@ -196,7 +196,7 @@ class EditBibleForm(QtGui.QDialog, Ui_EditBibleDialog):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/plugins/bibles/lib/db.py b/openlp/plugins/bibles/lib/db.py
index b8e85afaf..69ec72ee5 100644
--- a/openlp/plugins/bibles/lib/db.py
+++ b/openlp/plugins/bibles/lib/db.py
@@ -548,7 +548,7 @@ class BibleDB(QtCore.QObject, Manager):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/plugins/bibles/lib/http.py b/openlp/plugins/bibles/lib/http.py
index 7e441becc..624836860 100644
--- a/openlp/plugins/bibles/lib/http.py
+++ b/openlp/plugins/bibles/lib/http.py
@@ -306,7 +306,7 @@ class BGExtract(object):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
@@ -386,7 +386,7 @@ class BSExtract(object):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
@@ -490,7 +490,7 @@ class CWExtract(object):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
@@ -684,7 +684,7 @@ class HTTPBible(BibleDB):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/plugins/songs/forms/duplicatesongremovalform.py b/openlp/plugins/songs/forms/duplicatesongremovalform.py
index 0b2b2457a..4427bbb31 100644
--- a/openlp/plugins/songs/forms/duplicatesongremovalform.py
+++ b/openlp/plugins/songs/forms/duplicatesongremovalform.py
@@ -353,7 +353,7 @@ class DuplicateSongRemovalForm(OpenLPWizard):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/plugins/songs/forms/songmaintenanceform.py b/openlp/plugins/songs/forms/songmaintenanceform.py
index ff9e2d562..2eb895a9a 100644
--- a/openlp/plugins/songs/forms/songmaintenanceform.py
+++ b/openlp/plugins/songs/forms/songmaintenanceform.py
@@ -530,7 +530,7 @@ class SongMaintenanceForm(QtGui.QDialog, Ui_SongMaintenanceDialog):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')
diff --git a/openlp/plugins/songs/lib/openlyricsexport.py b/openlp/plugins/songs/lib/openlyricsexport.py
index be68a55c6..35a9a29ec 100644
--- a/openlp/plugins/songs/lib/openlyricsexport.py
+++ b/openlp/plugins/songs/lib/openlyricsexport.py
@@ -88,7 +88,7 @@ class OpenLyricsExport(object):
         Windows needs to access the application in a dynamic manner.
         """
         if os.name == u'nt':
-            Registry().get(u'application')
+            return Registry().get(u'application')
         else:
             if not hasattr(self, u'_application'):
                 self._application = Registry().get(u'application')

From b6954eba0f4f6d6314ed51d1ef19dc89115014f4 Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Tue, 25 Jun 2013 15:44:56 +0100
Subject: [PATCH 26/39] Rename tag

---
 openlp/core/lib/formattingtags.py | 12 +++++++-----
 openlp/core/lib/imagemanager.py   |  7 ++++++-
 openlp/core/lib/settings.py       |  3 ++-
 3 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/openlp/core/lib/formattingtags.py b/openlp/core/lib/formattingtags.py
index c136f00aa..96591e1c4 100644
--- a/openlp/core/lib/formattingtags.py
+++ b/openlp/core/lib/formattingtags.py
@@ -67,8 +67,8 @@ class FormattingTags(object):
                     if isinstance(tag[element], unicode):
                         tag[element] = tag[element].encode('utf8')
         # Formatting Tags were also known as display tags.
-        Settings().setValue(u'displayTags/html_tags', json.dumps(tags) if tags else u'')
-        Settings().setValue(u'displayTags/html_tags_json', True)
+        Settings().setValue(u'formattingTags/html_tags', json.dumps(tags) if tags else u'')
+        Settings().setValue(u'formattingTags/html_tags_json', True)
 
     @staticmethod
     def load_tags():
@@ -160,10 +160,12 @@ class FormattingTags(object):
         FormattingTags.add_html_tags(temporary_tags)
 
         # Formatting Tags were also known as display tags.
-        user_expands = Settings().value(u'displayTags/html_tags')
-        user_format_json = Settings().value(u'displayTags/html_tags_json')
+        user_format_json = Settings().value(u'formattingTags/html_tags_json')
         # cPickle only accepts str not unicode strings
-        user_expands_string = str(user_expands)
+        if user_format_json:
+            user_expands_string = str(Settings().value(u'formattingTags/html_tags'))
+        else:
+            user_expands_string = str(Settings().value(u'displayTags/html_tags'))
         if user_expands_string:
             if user_format_json:
                 user_tags = json.loads(user_expands_string)
diff --git a/openlp/core/lib/imagemanager.py b/openlp/core/lib/imagemanager.py
index 412eddd1e..f8af5d0a0 100644
--- a/openlp/core/lib/imagemanager.py
+++ b/openlp/core/lib/imagemanager.py
@@ -1,9 +1,14 @@
+
 # -*- coding: utf-8 -*-
 # vim: autoindent shiftwidth=4 expandtab textwidth=120 tabstop=4 softtabstop=4
 
 ###############################################################################
 # OpenLP - Open Source Lyrics Projection                                      #
-# --------------------------------------------------------------------------- #
+
+
+
+
+    # --------------------------------------------------------------------------- #
 # Copyright (c) 2008-2013 Raoul Snyman                                        #
 # Portions copyright (c) 2008-2013 Tim Bentley, Gerald Britton, Jonathan      #
 # Corwin, Samuel Findlay, Michael Gorven, Scott Guerrieri, Matthias Hub,      #
diff --git a/openlp/core/lib/settings.py b/openlp/core/lib/settings.py
index 427c24c58..48a3a72c1 100644
--- a/openlp/core/lib/settings.py
+++ b/openlp/core/lib/settings.py
@@ -116,7 +116,8 @@ class Settings(QtCore.QSettings):
         u'advanced/x11 bypass wm': X11_BYPASS_DEFAULT,
         u'crashreport/last directory': u'',
         u'displayTags/html_tags': u'',
-        u'displayTags/html_tags_json': False,
+        u'formattingTags/html_tags': u'',
+        u'formattingTags/html_tags_json': False,
         u'core/audio repeat list': False,
         u'core/auto open': False,
         u'core/auto preview': False,

From f0693e81630ffe94fa1d56b7f43a17941fe59d9e Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Wed, 26 Jun 2013 20:54:21 +0200
Subject: [PATCH 27/39] fixed bs4.__version__

---
 openlp/core/ui/exceptionform.py | 4 ++--
 scripts/check_dependencies.py   | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/openlp/core/ui/exceptionform.py b/openlp/core/ui/exceptionform.py
index af1744424..49d6b0bef 100644
--- a/openlp/core/ui/exceptionform.py
+++ b/openlp/core/ui/exceptionform.py
@@ -34,8 +34,8 @@ import re
 import os
 import platform
 
+import bs4
 import sqlalchemy
-from bs4 import BeautifulSoup
 from lxml import etree
 from PyQt4 import Qt, QtCore, QtGui, QtWebKit
 
@@ -145,7 +145,7 @@ class ExceptionForm(QtGui.QDialog, Ui_ExceptionDialog):
             u'QtWebkit: %s\n' % WEBKIT_VERSION + \
             u'SQLAlchemy: %s\n' % sqlalchemy.__version__ + \
             u'SQLAlchemy Migrate: %s\n' % MIGRATE_VERSION + \
-            u'BeautifulSoup: %s\n' % BeautifulSoup.__version__ + \
+            u'BeautifulSoup: %s\n' % bs4.__version__ + \
             u'lxml: %s\n' % etree.__version__ + \
             u'Chardet: %s\n' % CHARDET_VERSION + \
             u'PyEnchant: %s\n' % ENCHANT_VERSION + \
diff --git a/scripts/check_dependencies.py b/scripts/check_dependencies.py
index 6c1d528ac..e3849d449 100755
--- a/scripts/check_dependencies.py
+++ b/scripts/check_dependencies.py
@@ -85,6 +85,7 @@ MODULES = [
     'migrate',
     'uno',
     'icu',
+    'bs4',
 ]
 
 

From 96fbb5ddbb216037ad2fb6fcd95dcede2712e0bd Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Sun, 30 Jun 2013 06:30:42 +0100
Subject: [PATCH 28/39] clean up tags

---
 openlp/core/lib/formattingtags.py | 13 ++-----------
 openlp/core/lib/settings.py       |  2 --
 2 files changed, 2 insertions(+), 13 deletions(-)

diff --git a/openlp/core/lib/formattingtags.py b/openlp/core/lib/formattingtags.py
index 96591e1c4..2dcae334f 100644
--- a/openlp/core/lib/formattingtags.py
+++ b/openlp/core/lib/formattingtags.py
@@ -158,19 +158,10 @@ class FormattingTags(object):
             u'end html': u'', u'protected': True, u'temporary': False})
         FormattingTags.add_html_tags(base_tags)
         FormattingTags.add_html_tags(temporary_tags)
-
         # Formatting Tags were also known as display tags.
-        user_format_json = Settings().value(u'formattingTags/html_tags_json')
-        # cPickle only accepts str not unicode strings
-        if user_format_json:
-            user_expands_string = str(Settings().value(u'formattingTags/html_tags'))
-        else:
-            user_expands_string = str(Settings().value(u'displayTags/html_tags'))
+        user_expands_string = str(Settings().value(u'formattingTags/html_tags'))
         if user_expands_string:
-            if user_format_json:
-                user_tags = json.loads(user_expands_string)
-            else:
-                user_tags = cPickle.loads(user_expands_string)
+            user_tags = json.loads(user_expands_string)
             for tag in user_tags:
                 for element in tag:
                     if isinstance(tag[element], str):
diff --git a/openlp/core/lib/settings.py b/openlp/core/lib/settings.py
index 48a3a72c1..a20e3e1a8 100644
--- a/openlp/core/lib/settings.py
+++ b/openlp/core/lib/settings.py
@@ -115,9 +115,7 @@ class Settings(QtCore.QSettings):
         u'advanced/single click preview': False,
         u'advanced/x11 bypass wm': X11_BYPASS_DEFAULT,
         u'crashreport/last directory': u'',
-        u'displayTags/html_tags': u'',
         u'formattingTags/html_tags': u'',
-        u'formattingTags/html_tags_json': False,
         u'core/audio repeat list': False,
         u'core/auto open': False,
         u'core/auto preview': False,

From 74432f93e27bca26645396b191d79009178e186a Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Sun, 30 Jun 2013 06:35:57 +0100
Subject: [PATCH 29/39] oops

---
 openlp/core/lib/imagemanager.py | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/openlp/core/lib/imagemanager.py b/openlp/core/lib/imagemanager.py
index f8af5d0a0..dc535a665 100644
--- a/openlp/core/lib/imagemanager.py
+++ b/openlp/core/lib/imagemanager.py
@@ -4,11 +4,7 @@
 
 ###############################################################################
 # OpenLP - Open Source Lyrics Projection                                      #
-
-
-
-
-    # --------------------------------------------------------------------------- #
+# --------------------------------------------------------------------------- #
 # Copyright (c) 2008-2013 Raoul Snyman                                        #
 # Portions copyright (c) 2008-2013 Tim Bentley, Gerald Britton, Jonathan      #
 # Corwin, Samuel Findlay, Michael Gorven, Scott Guerrieri, Matthias Hub,      #

From 5af820b7cd4596caaeac9bf4b399ab7c8dadc02a Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Sun, 30 Jun 2013 06:45:34 +0100
Subject: [PATCH 30/39] more cleanups

---
 openlp/core/lib/formattingtags.py |  1 -
 openlp/plugins/bibles/lib/http.py | 20 ++++++++------------
 2 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/openlp/core/lib/formattingtags.py b/openlp/core/lib/formattingtags.py
index 2dcae334f..be9cbe82e 100644
--- a/openlp/core/lib/formattingtags.py
+++ b/openlp/core/lib/formattingtags.py
@@ -68,7 +68,6 @@ class FormattingTags(object):
                         tag[element] = tag[element].encode('utf8')
         # Formatting Tags were also known as display tags.
         Settings().setValue(u'formattingTags/html_tags', json.dumps(tags) if tags else u'')
-        Settings().setValue(u'formattingTags/html_tags_json', True)
 
     @staticmethod
     def load_tags():
diff --git a/openlp/plugins/bibles/lib/http.py b/openlp/plugins/bibles/lib/http.py
index 624836860..de48b2617 100644
--- a/openlp/plugins/bibles/lib/http.py
+++ b/openlp/plugins/bibles/lib/http.py
@@ -367,8 +367,8 @@ class BSExtract(object):
             The version of the Bible like NIV for New International Version
         """
         log.debug(u'BSExtract.get_books_from_http("%s")', version)
-        urlversion = urllib.quote(version.encode("utf-8"))
-        chapter_url = u'http://m.bibleserver.com/overlay/selectBook?translation=%s' % (urlversion)
+        url_version = urllib.quote(version.encode("utf-8"))
+        chapter_url = u'http://m.bibleserver.com/overlay/selectBook?translation=%s' % (url_version)
         soup = get_soup_for_bible_ref(chapter_url)
         if not soup:
             return None
@@ -611,9 +611,8 @@ class HTTPBible(BibleDB):
                 if show_error:
                     critical_error_message_box(
                         translate('BiblesPlugin', 'No Book Found'),
-                        translate('BiblesPlugin', 'No matching '
-                        'book could be found in this Bible. Check that you '
-                        'have spelled the name of the book correctly.'))
+                        translate('BiblesPlugin', 'No matching book could be found in this Bible. Check that you have '
+                        'spelled the name of the book correctly.'))
                 return []
             book = db_book.name
             if BibleDB.get_verse_count(self, book_id, reference[1]) == 0:
@@ -742,13 +741,10 @@ def send_error_message(error_type):
     if error_type == u'download':
         critical_error_message_box(
             translate('BiblesPlugin.HTTPBible', 'Download Error'),
-            translate('BiblesPlugin.HTTPBible', 'There was a '
-            'problem downloading your verse selection. Please check your '
-            'Internet connection, and if this error continues to occur '
-            'please consider reporting a bug.'))
+            translate('BiblesPlugin.HTTPBible', 'There was a problem downloading your verse selection. Please check '
+                'your Internet connection, and if this error continues to occur please consider reporting a bug.'))
     elif error_type == u'parse':
         critical_error_message_box(
             translate('BiblesPlugin.HTTPBible', 'Parse Error'),
-            translate('BiblesPlugin.HTTPBible', 'There was a '
-            'problem extracting your verse selection. If this error continues '
-            'to occur please consider reporting a bug.'))
+            translate('BiblesPlugin.HTTPBible', 'There was a problem extracting your verse selection. If this error '
+                'continues to occur please consider reporting a bug.'))

From 8dd068b9c1beece6dfcea22abad0c270901445cc Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Sun, 30 Jun 2013 20:03:26 +0200
Subject: [PATCH 31/39] print func

---
 .../plugins/presentations/lib/pptviewlib/ppttest.py  | 12 ++++++------
 openlp/plugins/songs/lib/test/test_opensongimport.py | 10 +++++-----
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/openlp/plugins/presentations/lib/pptviewlib/ppttest.py b/openlp/plugins/presentations/lib/pptviewlib/ppttest.py
index c8573c807..f07468bb8 100644
--- a/openlp/plugins/presentations/lib/pptviewlib/ppttest.py
+++ b/openlp/plugins/presentations/lib/pptviewlib/ppttest.py
@@ -174,13 +174,13 @@ class PPTViewer(QtGui.QWidget):
             int(self.widthEdit.text()), int(self.heightEdit.text()))
         filename = str(self.pptEdit.text().replace(u'/', u'\\'))
         folder = str(self.folderEdit.text().replace(u'/', u'\\'))
-        print filename, folder
+        print(filename, folder)
         self.pptid = self.pptdll.OpenPPT(filename, None, rect, folder)
-        print u'id: ' + unicode(self.pptid)
+        print(u'id: ' + unicode(self.pptid))
         if oldid >= 0:
             self.pptdll.ClosePPT(oldid);
         slides = self.pptdll.GetSlideCount(self.pptid)
-        print u'slidecount: ' + unicode(slides)
+        print(u'slidecount: ' + unicode(slides))
         self.total.setNum(self.pptdll.GetSlideCount(self.pptid))
         self.updateCurrSlide()
 
@@ -188,14 +188,14 @@ class PPTViewer(QtGui.QWidget):
         if self.pptid < 0:
             return
         slide = unicode(self.pptdll.GetCurrentSlide(self.pptid))
-        print u'currslide: ' + slide
+        print(u'currslide: ' + slide)
         self.slideEdit.setText(slide)
         app.processEvents()
 
     def gotoClick(self):
         if self.pptid < 0:
             return
-        print self.slideEdit.text()
+        print(self.slideEdit.text())
         self.pptdll.GotoSlide(self.pptid, int(self.slideEdit.text()))
         self.updateCurrSlide()
         app.processEvents()
@@ -207,7 +207,7 @@ class PPTViewer(QtGui.QWidget):
 if __name__ == '__main__':
     pptdll = cdll.LoadLibrary(r'pptviewlib.dll')
     pptdll.SetDebug(1)
-    print u'Begin...'
+    print(u'Begin...')
     app = QtGui.QApplication(sys.argv)
     window = PPTViewer()
     window.pptdll = pptdll
diff --git a/openlp/plugins/songs/lib/test/test_opensongimport.py b/openlp/plugins/songs/lib/test/test_opensongimport.py
index 8f09ea875..3d367a166 100644
--- a/openlp/plugins/songs/lib/test/test_opensongimport.py
+++ b/openlp/plugins/songs/lib/test/test_opensongimport.py
@@ -89,7 +89,7 @@ def test():
     assert not [u'C3', u'Chorus 3'] in o.verses
     assert [u'V1', u'v1 Line 1\nV1 Line 2'] in o.verses
     assert [u'V2', u'v2 Line 1\nV2 Line 2'] in o.verses
-    print o.verse_order_list
+    print(o.verse_order_list)
     assert o.verse_order_list == [u'V1', u'C1', u'V2', u'C2', u'V3', u'B1', u'V1']
 
     o.filenames = [u'test2.opensong']
@@ -101,14 +101,14 @@ def test():
     assert o.title == u'Martins 2nd Test'
     assert o.alternate_title == u''
     assert o.song_number == u'2'
-    print o.verses
+    print(o.verses)
     assert [u'B1', u'Bridge 1\nBridge 1 line 2'] in o.verses
     assert [u'C1', u'Chorus 1'] in o.verses
     assert [u'C2', u'Chorus 2'] in o.verses
     assert not [u'C3', u'Chorus 3'] in o.verses
     assert [u'V1', u'v1 Line 1\nV1 Line 2'] in o.verses
     assert [u'V2', u'v2 Line 1\nV2 Line 2'] in o.verses
-    print o.verse_order_list
+    print(o.verse_order_list)
     assert o.verse_order_list == [u'V1', u'V2', u'B1', u'C1', u'C2']
 
     o.filenames = [u'test3.opensong']
@@ -122,10 +122,10 @@ def test():
     assert o.ccli_number == u'123456'
     assert o.verse_order_list == [u'V1']
     assert o.topics == [u'Worship: Declaration']
-    print o.verses[0]
+    print(o.verses[0])
     assert [u'V1', u'Line 1\nLine 2'] in o.verses
 
-    print "Tests passed"
+    print("Tests passed")
 
 if __name__ == "__main__":
     test()

From aa2df0c5d1d9c5beacf5ff4384f1e3d137e01f06 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Sun, 30 Jun 2013 20:08:50 +0200
Subject: [PATCH 32/39] removed test

---
 .../songs/lib/test/test_opensongimport.py     | 131 ------------------
 1 file changed, 131 deletions(-)
 delete mode 100644 openlp/plugins/songs/lib/test/test_opensongimport.py

diff --git a/openlp/plugins/songs/lib/test/test_opensongimport.py b/openlp/plugins/songs/lib/test/test_opensongimport.py
deleted file mode 100644
index 3d367a166..000000000
--- a/openlp/plugins/songs/lib/test/test_opensongimport.py
+++ /dev/null
@@ -1,131 +0,0 @@
-# -*- coding: utf-8 -*-
-# vim: autoindent shiftwidth=4 expandtab textwidth=80 tabstop=4 softtabstop=4
-
-###############################################################################
-# OpenLP - Open Source Lyrics Projection                                      #
-# --------------------------------------------------------------------------- #
-# Copyright (c) 2008-2013 Raoul Snyman                                        #
-# Portions copyright (c) 2008-2013 Tim Bentley, Gerald Britton, Jonathan      #
-# Corwin, Samuel Findlay, Michael Gorven, Scott Guerrieri, Matthias Hub,      #
-# Meinert Jordan, Armin Köhler, Erik Lundin, Edwin Lunando, Brian T. Meyer.   #
-# Joshua Miller, Stevan Pettit, Andreas Preikschat, Mattias Põldaru,          #
-# Christian Richter, Philip Ridout, Simon Scudder, Jeffrey Smith,             #
-# Maikel Stuivenberg, Martin Thompson, Jon Tibble, Dave Warnock,              #
-# Frode Woldsund, Martin Zibricky, Patrick Zimmermann                         #
-# --------------------------------------------------------------------------- #
-# This program is free software; you can redistribute it and/or modify it     #
-# under the terms of the GNU General Public License as published by the Free  #
-# Software Foundation; version 2 of the License.                              #
-#                                                                             #
-# This program is distributed in the hope that it will be useful, but WITHOUT #
-# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or       #
-# FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for    #
-# more details.                                                               #
-#                                                                             #
-# You should have received a copy of the GNU General Public License along     #
-# with this program; if not, write to the Free Software Foundation, Inc., 59  #
-# Temple Place, Suite 330, Boston, MA 02111-1307 USA                          #
-###############################################################################
-
-from openlp.plugins.songs.lib.opensongimport import OpenSongImport
-from openlp.core.lib.db import Manager
-from openlp.plugins.songs.lib.db import init_schema
-
-import logging
-LOG_FILENAME = 'test.log'
-logging.basicConfig(filename=LOG_FILENAME,level=logging.INFO)
-
-# Stubs to replace the UI functions for raw testing
-class wizard_stub:
-    def __init__(self):
-        self.progressBar=progbar_stub()
-    def incrementProgressBar(self, str):
-        pass
-class progbar_stub:
-    def __init__(self):
-        pass
-    def setMaximum(self, arg):
-        pass
-
-def test():
-    manager = Manager(u'songs', init_schema)
-    o = OpenSongImport(manager, filenames=[u'test.opensong'])
-    o.import_wizard = wizard_stub()
-    o.commit = False
-    o.do_import()
-    o.print_song()
-    assert o.copyright == u'2010 Martin Thompson'
-    assert o.authors == [u'MartiÑ Thómpson', u'Martin2 Thómpson']
-    assert o.title == u'Martins Test'
-    assert o.alternate_title == u''
-    assert o.song_number == u'1'
-    assert [u'C1', u'Chorus 1'] in o.verses
-    assert [u'C2', u'Chorus 2'] in o.verses
-    assert not [u'C3', u'Chorus 3'] in o.verses
-    assert [u'B1', u'Bridge 1\nBridge 1 line 2'] in o.verses
-    assert [u'V1', u'v1 Line 1\nV1 Line 2'] in o.verses
-    assert [u'V2', u'v2 Line 1\nV2 Line 2'] in o.verses
-    assert [u'V3A', u'V3 Line 1\nV3 Line 2'] in o.verses
-    assert [u'RAP1', u'Rap 1 Line 1\nRap 1 Line 2'] in o.verses
-    assert [u'RAP2', u'Rap 2 Line 1\nRap 2 Line 2'] in o.verses
-    assert [u'RAP3', u'Rap 3 Line 1\nRap 3 Line 2'] in o.verses
-    assert [u'X1', u'Unreferenced verse line 1'] in o.verses
-    assert o.verse_order_list == [u'V1', u'C1', u'V2', u'C2', u'V3A', u'B1', u'V1', u'T1', u'RAP1', u'RAP2', u'RAP3']
-    assert o.ccli_number == u'Blah'
-    assert o.topics == [u'TestTheme', u'TestAltTheme']
-
-    o.filenames = [u'test.opensong.zip']
-    o.set_defaults()
-    o.do_import()
-    o.print_song()
-    assert o.copyright == u'2010 Martin Thompson'
-    assert o.authors == [u'MartiÑ Thómpson']
-    assert o.title == u'Martins Test'
-    assert o.alternate_title == u''
-    assert o.song_number == u'1'
-    assert [u'B1', u'Bridge 1\nBridge 1 line 2'] in o.verses
-    assert [u'C1', u'Chorus 1'] in o.verses
-    assert [u'C2', u'Chorus 2'] in o.verses
-    assert not [u'C3', u'Chorus 3'] in o.verses
-    assert [u'V1', u'v1 Line 1\nV1 Line 2'] in o.verses
-    assert [u'V2', u'v2 Line 1\nV2 Line 2'] in o.verses
-    print(o.verse_order_list)
-    assert o.verse_order_list == [u'V1', u'C1', u'V2', u'C2', u'V3', u'B1', u'V1']
-
-    o.filenames = [u'test2.opensong']
-    o.set_defaults()
-    o.do_import()
-    o.print_song()
-    assert o.copyright == u'2010 Martin Thompson'
-    assert o.authors == [u'Martin Thompson']
-    assert o.title == u'Martins 2nd Test'
-    assert o.alternate_title == u''
-    assert o.song_number == u'2'
-    print(o.verses)
-    assert [u'B1', u'Bridge 1\nBridge 1 line 2'] in o.verses
-    assert [u'C1', u'Chorus 1'] in o.verses
-    assert [u'C2', u'Chorus 2'] in o.verses
-    assert not [u'C3', u'Chorus 3'] in o.verses
-    assert [u'V1', u'v1 Line 1\nV1 Line 2'] in o.verses
-    assert [u'V2', u'v2 Line 1\nV2 Line 2'] in o.verses
-    print(o.verse_order_list)
-    assert o.verse_order_list == [u'V1', u'V2', u'B1', u'C1', u'C2']
-
-    o.filenames = [u'test3.opensong']
-    o.set_defaults()
-    o.do_import()
-    o.print_song()
-    assert o.copyright == u'2010'
-    assert o.authors == [u'Martin Thompson']
-    assert o.title == u'Test single verse'
-    assert o.alternate_title == u''
-    assert o.ccli_number == u'123456'
-    assert o.verse_order_list == [u'V1']
-    assert o.topics == [u'Worship: Declaration']
-    print(o.verses[0])
-    assert [u'V1', u'Line 1\nLine 2'] in o.verses
-
-    print("Tests passed")
-
-if __name__ == "__main__":
-    test()

From 143ce0015f06f07e38db4d03d0b0f98389997975 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Sun, 30 Jun 2013 20:33:41 +0200
Subject: [PATCH 33/39] added __hash__ methods

---
 openlp/core/lib/serviceitem.py  | 6 ++++++
 openlp/plugins/custom/lib/db.py | 9 +++++++--
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/openlp/core/lib/serviceitem.py b/openlp/core/lib/serviceitem.py
index b32e1aaf0..ee1a5c588 100644
--- a/openlp/core/lib/serviceitem.py
+++ b/openlp/core/lib/serviceitem.py
@@ -485,6 +485,12 @@ class ServiceItem(object):
         """
         return self.unique_identifier != other.unique_identifier
 
+    def __hash__(self):
+        """
+        Return the hash for the service item.
+        """
+        return self.unique_identifier
+
     def is_media(self):
         """
         Confirms if the ServiceItem is media
diff --git a/openlp/plugins/custom/lib/db.py b/openlp/plugins/custom/lib/db.py
index 253ca5432..2f6eb5d6d 100644
--- a/openlp/plugins/custom/lib/db.py
+++ b/openlp/plugins/custom/lib/db.py
@@ -41,14 +41,19 @@ class CustomSlide(BaseModel):
     """
     CustomSlide model
     """
-    # By default sort the customs by its title considering language specific
-    # characters.
+    # By default sort the customs by its title considering language specific characters.
     def __lt__(self, other):
         return get_locale_key(self.title) < get_locale_key(other.title)
 
     def __eq__(self, other):
         return get_locale_key(self.title) == get_locale_key(other.title)
 
+    def __hash__(self):
+        """
+        Return the hash for a custom slide.
+        """
+        return self.id
+
 
 def init_schema(url):
     """

From 1d571507dbc08edefd9390bfad13292cad8ace71 Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Sun, 30 Jun 2013 19:36:52 +0100
Subject: [PATCH 34/39] remove import

---
 openlp/core/lib/formattingtags.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/openlp/core/lib/formattingtags.py b/openlp/core/lib/formattingtags.py
index be9cbe82e..1984b08ba 100644
--- a/openlp/core/lib/formattingtags.py
+++ b/openlp/core/lib/formattingtags.py
@@ -29,7 +29,6 @@
 """
 Provide HTML Tag management and Formatting Tag access class
 """
-import cPickle
 import json
 
 from openlp.core.lib import Settings, translate

From 72933138a29b5e8c1ab8383f25e133740e8da793 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <googol@lavabit.com>
Date: Sun, 30 Jun 2013 20:37:12 +0200
Subject: [PATCH 35/39] clean up

---
 openlp/plugins/custom/forms/editcustomform.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/openlp/plugins/custom/forms/editcustomform.py b/openlp/plugins/custom/forms/editcustomform.py
index 580fbde07..1009ea8d3 100644
--- a/openlp/plugins/custom/forms/editcustomform.py
+++ b/openlp/plugins/custom/forms/editcustomform.py
@@ -100,8 +100,7 @@ class EditCustomForm(QtGui.QDialog, Ui_CustomEditDialog):
             self.credit_edit.setText(self.custom_slide.credits)
             custom_XML = CustomXMLParser(self.custom_slide.text)
             slide_list = custom_XML.get_verses()
-            for slide in slide_list:
-                self.slide_list_view.addItem(slide[1])
+            self.slide_list_view.addItems([slide[1] for slide in slide_list])
             theme = self.custom_slide.theme_name
             find_and_set_in_combo_box(self.theme_combo_box, theme)
         self.title_edit.setFocus()

From f583be0c76725a0e7d2b884ab2108f52a672a066 Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Sun, 30 Jun 2013 19:39:38 +0100
Subject: [PATCH 36/39] fix registry call

---
 openlp/core/ui/mainwindow.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openlp/core/ui/mainwindow.py b/openlp/core/ui/mainwindow.py
index ffc0ebf16..c14a50d69 100644
--- a/openlp/core/ui/mainwindow.py
+++ b/openlp/core/ui/mainwindow.py
@@ -1066,7 +1066,7 @@ class MainWindow(QtGui.QMainWindow, Ui_MainWindow):
             self.live_controller.display = None
         if os.name == u'nt':
             # Needed for Windows to stop crashes on exit
-            Registry.remove(u'application')
+            Registry().remove(u'application')
 
     def service_changed(self, reset=False, serviceName=None):
         """

From 538e885cfb3e02d04e2ff2c18fc1f8099bcbf5fc Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Wed, 3 Jul 2013 17:34:02 +0100
Subject: [PATCH 38/39] Fix broken test

---
 tests/functional/openlp_core_lib/test_serviceitem.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/functional/openlp_core_lib/test_serviceitem.py b/tests/functional/openlp_core_lib/test_serviceitem.py
index 4e777b953..f970eba22 100644
--- a/tests/functional/openlp_core_lib/test_serviceitem.py
+++ b/tests/functional/openlp_core_lib/test_serviceitem.py
@@ -272,13 +272,13 @@ class TestServiceItem(TestCase):
         service_item.add_icon = MagicMock()
 
         # WHEN: adding an media from a saved Service and mocked exists
-        line = self.convert_file_service_item(u'migrate_video_20_22.osd')
+        line = read_service_from_file(u'migrate_video_20_22.osd')
         with patch('os.path.exists'):
-            service_item.set_from_service(line, TEST_PATH)
+            service_item.set_from_service(line[0], TEST_RESOURCES_PATH)
 
         # THEN: We should get back a converted service item
         assert service_item.is_valid is True, u'The new service item should be valid'
-        assert service_item.processor is None, u'The Processor should have been set'
-        assert service_item.title is None, u'The title should be set to a value'
+        assert service_item.processor is not None, u'The Processor should have been set'
+        assert service_item.title is not None, u'The title should be set to a value'
         assert service_item.is_capable(ItemCapabilities.HasDetailedTitleDisplay) is False, \
             u'The Capability should have been removed'

From ab3fd38551044f4530eafb2dda88b54dbca23522 Mon Sep 17 00:00:00 2001
From: Tim Bentley <tim.bentley@gmail.com>
Date: Wed, 3 Jul 2013 17:34:40 +0100
Subject: [PATCH 39/39] Fix broken test

---
 tests/functional/openlp_core_lib/test_serviceitem.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/functional/openlp_core_lib/test_serviceitem.py b/tests/functional/openlp_core_lib/test_serviceitem.py
index f970eba22..fdeb081b3 100644
--- a/tests/functional/openlp_core_lib/test_serviceitem.py
+++ b/tests/functional/openlp_core_lib/test_serviceitem.py
@@ -278,7 +278,7 @@ class TestServiceItem(TestCase):
 
         # THEN: We should get back a converted service item
         assert service_item.is_valid is True, u'The new service item should be valid'
-        assert service_item.processor is not None, u'The Processor should have been set'
+        assert service_item.processor == u'VLC', u'The Processor should have been set'
         assert service_item.title is not None, u'The title should be set to a value'
         assert service_item.is_capable(ItemCapabilities.HasDetailedTitleDisplay) is False, \
             u'The Capability should have been removed'