[Laszlo-checkins] r6936 - openlaszlo/trunk/WEB-INF/lps/lfc/data

bargull@openlaszlo.org bargull at openlaszlo.org
Sat Oct 20 06:42:45 PDT 2007


Author: bargull
Date: 2007-10-20 06:42:43 -0700 (Sat, 20 Oct 2007)
New Revision: 6936

Modified:
   openlaszlo/trunk/WEB-INF/lps/lfc/data/LzReplicationManager.lzs
Log:
Change 20071020-bargull-4 by bargull at dell--p4--2-53 on 2007-10-20 15:16:25
    in /home/Admin/src/svn/openlaszlo/trunk
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: LzReplicationManager needs to check immediateparent for implicit datapath 

New Features:

Bugs Fixed:
LPP-4654 - "LzReplicationManager doesn't handle relative xpath appropriately"

Technical Reviewer: hminsky
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:
LzDatapath#setDataContext(..) checks all immediateparents of its immediateparent for an implicit datapath, but LzReplicationManager must also check its immediateparent for an implicit datapath. 
We cannot directly check in LzDatapath for an implicit datapath on its immediateparent, because this would lead to a circular logic as the datapath of the immediateparent of a datapath is the same datapath.

Release Notes:

Details:
    

Tests:



Modified: openlaszlo/trunk/WEB-INF/lps/lfc/data/LzReplicationManager.lzs
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/data/LzReplicationManager.lzs	2007-10-20 13:31:44 UTC (rev 6935)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/data/LzReplicationManager.lzs	2007-10-20 13:42:43 UTC (rev 6936)
@@ -289,6 +289,18 @@
 /**
   * @access private
   */
+function setDataContext ( p , implicit ){
+    var args = arguments; 
+    if (p == null && this.immediateparent != null && 'datapath' in this.immediateparent && this.immediateparent.datapath != null){
+        args[0] = this.immediateparent.datapath;
+        args[1] = true;
+    }
+    super.setDataContext.apply(this, args);
+}
+
+/**
+  * @access private
+  */
 function getNodeOffset ( p ){
     if (this.nodes != null) {
         var l = this.nodes.length; 



More information about the Laszlo-checkins mailing list