openlp-core team mailing list archive

[Phill <phill.ridout@xxxxxxxxxxxxxx>]

Phill has proposed merging lp:~phill-ridout/openlp/bug1135392_2.0 into lp:openlp/2.0.

Requested reviews:
  OpenLP Core (openlp-core)
Related bugs:
  Bug #1135392 in OpenLP: "Catch all error trap in editsongform.py, _processLyrics"
  https://bugs.launchpad.net/openlp/+bug/1135392

For more details, see:
https://code.launchpad.net/~phill-ridout/openlp/bug1135392_2.0/+merge/151622

Reraise the error so that we can catch the original cause and hopefully solve some bugs.

Propose for the correct target for this time.
-- 
https://code.launchpad.net/~phill-ridout/openlp/bug1135392_2.0/+merge/151622
Your team OpenLP Core is requested to review the proposed merge of lp:~phill-ridout/openlp/bug1135392_2.0 into lp:openlp/2.0.

=== modified file 'openlp/plugins/songs/forms/editsongform.py'
--- openlp/plugins/songs/forms/editsongform.py	2013-02-11 08:37:16 +0000
+++ openlp/plugins/songs/forms/editsongform.py	2013-03-04 21:22:34 +0000
@@ -958,4 +958,5 @@
         except:
             log.exception(u'Problem processing song Lyrics \n%s',
                 sxml.dump_xml())
+            raise