amyroh      2003/02/09 20:25:26

  Modified:    webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve
                        SaveAccessLogValveAction.java
                        SaveRemoteAddrValveAction.java
                        SaveRemoteHostValveAction.java
                        SaveRequestDumperValveAction.java
                        SaveSingleSignOnValveAction.java ValveUtil.java
  Log:
  Correct the behavior of valve creation - it wasn't saving its attributes during 
creation.
  
  Revision  Changes    Path
  1.4       +7 -7      
jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveAccessLogValveAction.java
  
  Index: SaveAccessLogValveAction.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveAccessLogValveAction.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- SaveAccessLogValveAction.java     22 Nov 2002 21:49:46 -0000      1.3
  +++ SaveAccessLogValveAction.java     10 Feb 2003 04:25:26 -0000      1.4
  @@ -155,9 +155,9 @@
           // Perform a "Create Valve" transaction (if requested)
           if ("Create".equals(adminAction)) {
           
  -           return ValveUtil.
  -                  createValve(parent, valveType, response, request, mapping,
  -                  (ApplicationServlet) getServlet());
  +            vObjectName = ValveUtil.createValve(parent, valveType, 
  +                                response, request, mapping, 
  +                                (ApplicationServlet) getServlet());
              
           }
   
  
  
  
  1.4       +8 -7      
jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRemoteAddrValveAction.java
  
  Index: SaveRemoteAddrValveAction.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRemoteAddrValveAction.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- SaveRemoteAddrValveAction.java    12 Apr 2002 22:06:30 -0000      1.3
  +++ SaveRemoteAddrValveAction.java    10 Feb 2003 04:25:26 -0000      1.4
  @@ -154,9 +154,10 @@
                  
           // Perform a "Create Valve" transaction (if requested)
           if ("Create".equals(adminAction)) {
  -           return ValveUtil.
  -                  createValve(parent, valveType, response, request, mapping,
  -                  (ApplicationServlet) getServlet());
  +
  +            vObjectName = ValveUtil.createValve(parent, valveType, 
  +                                response, request, mapping, 
  +                                (ApplicationServlet) getServlet());
              
           }
   
  
  
  
  1.4       +7 -7      
jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRemoteHostValveAction.java
  
  Index: SaveRemoteHostValveAction.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRemoteHostValveAction.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- SaveRemoteHostValveAction.java    12 Apr 2002 22:06:30 -0000      1.3
  +++ SaveRemoteHostValveAction.java    10 Feb 2003 04:25:26 -0000      1.4
  @@ -155,9 +155,9 @@
           // Perform a "Create Valve" transaction (if requested)
           if ("Create".equals(adminAction)) {
   
  -           return ValveUtil.
  -                  createValve(parent, valveType, response, request, mapping,
  -                  (ApplicationServlet) getServlet());
  +            vObjectName = ValveUtil.createValve(parent, valveType, 
  +                                response, request, mapping, 
  +                                (ApplicationServlet) getServlet());
              
           }
   
  
  
  
  1.3       +7 -7      
jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRequestDumperValveAction.java
  
  Index: SaveRequestDumperValveAction.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRequestDumperValveAction.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- SaveRequestDumperValveAction.java 12 Apr 2002 22:06:30 -0000      1.2
  +++ SaveRequestDumperValveAction.java 10 Feb 2003 04:25:26 -0000      1.3
  @@ -155,9 +155,9 @@
           // Perform a "Create Valve" transaction (if requested)
           if ("Create".equals(adminAction)) {
           
  -           return ValveUtil.
  -                  createValve(parent, valveType, response, request, mapping,
  -                  (ApplicationServlet) getServlet());
  +            vObjectName = ValveUtil.createValve(parent, valveType, 
  +                                response, request, mapping, 
  +                                (ApplicationServlet) getServlet());
              
           }
   
  
  
  
  1.5       +7 -7      
jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveSingleSignOnValveAction.java
  
  Index: SaveSingleSignOnValveAction.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveSingleSignOnValveAction.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- SaveSingleSignOnValveAction.java  18 Apr 2002 21:39:45 -0000      1.4
  +++ SaveSingleSignOnValveAction.java  10 Feb 2003 04:25:26 -0000      1.5
  @@ -156,9 +156,9 @@
           // Perform a "Create Valve" transaction (if requested)
           if ("Create".equals(adminAction)) {
   
  -           return ValveUtil.
  -                  createValve(parent, valveType, response, request, mapping,
  -                  (ApplicationServlet) getServlet());
  +            vObjectName = ValveUtil.createValve(parent, valveType, 
  +                                response, request, mapping, 
  +                                (ApplicationServlet) getServlet());
                         
           }
   
  
  
  
  1.8       +10 -9     
jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/ValveUtil.java
  
  Index: ValveUtil.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/ValveUtil.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- ValveUtil.java    9 May 2002 01:07:31 -0000       1.7
  +++ ValveUtil.java    10 Feb 2003 04:25:26 -0000      1.8
  @@ -112,7 +112,7 @@
       
       // --------------------------------------------------------- Public Methods
       
  -    public static ActionForward createValve(String parent, String valveType,
  +    public static String createValve(String parent, String valveType,
       HttpServletResponse response, HttpServletRequest request,
       ActionMapping mapping, ApplicationServlet servlet)
       throws IOException, ServletException {
  @@ -132,6 +132,7 @@
           
           String operation = null;
           String values[] = null;
  +        String vObjectName = null;
           
           try {
               
  @@ -157,8 +158,7 @@
                   servlet.log(message);
                   response.sendError(HttpServletResponse.SC_BAD_REQUEST, message);
                   return (null);
  -            }
  -                        
  +            }    
               // Ensure that the requested valve name is unique
               
               // TBD -- do we need this check?
  @@ -187,7 +187,8 @@
               operation = "create" + valveType;
               if ("AccessLogValve".equalsIgnoreCase(valveType))
                   operation = "createAccessLoggerValve";
  -            String vObjectName = (String)
  +                
  +            vObjectName = (String)
                           mBServer.invoke(fname, operation, values, 
createStandardValveTypes);
               
               // Add the new Valve to our tree control node
  @@ -235,6 +236,6 @@
           
           // Forward to the success reporting page
           session.removeAttribute(mapping.getAttribute());
  -        return (mapping.findForward("Save Successful"));
  +        return vObjectName;
       }
   }
  
  
  

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

Reply via email to