Code review comment for lp:~trb143/openlp/bug-772523

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

This isn't all your code, but I did just need to point it out...

    editId, self.loadItem_uuid, temporary = message.split(u':')
    self.loadItem_editId = int(editId)
    self.loadItem_temporary = str_to_bool(temporary)

Um, two naming conventions in the same variable? It should either be self.load_item_edit_id or self.loadItemEditId. Or if you need a bunch of related things, what about a dictionary?

Also, is that first line backwards compatible with existing service files?

review: Needs Fixing

« Back to merge proposal