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:57 UTC
svn commit: r575841 -
/struts/struts2/trunk/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java
Author: jholmes
Date: Fri Sep 14 18:05:56 2007
New Revision: 575841
URL: http://svn.apache.org/viewvc?rev=575841&view=rev
Log:
WW-2183 XSLT result type is extremely slow for actions that produce a large xml document
Modified:
struts/struts2/trunk/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java
Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java
URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java?rev=575841&r1=575840&r2=575841&view=diff
==============================================================================
--- struts/struts2/trunk/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java (original)
+++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/views/xslt/AbstractAdapterNode.java Fri Sep 14 18:05:56 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() {