OpenSong importer crashes at v1a verse markup

[mahfiaz]

Importing the following OpenSong song fails:
[V1a]
Suur on su ustavus
[V1b]
suur on su ustavus

Such markup is totally valid for OpenSong:
http://sourceforge.net/tracker/?func=detail&aid=3459158&group_id=108754&atid=651379

The original report is at:
http://support.openlp.org/issues/347

Traceback (most recent call last):
  File "/home/mattias/Arendus/OpenLP/openlp/openlp/core/ui/wizard.py", line 191, in onCurrentIdChanged
    self.performWizard()
  File "/home/mattias/Arendus/OpenLP/openlp/openlp/plugins/songs/forms/songimportform.py", line 771, in performWizard
    importer.doImport()
  File "/home/mattias/Arendus/OpenLP/openlp/openlp/plugins/songs/lib/opensongimport.py", line 116, in doImport
    self.doImportFile(song_file)
  File "/home/mattias/Arendus/OpenLP/openlp/openlp/plugins/songs/lib/opensongimport.py", line 226, in doImportFile
    self.addVerse(lines, verse_def)
  File "/home/mattias/Arendus/OpenLP/openlp/openlp/plugins/songs/lib/songimport.py", line 256, in addVerse
    elif int(verse_def[1:]) > self.verseCounts[verse_def[0]]:
ValueError: invalid literal for int() with base 10: '1a'

[Jonathan Corwin (j-corwin)]

Also reported here: http://openlp.org/en/forums/general-support/troubleshooting/mac-osx-issues#comment-4324

[Phill (phill-ridout)]

Other variations on this bug:
 http://support.openlp.org/issues/492
 http://support.openlp.org/issues/475
 http://support.openlp.org/issues/441

[Jonathan Corwin (j-corwin)]

Wondering who linked this to related branch lp:openlp?
If we do link, don't we normally link to the branch it was fixed in e.g. lp:~m2j/openlp/bug-953240 ?

[Phill (phill-ridout)]

I've just spotted this bug. Has this made it in to trunk? If so, I still get these issues! My branch https://code.launchpad.net/~phill-ridout/openlp/opensong_fix/+merge/110890 deals with this.