[ 
https://issues.apache.org/jira/browse/TAP5-2070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13729783#comment-13729783
 ] 

Hudson commented on TAP5-2070:
------------------------------

FAILURE: Integrated in tapestry-trunk-freestyle #1127 (See 
[https://builds.apache.org/job/tapestry-trunk-freestyle/1127/])
TAP5-2070 better to take no activation handler method as a way to accept empty 
activation context (mlusetti: rev 28e3321f7865fb74cd36c78f3911bfbd9750ae77)
* 
tapestry-core/src/main/java/org/apache/tapestry5/internal/services/PageActivatorImpl.java
TAP5-2070 tests on basic check on activation context with new annotation 
(mlusetti: rev 0ae1d1170f93f7570a3340a0d07185c79a300acb)
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/pages/Index.java
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/pages/UnknownActivationContextDemo.java
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/CoreBehaviorsTests.java
* 
tapestry-core/src/test/resources/org/apache/tapestry5/integration/app1/pages/UnknownActivationContextDemo.tml
TAP5-2070 new page for base tests (mlusetti: rev 
7de82f531c875bf916385c5dc1fedd64687af7d6)
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/pages/KnownActivationContextDemo.java
* 
tapestry-core/src/test/resources/org/apache/tapestry5/integration/app1/pages/KnownActivationContextDemo.tml
TAP5-2070 show new pages in main intergation test app (mlusetti: rev 
83fcdca312281c36399ed69773f55d677e05f6fd)
* tapestry-core/src/test/app1/Index.tml
TAP5-2070 new test integration app for activation context check (mlusetti: rev 
7e058593bf074416bd8af97a66a1dc770021e8bd)
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx/services/AppModule.java
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx/pages/Index.java
* 
tapestry-core/src/test/resources/org/apache/tapestry5/integration/activationctx/pages/Index.tml
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx/pages/OneContext.java
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx/pages/NoContext.java
* 
tapestry-core/src/test/resources/org/apache/tapestry5/integration/activationctx/pages/NoContext.tml
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx/components/Layout.java
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx/pages/TwoContext.java
* tapestry-core/src/test/activationctx/WEB-INF/web.xml
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx/ActivationContextintegrationTests.java
* 
tapestry-core/src/test/resources/org/apache/tapestry5/integration/activationctx/pages/OneContext.tml
* 
tapestry-core/src/test/resources/org/apache/tapestry5/integration/activationctx/components/Layout.tml
* 
tapestry-core/src/test/resources/org/apache/tapestry5/integration/activationctx/pages/TwoContext.tml
TAP5-2070 new test integration app for activation context check with inverse 
behavior (all enabled, some excluded) (mlusetti: rev 
2d5fb4b77e7250b5d12ba23fc1a48075b429d034)
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx2/pages/ContextUnchecked.java
* 
tapestry-core/src/test/resources/org/apache/tapestry5/integration/activationctx2/pages/Index.tml
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx2/services/AppModule.java
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx2/pages/ContextChecked.java
* 
tapestry-core/src/test/resources/org/apache/tapestry5/integration/activationctx2/pages/ContextUnchecked.tml
* 
tapestry-core/src/test/resources/org/apache/tapestry5/integration/activationctx2/pages/ContextChecked.tml
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx2/pages/Index.java
* tapestry-core/src/test/activationctx2/WEB-INF/web.xml
* 
tapestry-core/src/test/resources/org/apache/tapestry5/integration/activationctx2/components/Layout.tml
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx2/ActivationContextIntegrationTests2.java
* 
tapestry-core/src/test/java/org/apache/tapestry5/integration/activationctx2/components/Layout.java
TAP5-2070 hook new test apps into test chain (mlusetti: rev 
a9173f9a7c86f2b65fa2b4d9f1b430f2e40a161b)
* tapestry-core/src/test/conf/testng.xml

                
> Respond with a HTTP 404 if the page is activated with an activation context 
> not explicitly supported by the page itself
> -----------------------------------------------------------------------------------------------------------------------
>
>                 Key: TAP5-2070
>                 URL: https://issues.apache.org/jira/browse/TAP5-2070
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.4
>            Reporter: Massimo Lusetti
>            Assignee: Massimo Lusetti
>
> The behavior from day one for pages within Tapestry is to being able to serve 
> requests that come with an activation context with more parameters that the 
> ones declared by the page itself.
> This feature lead the framework to serve requests that comes to URL which are 
> not explicitly declared by the pages, for example a page named: Super with an 
> activation event handler method like this one:
> onActivate(String character) { ... }
> will finely serve requests for:
> /super/mario
> /super/luigi
> /super/wario
> /super/waluigi
> but it will also serve requests for:
> /super/mario/luigi/wario/waluigi
> This issue is to change that behavior.
> If the page does declare a specific activation event handler method the 
> corresponding serving URLs should adhere strictly, otherwise a HTTP 404 is 
> raised.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to