SongShowPlus Import fails when CCLI number has text in it
Bug #1310623 reported by
Samuel Mehrbrodt
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 |
Bug Description
From http://
We need to clear the text from the string before converting to int.
Traceback (most recent call last):
File "/usr/share/
onCurrentIdChanged
self.performWiz
File "/usr/share/
line 391, in performWizard
importer.doImport()
File "/usr/share/
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 modifiedopenlp/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)
Changed in openlp: | |
importance: | Undecided → Medium |
To post a comment you must log in.