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 2004/09/10 00:06:31 UTC
svn commit: rev 43633 - in cocoon/trunk: . src/java/org/apache/cocoon/components/modules/input
Author: antonio
Date: Thu Sep 9 15:06:30 2004
New Revision: 43633
Modified:
cocoon/trunk/src/java/org/apache/cocoon/components/modules/input/JXPathMetaModule.java
cocoon/trunk/status.xml
Log:
Fix bug 31134, thanks to Ralph Goers
Modified: cocoon/trunk/src/java/org/apache/cocoon/components/modules/input/JXPathMetaModule.java
==============================================================================
--- cocoon/trunk/src/java/org/apache/cocoon/components/modules/input/JXPathMetaModule.java (original)
+++ cocoon/trunk/src/java/org/apache/cocoon/components/modules/input/JXPathMetaModule.java Thu Sep 9 15:06:30 2004
@@ -66,7 +66,7 @@
*
* @author <a href="mailto:kpiroumian@apache.org">Konstantin Piroumian</a>
* @author <a href="mailto:haul@apache.org">Christian Haul</a>
- * @version CVS $Id: JXPathMetaModule.java,v 1.7 2004/03/05 13:02:48 bdelacretaz Exp $
+ * @version CVS $Id$
*/
public class JXPathMetaModule extends AbstractMetaModule implements Configurable, ThreadSafe {
@@ -114,7 +114,7 @@
Object contextObj = getContextObject(modeConf, objectModel);
if (modeConf != null) {
- name = modeConf.getChild("parameter").getValue(this.parameter != null ? this.parameter : name);
+ name = modeConf.getChild("parameter").getValue(!this.parameter.equals("") ? this.parameter : name);
}
return JXPathHelper.getAttribute(name, modeConf, this.configuration, contextObj);
}
@@ -133,7 +133,7 @@
Object contextObj = getContextObject(modeConf, objectModel);
if (modeConf != null) {
- name = modeConf.getChild("parameter").getValue(this.parameter != null ? this.parameter : name);
+ name = modeConf.getChild("parameter").getValue(!this.parameter.equals("") ? this.parameter : name);
}
return JXPathHelper.getAttributeValues(name, modeConf, this.configuration, contextObj);
}
Modified: cocoon/trunk/status.xml
==============================================================================
--- cocoon/trunk/status.xml (original)
+++ cocoon/trunk/status.xml Thu Sep 9 15:06:30 2004
@@ -204,6 +204,9 @@
<changes>
<release version="@version@" date="@date@">
+ <action dev="AG" type="fix" fixes-bug="31134" due-to="Ralph Goers" due-to-email="Ralph_Goers@dslextreme.com">
+ JXPathMetaModule incorrectly checks for null parameter (can't happen) instead of empty string.
+ </action>
<action dev="AG" type="update">
Update jexl to 1.0.
</action>