DisplayTag
  1. DisplayTag
  2. DISPL-222

Ability to use a custom DecoratorFactory

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0
    • Fix Version/s: 1.1
    • Component/s: Decorators
    • Labels:
      None

      Description

      Currently, the DecoratorFactory is a utility class that has 2 static methods. It would be nice if it was a user specifiable implementation of a simple interface with those two methods. (You could store it in the servlet context to avoid having to reinstanciating it everytime).

      Doing this would allow to have (for instance) a DecoratorFactory that retrieves decorators from an ioc container, allowing them to enjoy the power of dependency injection. (One could also have a caching implementation, which would for instance keep an instance of each decorator per thread (using a ThreadLocal), avoiding many reinstanciations of the decorator)

        Issue Links

          Activity

          Hide
          fabrizio giustina added a comment -
          planning for 1.1
          Show
          fabrizio giustina added a comment - planning for 1.1
          Hide
          fabrizio giustina added a comment -
          will be solved by custom decorator factories
          Show
          fabrizio giustina added a comment - will be solved by custom decorator factories
          Hide
          fabrizio giustina added a comment -
          implemented in 1.1 with a new factory.decorator property
          Show
          fabrizio giustina added a comment - implemented in 1.1 with a new factory.decorator property

            People

            • Reporter:
              Anonymous
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: