Tapestry
  1. Tapestry
  2. TAPESTRY-1336

Refactor ApplicationGlobals's store methods

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 5.0.3
    • Fix Version/s: 5.0.11
    • Component/s: tapestry-core
    • Labels:
      None

      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!

        Activity

        There are no comments yet on this issue.

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            liigo
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development