Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-2621

Extract interface from MessagesManager

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: 4.0
    • Component/s: core
    • Labels:
      None
    • Patch included:
      Yes

      Description

      Hi.
      We need to extends MessagesManager. I see (MAGNOLIA-2273) you've introduces the getInstance() by using FactoryUtils, but the problem is that the resulting object is casted to MessagesManager itself, which is final, no allowing subclassing.
      I attach a patch in which i create a MessagesProvider interface, apply that to MessagesManager and use it for the MessagesManager.getInstance()

      I think that MessagesManager could be improved by allowing, via configuration, to add Messages implementations to be (on startup or on configuration changes) chained in newMessages method.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              fgiust Fabrizio Giustina
              Reporter:
              manuelmolaschi Manuel Molaschi
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Date of First Response: