You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Igor Vaynberg (JIRA)" <ji...@apache.org> on 2007/12/11 06:34:43 UTC

[jira] Assigned: (WICKET-1180) Converters : final vs non final

     [ https://issues.apache.org/jira/browse/WICKET-1180?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Igor Vaynberg reassigned WICKET-1180:
-------------------------------------

    Assignee: Eelco Hillenius

go eelco go!

> Converters : final vs non final
> -------------------------------
>
>                 Key: WICKET-1180
>                 URL: https://issues.apache.org/jira/browse/WICKET-1180
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.3.0-rc1
>            Reporter: Stephane Bailliez
>            Assignee: Eelco Hillenius
>            Priority: Minor
>
> I can understand the reason not to put anything extensible but then there is a consistency problem as in the converters:
> all converters are final except: DateConverter, SqlDateConverter, SqlTimeConverter, SqlTimestampConverter
> which then makes you wonder why the rest is actually final, DateConverter is obviously not final to support inherirtance by the sql* converter, so if we wanted to restrict it, the base class should have been abstract and the other final, but it really does not protect  any significant api changes from the client as the code is not so complex that it deserves to do so
> For instance I wanted to subclass ZeroPaddingConverter to work around the null support and I realize it's final, so I  basically end up copying the whole code.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.