Merge lp://qastaging/~tomasgroth/openlp/videopsalm-fix-24 into lp://qastaging/openlp/2.4
Proposed by
Tomas Groth
Status: | Merged |
---|---|
Merged at revision: | 2640 |
Proposed branch: | lp://qastaging/~tomasgroth/openlp/videopsalm-fix-24 |
Merge into: | lp://qastaging/openlp/2.4 |
Diff against target: |
120 lines (+92/-0) 4 files modified
openlp/plugins/songs/lib/importers/videopsalm.py (+8/-0) tests/functional/openlp_plugins/songs/test_videopsalm.py (+2/-0) tests/resources/videopsalmsongs/as-safe-a-stronghold2.json (+35/-0) tests/resources/videopsalmsongs/videopsalm-as-safe-a-stronghold2.json (+47/-0) |
To merge this branch: | bzr merge lp://qastaging/~tomasgroth/openlp/videopsalm-fix-24 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Raoul Snyman | Approve | ||
Tim Bentley | Approve | ||
Review via email: mp+298065@code.qastaging.launchpad.net |
Description of the change
Fix handeling of control chars and escaped chars in VideoPsalm import. Fixes bug 1594945.
To post a comment you must log in.
lp:~tomasgroth/openlp/videopsalm-fix-24 (revision 2638) /ci.openlp. io/job/ Branch- 01-Pull/ 1631/ /ci.openlp. io/job/ Branch- 02-Functional- Tests/1542/ /ci.openlp. io/job/ Branch- 03-Interface- Tests/1480/ /ci.openlp. io/job/ Branch- 04a-Windows_ Functional_ Tests/1249/ /ci.openlp. io/job/ Branch- 04b-Windows_ Interface_ Tests/839/ /ci.openlp. io/job/ Branch- 05a-Code_ Analysis/ 907/ /ci.openlp. io/job/ Branch- 05b-Test_ Coverage/ 775/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/