Review: Needs Fixing
Just one thing...

42      if self.author == None:

should read:

42      if self.author is None:

There is a small, but significant, difference between == and is when it comes 
to None.
-- 
https://code.launchpad.net/~thealok/openlp/authorform/+merge/9118
Your team openlp.org Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to