luehe       2004/03/25 13:53:13

  Modified:    jasper2/src/share/org/apache/jasper/compiler Generator.java
  Log:
  Fixed Bugzilla 27948 ("NPE with null fragment")
  
  Revision  Changes    Path
  1.229     +8 -8      
jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/compiler/Generator.java
  
  Index: Generator.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/compiler/Generator.java,v
  retrieving revision 1.228
  retrieving revision 1.229
  diff -u -r1.228 -r1.229
  --- Generator.java    25 Mar 2004 21:44:35 -0000      1.228
  +++ Generator.java    25 Mar 2004 21:53:12 -0000      1.229
  @@ -1986,11 +1986,6 @@
   
               n.setBeginJavaLine(out.getJavaLine());
   
  -            // Invoke fragment, unless fragment is null
  -            out.printin("if (");
  -            out.print(toGetterMethod(n.getTextAttribute("fragment")));
  -            out.println(" != null) {");
  -            out.pushIndent();
               // Copy virtual page scope of tag file to page scope of invoking
               // page
               out.printil(
  @@ -2002,8 +1997,16 @@
               } else {
                   out.printil("_jspx_sout = null;");
               }
  +
  +            // Invoke fragment, unless fragment is null
  +            out.printin("if (");
  +            out.print(toGetterMethod(n.getTextAttribute("fragment")));
  +            out.println(" != null) {");
  +            out.pushIndent();
               out.printin(toGetterMethod(n.getTextAttribute("fragment")));
               out.println(".invoke(_jspx_sout);");
  +            out.popIndent();
  +            out.printil("}");
   
               // Store varReader in appropriate scope
               if (varReaderAttr != null || varAttr != null) {
  @@ -2023,9 +2026,6 @@
                   }
                   out.println(");");
               }
  -
  -            out.popIndent();
  -            out.printil("}");
   
               n.setEndJavaLine(out.getJavaLine());
           }
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to