Review: Needs Fixing I think line 41 needs to be an "if not windows and if not osx" check, rather than an if linux. (FreeBSD etc)
However is the if check needed at all. What happens if you create default_translator regardless of system, and app.installTranslator is given a folder that doesn't exist? Does it crash? (Note I agree the system languages will still need to be part of the install package on Windows/OSX) Also as it stands currently on Windows/OSX it will do an app.installTranslator on the same path twice. Is this necessary? -- https://code.launchpad.net/~googol-hush/openlp/fixes2/+merge/59681 Your team OpenLP 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
