Ooops... I resubmitted because of a change in my code (not as a response to 
you).

> Although wondering what verseId on line 162 did to upset you :)

Local variables use underscore_variable_names

>Rather than splitting the title if it contains any \n's (which will cause a 
>blank title if there is 
> a leading blank line), how about replacing them with a space and left 
> trimming?

That's another story, I was just refactoring.......
-- 
https://code.launchpad.net/~googol/openlp/trivial/+merge/110664
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to