Tapestry 5
  1. Tapestry 5
  2. TAP5-76

Provide decorators service matching based on annotations

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 5.0.15
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      It would be very useful if we could match services for decoration using the services' annotations. In spite of documentation saying it can be done, it can't:

      Tapestry IoC documentation:

      "Note: It is not possible to decorate the services of the TapestryIOCModule."

      "Note: Another idea will be other ways of matching services: base on inheritance of the service interface and/or based on the presence of particular class annotations on the service interface. None of this has been implemented yet, and can readily be accompllished inside the decorator method (which will return null if it decides the service doesn't need decoration)."

      If you write a @Match("*"), Tapestry-IoC throws an exception telling that you cannot decorate internal Tapestry-IoC services. Therefore, there's no way to select services to decorate using only annotations: we need to match services based on name patterns in all situations.

      A suggestions is the creation of a @MatchAnnotation() annotation.

        Activity

        Hide
        Howard M. Lewis Ship added a comment -

        Due to limitations in JIRA, the resolution on some issues had to be changed, as part of removing the fix release from the issue. Only issues that are actually fixed should have a fix release.

        Show
        Howard M. Lewis Ship added a comment - Due to limitations in JIRA, the resolution on some issues had to be changed, as part of removing the fix release from the issue. Only issues that are actually fixed should have a fix release.
        Hide
        Howard M. Lewis Ship added a comment -

        MARKER: INVALID FIX RELEASE

        Show
        Howard M. Lewis Ship added a comment - MARKER: INVALID FIX RELEASE
        Hide
        Massimo Lusetti added a comment -

        Please open a new one for 5.3 if this still applicable

        Show
        Massimo Lusetti added a comment - Please open a new one for 5.3 if this still applicable

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Thiago H. de Paula Figueiredo
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development