You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by "Howard M. Lewis Ship (JIRA)" <ji...@apache.org> on 2010/02/10 16:17:27 UTC
[jira] Created: (TAP5-1011) When using @PageActivationContext and
providing a no-args activate event handler, the handler may be called too
soon
When using @PageActivationContext and providing a no-args activate event handler, the handler may be called too soon
--------------------------------------------------------------------------------------------------------------------
Key: TAP5-1011
URL: https://issues.apache.org/jira/browse/TAP5-1011
Project: Tapestry 5
Issue Type: Bug
Components: tapestry-core
Affects Versions: 5.1.0.5
Reporter: Howard M. Lewis Ship
It appears that the onActivate() method is called before the @PageActivationContext logic, which means the following can fail:
@PageActivationContext
private MyEntity entity;
void onActivate()
{
if (entity == null) throw new RuntimeException("Entity not found.");
}
The RuntimeException is thrown even when a valid Entity is in the page activation context.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (TAP5-1011) When using @PageActivationContext and
providing a no-args activate event handler, the handler may be called too
soon
Posted by "Alexander Gavrilov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/TAP5-1011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12915378#action_12915378 ]
Alexander Gavrilov commented on TAP5-1011:
------------------------------------------
It's seems like tapestry skips OnEvent advice for ConcreteClass method and create one for method in BasePage. For PageActivationContext correct work advices must be invoked in the sequence "OnEvent advice" -> "PageActivationContext advice", but in this case the sequence is different.
> When using @PageActivationContext and providing a no-args activate event handler, the handler may be called too soon
> --------------------------------------------------------------------------------------------------------------------
>
> Key: TAP5-1011
> URL: https://issues.apache.org/jira/browse/TAP5-1011
> Project: Tapestry 5
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.1.0.5
> Reporter: Howard M. Lewis Ship
> Assignee: Howard M. Lewis Ship
> Fix For: 5.2.1
>
>
> It appears that the onActivate() method is called before the @PageActivationContext logic, which means the following can fail:
> @PageActivationContext
> private MyEntity entity;
> void onActivate()
> {
> if (entity == null) throw new RuntimeException("Entity not found.");
> }
> The RuntimeException is thrown even when a valid Entity is in the page activation context.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Assigned: (TAP5-1011) When using @PageActivationContext and
providing a no-args activate event handler, the handler may be called too
soon
Posted by "Howard M. Lewis Ship (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/TAP5-1011?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Howard M. Lewis Ship reassigned TAP5-1011:
------------------------------------------
Assignee: Howard M. Lewis Ship
> When using @PageActivationContext and providing a no-args activate event handler, the handler may be called too soon
> --------------------------------------------------------------------------------------------------------------------
>
> Key: TAP5-1011
> URL: https://issues.apache.org/jira/browse/TAP5-1011
> Project: Tapestry 5
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.1.0.5
> Reporter: Howard M. Lewis Ship
> Assignee: Howard M. Lewis Ship
>
> It appears that the onActivate() method is called before the @PageActivationContext logic, which means the following can fail:
> @PageActivationContext
> private MyEntity entity;
> void onActivate()
> {
> if (entity == null) throw new RuntimeException("Entity not found.");
> }
> The RuntimeException is thrown even when a valid Entity is in the page activation context.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Closed: (TAP5-1011) When using @PageActivationContext and
providing a no-args activate event handler, the handler may be called too
soon
Posted by "Howard M. Lewis Ship (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/TAP5-1011?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Howard M. Lewis Ship closed TAP5-1011.
--------------------------------------
Fix Version/s: 5.2.1
Resolution: Fixed
> When using @PageActivationContext and providing a no-args activate event handler, the handler may be called too soon
> --------------------------------------------------------------------------------------------------------------------
>
> Key: TAP5-1011
> URL: https://issues.apache.org/jira/browse/TAP5-1011
> Project: Tapestry 5
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.1.0.5
> Reporter: Howard M. Lewis Ship
> Assignee: Howard M. Lewis Ship
> Fix For: 5.2.1
>
>
> It appears that the onActivate() method is called before the @PageActivationContext logic, which means the following can fail:
> @PageActivationContext
> private MyEntity entity;
> void onActivate()
> {
> if (entity == null) throw new RuntimeException("Entity not found.");
> }
> The RuntimeException is thrown even when a valid Entity is in the page activation context.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (TAP5-1011) When using @PageActivationContext and
providing a no-args activate event handler, the handler may be called too
soon
Posted by "Alexander Gavrilov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/TAP5-1011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12915378#action_12915378 ]
Alexander Gavrilov commented on TAP5-1011:
------------------------------------------
It's seems like tapestry skips OnEvent advice for ConcreteClass method and create one for method in BasePage. For PageActivationContext correct work advices must be invoked in the sequence "OnEvent advice" -> "PageActivationContext advice", but in this case the sequence is different.
> When using @PageActivationContext and providing a no-args activate event handler, the handler may be called too soon
> --------------------------------------------------------------------------------------------------------------------
>
> Key: TAP5-1011
> URL: https://issues.apache.org/jira/browse/TAP5-1011
> Project: Tapestry 5
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.1.0.5
> Reporter: Howard M. Lewis Ship
> Assignee: Howard M. Lewis Ship
> Fix For: 5.2.1
>
>
> It appears that the onActivate() method is called before the @PageActivationContext logic, which means the following can fail:
> @PageActivationContext
> private MyEntity entity;
> void onActivate()
> {
> if (entity == null) throw new RuntimeException("Entity not found.");
> }
> The RuntimeException is thrown even when a valid Entity is in the page activation context.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Closed: (TAP5-1011) When using @PageActivationContext and
providing a no-args activate event handler, the handler may be called too
soon
Posted by "Howard M. Lewis Ship (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/TAP5-1011?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Howard M. Lewis Ship closed TAP5-1011.
--------------------------------------
Fix Version/s: 5.2.1
Resolution: Fixed
> When using @PageActivationContext and providing a no-args activate event handler, the handler may be called too soon
> --------------------------------------------------------------------------------------------------------------------
>
> Key: TAP5-1011
> URL: https://issues.apache.org/jira/browse/TAP5-1011
> Project: Tapestry 5
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.1.0.5
> Reporter: Howard M. Lewis Ship
> Assignee: Howard M. Lewis Ship
> Fix For: 5.2.1
>
>
> It appears that the onActivate() method is called before the @PageActivationContext logic, which means the following can fail:
> @PageActivationContext
> private MyEntity entity;
> void onActivate()
> {
> if (entity == null) throw new RuntimeException("Entity not found.");
> }
> The RuntimeException is thrown even when a valid Entity is in the page activation context.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (TAP5-1011) When using @PageActivationContext and
providing a no-args activate event handler, the handler may be called too
soon
Posted by "Alexander Gavrilov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/TAP5-1011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12915351#action_12915351 ]
Alexander Gavrilov commented on TAP5-1011:
------------------------------------------
Its seems like the following case still fail
public class BasePage {
private MyEntity entity;
protected void setEntity(MyEntity entity) {
this.entity = entity;
}
protected void onActivate() {
if (entity == null) throw new RuntimeException("Entity not found.");
}
}
public class ConcreteClass {
@PageActivationContext
private MyEntity entity;
protected void onActivate() {
setEntity(entity);
super.onActivate();
}
}
Shold i clone this issue or you can reopet it?
> When using @PageActivationContext and providing a no-args activate event handler, the handler may be called too soon
> --------------------------------------------------------------------------------------------------------------------
>
> Key: TAP5-1011
> URL: https://issues.apache.org/jira/browse/TAP5-1011
> Project: Tapestry 5
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.1.0.5
> Reporter: Howard M. Lewis Ship
> Assignee: Howard M. Lewis Ship
> Fix For: 5.2.1
>
>
> It appears that the onActivate() method is called before the @PageActivationContext logic, which means the following can fail:
> @PageActivationContext
> private MyEntity entity;
> void onActivate()
> {
> if (entity == null) throw new RuntimeException("Entity not found.");
> }
> The RuntimeException is thrown even when a valid Entity is in the page activation context.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (TAP5-1011) When using @PageActivationContext and
providing a no-args activate event handler, the handler may be called too
soon
Posted by "Hudson (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/TAP5-1011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12912722#action_12912722 ]
Hudson commented on TAP5-1011:
------------------------------
Integrated in tapestry-5.2-freestyle #194 (See [https://hudson.apache.org/hudson/job/tapestry-5.2-freestyle/194/])
Merge branch 'TAP5-1011' into trunk
> When using @PageActivationContext and providing a no-args activate event handler, the handler may be called too soon
> --------------------------------------------------------------------------------------------------------------------
>
> Key: TAP5-1011
> URL: https://issues.apache.org/jira/browse/TAP5-1011
> Project: Tapestry 5
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.1.0.5
> Reporter: Howard M. Lewis Ship
> Assignee: Howard M. Lewis Ship
> Fix For: 5.2.1
>
>
> It appears that the onActivate() method is called before the @PageActivationContext logic, which means the following can fail:
> @PageActivationContext
> private MyEntity entity;
> void onActivate()
> {
> if (entity == null) throw new RuntimeException("Entity not found.");
> }
> The RuntimeException is thrown even when a valid Entity is in the page activation context.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (TAP5-1011) When using @PageActivationContext and
providing a no-args activate event handler, the handler may be called too
soon
Posted by "Alexander Gavrilov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/TAP5-1011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12915351#action_12915351 ]
Alexander Gavrilov commented on TAP5-1011:
------------------------------------------
Its seems like the following case still fail
public class BasePage {
private MyEntity entity;
protected void setEntity(MyEntity entity) {
this.entity = entity;
}
protected void onActivate() {
if (entity == null) throw new RuntimeException("Entity not found.");
}
}
public class ConcreteClass {
@PageActivationContext
private MyEntity entity;
protected void onActivate() {
setEntity(entity);
super.onActivate();
}
}
Shold i clone this issue or you can reopet it?
> When using @PageActivationContext and providing a no-args activate event handler, the handler may be called too soon
> --------------------------------------------------------------------------------------------------------------------
>
> Key: TAP5-1011
> URL: https://issues.apache.org/jira/browse/TAP5-1011
> Project: Tapestry 5
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.1.0.5
> Reporter: Howard M. Lewis Ship
> Assignee: Howard M. Lewis Ship
> Fix For: 5.2.1
>
>
> It appears that the onActivate() method is called before the @PageActivationContext logic, which means the following can fail:
> @PageActivationContext
> private MyEntity entity;
> void onActivate()
> {
> if (entity == null) throw new RuntimeException("Entity not found.");
> }
> The RuntimeException is thrown even when a valid Entity is in the page activation context.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Assigned: (TAP5-1011) When using @PageActivationContext and
providing a no-args activate event handler, the handler may be called too
soon
Posted by "Howard M. Lewis Ship (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/TAP5-1011?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Howard M. Lewis Ship reassigned TAP5-1011:
------------------------------------------
Assignee: Howard M. Lewis Ship
> When using @PageActivationContext and providing a no-args activate event handler, the handler may be called too soon
> --------------------------------------------------------------------------------------------------------------------
>
> Key: TAP5-1011
> URL: https://issues.apache.org/jira/browse/TAP5-1011
> Project: Tapestry 5
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.1.0.5
> Reporter: Howard M. Lewis Ship
> Assignee: Howard M. Lewis Ship
>
> It appears that the onActivate() method is called before the @PageActivationContext logic, which means the following can fail:
> @PageActivationContext
> private MyEntity entity;
> void onActivate()
> {
> if (entity == null) throw new RuntimeException("Entity not found.");
> }
> The RuntimeException is thrown even when a valid Entity is in the page activation context.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (TAP5-1011) When using @PageActivationContext and
providing a no-args activate event handler, the handler may be called too
soon
Posted by "Hudson (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/TAP5-1011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12912722#action_12912722 ]
Hudson commented on TAP5-1011:
------------------------------
Integrated in tapestry-5.2-freestyle #194 (See [https://hudson.apache.org/hudson/job/tapestry-5.2-freestyle/194/])
Merge branch 'TAP5-1011' into trunk
> When using @PageActivationContext and providing a no-args activate event handler, the handler may be called too soon
> --------------------------------------------------------------------------------------------------------------------
>
> Key: TAP5-1011
> URL: https://issues.apache.org/jira/browse/TAP5-1011
> Project: Tapestry 5
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.1.0.5
> Reporter: Howard M. Lewis Ship
> Assignee: Howard M. Lewis Ship
> Fix For: 5.2.1
>
>
> It appears that the onActivate() method is called before the @PageActivationContext logic, which means the following can fail:
> @PageActivationContext
> private MyEntity entity;
> void onActivate()
> {
> if (entity == null) throw new RuntimeException("Entity not found.");
> }
> The RuntimeException is thrown even when a valid Entity is in the page activation context.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.