Author: rich
Date: Thu Feb 10 20:44:00 2005
New Revision: 153352

URL: http://svn.apache.org/viewcvs?view=rev&rev=153352
Log:
This is a contribution from Carlin Rogers:

    "Change the URLRewriterService API to replace the method rewriteName() with 
another method getNamePrefix(). The new method should get the prefix to use to 
rewrite a URL query parameter name without requiring the name as an argument to 
the URLRewriter method implementations."

DRT: netui (WinXP)
BB: self (linux)


Modified:
    
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/PageFlowRequestProcessor.java
    
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/internal/DefaultURLRewriter.java
    
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/util/OldURLRewriterWrapper.java
    
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/util/URLRewriterService.java
    
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/AbstractClassicTag.java
    
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/html/Button.java
    
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/html/Form.java
    
incubator/beehive/trunk/netui/src/util/org/apache/beehive/netui/core/urls/URLRewriter.java
    
incubator/beehive/trunk/netui/src/util/org/apache/beehive/netui/core/urls/URLRewriterService.java

Modified: 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/PageFlowRequestProcessor.java
URL: 
http://svn.apache.org/viewcvs/incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/PageFlowRequestProcessor.java?view=diff&r1=153351&r2=153352
==============================================================================
--- 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/PageFlowRequestProcessor.java
 (original)
+++ 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/PageFlowRequestProcessor.java
 Thu Feb 10 20:44:00 2005
@@ -1632,8 +1632,8 @@
             String hash = Integer.toString( request.hashCode() );
             String key = makeRedirectedRequestAttrsKey( webappRelativeURI, 
hash );
             request.getSession().setAttribute( key, attrs );
-            queryString = URLRewriterService.rewriteName( servletContext, 
request, REDIRECT_REQUEST_ATTRS_PARAM )
-                            + '=' + hash;
+            queryString = URLRewriterService.getNamePrefix( servletContext, 
request )
+                          + REDIRECT_REQUEST_ATTRS_PARAM + '=' + hash;
         }
 
         

Modified: 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/internal/DefaultURLRewriter.java
URL: 
http://svn.apache.org/viewcvs/incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/internal/DefaultURLRewriter.java?view=diff&r1=153351&r2=153352
==============================================================================
--- 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/internal/DefaultURLRewriter.java
 (original)
+++ 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/internal/DefaultURLRewriter.java
 Thu Feb 10 20:44:00 2005
