I was looking at the history of revisions for FormTag in the taglib sub project using ViewCVS - there is no record of Martin's Revision 123447.
However looking into it further - the code changes to FormTag are there but they're showing under revision 12703 - which was Don's revision extracting taglibs to their own sub project. So looks like we lost the history? Niall ----- Original Message ----- From: "James Mitchell" <[EMAIL PROTECTED]> To: "Struts Developers List" <dev@struts.apache.org> Sent: Wednesday, February 02, 2005 12:36 AM Subject: Re: Missing svn commit: r123447 > What do you mean? The files are there, did we lose the history or > properties or something? > > -- > James Mitchell > Software Engineer / Open Source Evangelist > EdgeTech, Inc. > 678.910.8017 > AIM: jmitchtx > > ----- Original Message ----- > From: "Niall Pemberton" <[EMAIL PROTECTED]> > To: "Struts Developers List" <dev@struts.apache.org> > Sent: Tuesday, February 01, 2005 12:56 PM > Subject: Missing svn commit: r123447 > > > > This commit seems to have gone missing - I don't see it in the new > > "taglib" > > sub-project. I'm guessing this happened when Don extracted the taglibs out > > of core and into their own sub-project? All I can see from the commit > > message when taglibs were moved on 30/12/2004 is the following... > > > > struts/taglib/trunk/src/java/org/apache/struts/taglib/html/ > > - copied from r122972, > > struts/core/trunk/src/share/org/apache/struts/taglib/html/ > > > > Am wondering if we've lost anything else in the re-organisation? > > > > Niall > > > > ----- Original Message ----- > > From: <[EMAIL PROTECTED]> > > To: <[EMAIL PROTECTED]> > > Sent: Monday, December 27, 2004 8:20 PM > > Subject: svn commit: r123447 - > > /struts/core/trunk/src/share/org/apache/struts/taglib/html/FormTag.java > > > > > >> Author: martinc > >> Date: Mon Dec 27 12:20:32 2004 > >> New Revision: 123447 > >> > >> URL: http://svn.apache.org/viewcvs?view=rev&rev=123447 > >> Log: > >> Bug #32016: Wrap the hidden token field in a <div> element for HTML 4.01 > > Strict compliance. > >> Modified: > >> struts/core/trunk/src/share/org/apache/struts/taglib/html/FormTag.java > >> > >> Modified: > > struts/core/trunk/src/share/org/apache/struts/taglib/html/FormTag.java > >> Url: > > http://svn.apache.org/viewcvs/struts/core/trunk/src/share/org/apache/struts/taglib/html/FormTag.java?view=diff&rev=123447&p1=struts/core/trunk/src/share/org/apache/struts/taglib/html/FormTag.java&r1=123446&p2=struts/core/trunk/src/share/org/apache/struts/taglib/html/FormTag.java&r2=123447 > >> > > ============================================================================ > > == > >> --- > >> struts/core/trunk/src/share/org/apache/struts/taglib/html/FormTag.java > > (original) > >> +++ > >> struts/core/trunk/src/share/org/apache/struts/taglib/html/FormTag.java > > Mon Dec 27 12:20:32 2004 > >> @@ -529,7 +529,8 @@ > >> } > >> > >> /** > >> - * Generates a hidden input field with token information, if any. > >> + * Generates a hidden input field with token information, if any. > >> The > >> + * field is added within a div element for HTML 4.01 Strict > > compliance. > >> * @return A hidden input field containing the token. > >> * @since Struts 1.1 > >> */ > >> @@ -542,7 +543,7 @@ > >> (String) > > session.getAttribute(Globals.TRANSACTION_TOKEN_KEY); > >> > >> if (token != null) { > >> - results.append("<input type=\"hidden\" name=\""); > >> + results.append("<div><input type=\"hidden\" name=\""); > >> results.append(Constants.TOKEN_KEY); > >> results.append("\" value=\""); > >> results.append(token); > >> @@ -551,6 +552,7 @@ > >> } else { > >> results.append("\">"); > >> } > >> + results.append("</div>"); > >> } > >> } > >> > > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]