You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@isis.apache.org by "Daniel Keir Haywood (Jira)" <ji...@apache.org> on 2021/07/26 12:41:00 UTC
[jira] [Updated] (ISIS-2749) 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 ]
Daniel Keir Haywood updated ISIS-2749:
--------------------------------------
Fix Version/s: (was: 2.0.0-M6)
2.0.0
> 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
>
> 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.3.4#803005)