Merge lp://qastaging/~trb143/openlp/reporting into lp://qastaging/openlp
Status: | Superseded | ||||
---|---|---|---|---|---|
Proposed branch: | lp://qastaging/~trb143/openlp/reporting | ||||
Merge into: | lp://qastaging/openlp | ||||
Diff against target: |
411 lines (+215/-29) 7 files modified
openlp/core/common/settings.py (+1/-0) openlp/plugins/custom/lib/mediaitem.py (+1/-1) openlp/plugins/songs/forms/editsongform.py (+1/-1) openlp/plugins/songs/lib/songcompare.py (+3/-3) openlp/plugins/songs/reporting.py (+102/-0) openlp/plugins/songs/songsplugin.py (+27/-8) tests/functional/openlp_core_ui/test_servicemanager.py (+80/-16) |
||||
To merge this branch: | bzr merge lp://qastaging/~trb143/openlp/reporting | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Raoul Snyman | Pending | ||
Review via email:
|
This proposal supersedes a proposal from 2016-09-08.
This proposal has been superseded by a proposal from 2016-09-21.
Description of the change
My dad needed a report of all the songs on their database, they had 1800.
Made this into a reporting option and cleaned up the menu.
Fixed some errors spotted as well
Fixed issues and comments
1800 songs takes about 3 secs to run on my i7
lp:~trb143/openlp/reporting (revision 2699)
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
1. There's a "csv" module, use it.
2. I should be able to specify the file name, not just where to save it to.
3. Have you tested this with > 1000 songs? How long does it take? Some sort of progress window necessary?
4. You call it a report internally, but you're not very specific for the user. Rather call it a "Song List Report".
More comments inline.