You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@isis.apache.org by "Andi Huber (Jira)" <ji...@apache.org> on 2021/12/10 08:46:00 UTC
[jira] [Closed] (ISIS-2749) [Duplicate] Convert IsisConverterLocator into a domain service and allow Wicket Converters to be substituted in.
[ https://issues.apache.org/jira/browse/ISIS-2749?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Andi Huber closed ISIS-2749.
----------------------------
Resolution: Duplicate
> [Duplicate] Convert IsisConverterLocator into a domain service and allow Wicket Converters to be substituted in.
> ----------------------------------------------------------------------------------------------------------------
>
> Key: ISIS-2749
> URL: https://issues.apache.org/jira/browse/ISIS-2749
> Project: Isis
> Issue Type: Improvement
> Components: Isis Core
> Affects Versions: 2.0.0-M5
> Reporter: Daniel Keir Haywood
> Priority: Minor
> Fix For: 2.0.0-M7
>
> Attachments: image-2021-06-17-12-50-57-240.png
>
>
> For example, if enter 'abc' into a BigDecimal' then the error message returned is effectively hardcoded because in IsisWicketApplication we just use Wicket's default ConverterLocator (fine tuned with a couple of extra types to convert). That means we get:
> !image-2021-06-17-12-50-57-240.png|width=582,height=120!
> We also have a bit of technical debt, because there is an IsisConverterLocator that _doesn't_ implement Wicket's IConvertLocator, and is just called from one place, statically, to render an object as a string.
> So we should perhaps:
> * provide our own impl of IConveterLocator, probably subclassed from Wicket's
> * make it a Spring Bean
> * get rid of IsisConverterLocator (roll it into aforementioned IConverterLocator)
> * decide on a way to allow new/replacement converters to be registered.
> We might also want to look at the ValueTypeRegistry we have, just in case there's overlap.
>
>
--
This message was sent by Atlassian Jira
(v8.20.1#820001)