openlp-core team mailing list archive

[Raoul Snyman <raoul.snyman@xxxxxxxxxxxxxxxxxxxxxxxx>]

Review: Needs Fixing
Shouldn't you use "os.path.exists" rather than "os.path.isfile"?

You've commwnted out a whole section of code that looks quite important. Did you mean to do this? If it should not be there, please remove the comments.
-- 
https://code.launchpad.net/~smpettit/openlp/save_service/+merge/70356
Your team OpenLP Core is subscribed to branch lp:openlp.