You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by an...@apache.org on 2005/04/23 13:41:36 UTC

svn commit: r164374 - /cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/binding/JavaScriptJXPathBinding.java

Author: antonio
Date: Sat Apr 23 04:41:34 2005
New Revision: 164374

URL: http://svn.apache.org/viewcvs?rev=164374&view=rev
Log:
cleaning up

Modified:
    cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/binding/JavaScriptJXPathBinding.java

Modified: cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/binding/JavaScriptJXPathBinding.java
URL: http://svn.apache.org/viewcvs/cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/binding/JavaScriptJXPathBinding.java?rev=164374&r1=164373&r2=164374&view=diff
==============================================================================
--- cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/binding/JavaScriptJXPathBinding.java (original)
+++ cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/binding/JavaScriptJXPathBinding.java Sat Apr 23 04:41:34 2005
@@ -36,7 +36,7 @@
  */
 public class JavaScriptJXPathBinding extends JXPathBindingBase {
 
-	private final Context avalonContext;
+    private final Context avalonContext;
     private final String id;
     private final String path;
     private final Function loadScript;
@@ -47,7 +47,7 @@
     final static String[] SAVE_PARAMS = { "widget", "jxpathPointer", "jxpathContext", "childBindings" };
 
     public JavaScriptJXPathBinding(
-    		    Context context, JXPathBindingBuilderBase.CommonAttributes commonAtts, String id,
+    	    Context context, JXPathBindingBuilderBase.CommonAttributes commonAtts, String id,
             String path, Function loadScript, Function saveScript, Map childBindings) {
         super(commonAtts);
         this.id = id;
@@ -58,9 +58,8 @@
         
         // Set parent on child bindings
         for(Iterator iter = childBindings.values().iterator(); iter.hasNext(); ) {
-        		((Binding)iter.next()).setParent(this);
+       	    ((Binding)iter.next()).setParent(this);
         }
-        
         this.childBindings = new ScriptableMap(childBindings);
     }
 
@@ -74,14 +73,6 @@
             Map objectModel = ContextHelper.getObjectModel(this.avalonContext);
 
             try {
-//                Map values = new HashMap(3);
-//                values.put("widget", widget);
-//                values.put("jxpathPointer", pointer);
-//                if (pointer.getNode() != null) {
-//                    values.put("jxpathContext", jctx.getRelativeContext(pointer));
-//                }
-//                values.put("childBindings", this.childBindings);
-                
                 JXPathContext newCtx = pointer.getNode() == null ? null :
                 	    jctx.getRelativeContext(pointer);
 
@@ -110,12 +101,6 @@
             JXPathContext widgetCtx = jctx.getRelativeContext(pointer);
             try {
                 Map objectModel = ContextHelper.getObjectModel(this.avalonContext);
-
-//                Map values = new HashMap();
-//                values.put("widget", widget);
-//                values.put("jxpathContext", widgetCtx);
-//                values.put("jxpathPointer", pointer);
-//                values.put("childBindings", this.childBindings);
 
                 JavaScriptHelper.callFunction(this.saveScript, frmModel,
                 		new Object[] {widget, pointer, widgetCtx, this.childBindings}, objectModel);