openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #02525
Re: [Merge] lp:~mjthompson/openlp/opensong_import into lp:openlp
Fix __init__.py as you're no longer renaming the file
Have you come across a problem requiring lines 305/306? If not the DB returning None should suffice, if so then should the test probably be applied to song_book and author too?
Line 307 shouldn't have the carriage return removed - makes it too long.
Still don't need spaces around "=" when passing arguments and check 948 isn't now too long.
--
https://code.launchpad.net/~mjthompson/openlp/opensong_import/+merge/30047
Your team OpenLP Core is subscribed to branch lp:openlp.
Follow ups
References