|Summary:||org.apache.catalina.session.StoreBase has a hidden dependency on PersistentManager|
|Product:||Tomcat 7||Reporter:||Tiago Batista <tsbatista>|
|Component:||Catalina||Assignee:||Tomcat Developers Mailing List <dev>|
Removes hidden dependency on PersistentStore
Removes the isLoaded method from PersistentManagerBase
Description Tiago Batista 2010-11-08 05:09:39 UTC
Created attachment 26270 [details] Removes hidden dependency on PersistentStore StoreBase.java depends explicitly on the Manager interface, however, it later casts the manager object to the PersistentManager class with out any check. This removes that dependency, therefore allowing the usage of this store with any manager that does not extend PersistentManager. I will later attach a path that removes the offending isLoaded() method as that may induce the same problem further along...
Comment 1 Tiago Batista 2010-11-08 05:19:24 UTC
Created attachment 26271 [details] Removes the isLoaded method from PersistentManagerBase This patch removes the isLoaded method that was previously called only by StoreBase. This is optional, but as that particular path of code is no longer used, it seemed like the right thing to do.
Comment 2 Mark Thomas 2010-11-09 10:32:48 UTC
Thanks for the patches. Fixed in trunk (with a slight modification) and will be included in 7.0.5 onwards.