← Back to team overview

openlp-core team mailing list archive

[Merge] lp:~phill-ridout/openlp/1184869 into lp:openlp

 

Phill has proposed merging lp:~phill-ridout/openlp/1184869 into lp:openlp.

Requested reviews:
  OpenLP Core (openlp-core)
Related bugs:
  Bug #1184869 in OpenLP: "Webbible import fails"
  https://bugs.launchpad.net/openlp/+bug/1184869

For more details, see:
https://code.launchpad.net/~phill-ridout/openlp/1184869/+merge/178394

Fixes #1184869 Ensures that there is text in the anchor tag
-- 
https://code.launchpad.net/~phill-ridout/openlp/1184869/+merge/178394
Your team OpenLP Core is requested to review the proposed merge of lp:~phill-ridout/openlp/1184869 into lp:openlp.
=== modified file 'openlp/plugins/bibles/lib/http.py'
--- openlp/plugins/bibles/lib/http.py	2013-06-30 05:45:34 +0000
+++ openlp/plugins/bibles/lib/http.py	2013-08-02 21:55:35 +0000
@@ -378,7 +378,7 @@
             send_error_message(u'parse')
             return None
         content = content.find_all(u'li')
-        return [book.contents[0].contents[0] for book in content]
+        return [book.contents[0].contents[0] for book in content if len(book.contents[0].contents)]
 
     def _get_application(self):
         """


Follow ups