Review: Needs Fixing
708     + searchPlugins: function (event) {

Does this method actually perform the search, or just return a list of 
searchable plugins? If the latter, please rename it to something like 
"getSearchablePlugins".

799     + def plugin(self, action):

Possibly rename this one too?
-- 
https://code.launchpad.net/~j-corwin/openlp/general/+merge/61047
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

Reply via email to