openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #22744
Re: [Merge] lp:~tomasgroth/openlp/better-remote into lp:openlp
Review: Needs Fixing
23 - return Settings.__default_settings__[key]
24 + return Settings.__default_settings__.get(key, '')
The reason line 23 (and all the rest like them) was not working was because Felipe was doing it wrong. That line MUST fail if you're writing your code wrong.
What you need to do is to specify a list of DEFAULT SETTINGS in the __default_settings__ dictionary in <blah>plugin.py. See songsplugin.py for an example.
--
https://code.launchpad.net/~tomasgroth/openlp/better-remote/+merge/211631
Your team OpenLP Core is subscribed to branch lp:openlp.
References