On Wed, 27 Jul 2011 10:26:06 -0300, Markus Jung <[email protected]>
wrote:

Hi,

Hi!

I found a way to enable mixin based protection of the event links rendered by an AjaxFormLoop component. If there is access to the inits map of the
JavaScriptSupportImpl class, I can modify the links in the mixin.

Couldn't you find a solution that doesn't involve changing the inits?
Using addScript() isntead of addInit()?

I think adding this getter returning the Map breaks encapsulation too hard
and opens Tapestry for subtle, difficult to track bugs. I'd add add() and
set() methods delegating to the inits fields instead if access to them is
really, really necessary.

--
Thiago H. de Paula Figueiredo
Independent Java, Apache Tapestry 5 and Hibernate consultant, developer,
and instructor
Owner, Ars Machina Tecnologia da Informação Ltda.
http://www.arsmachina.com.br

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to