Merge lp:~raoul-snyman/openlp/support-322 into lp:openlp

Proposed by Raoul Snyman
Status: Merged
Approved by: Tim Bentley
Approved revision: 1806
Merged at revision: 1806
Proposed branch: lp:~raoul-snyman/openlp/support-322
Merge into: lp:openlp
Diff against target: 19 lines (+5/-4)
1 file modified
openlp/plugins/remotes/html/stage.js (+5/-4)
To merge this branch: bzr merge lp:~raoul-snyman/openlp/support-322
Reviewer Review Type Date Requested Status
Tim Bentley Approve
Review via email: mp+83525@code.launchpad.net

Description of the change

Service notes are not displayed on stage remote view when the current service item is the last.

(Fix from Jérémie Nau)

To post a comment you must log in.
Revision history for this message
Tim Bentley (trb143) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'openlp/plugins/remotes/html/stage.js'
2--- openlp/plugins/remotes/html/stage.js 2011-10-01 12:04:09 +0000
3+++ openlp/plugins/remotes/html/stage.js 2011-11-27 19:47:24 +0000
4@@ -30,10 +30,11 @@
5 $("#notes").html("");
6 for (idx in data.results.items) {
7 idx = parseInt(idx, 10);
8- if ((data.results.items[idx]["selected"]) &&
9- (data.results.items.length > idx + 1)) {
10- $("#notes").html(data.results.items[idx]["notes"]);
11- OpenLP.nextSong = data.results.items[idx + 1]["title"];
12+ if (data.results.items[idx]["selected"]) {
13+ $("#notes").html(data.results.items[idx]["notes"].replace(/\n/g, "<br />"));
14+ if (data.results.items.length > idx + 1) {
15+ OpenLP.nextSong = data.results.items[idx + 1]["title"];
16+ }
17 break;
18 }
19 }