Bug in RequestLocaleManagerImpl?

["Michael J. Long" <Mi...@cornerstone.net>]

I believe I have found a bug in RequestLocaleManagerImpl due to it  
being a pooled service.

There is a member variable, _lastPersisted, which stores the last  
Locale that was persisted (i.e., stored as a cookie). However, since  
RequestLocaleManager is a pooled resource, that instance may be used  
by another request that does not come from the same client (i.e., the  
_lastPersisted does not apply).

I believe the RequestLocaleManagerImpl should implement the  
PoolManageable interface and set the _lastPersisted to null in the  
passivateService method. Is this the correct solution?


---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org