← Back to team overview

openlp-core team mailing list archive

[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