← Back to team overview

openlp-core team mailing list archive

[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