← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~smpettit/openlp/ftw into lp:openlp

 

> Why do we need 91-100?
> A comment would be helpful as I am confused.  That code is for version updates
> and will have run at start up so will not be relevant if FTW runs again.

91-100 handles the changes to the "active" plugins that can be made from the wizard.  

With the re-running of the wizard, the user can change which plugins they want.  The code here is similar to the "Active/Inactive" code that is pluginform.onStatusComboBoxChanged with the inclusion of the For loop to check all plugins.

I can add comments if you want?

Added note:  NO songs are deleted from the songs.db with the re-running of the wizard.  Selected songs, themes and bibles are merged.  (I have tested numerous times).
-- 
https://code.launchpad.net/~smpettit/openlp/ftw/+merge/68968
Your team OpenLP Core is subscribed to branch lp:openlp.


References