You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by dk...@apache.org on 2008/09/12 18:00:56 UTC

svn commit: r694717 - in /cxf/branches/2.1.x-fixes: ./ maven-plugins/xml2fastinfoset-plugin/src/main/java/org/apache/cxf/maven_plugin/xml2fastinfoset/ rt/frontend/jaxws/src/main/resources/META-INF/cxf/ rt/transports/http/src/main/java/org/apache/cxf/tr...

Author: dkulp
Date: Fri Sep 12 09:00:56 2008
New Revision: 694717

URL: http://svn.apache.org/viewvc?rev=694717&view=rev
Log:
Merged revisions 694716 via svnmerge from 
https://svn.apache.org/repos/asf/cxf/trunk

........
  r694716 | dkulp | 2008-09-12 11:59:00 -0400 (Fri, 12 Sep 2008) | 3 lines
  
  [CXF-1782, CXF-1796] Updates to Servlet controller stuff to make it more subclassable
  Reduce logging in fastinfoset plugin
........

Modified:
    cxf/branches/2.1.x-fixes/   (props changed)
    cxf/branches/2.1.x-fixes/maven-plugins/xml2fastinfoset-plugin/src/main/java/org/apache/cxf/maven_plugin/xml2fastinfoset/XML2FastInfosetCompilerMojo.java
    cxf/branches/2.1.x-fixes/rt/frontend/jaxws/src/main/resources/META-INF/cxf/cxf-extension-jaxws.xml
    cxf/branches/2.1.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractCXFServlet.java
    cxf/branches/2.1.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/ServletController.java

Propchange: cxf/branches/2.1.x-fixes/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Fri Sep 12 09:00:56 2008
@@ -1 +1 @@
-/cxf/trunk:686333-686363,686764,686820,687096,687194,687363,687387,687463,687543,687722,687798,687814,687817,687891,687910,687914,688086,688102,688133,688596,688735,688870,689572,689596,689855,689924,690067,690289,691246,691271,691295,691355,691488,691602,691646,691706,691728,692116,692157,692310,692466,692499,693653,693819,694179,694263,694417
+/cxf/trunk:686333-686363,686764,686820,687096,687194,687363,687387,687463,687543,687722,687798,687814,687817,687891,687910,687914,688086,688102,688133,688596,688735,688870,689572,689596,689855,689924,690067,690289,691246,691271,691295,691355,691488,691602,691646,691706,691728,692116,692157,692310,692466,692499,693653,693819,694179,694263,694417,694716

Propchange: cxf/branches/2.1.x-fixes/
------------------------------------------------------------------------------
Binary property 'svnmerge-integrated' - no diff available.

Modified: cxf/branches/2.1.x-fixes/maven-plugins/xml2fastinfoset-plugin/src/main/java/org/apache/cxf/maven_plugin/xml2fastinfoset/XML2FastInfosetCompilerMojo.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.1.x-fixes/maven-plugins/xml2fastinfoset-plugin/src/main/java/org/apache/cxf/maven_plugin/xml2fastinfoset/XML2FastInfosetCompilerMojo.java?rev=694717&r1=694716&r2=694717&view=diff
==============================================================================
--- cxf/branches/2.1.x-fixes/maven-plugins/xml2fastinfoset-plugin/src/main/java/org/apache/cxf/maven_plugin/xml2fastinfoset/XML2FastInfosetCompilerMojo.java (original)
+++ cxf/branches/2.1.x-fixes/maven-plugins/xml2fastinfoset-plugin/src/main/java/org/apache/cxf/maven_plugin/xml2fastinfoset/XML2FastInfosetCompilerMojo.java Fri Sep 12 09:00:56 2008
@@ -106,7 +106,7 @@
             }
 
             if (!resourceDirectory.exists()) {
-                getLog().info("Resource directory does not exist: " + resourceDirectory);
+                getLog().debug("Resource directory does not exist: " + resourceDirectory);
                 continue;
             }
 
@@ -136,8 +136,7 @@
 
             List includedFiles = Arrays.asList(scanner.getIncludedFiles());
 
