Uploaded image for project: 'Tapestry 5'
  1. Tapestry 5
  2. TAP5-216

Many methods of the Page interface could be moved to PageResources (which should be renamed to ComponentPageElementResources)

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 5.1.0.0
    • Fix Version/s: 5.1.0.0
    • Component/s: None
    • Labels:
      None

      Description

      Just an observation; many methods on Page were done to cut down on the number of injections passed to each ComponentPageElementImpl ... however, since then the PageResources interface was introduced to act as a facade over services needed by the ComponenPageElementImpl.

      I think several methods of Page could be moved to PageResources, which would simplify PageImpl and the (increasing number of) dependencies passed to it.

      Also, PageResourcs could be renamed ComponentPageElementResources, and a new PageResources facaded could be introduced for PageImpl.

        Attachments

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved: