← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~trb143/openlp/audit into lp:openlp

 

I haven't looked too closely, but it looks like you're outputting to a comma separated list.
E.g. 
openlp/plugins/songs/lib/mediaitem.py
  service_item.audit = u'%s,%s,%s' % (service_item.title, author_list, ccl) 

I expect the title and author_list could potentially contain commas also, so it could probably do with quoting the fields, something like the following, unless this is handled elsewhere?

service_item.audit = u'"%s","%s",%s' % (service_item.title.replace(u'"', u'""'), author_list.replace(u'"', u'""'), ccl) 
-- 
https://code.launchpad.net/~trb143/openlp/audit/+merge/12043
Your team openlp.org Core is subscribed to branch lp:openlp.



References