OpenLP

SongShowPlus Import fails when CCLI number has text in it

Bug #1310623 reported by Samuel Mehrbrodt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenLP
Status tracked in Trunk
2.0
Won't Fix
Medium
Unassigned
Trunk
Fix Released
Medium
Samuel Mehrbrodt
OpenLP 2.1.2

Bug Description

From http://support.openlp.org/scp/tickets.php?id=2314

We need to clear the text from the string before converting to int.

Traceback (most recent call last):
File "/usr/share/openlp/openlp/core/ui/wizard.py", line 188, in
onCurrentIdChanged
self.performWizard()
File "/usr/share/openlp/openlp/plugins/songs/forms/songimportform.py",
line 391, in performWizard
importer.doImport()
File "/usr/share/openlp/openlp/plugins/songs/lib/songshowplusimport.py",
line 155, in doImport
self.ccliNumber = int(data)
ValueError: invalid literal for int() with base 10: 'CCL Licence number:
129265'

Related branches

lp:~sam92/openlp/bug-1310623
Tim Bentley: Approve
Raoul Snyman: Approve
Andreas Preikschat: Pending requested
Diff: 78 lines (+40/-2)
3 files modified
openlp/plugins/songs/lib/songshowplusimport.py (+7/-2)
tests/functional/openlp_plugins/songs/test_songshowplusimport.py (+2/-0)
tests/resources/songshowplussongs/a mighty fortress is our god.json (+31/-0)
Samuel Mehrbrodt (sam92)
Changed in openlp:
importance: Undecided → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.