-            getLog().info(
-                          "FastInfosetting " + includedFiles.size() + " resource"
+            getLog().debug("FastInfosetting " + includedFiles.size() + " resource"
                               + (includedFiles.size() > 1 ? "s" : "")
                               + (targetPath == null ? "" : " to " + targetPath));
 

Modified: cxf/branches/2.1.x-fixes/rt/frontend/jaxws/src/main/resources/META-INF/cxf/cxf-extension-jaxws.xml
URL: http://svn.apache.org/viewvc/cxf/branches/2.1.x-fixes/rt/frontend/jaxws/src/main/resources/META-INF/cxf/cxf-extension-jaxws.xml?rev=694717&r1=694716&r2=694717&view=diff
==============================================================================
--- cxf/branches/2.1.x-fixes/rt/frontend/jaxws/src/main/resources/META-INF/cxf/cxf-extension-jaxws.xml (original)
+++ cxf/branches/2.1.x-fixes/rt/frontend/jaxws/src/main/resources/META-INF/cxf/cxf-extension-jaxws.xml Fri Sep 12 09:00:56 2008
@@ -27,5 +27,7 @@
     <bean class="org.apache.cxf.jaxws.context.WebServiceContextResourceResolver" 
 	  id="org.apache.cxf.jaxws.context.WebServiceContextResourceResolver" lazy-init="false"/>
     <bean class="org.apache.cxf.jaxws.context.WebServiceContextImpl" 
-	  id="org.apache.cxf.jaxws.context.WebServiceContextImpl"/>
+	  id="org.apache.cxf.jaxws.context.WebServiceContextImpl" lazy-init="true"/>
+    <bean class="org.apache.cxf.jaxws.context.WebServiceContextImpl" 
+          id="javax.xml.ws.WebServiceContext" lazy-init="true"/>
 </beans>

Modified: cxf/branches/2.1.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractCXFServlet.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.1.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractCXFServlet.java?rev=694717&r1=694716&r2=694717&view=diff
==============================================================================
--- cxf/branches/2.1.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractCXFServlet.java (original)
+++ cxf/branches/2.1.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/AbstractCXFServlet.java Fri Sep 12 09:00:56 2008
@@ -53,7 +53,8 @@
     
     public ServletController createServletController(ServletConfig servletConfig) {
         String hideServiceList = servletConfig.getInitParameter("hide-service-list-page");
-        ServletController newController = new ServletController(servletTransportFactory, this);
+        ServletController newController =
+            new ServletController(servletTransportFactory, this.getServletContext(), bus);
         if (hideServiceList != null) {
             newController.setHideServiceList(Boolean.valueOf(hideServiceList));
         }

Modified: cxf/branches/2.1.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/ServletController.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.1.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/ServletController.java?rev=694717&r1=694716&r2=694717&view=diff
==============================================================================
--- cxf/branches/2.1.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/ServletController.java (original)
+++ cxf/branches/2.1.x-fixes/rt/transports/http/src/main/java/org/apache/cxf/transport/servlet/ServletController.java Fri Sep 12 09:00:56 2008
@@ -29,6 +29,7 @@
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
+import javax.servlet.ServletContext;
 import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
@@ -48,16 +49,18 @@
     private static final Logger LOG = LogUtils.getL7dLogger(ServletController.class);
 
     private ServletTransportFactory transport;
-    private AbstractCXFServlet cxfServlet;
+    private ServletContext servletContext;
+    private Bus bus;
     private String lastBase = "";
     private boolean isHideServiceList;
     private boolean disableAddressUpdates;
     private String forcedBaseAddress;
     private String serviceListStyleSheet;
  
-    public ServletController(ServletTransportFactory df, AbstractCXFServlet servlet) {
+    public ServletController(ServletTransportFactory df, ServletContext context, Bus b) {
         this.transport = df;
-        this.cxfServlet = servlet;
+        this.servletContext = context;
+        this.bus = b;
         df.setServletController(this);
     }
     
@@ -130,7 +133,6 @@
                 }
             } else {
                 ei = d.getEndpointInfo();
-                Bus bus = cxfServlet.getBus();
                 if (null != request.getQueryString() 
                     && request.getQueryString().length() > 0
                     && bus.getExtension(QueryHandlerRegistry.class) != null) {                    
@@ -182,7 +184,7 @@
         return null; 
     }
     
-    private void generateServiceList(HttpServletRequest request, HttpServletResponse response)
+    protected void generateServiceList(HttpServletRequest request, HttpServletResponse response)
         throws IOException {
         
         if (request.getParameter("stylesheet") != null) {
@@ -280,7 +282,7 @@
         }
 
         try {
-            d.invoke(cxfServlet.getServletContext(), request, response);
+            d.invoke(servletContext, request, response);
         } catch (IOException e) {
             throw new ServletException(e);
         } finally {