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

Injection via Marker Annotations

Details

    • New Feature
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 5.0.5
    • 5.0.6
    • tapestry-ioc
    • None

    Description

      Currently in IoC, when injecting a service, you are limited to two options:

      • Inject by type, assuming exactly one match
      • Inject by explicit service id

      In a fully featured system both of these are imperfect.

      When you take into account the idea that there will be overrides for built in services, then you can't rely on their being just a single match. That's what the Alias service in tapestry-core is all about: contributing to the MasterObjectProvider to handle overrides. Unfortunately, contributions to Alias are clumsy, as they have to rely on @InjectService for disambiguation.

      Having to know the service id is a problem, since service ids are not refactoring safe: refactoring the service interface name will likely change the service id, but @InjectService annotation values will still point to the old name.

      Imagine instead a marker annotation; no attributes, attached to a type. Maybe it's attached to the service implementation class, maybe its specified to the ServiceBinder.

      When injecting, the parameter type and annotations are checked for any known marker annotation (all the marker annotation will be identified at Registry start up). The parameter type is matched against services with the marker, at must be unique for just those services.

      An idiom involving a nested annotation named "Local" is likely, thus:

      public class MyModule
      {

      @interface Local { }

      @MarkedBy(Local.class)
      public MyService buildMyService(@Local OtherService other)

      { ... }

      I'm fairly certain Guice has something quite similiar.

      The question is: can we get rid of service ids entirely?

      Attachments

        Activity

          People

            hlship Howard Lewis Ship
            hlship Howard Lewis Ship
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment