Thanks, I've added that one too. (These should appear at http://tapestry.apache.org/release-notes-52.html within an hour or two, after the documentation site export occurs.)
On Mon, Jan 17, 2011 at 4:13 PM, Donny Nadolny <donny.nado...@gmail.com> wrote: > There's another one I came across looking through JIRA. > > Copied from https://issues.apache.org/jira/browse/TAP5-1401 > > In Tapestry 5.1.0.5, you can have code such as: > > @Component > private Form inputs; > ... > if (inputs.getDefaultTracker().getHasErrors()) > ... > > This fails in 5.2.4 because getDefaultTracker() returns null if there are no > errors. > > The fix is simple - call inputs.getHasErrors() directly, rather than getting > the tracker and calling getHasErrors() on that. > > On Mon, Jan 17, 2011 at 3:55 PM, Bob Harner <bobhar...@gmail.com> wrote: > >> I took Donny's recommendations from last month as the basis for a new >> "Breaking Changes" section at the top of the >> https://cwiki.apache.org/confluence/display/TAPESTRY/Release+Notes+5.2 >> page. Does anybody know of any other "breaking" changes going from >> 5.1 to 5.2? >> >> On Fri, Dec 17, 2010 at 12:29 PM, Donny Nadolny <donny.nado...@gmail.com> >> wrote: >> > There's been a few backwards-incompatible changes from 5.1 to 5.2, I >> > think it would be helpful to have a place where these are documented. >> > Since you can't edit the wiki until you sign and mail/fax/scan & email >> > a license agreement, we can document them here and let someone with >> > access make the wiki page for it. >> > >> > This should probably be split in to a few sections, maybe like this: >> > 1. necessary changes for all applications (eg changed behavior of >> > Link.toAbsoluteURI()) >> > 2. suggested changes for all applications (eg use IncludeStylesheet is >> > deprecated, use Import) >> > >> > I'm not sure if there are any changes that meet these criteria, but if >> > there are they should be split up too: >> > 3. necessary changes for applications that implemented/extended public >> apis >> > 4. suggested changes for applications that implemented/extended public >> apis >> > >> > Here are the ones I know of right now: >> > 1. Necessary changes for all applications >> > - Page classes with instance variables that are not thread safe must >> > be created in a method rather than declared. For example, creating an >> > instance variable "private final DateFormat format = >> > DateFormat.getDateInstance(DateFormat.MEDIUM, locale);" in a page and >> > using it will cause problems because DateFormat is not thread safe. >> > Instead, you must create the DateFormat in a method. See >> > https://cwiki.apache.org/confluence/display/TAPESTRY/Release+Notes+5.2.0 >> > for details. >> > >> > - Link.toAbsoluteURI() now returns a full URI rather than just the >> > path. Previously it would return, for example, "/myapp/viewproduct/4". >> > Now it returns "http://example.com:8080/myapp/viewproduct/4". All >> > calls to toAbsoluteURI() should be replaced with toURI(). (I got hit >> > with this problem - I was generating links for an RSS feed and >> > prepending http://mydomain.com, so after upgrading the URLs were >> > http://mydomain.comhttp://mydomain.com:8080/viewproduct/4) >> > >> > - Label component no longer outputs an id (credit to Andy Blower for >> > noticing). Previously valid code in 5.1.0.5: >> > <t:form> >> > <t:label for="search"/> >> > <t:textfield t:id="search" size="50"/> >> > </t:form> >> > Combined with Javascript that references the id: >> > jQuery('#search-label').text('This is the search label'); >> > >> > New javascript code for 5.2: >> > jQuery('label[for=search]').text('This is the search label'); >> > >> > 2. Suggested changes for all applications: >> > - Replace @IncludeStylesheet and @IncludeJavaScriptLibrary with @Import. >> > >> > Donny >> > >> > --------------------------------------------------------------------- >> > To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org >> > For additional commands, e-mail: users-h...@tapestry.apache.org >> > >> > >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org >> For additional commands, e-mail: users-h...@tapestry.apache.org >> >> > --------------------------------------------------------------------- To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org For additional commands, e-mail: users-h...@tapestry.apache.org