sylvain 01/09/19 07:46:37 Modified: src/org/apache/cocoon/components/language/markup/xsp/java xsp.xsl src/org/apache/cocoon/matching BrowserMatcherFactory.java WildcardHeaderMatcherFactory.java WildcardParameterValueMatcherFactory.java WildcardSessionAttributeMatcherFactory.java src/org/apache/cocoon/selection BrowserSelectorFactory.java HeaderSelectorFactory.java HostSelectorFactory.java RequestSelectorFactory.java SessionAttributeSelectorFactory.java Log: Fix for bug #3138 : test for null nodes in factories configuration NodeList Revision Changes Path 1.9 +2 -2 xml-cocoon2/src/org/apache/cocoon/components/language/markup/xsp/java/xsp.xsl Index: xsp.xsl =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/components/language/markup/xsp/java/xsp.xsl,v retrieving revision 1.8 retrieving revision 1.9 diff -u -r1.8 -r1.9 --- xsp.xsl 2001/09/14 12:58:33 1.8 +++ xsp.xsl 2001/09/19 14:46:37 1.9 @@ -12,7 +12,7 @@ <!-- * @author <a href="mailto:[EMAIL PROTECTED]>Ricardo Rocha</a> * @author <a href="[EMAIL PROTECTED]">Sylvain Wallez</a> - * @version CVS $Revision: 1.8 $ $Date: 2001/09/14 12:58:33 $ + * @version CVS $Revision: 1.9 $ $Date: 2001/09/19 14:46:37 $ --> <!-- XSP Core logicsheet for the Java language --> @@ -145,7 +145,7 @@ <xsl:for-each select="$content/namespace::*"> <xsl:variable name="ns-prefix" select="local-name(.)"/> <xsl:variable name="ns-uri" select="string(.)"/> - <xsl:if test="$parent-element/namespace::*[local-name(.) = $ns-prefix and string(.) = $ns-uri]"> + <xsl:if test="($ns-prefix != 'xmlns') and $parent-element/namespace::*[local-name(.) = $ns-prefix and string(.) = $ns-uri]"> this.contentHandler.endPrefixMapping( "<xsl:value-of select="local-name(.)"/>" ); 1.4 +3 -1 xml-cocoon2/src/org/apache/cocoon/matching/BrowserMatcherFactory.java Index: BrowserMatcherFactory.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/matching/BrowserMatcherFactory.java,v retrieving revision 1.3 retrieving revision 1.4 diff -u -r1.3 -r1.4 --- BrowserMatcherFactory.java 2001/08/20 13:55:16 1.3 +++ BrowserMatcherFactory.java 2001/09/19 14:46:37 1.4 @@ -19,7 +19,7 @@ * * @author <a href="mailto:[EMAIL PROTECTED]">Giacomo Pati</a> * @author <a href="mailto:[EMAIL PROTECTED]">Berin Loritsch</a> - * @version CVS $Revision: 1.3 $ $Date: 2001/08/20 13:55:16 $ + * @version CVS $Revision: 1.4 $ $Date: 2001/09/19 14:46:37 $ */ public class BrowserMatcherFactory implements CodeFactory { @@ -35,6 +35,8 @@ int count = conf.getLength(); for(int k = 0; k < count;k++) { node = conf.item(k); + if (node == null) + continue; sb.append("name=") .append(node.getNodeName()) .append(" type=") 1.5 +3 -2 xml-cocoon2/src/org/apache/cocoon/matching/WildcardHeaderMatcherFactory.java Index: WildcardHeaderMatcherFactory.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/matching/WildcardHeaderMatcherFactory.java,v retrieving revision 1.4 retrieving revision 1.5 diff -u -r1.4 -r1.5 --- WildcardHeaderMatcherFactory.java 2001/09/18 14:47:00 1.4 +++ WildcardHeaderMatcherFactory.java 2001/09/19 14:46:37 1.5 @@ -22,7 +22,7 @@ * </table> * * @author <a href="mailto:[EMAIL PROTECTED]">Christian Haul</a> - * @version CVS $Revision: 1.4 $ $Date: 2001/09/18 14:47:00 $ + * @version CVS $Revision: 1.5 $ $Date: 2001/09/19 14:46:37 $ */ public class WildcardHeaderMatcherFactory extends WildcardURIMatcherFactory { @@ -62,7 +62,8 @@ int count = conf.getLength(); for(int k = 0; k < count;k++) { Node node = conf.item(k); - if (node.getNodeName() != null && + if (node != null && + node.getNodeName() != null && node.getNodeName().equals("parameter-name")) { Node textNode = node.getFirstChild(); 1.5 +3 -2 xml-cocoon2/src/org/apache/cocoon/matching/WildcardParameterValueMatcherFactory.java Index: WildcardParameterValueMatcherFactory.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/matching/WildcardParameterValueMatcherFactory.java,v retrieving revision 1.4 retrieving revision 1.5 diff -u -r1.4 -r1.5 --- WildcardParameterValueMatcherFactory.java 2001/09/18 14:47:00 1.4 +++ WildcardParameterValueMatcherFactory.java 2001/09/19 14:46:37 1.5 @@ -26,7 +26,7 @@ * </table> * * @author <a href="mailto:[EMAIL PROTECTED]">Christian Haul</a> - * @version CVS $Revision: 1.4 $ $Date: 2001/09/18 14:47:00 $ */ + * @version CVS $Revision: 1.5 $ $Date: 2001/09/19 14:46:37 $ */ public class WildcardParameterValueMatcherFactory extends WildcardURIMatcherFactory { @@ -41,7 +41,8 @@ int count = conf.getLength(); for(int k = 0; k < count;k++) { Node node = conf.item(k); - if (node.getNodeName() != null && + if (node != null && + node.getNodeName() != null && node.getNodeName().equals("parameter-name")) { Node textNode = node.getFirstChild(); 1.2 +3 -2 xml-cocoon2/src/org/apache/cocoon/matching/WildcardSessionAttributeMatcherFactory.java Index: WildcardSessionAttributeMatcherFactory.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/matching/WildcardSessionAttributeMatcherFactory.java,v retrieving revision 1.1 retrieving revision 1.2 diff -u -r1.1 -r1.2 --- WildcardSessionAttributeMatcherFactory.java 2001/09/18 14:47:00 1.1 +++ WildcardSessionAttributeMatcherFactory.java 2001/09/19 14:46:37 1.2 @@ -21,7 +21,7 @@ * </table> * * @author <a href="mailto:[EMAIL PROTECTED]">Christian Haul</a> - * @version CVS $Revision: 1.1 $ $Date: 2001/09/18 14:47:00 $ + * @version CVS $Revision: 1.2 $ $Date: 2001/09/19 14:46:37 $ */ public class WildcardSessionAttributeMatcherFactory extends WildcardURIMatcherFactory { @@ -37,7 +37,8 @@ int count = conf.getLength(); for(int k = 0; k < count;k++) { Node node = conf.item(k); - if (node.getNodeName() != null && + if (node != null && + node.getNodeName() != null && node.getNodeName().equals("attribute-name")) { Node textNode = node.getFirstChild(); 1.3 +3 -2 xml-cocoon2/src/org/apache/cocoon/selection/BrowserSelectorFactory.java Index: BrowserSelectorFactory.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/selection/BrowserSelectorFactory.java,v retrieving revision 1.2 retrieving revision 1.3 diff -u -r1.2 -r1.3 --- BrowserSelectorFactory.java 2001/08/20 13:55:16 1.2 +++ BrowserSelectorFactory.java 2001/09/19 14:46:37 1.3 @@ -20,7 +20,7 @@ * @author <a href="mailto:[EMAIL PROTECTED]">Carsten Ziegeler</a> * @author <a href="mailto:[EMAIL PROTECTED]">Giacomo Pati</a> * @author <a href="mailto:[EMAIL PROTECTED]">Berin Loritsch</a> - * @version CVS $Revision: 1.2 $ $Date: 2001/08/20 13:55:16 $ + * @version CVS $Revision: 1.3 $ $Date: 2001/09/19 14:46:37 $ */ @@ -45,7 +45,8 @@ int count = conf.getLength(); for(int k = 0; k < count;k++) { node = conf.item(k); - if (node.getNodeName().equals("browser") && + if (node != null && + node.getNodeName().equals("browser") && node.getNodeType() == Node.ELEMENT_NODE) { nm = node.getAttributes(); if (nm != null) { 1.5 +3 -2 xml-cocoon2/src/org/apache/cocoon/selection/HeaderSelectorFactory.java Index: HeaderSelectorFactory.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/selection/HeaderSelectorFactory.java,v retrieving revision 1.4 retrieving revision 1.5 diff -u -r1.4 -r1.5 --- HeaderSelectorFactory.java 2001/09/18 14:47:00 1.4 +++ HeaderSelectorFactory.java 2001/09/19 14:46:37 1.5 @@ -22,7 +22,7 @@ * </table> * * @author <a href="mailto:[EMAIL PROTECTED]">Christian Haul</a> - * @version CVS $Revision: 1.4 $ $Date: 2001/09/18 14:47:00 $ + * @version CVS $Revision: 1.5 $ $Date: 2001/09/19 14:46:37 $ */ public class HeaderSelectorFactory extends ParameterSelectorFactory { @@ -34,7 +34,8 @@ int count = conf.getLength(); for(int k = 0; k < count;k++) { Node node = conf.item(k); - if (node.getNodeName() != null && + if (node != null && + node.getNodeName() != null && node.getNodeName().equals("parameter-name")) { Node textNode = node.getFirstChild(); 1.4 +3 -2 xml-cocoon2/src/org/apache/cocoon/selection/HostSelectorFactory.java Index: HostSelectorFactory.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/selection/HostSelectorFactory.java,v retrieving revision 1.3 retrieving revision 1.4 diff -u -r1.3 -r1.4 --- HostSelectorFactory.java 2001/08/20 13:55:16 1.3 +++ HostSelectorFactory.java 2001/09/19 14:46:37 1.4 @@ -32,7 +32,7 @@ * </map:select> * * @author <a href="mailto:[EMAIL PROTECTED]">Colin Britton</a> - * @version CVS $Revision: 1.3 $ $Date: 2001/08/20 13:55:16 $ + * @version CVS $Revision: 1.4 $ $Date: 2001/09/19 14:46:37 $ */ @@ -57,7 +57,8 @@ int count = conf.getLength(); for(int k = 0; k < count;k++) { node = conf.item(k); - if (node.getNodeName().equals("host") && + if (node != null && + node.getNodeName().equals("host") && node.getNodeType() == Node.ELEMENT_NODE) { nm = node.getAttributes(); if (nm != null) { 1.5 +3 -2 xml-cocoon2/src/org/apache/cocoon/selection/RequestSelectorFactory.java Index: RequestSelectorFactory.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/selection/RequestSelectorFactory.java,v retrieving revision 1.4 retrieving revision 1.5 diff -u -r1.4 -r1.5 --- RequestSelectorFactory.java 2001/09/18 14:47:00 1.4 +++ RequestSelectorFactory.java 2001/09/19 14:46:37 1.5 @@ -22,7 +22,7 @@ * </table> * * @author <a href="mailto:[EMAIL PROTECTED]">Christian Haul</a> - * @version CVS $Revision: 1.4 $ $Date: 2001/09/18 14:47:00 $ + * @version CVS $Revision: 1.5 $ $Date: 2001/09/19 14:46:37 $ */ public class RequestSelectorFactory extends ParameterSelectorFactory { @@ -34,7 +34,8 @@ int count = conf.getLength(); for(int k = 0; k < count;k++) { Node node = conf.item(k); - if (node.getNodeName() != null && + if (node != null && + node.getNodeName() != null && node.getNodeName().equals("parameter-name")) { Node textNode = node.getFirstChild(); 1.2 +4 -2 xml-cocoon2/src/org/apache/cocoon/selection/SessionAttributeSelectorFactory.java Index: SessionAttributeSelectorFactory.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/selection/SessionAttributeSelectorFactory.java,v retrieving revision 1.1 retrieving revision 1.2 diff -u -r1.1 -r1.2 --- SessionAttributeSelectorFactory.java 2001/09/18 14:47:00 1.1 +++ SessionAttributeSelectorFactory.java 2001/09/19 14:46:37 1.2 @@ -21,7 +21,8 @@ * </table> * * @author <a href="mailto:[EMAIL PROTECTED]">Christian Haul</a> - * @version CVS $Revision: 1.1 $ $Date: 2001/09/18 14:47:00 $ */ + * @version CVS $Revision: 1.2 $ $Date: 2001/09/19 14:46:37 $ + */ public class SessionAttributeSelectorFactory extends ParameterSelectorFactory { public String generateMethodSource (NodeList conf) @@ -32,7 +33,8 @@ int count = conf.getLength(); for(int k = 0; k < count;k++) { Node node = conf.item(k); - if (node.getNodeName() != null && + if (node != null && + node.getNodeName() != null && node.getNodeName().equals("attribute-name")) { Node textNode = node.getFirstChild(); ---------------------------------------------------------------------- In case of troubles, e-mail: [EMAIL PROTECTED] To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]