Merge lp:~sam92/openlp/bug-1552563 into lp:openlp
Proposed by
Samuel Mehrbrodt
Status: | Merged |
---|---|
Approved by: | Tim Bentley |
Approved revision: | 2659 |
Merged at revision: | 2655 |
Proposed branch: | lp:~sam92/openlp/bug-1552563 |
Merge into: | lp:openlp |
Diff against target: |
144 lines (+39/-37) 3 files modified
openlp/plugins/songs/lib/mediaitem.py (+10/-12) tests/functional/openlp_plugins/songs/test_mediaitem.py (+29/-24) tests/functional/openlp_plugins/songs/test_openlpimporter.py (+0/-1) |
To merge this branch: | bzr merge lp:~sam92/openlp/bug-1552563 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Bentley | Approve | ||
Raoul Snyman | Approve | ||
Review via email: mp+293427@code.launchpad.net |
This proposal supersedes a proposal from 2016-04-29.
Description of the change
Fix performance regression with Songbook search
The problem was that in each iteration the database was accessed (the song object).
Fixed this by loading all neccessary information directly in the query.
To post a comment you must log in.
lp:~sam92/openlp/bug-1552563 (revision 2659) /ci.openlp. io/job/ Branch- 01-Pull/ 1508/ /ci.openlp. io/job/ Branch- 02-Functional- Tests/1419/ /ci.openlp. io/job/ Branch- 03-Interface- Tests/1357/ /ci.openlp. io/job/ Branch- 04a-Windows_ Functional_ Tests/1153/ /ci.openlp. io/job/ Branch- 04b-Windows_ Interface_ Tests/744/ /ci.openlp. io/job/ Branch- 05a-Code_ Analysis/ 811/ /ci.openlp. io/job/ Branch- 05b-Test_ Coverage/ 679/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/