[prev in list] [next in list] [prev in thread] [next in thread] 

List:       struts-dev
Subject:    Re: [shale] class type of #{dialog.data} contents
From:       Sean Schofield <sean.schofield () gmail ! com>
Date:       2005-09-27 16:25:54
Message-ID: 2387fbc5050927092520290442 () mail ! gmail ! com
[Download RAW message or body]

Just a quick update:  The RI works perfectly with my code so I think I
have interpreted the spec correctly.  I working on a fix in MyFaces
but its slow going due to my unfamiliarity with the EL implementation.

sean

On 9/26/05, Craig McClanahan <craigmcc@apache.org> wrote:
>
>
> On 9/26/05, Sean Schofield <sean.schofield@gmail.com> wrote:
> > Craig,
> >
> > I am using a explicity defined converter and everything is converting
> > fine through the Process Validations phase.  Its during Update Model
> > phase that things go funky.  Converters don't matter at this point do
> > they?
>
>  No ... by that time, the converted value will have been stored with
> setLocalValue(), which  should be of the appropriate data type to be stored
> during Update Model Values.
>
> > I think there might be a bug in MyFaces
> > ( http://issues.apache.org/jira/browse/MYFACES-623).
> When I examined
> > the ValueBindingImpl using my debugger it showed that it was
> > specifically trying to coerce the value to class associated with
> > java.lang.String .
>
>  Hmm ... what does the RI do?
>
> > It looks like PropertyResolverImpl in MyFaces is checking if
> > StatusImpl is an instanceof Map when it should be checking the result
> > of StatusImpl.getData().  If it were checking the right object then
> > we'd be ok (because the class that it tries to coerce to is the
> > object's class.)
>
>  Yah, the base object should be of type StatusImpl for the *first* property
> resolution (dialog.data) but not for the second (data['foo']).
>
> > sean
>
>  Craig
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic