costin      00/11/02 13:51:47

  Modified:    src/facade22/org/apache/tomcat/facade
                        HttpServletRequestFacade.java
                        HttpServletResponseFacade.java
                        RequestDispatcherImpl.java
                        Servlet22Interceptor.java ServletWrapper.java
               src/facade22/org/apache/tomcat/modules/facade22
                        JspInterceptor.java
               src/facade23/org/apache/tomcat/facade23
                        HttpServletRequestFacade.java
                        HttpServletResponseFacade.java
                        RequestDispatcherImpl.java ServletWrapper.java
  Log:
  - Changes required to support String->MessageBytes and other core changes.
  
  JspInterceptor now supports jikes :-) !!
  ( you need to enable it with compiler="jikes" - and it's fast !)
  
  Revision  Changes    Path
  1.8       +4 -4      
jakarta-tomcat/src/facade22/org/apache/tomcat/facade/HttpServletRequestFacade.java
  
  Index: HttpServletRequestFacade.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/facade/HttpServletRequestFacade.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- HttpServletRequestFacade.java     2000/10/08 21:28:56     1.7
  +++ HttpServletRequestFacade.java     2000/11/02 21:51:34     1.8
  @@ -224,7 +224,7 @@
       }
       
       public String getMethod() {
  -        return request.getMethod();
  +        return request.method().toString();
       }
   
       /** Adapter: Request doesn't deal with this servlet convention
  @@ -263,7 +263,7 @@
       }
   
       public String getQueryString() {
  -        return request.getQueryString();
  +        return request.queryString().toString();
       }
   
       public String getRemoteUser() {
  @@ -271,7 +271,7 @@
       }
   
       public String getScheme() {
  -        return request.getScheme();
  +        return request.scheme().toString();
       }
   
       public String getServerName() {
  @@ -320,7 +320,7 @@
       }
   
       public String getRequestURI() {
  -        return request.getRequestURI();
  +        return request.requestURI().toString();
       }
   
       /** Facade: we delegate to the right object ( the context )
  
  
  
  1.8       +1 -1      
jakarta-tomcat/src/facade22/org/apache/tomcat/facade/HttpServletResponseFacade.java
  
  Index: HttpServletResponseFacade.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/facade/HttpServletResponseFacade.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- HttpServletResponseFacade.java    2000/10/27 20:50:50     1.7
  +++ HttpServletResponseFacade.java    2000/11/02 21:51:37     1.8
  @@ -364,7 +364,7 @@
        }
   
        // Does this URL match down to (and including) the context path?
  -     if (!request.getScheme().equalsIgnoreCase(url.getProtocol()))
  +     if (!request.scheme().equalsIgnoreCase(url.getProtocol()))
            return (false);
        if (!request.getServerName().equalsIgnoreCase(url.getHost()))
            return (false);
  
  
  
  1.4       +3 -3      
jakarta-tomcat/src/facade22/org/apache/tomcat/facade/RequestDispatcherImpl.java
  
  Index: RequestDispatcherImpl.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/facade/RequestDispatcherImpl.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- RequestDispatcherImpl.java        2000/09/28 03:25:44     1.3
  +++ RequestDispatcherImpl.java        2000/11/02 21:51:37     1.4
  @@ -168,7 +168,7 @@
        // set the context - no need to fire context parsing again
        realRequest.setContext( context );
   
  -     realRequest.setRequestURI( context.getPath() + path );
  +     realRequest.requestURI().setString( context.getPath() + path );
   
        // merge query string as specified in specs - before, it may affect
        // the way the request is handled by special interceptors
  @@ -187,7 +187,7 @@
   
   
        // CM should have set the wrapper - call it
  -     Handler wr=realRequest.getWrapper();
  +     Handler wr=realRequest.getHandler();
        if( wr!=null ) wr.service(realRequest, realResponse);
   
        // close the response - output after this point will be discarded.
  @@ -312,7 +312,7 @@
        // now it's really strange: we call the wrapper on the subrequest
        // for the realRequest ( since the real request will still have the
        // original handler/wrapper )
  -     Handler wr=subRequest.getWrapper();
  +     Handler wr=subRequest.getHandler();
        if( wr!=null ) wr.service(realRequest, realResponse);
   
        // After request, we want to restore the include attributes - for
  
  
  
  1.5       +20 -1     
jakarta-tomcat/src/facade22/org/apache/tomcat/facade/Servlet22Interceptor.java
  
  Index: Servlet22Interceptor.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/facade/Servlet22Interceptor.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- Servlet22Interceptor.java 2000/10/01 06:37:42     1.4
  +++ Servlet22Interceptor.java 2000/11/02 21:51:39     1.5
  @@ -105,7 +105,26 @@
        ctx.setEngineHeader( engineHeader );
       }
   
  -
  +    /** Call servlet.destroy() for all servlets, as required
  +     by the spec
  +    */
  +    public void contextShutdown( Context ctx )
  +     throws TomcatException
  +    {
  +     // shut down and servlets
  +     Enumeration enum = ctx.getServletNames();
  +     while (enum.hasMoreElements()) {
  +         String key = (String)enum.nextElement();
  +         Handler wrapper = ctx.getServletByName( key );
  +         ctx.removeServletByName( key );
  +         try {
  +             wrapper.destroy();
  +         } catch(Exception ex ) {
  +             ctx.log( "Error in destroy ", ex);
  +         }
  +     }
  +    }
  +    
       public void addContext( ContextManager cm, Context ctx )
        throws TomcatException
       {
  
  
  
  1.9       +3 -2      
jakarta-tomcat/src/facade22/org/apache/tomcat/facade/ServletWrapper.java
  
  Index: ServletWrapper.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/facade/ServletWrapper.java,v
  retrieving revision 1.8
  retrieving revision 1.9
  diff -u -r1.8 -r1.9
  --- ServletWrapper.java       2000/10/01 06:37:42     1.8
  +++ ServletWrapper.java       2000/11/02 21:51:40     1.9
  @@ -252,7 +252,8 @@
   
            try {
                if( servlet!=null) {
  -                 BaseInterceptor 
cI[]=contextM.getInterceptors(context.getContainer());
  +                 BaseInterceptor cI[]=context.
  +                     getContainer().getInterceptors();
                    for( int i=0; i< cI.length; i++ ) {
                        try {
                            cI[i].preServletDestroy( context, this );
  @@ -323,7 +324,7 @@
        //}
   
        // Call pre, doInit and post
  -     BaseInterceptor cI[]=contextM.getInterceptors(context.getContainer());
  +     BaseInterceptor cI[]=context.getContainer().getInterceptors();
        for( int i=0; i< cI.length; i++ ) {
            try {
                cI[i].preServletInit( context, this );
  
  
  
  1.9       +18 -3     
jakarta-tomcat/src/facade22/org/apache/tomcat/modules/facade22/JspInterceptor.java
  
  Index: JspInterceptor.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/modules/facade22/JspInterceptor.java,v
  retrieving revision 1.8
  retrieving revision 1.9
  diff -u -r1.8 -r1.9
  --- JspInterceptor.java       2000/11/02 00:41:23     1.8
  +++ JspInterceptor.java       2000/11/02 21:51:44     1.9
  @@ -84,7 +84,22 @@
    */
   public class JspInterceptor extends BaseInterceptor {
       int jspInfoNOTE;
  +    TomcatOptions options=new TomcatOptions();
   
  +    static final String JIKES="org.apache.jasper.compiler.JikesJavaCompiler";
  +    
  +    public void setJavaCompiler( String type ) {
  +     // shortcut
  +     if( "jikes".equals( type ) )
  +         type=JIKES;
  +     
  +     try {
  +         options.jspCompilerPlugin=Class.forName(type);
  +     } catch(Exception ex ) {
  +         ex.printStackTrace();
  +     }
  +    }
  +
       public void engineInit(ContextManager cm )
        throws TomcatException
       {
  @@ -92,6 +107,7 @@
        jspInfoNOTE=cm.getNoteId( ContextManager.HANDLER_NOTE,
                                  "tomcat.jspInfoNote");
       }
  +
       
       public void addContext(ContextManager cm, Context ctx)
        throws TomcatException 
  @@ -119,7 +135,7 @@
       }
   
       public int requestMap( Request req ) {
  -     Handler wrapper=(Handler)req.getWrapper();
  +     Handler wrapper=(Handler)req.getHandler();
   
        //      log( "Try: " + req );
        
  @@ -208,7 +224,7 @@
            log("mapJspPage: request=" + req + ", jspInfo=" + jspInfo + ", 
servletName=" + servletName + ", classN=" + classN, ex);
            return ;
        }
  -     req.setWrapper( wrapper );
  +     req.setHandler( wrapper );
        if( debug>0) log("Wrapper " + wrapper);
       }
   
  @@ -225,7 +241,6 @@
            dir=new File( jspInfo.outputDir);
            dir.mkdirs();
            JspMangler mangler= new JspMangler(jspInfo);
  -         TomcatOptions options=new TomcatOptions();
            JspEngineContext1 ctxt = new JspEngineContext1(req, mangler);
            ctxt.setOptions( options );
            
  
  
  
  1.7       +4 -4      
jakarta-tomcat/src/facade23/org/apache/tomcat/facade23/HttpServletRequestFacade.java
  
  Index: HttpServletRequestFacade.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade23/org/apache/tomcat/facade23/HttpServletRequestFacade.java,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -u -r1.6 -r1.7
  --- HttpServletRequestFacade.java     2000/10/08 21:28:56     1.6
  +++ HttpServletRequestFacade.java     2000/11/02 21:51:44     1.7
  @@ -233,7 +233,7 @@
       }
       
       public String getMethod() {
  -        return request.getMethod();
  +        return request.method().toString();
       }
   
       /** Adapter: Request doesn't deal with this servlet convention
  @@ -272,7 +272,7 @@
       }
   
       public String getQueryString() {
  -        return request.getQueryString();
  +        return request.queryString().toString();
       }
   
       public String getRemoteUser() {
  @@ -280,7 +280,7 @@
       }
   
       public String getScheme() {
  -        return request.getScheme();
  +        return request.scheme().toString();
       }
   
       public String getServerName() {
  @@ -328,7 +328,7 @@
       }
   
       public String getRequestURI() {
  -        return request.getRequestURI();
  +        return request.requestURI().toString();
       }
   
       /** Facade: we delegate to the right object ( the context )
  
  
  
  1.6       +4 -4      
jakarta-tomcat/src/facade23/org/apache/tomcat/facade23/HttpServletResponseFacade.java
  
  Index: HttpServletResponseFacade.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade23/org/apache/tomcat/facade23/HttpServletResponseFacade.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- HttpServletResponseFacade.java    2000/09/25 17:17:52     1.5
  +++ HttpServletResponseFacade.java    2000/11/02 21:51:45     1.6
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-tomcat/src/facade23/org/apache/tomcat/facade23/HttpServletResponseFacade.java,v
 1.5 2000/09/25 17:17:52 costin Exp $
  - * $Revision: 1.5 $
  - * $Date: 2000/09/25 17:17:52 $
  + * $Header: 
/home/cvs/jakarta-tomcat/src/facade23/org/apache/tomcat/facade23/HttpServletResponseFacade.java,v
 1.6 2000/11/02 21:51:45 costin Exp $
  + * $Revision: 1.6 $
  + * $Date: 2000/11/02 21:51:45 $
    *
    * ====================================================================
    *
  @@ -365,7 +365,7 @@
        }
   
        // Does this URL match down to (and including) the context path?
  -     if (!request.getScheme().equalsIgnoreCase(url.getProtocol()))
  +     if (!request.scheme().equalsIgnoreCase(url.getProtocol()))
            return (false);
        if (!request.getServerName().equalsIgnoreCase(url.getHost()))
            return (false);
  
  
  
  1.2       +3 -3      
jakarta-tomcat/src/facade23/org/apache/tomcat/facade23/RequestDispatcherImpl.java
  
  Index: RequestDispatcherImpl.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade23/org/apache/tomcat/facade23/RequestDispatcherImpl.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- RequestDispatcherImpl.java        2000/08/23 04:06:42     1.1
  +++ RequestDispatcherImpl.java        2000/11/02 21:51:45     1.2
  @@ -166,7 +166,7 @@
        // set the context - no need to fire context parsing again
        realRequest.setContext( context );
   
  -     realRequest.setRequestURI( context.getPath() + path );
  +     realRequest.requestURI().setString( context.getPath() + path );
   
        // merge query string as specified in specs - before, it may affect
        // the way the request is handled by special interceptors
  @@ -185,7 +185,7 @@
   
   
        // CM should have set the wrapper - call it
  -     Handler wr=realRequest.getWrapper();
  +     Handler wr=realRequest.getHandler();
        if( wr!=null ) wr.service(realRequest, realResponse);
   
        // close the response - output after this point will be discarded.
  @@ -302,7 +302,7 @@
        // now it's really strange: we call the wrapper on the subrequest
        // for the realRequest ( since the real request will still have the
        // original handler/wrapper )
  -     Handler wr=subRequest.getWrapper();
  +     Handler wr=subRequest.getHandler();
        if( wr!=null ) wr.service(realRequest, realResponse);
   
        // After request, we want to restore the include attributes - for
  
  
  
  1.7       +3 -2      
jakarta-tomcat/src/facade23/org/apache/tomcat/facade23/ServletWrapper.java
  
  Index: ServletWrapper.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade23/org/apache/tomcat/facade23/ServletWrapper.java,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -u -r1.6 -r1.7
  --- ServletWrapper.java       2000/10/01 06:37:43     1.6
  +++ ServletWrapper.java       2000/11/02 21:51:45     1.7
  @@ -249,7 +249,8 @@
   
            try {
                if( servlet!=null) {
  -                 BaseInterceptor 
cI[]=contextM.getInterceptors(context.getContainer());
  +                 BaseInterceptor cI[]=context.
  +                     getContainer().getInterceptors();
                    for( int i=0; i< cI.length; i++ ) {
                        try {
                            cI[i].preServletDestroy( context, this );
  @@ -320,7 +321,7 @@
        //}
   
        // Call pre, doInit and post
  -     BaseInterceptor cI[]=contextM.getInterceptors(context.getContainer());
  +     BaseInterceptor cI[]=context.getContainer().getInterceptors();
        for( int i=0; i< cI.length; i++ ) {
            try {
                cI[i].preServletInit( context, this );
  
  
  

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

Reply via email to