openlp-core team mailing list archive

[Phill <phill.ridout@xxxxxxxxxxxxxx>]

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

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/+merge/151620

Reraise the error so that we can catch the original cause and hopefully solve some bugs.
-- 
https://code.launchpad.net/~phill-ridout/openlp/bug1135392/+merge/151620
Your team OpenLP Core is requested to review the proposed merge of lp:~phill-ridout/openlp/bug1135392 into lp:openlp.

=== modified file 'openlp/plugins/songs/forms/editsongform.py'
--- openlp/plugins/songs/forms/editsongform.py	2013-02-24 18:13:50 +0000
+++ openlp/plugins/songs/forms/editsongform.py	2013-03-04 21:20:36 +0000
@@ -923,6 +923,7 @@
                     self.song.verse_order)
         except:
             log.exception(u'Problem processing song Lyrics \n%s', sxml.dump_xml())
+            raise
 
     def _get_plugin_manager(self):
         """