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]

Reply via email to