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

Keijo Nurmes commented on MYFACES-736:
--------------------------------------

Olivier, this is your comment in issue 623.

------------------ 
"I checked the spec document and there is no word about coercion in the 5.1.4 
Set Value Semantics section. I also had a look at the RI sources and there is 
no coercion in ValueBindingImpl.setValue, either. So I suppose it's save to 
remove coercion in MyFaces' ValueBindingImpl, too. From my POV it's even 
required to do so to comply with the spec. 

In my interpretion of the spec it's up to the application developer to provide 
a converter where necessary, but it's not required for the JSF runtime to try 
to guess the right conversion to match UI and model."
------------------

How can this be against the spec if it's not mentioned in it ?
Didn't myfaces pass the TCK tests without this modification?
IMHO the last sentence in that comment pretty much ruins the usability of 
myfaces. 
(Imagine every component working like this.. )

If everybody agrees the removal of coercion in ValueBindingImpl.setValue (as it 
is now)
I ( or somebody else) have to figure out how to bring back some intelligence to 
convertNumber.

regards, Keijo


> Number converter not working with Number
> ----------------------------------------
>
>          Key: MYFACES-736
>          URL: http://issues.apache.org/jira/browse/MYFACES-736
>      Project: MyFaces
>         Type: Bug
>   Components: Implementation
>     Versions: Nightly
>     Reporter: Sylvain Vieujot

>
> If the backing bean setter takes a Number argument (Integer, Float, ...), the 
> converter fails :
> Conversion Error "XXX: Error during model data update.
> If the setter uses a primitive type, then it works fine.
> This worked with the trunk version from September 30.
> There must have been a change that broke it since.

-- 
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

Reply via email to