You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by ni...@apache.org on 2006/03/08 16:18:12 UTC

svn commit: r384235 - /struts/action/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/TagUtils.java

Author: niallp
Date: Wed Mar  8 07:18:11 2006
New Revision: 384235

URL: http://svn.apache.org/viewcvs?rev=384235&view=rev
Log:
Port Fix for Bug 37817 to 1.2.x branch - TagUtils doesn't create XHTML compliant URLs using forwards defined with redirect="true" - thanks to Jason A. Lunn

Modified:
    struts/action/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/TagUtils.java

Modified: struts/action/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/TagUtils.java
URL: http://svn.apache.org/viewcvs/struts/action/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/TagUtils.java?rev=384235&r1=384234&r2=384235&view=diff
==============================================================================
--- struts/action/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/TagUtils.java (original)
+++ struts/action/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/TagUtils.java Wed Mar  8 07:18:11 2006
@@ -1,7 +1,7 @@
 /*
  * $Id$
  *
- * Copyright 1999-2005 The Apache Software Foundation.
+ * Copyright 1999-2006 The Apache Software Foundation.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -429,9 +429,10 @@
             if (forwardConfig == null) {
                 throw new MalformedURLException(messages.getMessage("computeURL.forward", forward));
             }
-            if (forwardConfig.getRedirect()) {
-                redirect = true;
-            }
+            // **** removed - see bug 37817 ****
+            //  if (forwardConfig.getRedirect()) {
+            //      redirect = true;
+            //  }
             if (forwardConfig.getPath().startsWith("/")) {
                 url.append(request.getContextPath());
                 url.append(RequestUtils.forwardURL(request, forwardConfig, moduleConfig));



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org