Review: Needs Fixing 29 + for tryCount in range(3): 44 + if tryCount >= 2: 81 + for tryCount in range(3): 96 + if tryCount >= 2: 121 + for tryCount in range(3): 131 + if tryCount >= 2: 149 + for tryCount in range(3): 159 + if tryCount >= 2: 177 + for tryCount in range(3): 187 + if tryCount >= 2: 205 + for tryCount in range(3): 215 + if tryCount >= 2: 243 + for tryCount in range(3): 257 + if tryCount >= 2: 298 + for tryCount in range(3): 315 + if tryCount >= 2:
That should be "try_count". Local variables should use the PEP8 naming convention/syntax. -- https://code.launchpad.net/~dave-warnock/openlp/dbclean/+merge/109546 Your team OpenLP Core is subscribed to branch lp:openlp. _______________________________________________ Mailing list: https://launchpad.net/~openlp-core Post to : openlp-core@lists.launchpad.net Unsubscribe : https://launchpad.net/~openlp-core More help : https://help.launchpad.net/ListHelp