openlp-core team mailing list archive

[Raoul Snyman <raoul.snyman@xxxxxxxxxxxxxxxxxxxxxxxx>]

Raoul Snyman has proposed merging lp:~raoul-snyman/openlp/webclient into lp:openlp.

Requested reviews:
  OpenLP Core (openlp-core)
Related bugs:
  #693694 Web interface does not always send all necessary data
  https://bugs.launchpad.net/bugs/693694

For more details, see:
https://code.launchpad.net/~raoul-snyman/openlp/webclient/+merge/44552

Fixed bug #693694: Web interface does not always send all necessary data
-- 
https://code.launchpad.net/~raoul-snyman/openlp/webclient/+merge/44552
Your team OpenLP Core is requested to review the proposed merge of lp:~raoul-snyman/openlp/webclient into lp:openlp.

=== modified file 'openlp/plugins/remotes/html/openlp.js'
--- openlp/plugins/remotes/html/openlp.js	2010-09-14 18:18:47 +0000
+++ openlp/plugins/remotes/html/openlp.js	2010-12-23 08:44:11 +0000
@@ -206,8 +206,12 @@
     },
     sendLiveSet: function (e)
     {
-        var id = OpenLP.Events.getElement(e).parent().attr("value");
-        OpenLP.Remote.sendEvent("slidecontroller_live_set", id);
+        var tr = OpenLP.Events.getElement(e).parent();
+        if (tr[0].tagName != "TR")
+        {
+            tr = tr.parent();
+        }
+        OpenLP.Remote.sendEvent("slidecontroller_live_set", tr.attr("value"));
         return false;
     },
     sendSetItem: function (e)