[ 
https://issues.apache.org/jira/browse/MYFACES-1831?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12574755#action_12574755
 ] 

Simon Kitching commented on MYFACES-1831:
-----------------------------------------

Drat. I meant getCreated always returns *false* of course.

The problem *I* have is that the Orchestra examples use t:updateActionListener 
to push data from a master view into an access-scoped bean associated with a 
detail view. This orchestra example is therefore currently broken when run with 
"mvn -Djsf=12 jetty:run"; it splashes a big ugly NullPointerException 
stacktrace across the screen at an important point.

I cannot simply switch to f:setPropertyActionListener because then the 
Orchestra example will not run with JSF1.1.

And this orchestra example is my primary testbed for verifying that I haven't 
broken things when making changes in Orchestra, as our unit tests are currently 
.. how shall we say.. less than 100% coverage? In particular, I use it to check 
that orchestra is still 1.1 and 1.2 compatible. Unit tests that check JSF1.1 
*and* JSF1.2 compatibility would be tricky to write....

But anyway, many people will get bitten by this when upgrading existing 
applications from JSF1.1 to JSF1.2. The result is not an obvious crash, but a 
NullPointerException at some later time because the t:updateActionListener 
silently doesn't run so something doesn't get set but the problem only occurs 
later when that thing is *used*. That's not nice.

So for all those reasons I'll put this on the top of my spare-time to-do list.

> JSF12 UIComponentClassicTagBase.getCreated is broken, breaking 
> t:updateActionListener
> -------------------------------------------------------------------------------------
>
>                 Key: MYFACES-1831
>                 URL: https://issues.apache.org/jira/browse/MYFACES-1831
>             Project: MyFaces Core
>          Issue Type: Bug
>    Affects Versions: 1.2.2
>            Reporter: Simon Kitching
>
> In testing I have found that t:updateActionListener works fine with MyFaces 
> 1.1.x but is ignored with MyFaces 1.2.3-SNAPSHOT
> After debugging, I found that UIComponentClassicTagBase.getCreated is always 
> returning true. This means that UpdateActionListenerTag always thinks its 
> parent component is already configured, so does not attach an 
> UpdateActionListener instance to it.

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

Reply via email to