openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #19755
[Bug 1135392] Re: Catch all error trap in editsongform.py, _processLyrics
** Changed in: openlp/2.0
Assignee: (unassigned) => Phill (phill-ridout)
** Changed in: openlp/trunk
Assignee: (unassigned) => Phill (phill-ridout)
--
You received this bug notification because you are a member of OpenLP
Core, which is subscribed to OpenLP.
https://bugs.launchpad.net/bugs/1135392
Title:
Catch all error trap in editsongform.py, _processLyrics
Status in OpenLP - Worship Presentation Software:
In Progress
Status in OpenLP 2.0 series:
In Progress
Status in OpenLP trunk series:
In Progress
Bug description:
Hi,
I have been investigating a bug that has been reported on the support
tracker (http://support.openlp.org/issues/1906) and I have traced it
back to http://bazaar.launchpad.net/~openlp-
core/openlp/release-2.0/view/head:/openlp/plugins/songs/forms/editsongform.py#L941
I have seen reports like this before on the support tracker, but have
only just been able to narrow it down thanks to this comment:
https://bugs.launchpad.net/openlp/+bug/1051890/comments/4 by Jonathan
It appears that this catch all maybe responsible for us being unable
to fix several different bugs.
So as Jonathan says:
" We shouldn't ignore/hide these errors, and ideally want to fix the original cause."
"Either that or re-throw the error"
I've created this separate report as Jonathan's comments were kind of
hidden, but to me seem quite important.
To manage notifications about this bug go to:
https://bugs.launchpad.net/openlp/+bug/1135392/+subscriptions
References