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

Howard M. Lewis Ship updated TAP5-128:
--------------------------------------

    Assignee: Howard M. Lewis Ship
     Summary: Render phase short circuiting fails to abort the event when 
mixins are present on the component, resulting in an IllegalStateException when 
trying to store a subsequent result value  (was: Render phase short-circuiting 
is not implemented for methods of the same phase.)

> Render phase short circuiting fails to abort the event when mixins are 
> present on the component, resulting in an IllegalStateException when trying 
> to store a subsequent result value
> -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: TAP5-128
>                 URL: https://issues.apache.org/jira/browse/TAP5-128
>             Project: Tapestry 5
>          Issue Type: Bug
>            Reporter: Nick Westgate
>            Assignee: Howard M. Lewis Ship
>            Priority: Critical
>         Attachments: jira1662sample.tgz
>
>
> According to the docs on render phase short-circuiting for methods of the 
> same phase:
> http://tapestry.apache.org/tapestry5/tapestry-core/guide/rendering.html
> "If a method returns a true or false value, this will short circuit 
> processing.
> Other methods within the phase that would ordinarily be invoked will not be 
> invoked."
> This is currently unimplemented. (Only other "inner" phases are 
> short-circuited & not invoked.)
> Changes would primarily affect:
> org.apache.tapestry.internal.structure.ComponentCallback
> org.apache.tapestry.internal.structure.ComponentPageElementImpl
> In ComponentPageElementImpl.invoke(...), the loop:
>             while (i.hasNext())
>                 callback.run(i.next());
> needs to implement the short circuiting, for example:
>             while (i.hasNext())
>                 if (callback.run(i.next()))
>                     break;
> This change would cascade up to ComponentCallback and all its usage in 
> ComponentPageElementImpl.
> Cheers,
> Nick.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to