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

Unify injection; allow @Inject annotation on fields of service implementations

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 5.0.15
    • Fix Version/s: 5.0.16
    • Component/s: tapestry-ioc
    • Labels:
      None

      Description

      This has come up again and again, especially in recent training. Although I'm very strongly in favor of constructor injection (and the use of final fields to store dependencies), being allowed to inject into a field, even if it requires the use of reflection, would be a real boon.

        Activity

        Hide
        mlusetti Massimo Lusetti added a comment -

        I'm curious to know which are the situation that have lead you to consider this enhancement. I think this could be of interest to the community as well.

        Show
        mlusetti Massimo Lusetti added a comment - I'm curious to know which are the situation that have lead you to consider this enhancement. I think this could be of interest to the community as well.
        Hide
        hlship Howard M. Lewis Ship added a comment -

        Came out nicely.

        I've gotten lots of pushback from the last two training engagements that people are unhappy that there's so much difference between injection into components (via fields) and injection into services (via constructors). That's also been on the list a bit.

        Show
        hlship Howard M. Lewis Ship added a comment - Came out nicely. I've gotten lots of pushback from the last two training engagements that people are unhappy that there's so much difference between injection into components (via fields) and injection into services (via constructors). That's also been on the list a bit.

          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:

              Development