[ 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