This change undoes a change made by Craig to fix Bugzilla #10534 last Sunday. It looks like we need a different way to fix both Bugzilla #10534 and Tiles...
-- Martin Cooper > -----Original Message----- > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]] > Sent: Wednesday, July 10, 2002 7:57 AM > To: [EMAIL PROTECTED] > Subject: cvs commit: jakarta-struts/src/share/org/apache/struts/util > RequestUtils.java > > > cedric 2002/07/10 07:57:10 > > Modified: src/share/org/apache/struts/util RequestUtils.java > Log: > Remove the automatic '/' prefixing in forwardURL(). This > systematic prefixing > forbids the use of Tiles definition name as path in forward. > It is the user responsability to put the '/' before an URL. > > Revision Changes Path > 1.47 +10 -4 > jakarta-struts/src/share/org/apache/struts/util/RequestUtils.java > > Index: RequestUtils.java > =================================================================== > RCS file: > /home/cvs/jakarta-struts/src/share/org/apache/struts/util/Requ > estUtils.java,v > retrieving revision 1.46 > retrieving revision 1.47 > diff -u -r1.46 -r1.47 > --- RequestUtils.java 10 Jul 2002 00:00:19 -0000 1.46 > +++ RequestUtils.java 10 Jul 2002 14:57:10 -0000 1.47 > @@ -1191,9 +1191,11 @@ > // Handle a ForwardConfig marked as context relative > StringBuffer sb = new StringBuffer(); > if (forward.getContextRelative()) { > + /* Disabled to let Tiles definition names unchanged ! > if (!path.startsWith("/")) { > sb.append("/"); > } > + */ > sb.append(path); > return (sb.toString()); > } > @@ -1206,9 +1208,11 @@ > if (forwardPattern == null) { > // Performance optimization for previous > default behavior > sb.append(appConfig.getPrefix()); > + /* Disabled to let Tiles definition names unchanged ! > if (!path.startsWith("/")) { > sb.append("/"); > } > + */ > sb.append(path); > } else { > boolean dollar = false; > @@ -1220,9 +1224,11 @@ > sb.append(appConfig.getPrefix()); > break; > case 'P': > + /* Disabled to let Tiles > definition names unchanged ! > if (!path.startsWith("/")) { > sb.append("/"); > } > + */ > sb.append(path); > break; > case '$': > > > > > -- > To unsubscribe, e-mail: > <mailto:[EMAIL PROTECTED]> > For additional commands, e-mail: > <mailto:[EMAIL PROTECTED]> > > -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
