[TAPESTRY-1336] Refactor ApplicationGlobals's store methods

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 5.0.3
• Fix Version/s: 5.0.11
• Component/s: tapestry-core
• Assignee: Howard Lewis Ship
• Reporter: liigo
• Created: 09/Mar/07 10:30
• Updated: 03/Mar/08 17:10
• Resolved: 03/Mar/08 17:10

Description

there is three "store" methods (with diffrent parameter type) in ApplicationGlobals

void store(ServletContext context);
void store(Context context);
void store(String applicationRootPackage);

if we need another "store" method with a String parameter in the future, what to do with it? store(String) is already there!

so rename these methods to storeServletContext(), storeContext(), storeApplicationRootPackage() maybe ok, and clear its meaning.

Sorry for poor english!