Merge lp:~googol-deactivatedaccount/openlp/bug-789143 into lp:openlp
Status: | Merged |
---|---|
Merged at revision: | 1592 |
Proposed branch: | lp:~googol-deactivatedaccount/openlp/bug-789143 |
Merge into: | lp:openlp |
Diff against target: |
106 lines (+50/-45) 1 file modified
openlp/plugins/songs/lib/__init__.py (+50/-45) |
To merge this branch: | bzr merge lp:~googol-deactivatedaccount/openlp/bug-789143 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jonathan Corwin (community) | Approve | ||
Raoul Snyman | Approve | ||
Tim Bentley | Pending | ||
Review via email: mp+62790@code.launchpad.net |
This proposal supersedes a proposal from 2011-05-27.
Description of the change
Hello,
- "fixed" bug #789143
The "bug" lies in from_loose_input:
verse_index = None
if len(verse_name) > 1:
if verse_index is None:
if verse_index is None:
if verse_index is None:
return verse_index
If we pass "e" (Ending) to the method it first tries from_translated_tag (which is wrong). It seems that this causes problems when this method is used with English tags (but using a translated GUI).
A bit worried about removing so much code, but your testing seems to have been thorough!