Merge lp:~trb143/openlp/bug-772523 into lp:openlp
Status: | Merged |
---|---|
Merged at revision: | 1833 |
Proposed branch: | lp:~trb143/openlp/bug-772523 |
Merge into: | lp:openlp |
Diff against target: |
349 lines (+91/-21) 11 files modified
openlp/core/lib/db.py (+8/-1) openlp/core/lib/eventreceiver.py (+4/-0) openlp/core/lib/serviceitem.py (+2/-0) openlp/core/ui/maindisplay.py (+2/-2) openlp/core/ui/servicemanager.py (+25/-6) openlp/plugins/songs/forms/songexportform.py (+3/-0) openlp/plugins/songs/lib/db.py (+2/-1) openlp/plugins/songs/lib/mediaitem.py (+15/-3) openlp/plugins/songs/lib/upgrade.py (+12/-1) openlp/plugins/songs/lib/xml.py (+6/-7) openlp/plugins/songs/songsplugin.py (+12/-0) |
To merge this branch: | bzr merge lp:~trb143/openlp/bug-772523 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Raoul Snyman | Approve | ||
Andreas Preikschat | Pending | ||
Review via email: mp+85370@code.launchpad.net |
This proposal supersedes a proposal from 2011-12-12.
Description of the change
This patch changes your song database so back it up!
When a service is imported at present and we do not wish to save the songs we do not. This prevents the user from editing songs.
This change adds a "temporary" flag on the song database to allow those songs to be save and edited bit not via the plugin as the search will not show them.
On exit the songs will be deleted.
The code works until a service is loaded and then you get sqlalchemy errors.
I have no idea what is wrong but some help would be appreciated.
Amended code to add songs as per my email.
The code does set a correct value on existing databases.
Well, first remove your print statement.