Hi,
I promised myself the third time I found this annoying I would make this
patch.

So I would be grateful if someone applied it to the 5.3 and 5.4 branch.

I opened a JIRA without consulting the list, but because I considered this
trivial and straitforward I trusted my judgement and went ahead.

Cheers,
Dragan Sahpaski



On Thu, Mar 29, 2012 at 6:42 PM, Dragan Sahpaski (Updated) (JIRA) <
[email protected]> wrote:

>
>     [
> https://issues.apache.org/jira/browse/TAP5-1890?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel]
>
> Dragan Sahpaski updated TAP5-1890:
> ----------------------------------
>
>     Attachment: TAP-1890.patch
>
> > PlaceholderBlock should implement RenderCommand
> > -----------------------------------------------
> >
> >                 Key: TAP5-1890
> >                 URL: https://issues.apache.org/jira/browse/TAP5-1890
> >             Project: Tapestry 5
> >          Issue Type: Bug
> >          Components: tapestry-core
> >    Affects Versions: 5.3.2, 5.4
> >            Reporter: Dragan Sahpaski
> >            Priority: Trivial
> >              Labels: block, zone
> >         Attachments: TAP-1890.patch
> >
> >
> > If we have an empty zone like this:
> > <t:zone t:id="myZone" />
> > zone.getBody() fails to update the zone in an event handler with the
> following exception:
> > org.apache.tapestry5.ioc.util.UnknownValueException
> > A component event handler method returned the value <PlaceholderBlock>.
> Return type
> org.apache.tapestry5.internal.structure.ComponentPageElementImpl$PlaceholderBlock
> can not be handled.
> > This is because ComponentPageElementImpl.PlaceholderBlock does not
> implement RenderCommand.
> > A patch is provided with a test that fails.
> > I would highly appreciate it if a commiter applies it to the 5.4. branch
> and to the 5.3. branch.
> > Cheers,
> > Dragan Sahpaski
>
> --
> This message is automatically generated by JIRA.
> If you think it was sent incorrectly, please contact your JIRA
> administrators:
> https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
> For more information on JIRA, see: http://www.atlassian.com/software/jira
>
>
>

Reply via email to