← Back to team overview

openlp-core team mailing list archive

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

 

Review: Approve
+        log.debug(u'Handle event called with event %s' %event.event_type)

Pass event.event_type as a parameter.

+                if os.path.splitext (file) [1].lower ()  in [u'.xml']:

I'd prefer:
    if os.path.splitext(file)[1].lower() in (u'xml',):

 review approve

-- 
https://code.launchpad.net/~trb143/openlp/ThemeManager2/+merge/5302
Your team openlp.org Core is subscribed to branch lp:openlp.



References