Re: possible bug: onActivate/onDeactivate and ComponentActions

2007-12-01 Thread Fernando Padilla
The problem is that we develop with debug. And in tapestry it prints out the full generated source of the page as debug, basically making the first few requests on the webapp totally useless as far as debug goes... is there any way to make that source easier for us to parse out? move it to T

Re: possible bug: onActivate/onDeactivate and ComponentActions

2007-12-01 Thread Andy Huhn
I think console warnings are a great thing, and I hope you're not thinking about doing away with them! Maybe it's just because I'm a newbie, but I comb through those logs looking for problems if I get any unexpected behavior. Thanks, Andy On Sat, 2007-12-01 at 11:54 -0800, Howard Lewis Ship wrot

Re: possible bug: onActivate/onDeactivate and ComponentActions

2007-12-01 Thread Howard Lewis Ship
Tapestry generates warnings about any non-private fields it finds in a component class, so you might want to keep an eye on the console as well. Sometimes I think console warnings are useless as no one ever seems to see them, only by being "in your face" with a hard exception can you catch the dev

Re: possible bug: onActivate/onDeactivate and ComponentActions

2007-12-01 Thread Fernando Padilla
Yeah, they were public. Making all my variables private is one of those things I do without thinking, and I forgot to check that in this case. Funny. I don't remember reading about tapestry setting all private variables to null (or their default values) in the docs. Maybe I skimmed that part

Re: possible bug: onActivate/onDeactivate and ComponentActions

2007-11-30 Thread Howard Lewis Ship
What possible state would you have to clean up? Assuming you are using proper instance variables (all private), then Tapestry already does end-of-request cleanup, resetting all your fields to correct default values (for those that are not set to fixed values via an injection). On Nov 30, 2007 6:3

possible bug: onActivate/onDeactivate and ComponentActions

2007-11-30 Thread Fernando Padilla
The bug we're seeing is that the page context is being set by an onActivate call, but it's never being cleaned up. So the next page render still has the old page context setup, and we don't have a chance to re-null it. normally while handling a page: onActivate setupRender ... cleanupRender W