You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Martin Marinschek (JIRA)" <de...@myfaces.apache.org> on 2005/12/23 08:55:33 UTC

[jira] Commented: (MYFACES-210) Undesired dependency on ApplicationImpl in FacesConfigurator

    [ http://issues.apache.org/jira/browse/MYFACES-210?page=comments#action_12361174 ] 

Martin Marinschek commented on MYFACES-210:
-------------------------------------------

Adam, I don't understand your comment here either.

How would a tool provide a default-value for a property of a converter in JSF.1.1 if it can't inject it into the converter instance?

Is the tool supposed to automatically extend from the default converter and inject it there?

regards,

Martin

> Undesired dependency on ApplicationImpl in FacesConfigurator
> ------------------------------------------------------------
>
>          Key: MYFACES-210
>          URL: http://issues.apache.org/jira/browse/MYFACES-210
>      Project: MyFaces
>         Type: Bug
>   Components: Implementation
>     Versions: 1.0.9m9
>  Environment: Custom ApplicationFactory and Application Objects
>     Reporter: Walter Snel
>     Assignee: Martin Marinschek
>     Priority: Minor

>
> Manfred's recent additions related to the configurability of Converters created a dependency on a particular Application implementation in the FacesConfigurator.
> I noticed this because I'm using a custom ApplicationFactory that returns a custom Facade (with augmentations) to the ApplicationImpl class.
> In this case the FacesConfigurator doesn't call the addConverterConfiguration. This, later on, leads to horrible nullpointer exceptions when any of standard converters are used. Perhaps we could consider using relfection instead of the current 'instanceof' mechanism to find out if the addConverterConfiguration() method is supported ?

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira