openlp-core team mailing list archive

[Raoul Snyman <raoul.snyman@xxxxxxxxxxxxxxxxxxxxxxxx>]

849	+ if not self.edit:
850	+     if not self.thememanager.checkIfThemeExists(self.theme.theme_name):
851	+         return

I'm not too sure what's going on here, and I'm worried that something is going to slip through the cracks... what happens if the first statement is true, but the second is false? Should it just carry on? If they are meant to be used together, why not ``and`` them?


-- 
https://code.launchpad.net/~trb143/openlp/next/+merge/45641
Your team OpenLP Core is subscribed to branch lp:openlp.