openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #28113
[Merge] lp:~thelinuxguy/openlp/fix-1510572 into lp:openlp
Simon Hanna has proposed merging lp:~thelinuxguy/openlp/fix-1510572 into lp:openlp.
Requested reviews:
OpenLP Core (openlp-core)
Related bugs:
Bug #1510572 in OpenLP: "Double verses at end of song shows as 1 on stageview"
https://bugs.launchpad.net/openlp/+bug/1510572
For more details, see:
https://code.launchpad.net/~thelinuxguy/openlp/fix-1510572/+merge/281413
--
Your team OpenLP Core is requested to review the proposed merge of lp:~thelinuxguy/openlp/fix-1510572 into lp:openlp.
=== modified file 'openlp/plugins/remotes/html/stage.js'
--- openlp/plugins/remotes/html/stage.js 2015-01-18 13:39:21 +0000
+++ openlp/plugins/remotes/html/stage.js 2015-12-30 00:17:47 +0000
@@ -61,7 +61,7 @@
}
else {
if ((slide["text"] == data.results.slides[lastChange]["text"]) &&
- (data.results.slides.length > idx + (idx - lastChange))) {
+ (data.results.slides.length >= idx + (idx - lastChange))) {
// If the tag hasn't changed, check to see if the same verse
// has been repeated consecutively. Note the verse may have been
// split over several slides, so search through. If so, repeat the tag.
Follow ups