From 507934e85ceed0ce2ca0fbfaddeb135f2129cb2b Mon Sep 17 00:00:00 2001
From: Tomas Groth <second@tgc.dk>
Date: Wed, 4 Dec 2019 22:10:13 +0000
Subject: [PATCH] Disable BibleServer support for now since we haven't yet
 figured out how to extract available translations and books in translations.

---
 .../plugins/bibles/forms/bibleimportform.py   | 20 +++++++++++--------
 .../openlp_plugins/bibles/test_lib_http.py    |  3 ++-
 2 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/openlp/plugins/bibles/forms/bibleimportform.py b/openlp/plugins/bibles/forms/bibleimportform.py
index dcc7d93e0..62973c1d3 100644
--- a/openlp/plugins/bibles/forms/bibleimportform.py
+++ b/openlp/plugins/bibles/forms/bibleimportform.py
@@ -43,7 +43,7 @@ from openlp.core.lib.ui import critical_error_message_box
 from openlp.core.widgets.edits import PathEdit
 from openlp.core.widgets.wizard import OpenLPWizard, WizardStrings
 from openlp.plugins.bibles.lib.db import clean_filename
-from openlp.plugins.bibles.lib.importers.http import BGExtract, BSExtract, CWExtract
+from openlp.plugins.bibles.lib.importers.http import BGExtract, CWExtract
 from openlp.plugins.bibles.lib.manager import BibleFormat
 
 
@@ -59,7 +59,8 @@ class WebDownload(object):
     BibleGateway = 1
     Bibleserver = 2
 
-    Names = ['Crosswalk', 'BibleGateway', 'Bibleserver']
+    # NOTE: BibleServer support has been disabled since we can't currently parse it. Re-add if/when fixed.
+    Names = ['Crosswalk', 'BibleGateway']
 
 
 class BibleImportForm(OpenLPWizard):
@@ -227,7 +228,8 @@ class BibleImportForm(OpenLPWizard):
         self.web_bible_layout.setWidget(1, QtWidgets.QFormLayout.LabelRole, self.web_source_label)
         self.web_source_combo_box = QtWidgets.QComboBox(self.web_widget)
         self.web_source_combo_box.setObjectName('WebSourceComboBox')
-        self.web_source_combo_box.addItems(['', '', ''])
+        # NOTE: Set to 2 items since BibleServer has been disabled. Set to 3 if/when fixed
+        self.web_source_combo_box.addItems(['', ''])
         self.web_source_combo_box.setEnabled(False)
         self.web_bible_layout.setWidget(1, QtWidgets.QFormLayout.FieldRole, self.web_source_combo_box)
         self.web_translation_label = QtWidgets.QLabel(self.web_bible_tab)
@@ -239,7 +241,8 @@ class BibleImportForm(OpenLPWizard):
         self.web_translation_combo_box.setEnabled(False)
         self.web_bible_layout.setWidget(2, QtWidgets.QFormLayout.FieldRole, self.web_translation_combo_box)
         self.web_progress_bar = QtWidgets.QProgressBar(self)
-        self.web_progress_bar.setRange(0, 3)
+        # NOTE: Set to 2 since BibleServer has been disabled. Set to 3 if/when fixed
+        self.web_progress_bar.setRange(0, 2)
         self.web_progress_bar.setObjectName('WebTranslationProgressBar')
         self.web_progress_bar.setVisible(False)
         self.web_bible_layout.setWidget(3, QtWidgets.QFormLayout.SpanningRole, self.web_progress_bar)
@@ -400,8 +403,9 @@ class BibleImportForm(OpenLPWizard):
                                                                                'Crosswalk'))
         self.web_source_combo_box.setItemText(WebDownload.BibleGateway, translate('BiblesPlugin.ImportWizardForm',
                                                                                   'BibleGateway'))
-        self.web_source_combo_box.setItemText(WebDownload.Bibleserver, translate('BiblesPlugin.ImportWizardForm',
-                                                                                 'Bibleserver'))
+        # NOTE: BibleServer support has been disabled since we can't currently parse it. Re-add if/when fixed.
+        # self.web_source_combo_box.setItemText(WebDownload.Bibleserver, translate('BiblesPlugin.ImportWizardForm',
+        #                                                                         'Bibleserver'))
         self.web_translation_label.setText(translate('BiblesPlugin.ImportWizardForm', 'Bible:'))
         self.sword_bible_label.setText(translate('BiblesPlugin.ImportWizardForm', 'Bibles:'))
         self.sword_folder_label.setText(translate('BiblesPlugin.ImportWizardForm', 'SWORD data folder:'))
@@ -578,9 +582,9 @@ class BibleImportForm(OpenLPWizard):
         self.web_progress_bar.setVisible(True)
         self.web_progress_bar.setValue(0)
         # TODO: Where does critical_error_message_box get %s string from?
+        # NOTE: BibleServer support has been disabled since we can't currently parse it. Re-add if/when fixed.
         for (download_type, extractor) in ((WebDownload.Crosswalk, CWExtract()),
-                                           (WebDownload.BibleGateway, BGExtract()),
-                                           (WebDownload.Bibleserver, BSExtract())):
+                                           (WebDownload.BibleGateway, BGExtract())):
             try:
                 bibles = extractor.get_bibles_from_http()
             except (urllib.error.URLError, ConnectionError):
diff --git a/tests/interfaces/openlp_plugins/bibles/test_lib_http.py b/tests/interfaces/openlp_plugins/bibles/test_lib_http.py
index dd090c526..977aae31f 100644
--- a/tests/interfaces/openlp_plugins/bibles/test_lib_http.py
+++ b/tests/interfaces/openlp_plugins/bibles/test_lib_http.py
@@ -22,7 +22,7 @@
     Package to test the openlp.plugin.bible.lib.https package.
 """
 import os
-from unittest import TestCase, skipIf
+from unittest import TestCase, skipIf, skip
 from unittest.mock import MagicMock
 
 from openlp.core.common.registry import Registry
@@ -122,6 +122,7 @@ class TestBibleHTTP(TestCase):
         # THEN: We should get back a valid service item
         assert len(results.verse_list) == 36, 'The book of John should not have had any verses added or removed'
 
+    @skip("We can't currently parse BibelServer")
     def test_bibleserver_get_bibles(self):
         """
         Test getting list of bibles from BibleServer.com