Uploaded image for project: 'Tapestry'
  1. Tapestry
  2. TAPESTRY-1336

Refactor ApplicationGlobals's store methods

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: 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!

        Attachments

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved: