You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by jh...@apache.org on 2007/09/15 03:05:16 UTC

svn commit: r575840 - /struts/struts2/branches/STRUTS_2_0_X/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java

Author: jholmes
Date: Fri Sep 14 18:05:16 2007
New Revision: 575840

URL: http://svn.apache.org/viewvc?rev=575840&view=rev
Log:
WW-2183 XSLT result type is extremely slow for actions that produce a large xml document

Modified:
    struts/struts2/branches/STRUTS_2_0_X/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java

Modified: struts/struts2/branches/STRUTS_2_0_X/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java
URL: http://svn.apache.org/viewvc/struts/struts2/branches/STRUTS_2_0_X/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java?rev=575840&r1=575839&r2=575840&view=diff
==============================================================================
--- struts/struts2/branches/STRUTS_2_0_X/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java (original)
+++ struts/struts2/branches/STRUTS_2_0_X/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java Fri Sep 14 18:05:16 2007
@@ -131,8 +131,10 @@
     public Node getChildBeforeOrAfter(Node child, boolean before) {
         log.debug("getChildBeforeOrAfter: ");
         List adapters = getChildAdapters();
-        log.debug("childAdapters = " + adapters);
-        log.debug("child = " + child);
+        if (log.isDebugEnabled()) {
+            log.debug("childAdapters = " + adapters);
+            log.debug("child = " + child);
+        }
         int index = adapters.indexOf(child);
         if (index < 0)
             throw new StrutsException(child + " is no child of " + this);
@@ -251,7 +253,7 @@
                     + ((next == null) ? "null" : next.getNodeName()));
         }
 
-        return getParent().getChildAfter(this);
+        return next;
     }
 
     public Node getPreviousSibling() {