Details
-
Bug
-
Resolution: Fixed
-
Major
-
4.0
-
None
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.
Checklists
Acceptance criteria
Attachments
Issue Links
- is related to
-
MAGNOLIA-2273 Make MessagesManager easier to extend/replace
-
- Closed
-