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!

        Activity

        There are no comments yet on this issue.

          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:

              Development