Code review comment for lp:~trb143/openlp/ThemeManager2

Revision history for this message
Michael Gorven (mgorven) wrote :

+ 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

review: Approve

« Back to merge proposal