Nikolaos,
Excellent summary...
I was working against the 1.5.3 source for my initial pass adding the
attribute to the LayoutDefinition. The problem that stumped me was I was
placing the attribute in scope for use by the HtmlTagSupport and Form
classes. I first tried the Page scope. When that didn¹t work, I tried the
Request Scope. That didn¹t work either. I¹m fairly certain the problem is
that render pages using the definition did not have a page or request scope
for the attribute set.
I considered Session and even Application. Session doesn¹t feel right.
Application actually makes sense as an web app is going to want an output
format across the complete application. However, at application level, it
should be a web.xml configuration I think. This actually isn¹t a bad idea
thinking about it, it becomes a deployment decision and the Stripes code is
aware of this during load, and the deployment documentation could describe
this to the deployer.
My last submitted patch is attached to the ticket that shows the direction I
was taking; maybe someone smarter than me could show me what I was doing
wrong in the Page or Request scopes... Don¹t use the first one, it will
throw an NPE, I attached it to quickly without testing it. :)
Regards,
Tim
On 9/8/10 2:46 PM, "Nikolaos (JIRA)" <[email protected]> wrote:
> 1) Timothy's suggestion: Adding an additional optional argument to
> LayoutDefinition ala Struts that defaults to xhtml=true for backwards
> compatability.
>
> Cons: Not all output is HTML fragments and not all output occurs within a
> LayoutDefinition tag (e.g. someone may be using Tiles or SiteMesh); this is
> besides the issues of getting this work with nested Layouts (which are
> problematic in 1.5.4 Snapshot).
--
Timothy Stone | Director, Application Developer Technical Lead
Barclaycard | Business Technology Group, Solutions Delivery
125 S. West Street | Wilmington, DE | 19801
USA | www.barclaycardus.com
+1 302 255 8044 direct | +1 410 441 9450 cell
COMPANY CONFIDENTIAL
P Before printing this e-mail, or any other document, ask yourself whether
you need a hard copy
Barclays www.barclaycardus.com
This e-mail and any files transmitted with it may contain confidential and/or
proprietary information. It is intended solely for the use of the individual or
entity who is the intended recipient. Unauthorized use of this information is
prohibited. If you have received this in error, please contact the sender by
replying to this message and delete this material from any system it may be on.
------------------------------------------------------------------------------
This SF.net Dev2Dev email is sponsored by:
Show off your parallel programming skills.
Enter the Intel(R) Threading Challenge 2010.
http://p.sf.net/sfu/intel-thread-sfd
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development