Hi Jeremy,

Nice work. Please look into the small issues below:

[EMAIL PROTECTED] escribió:
Author: jeremy
Date: Sun Nov 12 07:28:08 2006
New Revision: 473974

URL: http://svn.apache.org/viewvc?view=rev&rev=473974
Log:
Adding support for Dojo's IframeIO to CForms.
This allows forms with file-upload fields to be submitted as a background AJAX 
process. Currently this is turned off for Safari where is is not working.

Added:
    
cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/resources/org/apache/cocoon/forms/resources/IframeTransport-bu-styling.xsl
Modified:
    
cocoon/trunk/blocks/cocoon-ajax/cocoon-ajax-impl/src/main/resources/org/apache/cocoon/ajax/resources/js/BUHandler.js
    
cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/resources/org/apache/cocoon/forms/resources/js/CFormsForm.js
    
cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/resources/org/apache/cocoon/forms/resources/js/common.js
    
cocoon/trunk/blocks/cocoon-forms/cocoon-forms-sample/src/main/resources/COB-INF/sitemap.xmap

Modified: 
cocoon/trunk/blocks/cocoon-ajax/cocoon-ajax-impl/src/main/resources/org/apache/cocoon/ajax/resources/js/BUHandler.js
URL: 
http://svn.apache.org/viewvc/cocoon/trunk/blocks/cocoon-ajax/cocoon-ajax-impl/src/main/resources/org/apache/cocoon/ajax/resources/js/BUHandler.js?view=diff&rev=473974&r1=473973&r2=473974
==============================================================================
--- 
cocoon/trunk/blocks/cocoon-ajax/cocoon-ajax-impl/src/main/resources/org/apache/cocoon/ajax/resources/js/BUHandler.js
 (original)
+++ 
cocoon/trunk/blocks/cocoon-ajax/cocoon-ajax-impl/src/main/resources/org/apache/cocoon/ajax/resources/js/BUHandler.js
 Sun Nov 12 07:28:08 2006
@@ -38,66 +38,71 @@
dojo.lang.extend(cocoon.ajax.BUHandler, {
     // Default highlight effect
-    highlight: null,
+       highlight: null,
- processResponse: function(doc, request) {
-           var nodes = doc.documentElement.childNodes;
-           for (var i = 0; i < nodes.length; i++) {
-               var node = nodes[i];
-               if (node.nodeType == dojo.dom.ELEMENT_NODE) {
-                   var handler;
-                   if (node.localName) {
-                       handler = node.localName;
-                   } else {
-                       // No DOM2 support (IE6)
-                       handler = node.nodeName.replace(/.*:/, "");
-                   }
-                   var handlerFunc = this.handlers[handler];
-                   if (handlerFunc) {
-                       handlerFunc(node);
-                   } else {
-                       this.handleError("No handler found for element " + 
handler, request);
-                   }
-               }
-           }
+  processResponse: function(doc, request) {
+               var base = doc.documentElement;
+               
+               var nodes = [];
+               if (base.nodeName.toLowerCase() == "bu:document") {
+                       nodes = base.childNodes;
+                       dojo.debug("got response using: XMLHTTPTransport");
+               } else {
+                       nodes = dojo.byId("browser-update", doc).childNodes;
+                       dojo.debug("got response using: IframeTransport");
+               }
+               for (var i = 0; i < nodes.length; i++) {
+                       var node = nodes[i];
+                       if (node.nodeType == dojo.dom.ELEMENT_NODE) {
+                               var handler = node.nodeName.replace(/.*:/, 
"").toLowerCase();
+                               if (handler == "textarea") handler = 
node.getAttribute("name");
+                               var handlerFunc = this.handlers[handler];
+                               if (handlerFunc) {
+                                       handlerFunc(node);
+                               } else {
+                                       this.handleError("No handler found for 
element " + handler, request);
+                               }
+                       }
+               }
        },
We use 4 spaces for tabs.

Modified: 
cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/resources/org/apache/cocoon/forms/resources/js/CFormsForm.js
URL: 
http://svn.apache.org/viewvc/cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/resources/org/apache/cocoon/forms/resources/js/CFormsForm.js?view=diff&rev=473974&r1=473973&r2=473974
==============================================================================
--- 
cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/resources/org/apache/cocoon/forms/resources/js/CFormsForm.js
 (original)
+++ 
cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/resources/org/apache/cocoon/forms/resources/js/CFormsForm.js
 Sun Nov 12 07:28:08 2006
@@ -1,4 +1,4 @@
-/*
+/*
Encoding typo? Please review your subversion settings.

Reply via email to