[
https://issues.apache.org/jira/browse/WICKET-7036?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17705401#comment-17705401
]
ASF GitHub Bot commented on WICKET-7036:
----------------------------------------
hosea commented on PR #564:
URL: https://github.com/apache/wicket/pull/564#issuecomment-1485306655
Hi @theigl , @martin-g ,
I agree with you, that one could easily extend MarkupFactory and achieve the
requirement. That's in fact, what we do now. I just want to explain, why it
would be helpful to have such extensions:
I have about 20 Wicket-Applications developed by several teams sharing
several libraries of wicket-components. These "central" libraries use
IInitializers. So the code
#Application.getMarkupSettings().setMarkupFactory(new AriaAwareMarkupFactory())
is now located in one of the libraries. But if I want to add processing of
MarkupFilters in another library, then I cannot use this initialization-code
anymore: two Initializers setting a new MarkupFacatory => only one wins and the
other functionality is lost. This means, that all 20 Applications have to
implement a new MarkupFactory that implements both functionalities, or I have
to add a dependency between libraries. Both solutions are not nice.
Adding to possibility to add IMarkupFilters without replacing the instance
of the MarkupFactory makes initialization much easier in such a case.
> Extend MarkupFactory, so that additional MarkupFilters for MarkupParser can
> be added
> ------------------------------------------------------------------------------------
>
> Key: WICKET-7036
> URL: https://issues.apache.org/jira/browse/WICKET-7036
> Project: Wicket
> Issue Type: Improvement
> Reporter: Hans Schäfer
> Priority: Minor
>
> Its intended to add additional MarkupFilters to a MarkupParser. But until now
> there is no possibility to configure the MarkupFactory, so that it creates
> MarkupParsers with additional MarkupFilters. Please add this functionality.
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
