myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Concini (JIRA)" <...@myfaces.apache.org>
Subject [jira] Resolved: (MYFACES-2671) MYFACES-2633 regresses the ezcomp spinner example
Date Tue, 27 Apr 2010 14:44:32 GMT

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

Michael Concini resolved MYFACES-2671.
--------------------------------------

         Assignee: Michael Concini
    Fix Version/s: 2.0.1-SNAPSHOT
       Resolution: Duplicate

Thanks to Jakob for the resolution via MYFACES-2677.

> MYFACES-2633 regresses the ezcomp spinner example
> -------------------------------------------------
>
>                 Key: MYFACES-2671
>                 URL: https://issues.apache.org/jira/browse/MYFACES-2671
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: JSR-314
>    Affects Versions: 2.0.0
>            Reporter: Michael Concini
>            Assignee: Michael Concini
>             Fix For: 2.0.1-SNAPSHOT
>
>
> The change to CompositeComponentRule committed by MYFACES-2633 actually does break the
ezcomp RI sample.  Specifically, in the spinner example it is causing the last value submitted
to not be updated when you click the reload button.  It has reverted to the same behavior
that was fixed by MYFACES-2340.  
> I did some playing around, and it seems that both adding the ValueExpression directly
to the map AND calling instance.setValueExpression() seems to resolve both cases.  
>       ((UIComponent) instance).getAttributes().put(_name, _attr.getValueExpression(ctx,
_type));  //right now this first line is commented out
>        ((UIComponent) instance).setValueExpression(_name, _attr.getValueExpression(ctx,
_type));
> As far as I can tell, I haven't seen any repercussions to leaving both calls in place,
but it feels like there should be a more elegant solution.   I'll leave this open for discussion
for a few days in case someone has another idea for a resolution before I check in the change.
  
> In the meantime, I'll do some additional testing to make sure there aren't any side affects
of making both calls.  

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


Mime
View raw message