setCacheNode wasn't necessary after all, instead overwrote setNode in ESUser

src/helma/framework/core/ESUser.java

@@ -61,6 +61,15 @@ public class ESUser extends ESNode {
 	cacheWrapper = new ESNode (cache, eval);
     }
 
+    public void setNode (INode node) {
+	if (node != null) {
+	    this.node = node;
+	    eval.objectcache.put (node, this);
+	    // we don't take over the transient cache from the node,
+	    // because we always use the one from the user object.
+	}
+    }
+
 
     public String toString () {
 	return ("UserNode "+node.getNameOrID ());

src/helma/objectmodel/INode.java

@@ -47,7 +47,6 @@ public interface INode {
     public String getPrototype ();
     public void setPrototype (String prototype);
     public INode getCacheNode ();
-    public void setCacheNode (INode cache);
 
     /**
      *  node-related methods

src/helma/objectmodel/Node.java

@@ -768,10 +768,6 @@ public class Node implements INode, Serializable {
 	return cacheNode;
     }
 
-    public synchronized void setCacheNode (INode cache) {
-	this.cacheNode = cache;
-    }
-
 }
 
 

src/helma/objectmodel/db/Node.java

@@ -1691,10 +1691,6 @@ public class Node implements INode, Serializable {
 	return cacheNode;
     }
 
-    public synchronized void setCacheNode (INode cache) {
-	this.cacheNode = cache;
-    }
-
     // walk down node path to the first non-virtual node and return its id.
     // limit max depth to 3, since there shouldn't be more then 2 layers of virtual nodes.
     public String getNonVirtualHomeID () {