Author: mrdon
Date: Sat Oct 27 05:05:20 2007
New Revision: 589083

URL: http://svn.apache.org/viewvc?rev=589083&view=rev
Log:
Renamed default extension variable to be more clear

Modified:
    
struts/sandbox/trunk/struts2-rest-plugin/src/main/java/org/apache/struts2/rest/ContentTypeHandlerManager.java
    
struts/sandbox/trunk/struts2-rest-plugin/src/main/resources/struts-plugin.xml
    
struts/sandbox/trunk/struts2-rest-plugin/src/test/java/org/apache/struts2/rest/ContentTypeHandlerManagerTest.java

Modified: 
struts/sandbox/trunk/struts2-rest-plugin/src/main/java/org/apache/struts2/rest/ContentTypeHandlerManager.java
URL: 
http://svn.apache.org/viewvc/struts/sandbox/trunk/struts2-rest-plugin/src/main/java/org/apache/struts2/rest/ContentTypeHandlerManager.java?rev=589083&r1=589082&r2=589083&view=diff
==============================================================================
--- 
struts/sandbox/trunk/struts2-rest-plugin/src/main/java/org/apache/struts2/rest/ContentTypeHandlerManager.java
 (original)
+++ 
struts/sandbox/trunk/struts2-rest-plugin/src/main/java/org/apache/struts2/rest/ContentTypeHandlerManager.java
 Sat Oct 27 05:05:20 2007
@@ -44,12 +44,12 @@
 public class ContentTypeHandlerManager {
 
     Map<String,ContentTypeHandler> handlers = new 
HashMap<String,ContentTypeHandler>();
-    String defaultHandlerName;
+    String defaultExtension;
     public static final String STRUTS_REST_HANDLER_OVERRIDE_PREFIX = 
"struts.rest.handlerOverride.";
 
-    @Inject("struts.rest.defaultHandlerName")
-    public void setDefaultHandlerName(String name) {
-        this.defaultHandlerName = name;
+    @Inject("struts.rest.defaultExtension")
+    public void setDefaultExtension(String name) {
+        this.defaultExtension = name;
     }
 
     @Inject
@@ -80,7 +80,7 @@
     public ContentTypeHandler getHandlerForRequest(HttpServletRequest req) {
         String extension = findExtension(req.getRequestURI());
         if (extension == null) {
-            extension = defaultHandlerName;
+            extension = defaultExtension;
         }
         return handlers.get(extension);
     }

Modified: 
struts/sandbox/trunk/struts2-rest-plugin/src/main/resources/struts-plugin.xml
URL: 
http://svn.apache.org/viewvc/struts/sandbox/trunk/struts2-rest-plugin/src/main/resources/struts-plugin.xml?rev=589083&r1=589082&r2=589083&view=diff
==============================================================================
--- 
struts/sandbox/trunk/struts2-rest-plugin/src/main/resources/struts-plugin.xml 
(original)
+++ 
struts/sandbox/trunk/struts2-rest-plugin/src/main/resources/struts-plugin.xml 
Sat Oct 27 05:05:20 2007
@@ -18,7 +18,7 @@
     <bean type="org.apache.struts2.rest.handler.ContentTypeHandler" 
name="html" class="org.apache.struts2.rest.handler.HtmlHandler" />
 
     <constant name="struts.actionProxyFactory" value="rest" />
-    <constant name="struts.rest.defaultHandlerName" value="xml" />
+    <constant name="struts.rest.defaultExtension" value="xml" />
     <constant name="struts.mapper.class" value="rest" />
     <constant name="struts.mapper.idParameterName" value="id" />
     <constant name="struts.action.extension" value="xhtml,,xml,json" />

Modified: 
struts/sandbox/trunk/struts2-rest-plugin/src/test/java/org/apache/struts2/rest/ContentTypeHandlerManagerTest.java
URL: 
http://svn.apache.org/viewvc/struts/sandbox/trunk/struts2-rest-plugin/src/test/java/org/apache/struts2/rest/ContentTypeHandlerManagerTest.java?rev=589083&r1=589082&r2=589083&view=diff
==============================================================================
--- 
struts/sandbox/trunk/struts2-rest-plugin/src/test/java/org/apache/struts2/rest/ContentTypeHandlerManagerTest.java
 (original)
+++ 
struts/sandbox/trunk/struts2-rest-plugin/src/test/java/org/apache/struts2/rest/ContentTypeHandlerManagerTest.java
 Sat Oct 27 05:05:20 2007
@@ -78,7 +78,7 @@
             public String getExtension() { return "foo"; }
         };
         mgr.handlers.put("xml", handler);
-        mgr.defaultHandlerName = "xml";
+        mgr.defaultExtension = "xml";
         mgr.handleResult(new ActionConfig(), new 
DefaultHttpHeaders().withStatus(SC_OK), obj);
 
         assertEquals(obj.getBytes().length, mockResponse.getContentLength());


Reply via email to