tapestry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Denis Delangle (JIRA)" <j...@apache.org>
Subject [jira] Created: (TAP5-1100) On ajax request, template body doesn't render if page is new in pagepool or if server restarted
Date Wed, 07 Apr 2010 13:17:33 GMT
On ajax request, template body doesn't render if page is new in pagepool or if server restarted

                 Key: TAP5-1100
                 URL: https://issues.apache.org/jira/browse/TAP5-1100
             Project: Tapestry 5
          Issue Type: Bug
          Components: tapestry-core
    Affects Versions:
            Reporter: Denis Delangle
            Priority: Critical

Bug process :
 * Write a component using ajax form and a <t:body />. When form is submitted in ajax,
the component renders its body.
 * Start jetty, open the page
 * Restart jetty, submit the ajax form
-> The form is refreshed without the body of the component

Attached to the ticket, a little project showing the problem (just open the index page and
click submit)

The problem is in ComponentPageElementImpl, the beforeRenderBodyPhase should be called for
the part of Index template inside my component, but as the page has never been rendered before,
beforeRenderBodyPhase is null.

We have the same issue if a new page instance is created to treat the ajax request. If the
page is taken from the page pool, then there is no problem as the page has already been initialized.

A patch that could fix it (didn't test it) :

Index: src/main/java/org/apache/tapestry5/internal/structure/ComponentPageElementImpl.java
--- src/main/java/org/apache/tapestry5/internal/structure/ComponentPageElementImpl.java	(revision
+++ src/main/java/org/apache/tapestry5/internal/structure/ComponentPageElementImpl.java	(working
@@ -834,7 +834,9 @@
     public void enqueueBeforeRenderBody(RenderQueue queue)
         // If no body, then no beforeRenderBody or afterRenderBody
+    	if(! renderPhasesInitalized) 
+    		initializeRenderPhases();
         if (bodyBlock != null)
             push(queue, beforeRenderBodyPhase);

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

View raw message