@@ -34,9 +34,9 @@
 {
     private static final Logger _log = Logger.getInstance( 
DefaultURLRewriter.class );
 
-    public String rewriteName( ServletContext servletContext, ServletRequest 
request, String name )
+    public String getNamePrefix( ServletContext servletContext, ServletRequest 
request )
     {
-        return name;
+        return "";
     }
 
     public void rewriteURL( ServletContext servletContext, ServletRequest 
request,

Modified: 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/util/OldURLRewriterWrapper.java
URL: 
http://svn.apache.org/viewcvs/incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/util/OldURLRewriterWrapper.java?view=diff&r1=153351&r2=153352
==============================================================================
--- 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/util/OldURLRewriterWrapper.java
 (original)
+++ 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/util/OldURLRewriterWrapper.java
 Thu Feb 10 20:44:00 2005
@@ -56,9 +56,9 @@
         return _oldURLRewriter;
     }
 
-    public String rewriteName( ServletContext servletContext, ServletRequest 
request, String name )
+    public String getNamePrefix( ServletContext servletContext, ServletRequest 
request )
     {
-        return _oldURLRewriter.rewriteName( servletContext, request, name );
+        return _oldURLRewriter.rewriteName( servletContext, request, "" );
     }
 
     public void rewriteURL( ServletContext servletContext, ServletRequest 
request,

Modified: 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/util/URLRewriterService.java
URL: 
http://svn.apache.org/viewcvs/incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/util/URLRewriterService.java?view=diff&r1=153351&r2=153352
==============================================================================
--- 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/util/URLRewriterService.java
 (original)
+++ 
incubator/beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/util/URLRewriterService.java
 Thu Feb 10 20:44:00 2005
@@ -60,7 +60,7 @@
      */ 
     public static String rewriteName(ServletContext servletContext, 
ServletRequest request, String name)
     {
-        return 
org.apache.beehive.netui.core.urls.URLRewriterService.rewriteName( 
servletContext, request, name );
+        return 
org.apache.beehive.netui.core.urls.URLRewriterService.getNamePrefix( 
servletContext, request ) + name;
     }
 
     /**

Modified: 
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/AbstractClassicTag.java
URL: 
http://svn.apache.org/viewcvs/incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/AbstractClassicTag.java?view=diff&r1=153351&r2=153352
==============================================================================
--- 
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/AbstractClassicTag.java
 (original)
+++ 
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/AbstractClassicTag.java
 Thu Feb 10 20:44:00 2005
@@ -274,7 +274,7 @@
      */
     final protected String rewriteName(String name)
     {
-        return URLRewriterService.rewriteName(pageContext.getServletContext(), 
pageContext.getRequest(), name);
+        return 
URLRewriterService.getNamePrefix(pageContext.getServletContext(), 
pageContext.getRequest()) + name;
     }
 
     /**

Modified: 
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/html/Button.java
URL: 
http://svn.apache.org/viewcvs/incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/html/Button.java?view=diff&r1=153351&r2=153352
==============================================================================
--- 
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/html/Button.java
 (original)
+++ 
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/html/Button.java
 Thu Feb 10 20:44:00 2005
@@ -277,8 +277,8 @@
 
                 String overrideAction = ACTION_OVERRIDE + _action;
                 overrideAction = HtmlUtils.addParams(overrideAction, _params, 
response.getCharacterEncoding());
-                String buttonOutput = 
URLRewriterService.rewriteName(pageContext.getServletContext(),
-                        pageContext.getRequest(), overrideAction);
+                String buttonOutput = 
URLRewriterService.getNamePrefix(pageContext.getServletContext(),
+                        pageContext.getRequest()) + overrideAction;
                 if (buttonOutput.indexOf(";") > -1) {
                     buttonOutput = buttonOutput.substring(0, 
buttonOutput.indexOf(";"));
                 }

Modified: 
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/html/Form.java
URL: 
http://svn.apache.org/viewcvs/incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/html/Form.java?view=diff&r1=153351&r2=153352
==============================================================================
--- 
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/html/Form.java
 (original)
+++ 
incubator/beehive/trunk/netui/src/tags-html/org/apache/beehive/netui/tags/html/Form.java
 Thu Feb 10 20:44:00 2005
@@ -764,12 +764,12 @@
                 if (paramValue instanceof String[]) {
                     String[] paramValArray = (String[]) paramValue;
                     for (int i = 0; i < paramValArray.length; i++) {
-                        String paramName = 
URLRewriterService.rewriteName(servletContext, request, paramKey.toString());
+                        String paramName = 
URLRewriterService.getNamePrefix(servletContext, request) + paramKey.toString();
                         writeHiddenParam(paramName, paramValArray[i], writer, 
request, true);
                     }
                 }
                 else {
-                    String paramName = 
URLRewriterService.rewriteName(servletContext, request, paramKey.toString());
+                    String paramName = 
URLRewriterService.getNamePrefix(servletContext, request) + paramKey.toString();
                     writeHiddenParam(paramName, paramValue.toString(), writer, 
request, true);
                 }
             }

Modified: 
incubator/beehive/trunk/netui/src/util/org/apache/beehive/netui/core/urls/URLRewriter.java
URL: 
http://svn.apache.org/viewcvs/incubator/beehive/trunk/netui/src/util/org/apache/beehive/netui/core/urls/URLRewriter.java?view=diff&r1=153351&r2=153352
==============================================================================
--- 
incubator/beehive/trunk/netui/src/util/org/apache/beehive/netui/core/urls/URLRewriter.java
 (original)
+++ 
incubator/beehive/trunk/netui/src/util/org/apache/beehive/netui/core/urls/URLRewriter.java
 Thu Feb 10 20:44:00 2005
@@ -51,14 +51,14 @@
     }
 
     /**
-     * Rewrite the given query parameter name.
-     * 
+     * Get the prefix to use when rewriting a query parameter name.
+     * Loops through the list of registered URLRewriters to build up a the 
prefix.
+     *
      * @param servletContext the current ServletContext.
      * @param request the current ServletRequest.
-     * @param name the query parameter name to rewrite.
-     * @return the rewritten query parameter name.
+     * @return a prefix to use to rewrite a query parameter name..
      */
-    public abstract String rewriteName( ServletContext servletContext, 
ServletRequest request, String name );
+    public abstract String getNamePrefix( ServletContext servletContext, 
ServletRequest request );
 
     /**
      * Rewrite the given URL.

Modified: 
incubator/beehive/trunk/netui/src/util/org/apache/beehive/netui/core/urls/URLRewriterService.java
URL: 
http://svn.apache.org/viewcvs/incubator/beehive/trunk/netui/src/util/org/apache/beehive/netui/core/urls/URLRewriterService.java?view=diff&r1=153351&r2=153352
==============================================================================
--- 
incubator/beehive/trunk/netui/src/util/org/apache/beehive/netui/core/urls/URLRewriterService.java
 (original)
+++ 
incubator/beehive/trunk/netui/src/util/org/apache/beehive/netui/core/urls/URLRewriterService.java
 Thu Feb 10 20:44:00 2005
@@ -57,26 +57,29 @@
     private static final DefaultTemplatedURLFormatter defaultFormatter = new 
DefaultTemplatedURLFormatter();
 
     /**
-     * Rewrite the given parameter name, looping through the list of 
registered URLRewriters.
+     * Get the prefix to use when rewriting a query parameter name.
+     * Loops through the list of registered URLRewriters to build up a the 
prefix.
      *
      * @param servletContext the current ServletContext.
      * @param request        the current ServletRequest.
-     * @param name           the parameter name to rewrite.
-     * @return the rewritten parameter name.
+     * @return a prefix to use to rewrite a query parameter name.
      */
-    public static String rewriteName( ServletContext servletContext, 
ServletRequest request, String name )
+    public static String getNamePrefix( ServletContext servletContext, 
ServletRequest request )
     {
         ArrayList< URLRewriter > rewriters = getRewriters( request );
 
+        StringBuilder prefix = new StringBuilder();
+
         if ( rewriters != null )
         {
             for ( URLRewriter rewriter : rewriters )
             {
-                name = rewriter.rewriteName( servletContext, request, name );
+                String nextPrefix = rewriter.getNamePrefix( servletContext, 
request );
+                if ( nextPrefix != null ) { prefix.append( nextPrefix ); }
             }
         }
 
-        return name;
+        return prefix.toString();
     }
 
     /**


Reply via email to