> 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.
> 

As a side note, if you want to upgrade but you're in a time crunch, there is a 
compatibility mode on this one:

public static void contributeApplicationDefaults(MappedConfiguration<String, 
String> config) {
    config.add(SymbolConstants.PAGE_POOL_ENABLED,"true");
}

This will restore the use of the page pool.

> - 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');
> 

See my previous e-mail reply to Andy for a way to restore the old behavior.

Robert

> 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

Reply via email to