You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@tapestry.apache.org by "Joost Schouten (ml)" <jo...@jsportal.com> on 2009/11/26 15:53:32 UTC
[T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Hi,
When using the AjaxFormLoop's AddRowLink in conjunction with onPassivate
an invalid url is generated [1]. When I remove onPassivate from my page
all works fine.
Am I missing something, is this work in progress (I'm using the SNAPSHOT
version) or shall I file a bug report?
Cheers,
Joost
[1] (shortened version but notice the double question mark)
http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org
Re: [T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Posted by "Joost Schouten (ml)" <jo...@jsportal.com>.
I have created: https://issues.apache.org/jira/browse/TAP5-937
whith a patch to fix the problem in the LinkImpl.
Thanks,
Joost
Ulrich Stärk wrote:
> I tested it with 5.2.0-SNAPSHOT and 5.1.0.5, but to be 100% sure, I'll
> clear out my local maven repo and test again this evening.
>
> Uli
>
> Am 27.11.2009 15:21 schrieb Joost Schouten (ml):
>> Ulrich,
>>
>> So in your case the add row url has appended the t:componentformid
>> and t:formid to the parameter list in the url with the already
>> available activation context param t:ac? And are you using the
>> 5.2.0-SNAPSHOT? This only started happening recently where before all
>> worked fine.
>>
>> Thanks for you help,
>> Joost
>>
>>
>> Ulrich Stärk wrote:
>>> I'm unable to reproduce any of the behaviour you are experiencing.
>>> The URL for the Ajax update generated by tapestry is valid and I
>>> don't get any errors.
>>>
>>> Uli
>>>
>>> Am 26.11.2009 17:15 schrieb Joost Schouten (ml):
>>>> Hi,
>>>>
>>>> It's not just passivate, its an activate and passivate combination.
>>>> The page [1] and tml [2] will reproduce the problem.
>>>>
>>>> Strangely enough when in this example you remove the activate and
>>>> passivate you get another weird error which looks like it could be
>>>> related to http://issues.apache.org/jira/browse/TAP5-733. My more
>>>> complex production code does not have this problem. Since it is
>>>> secondary to my current situation I'll just leave it for now.
>>>>
>>>> Cheers,
>>>> Joost
>>>>
>>>> [1]
>>>> public class TestPage {
>>>> @Property
>>>> private String loopValue;
>>>> @Component(parameters = {"source=['one','two','three']",
>>>> "value=loopValue"})
>>>> private AjaxFormLoop myAjaxLoop;
>>>>
>>>> @OnEvent(value=EventConstants.ADD_ROW)
>>>> private Object handleAddRow() {
>>>> return "the new row";
>>>> }
>>>> @OnEvent(value = EventConstants.PASSIVATE)
>>>> private Object passivate() {
>>>> return "passivate_value";
>>>> }
>>>>
>>>> @OnEvent(value = EventConstants.ACTIVATE)
>>>> private void activate(String value) {
>>>> }
>>>> }
>>>>
>>>>
>>>>
>>>> [2]
>>>> <html xmlns:t="http://tapestry.apache.org/schema/tapestry_5_0_0.xsd">
>>>> <head>
>>>> <title>test page</title>
>>>> </head>
>>>> <body>
>>>> <t:form>
>>>> <div t:id="myAjaxLoop">
>>>> <t:textfield value="loopValue"/>
>>>> </div>
>>>> </t:form>
>>>> </body>
>>>> </html>
>>>>
>>>>
>>>> Ulrich Stärk wrote:
>>>>> Could you provide a minimal example?
>>>>>
>>>>> Am 26.11.2009 15:53 schrieb Joost Schouten (ml):
>>>>>> Hi,
>>>>>>
>>>>>> When using the AjaxFormLoop's AddRowLink in conjunction with
>>>>>> onPassivate an invalid url is generated [1]. When I remove
>>>>>> onPassivate from my page all works fine.
>>>>>>
>>>>>> Am I missing something, is this work in progress (I'm using the
>>>>>> SNAPSHOT version) or shall I file a bug report?
>>>>>>
>>>>>> Cheers,
>>>>>> Joost
>>>>>>
>>>>>> [1] (shortened version but notice the double question mark)
>>>>>> http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
>>>>>>
>>>>>>
>>>>>> ---------------------------------------------------------------------
>>>>>>
>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>
>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>> For additional commands, e-mail: users-help@tapestry.apache.org
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org
Re: [T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Posted by "Joost Schouten (ml)" <jo...@jsportal.com>.
Before I forget, thanks a lot for your help on this issue! It caused me
to dive into the Tapestry source a bit deeper and makes me appreciate
the framework even more!
A few more weeks and I will bring a new Tapestry application to the
world ;-)
Ulrich Stärk wrote:
> That explains why I wasn't able to reproduce it. Great investigative
> work :)
>
> Uli
>
> Am 30.11.2009 08:55 schrieb Joost Schouten (ml):
>> Eureka! ;-) I figured it out (I think). The problem only exists when
>> using any UrlRewriteRule, activation context and additional
>> parameters. The t:ac seems to be added to the URL before the
>> UrlRewriting happens, where the AjaxFormLoop parameters seem to be
>> added after the rewriting which results in the buildURI being called
>> twice, causing the double question mark.
>>
>> My provided patch in https://issues.apache.org/jira/browse/TAP5-937
>> solves the issue by making the LinkImpl handle baseUrl's with already
>> provided parameters use "&" in stead of "?" for the first parameter
>> added.
>>
>> I look forward to seeing the patch implemented.
>> Cheers,
>> Joost
>>
>>
>> Ulrich Stärk wrote:
>>> Joost,
>>>
>>> I still can't reproduce your issue. The requested URL when clicking
>>> the add row link according to firebug is
>>> http://localhost:8080/testpage.myajaxloop.rowinjector:inject?t:ac=passivate_value&t:formcomponentid=TestPage:form&t:formid=form
>>> which is perfectly valid and is the same as generated by Tapestry
>>> for the responsible javascript.
>>> I tried with 5.2.0-SNAPSHOT and even directly from trunk.
>>>
>>> Do you have some dispatcher or filter in place that might be messing
>>> with the URL?
>>>
>>> Uli
>>>
>>> Am 27.11.2009 15:53 schrieb Joost Schouten (ml):
>>>> I just tested it against 5.1.0.5 and it does *not* have the problem
>>>> with the double printing of the question mark. There seems to be a
>>>> problem on the snapshot. Has any work been done on it lately?
>>>>
>>>> Ulrich Stärk wrote:
>>>>> I tested it with 5.2.0-SNAPSHOT and 5.1.0.5, but to be 100% sure,
>>>>> I'll clear out my local maven repo and test again this evening.
>>>>>
>>>>> Uli
>>>>>
>>>>> Am 27.11.2009 15:21 schrieb Joost Schouten (ml):
>>>>>> Ulrich,
>>>>>>
>>>>>> So in your case the add row url has appended the
>>>>>> t:componentformid and t:formid to the parameter list in the url
>>>>>> with the already available activation context param t:ac? And are
>>>>>> you using the 5.2.0-SNAPSHOT? This only started happening
>>>>>> recently where before all worked fine.
>>>>>>
>>>>>> Thanks for you help,
>>>>>> Joost
>>>>>>
>>>>>>
>>>>>> Ulrich Stärk wrote:
>>>>>>> I'm unable to reproduce any of the behaviour you are
>>>>>>> experiencing. The URL for the Ajax update generated by tapestry
>>>>>>> is valid and I don't get any errors.
>>>>>>>
>>>>>>> Uli
>>>>>>>
>>>>>>> Am 26.11.2009 17:15 schrieb Joost Schouten (ml):
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>> It's not just passivate, its an activate and passivate
>>>>>>>> combination. The page [1] and tml [2] will reproduce the problem.
>>>>>>>>
>>>>>>>> Strangely enough when in this example you remove the activate
>>>>>>>> and passivate you get another weird error which looks like it
>>>>>>>> could be related to
>>>>>>>> http://issues.apache.org/jira/browse/TAP5-733. My more complex
>>>>>>>> production code does not have this problem. Since it is
>>>>>>>> secondary to my current situation I'll just leave it for now.
>>>>>>>>
>>>>>>>> Cheers,
>>>>>>>> Joost
>>>>>>>>
>>>>>>>> [1]
>>>>>>>> public class TestPage {
>>>>>>>> @Property
>>>>>>>> private String loopValue;
>>>>>>>> @Component(parameters = {"source=['one','two','three']",
>>>>>>>> "value=loopValue"})
>>>>>>>> private AjaxFormLoop myAjaxLoop;
>>>>>>>>
>>>>>>>> @OnEvent(value=EventConstants.ADD_ROW)
>>>>>>>> private Object handleAddRow() {
>>>>>>>> return "the new row";
>>>>>>>> }
>>>>>>>> @OnEvent(value = EventConstants.PASSIVATE)
>>>>>>>> private Object passivate() {
>>>>>>>> return "passivate_value";
>>>>>>>> }
>>>>>>>>
>>>>>>>> @OnEvent(value = EventConstants.ACTIVATE)
>>>>>>>> private void activate(String value) {
>>>>>>>> }
>>>>>>>> }
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> [2]
>>>>>>>> <html
>>>>>>>> xmlns:t="http://tapestry.apache.org/schema/tapestry_5_0_0.xsd">
>>>>>>>> <head>
>>>>>>>> <title>test page</title>
>>>>>>>> </head>
>>>>>>>> <body>
>>>>>>>> <t:form>
>>>>>>>> <div t:id="myAjaxLoop">
>>>>>>>> <t:textfield value="loopValue"/>
>>>>>>>> </div>
>>>>>>>> </t:form>
>>>>>>>> </body>
>>>>>>>> </html>
>>>>>>>>
>>>>>>>>
>>>>>>>> Ulrich Stärk wrote:
>>>>>>>>> Could you provide a minimal example?
>>>>>>>>>
>>>>>>>>> Am 26.11.2009 15:53 schrieb Joost Schouten (ml):
>>>>>>>>>> Hi,
>>>>>>>>>>
>>>>>>>>>> When using the AjaxFormLoop's AddRowLink in conjunction with
>>>>>>>>>> onPassivate an invalid url is generated [1]. When I remove
>>>>>>>>>> onPassivate from my page all works fine.
>>>>>>>>>>
>>>>>>>>>> Am I missing something, is this work in progress (I'm using
>>>>>>>>>> the SNAPSHOT version) or shall I file a bug report?
>>>>>>>>>>
>>>>>>>>>> Cheers,
>>>>>>>>>> Joost
>>>>>>>>>>
>>>>>>>>>> [1] (shortened version but notice the double question mark)
>>>>>>>>>> http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> ---------------------------------------------------------------------
>>>>>>>>>>
>>>>>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>> ---------------------------------------------------------------------
>>>>>>>>>
>>>>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> ---------------------------------------------------------------------
>>>>>>>>
>>>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>>>
>>>>>>>
>>>>>>> ---------------------------------------------------------------------
>>>>>>>
>>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>>
>>>>>>
>>>>>>
>>>>>> ---------------------------------------------------------------------
>>>>>>
>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>
>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>> For additional commands, e-mail: users-help@tapestry.apache.org
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org
Re: [T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Posted by Ulrich Stärk <ul...@spielviel.de>.
That explains why I wasn't able to reproduce it. Great investigative work :)
Uli
Am 30.11.2009 08:55 schrieb Joost Schouten (ml):
> Eureka! ;-) I figured it out (I think). The problem only exists when
> using any UrlRewriteRule, activation context and additional parameters.
> The t:ac seems to be added to the URL before the UrlRewriting happens,
> where the AjaxFormLoop parameters seem to be added after the rewriting
> which results in the buildURI being called twice, causing the double
> question mark.
>
> My provided patch in https://issues.apache.org/jira/browse/TAP5-937
> solves the issue by making the LinkImpl handle baseUrl's with already
> provided parameters use "&" in stead of "?" for the first parameter added.
>
> I look forward to seeing the patch implemented.
> Cheers,
> Joost
>
>
> Ulrich Stärk wrote:
>> Joost,
>>
>> I still can't reproduce your issue. The requested URL when clicking
>> the add row link according to firebug is
>> http://localhost:8080/testpage.myajaxloop.rowinjector:inject?t:ac=passivate_value&t:formcomponentid=TestPage:form&t:formid=form
>> which is perfectly valid and is the same as generated by Tapestry for
>> the responsible javascript.
>> I tried with 5.2.0-SNAPSHOT and even directly from trunk.
>>
>> Do you have some dispatcher or filter in place that might be messing
>> with the URL?
>>
>> Uli
>>
>> Am 27.11.2009 15:53 schrieb Joost Schouten (ml):
>>> I just tested it against 5.1.0.5 and it does *not* have the problem
>>> with the double printing of the question mark. There seems to be a
>>> problem on the snapshot. Has any work been done on it lately?
>>>
>>> Ulrich Stärk wrote:
>>>> I tested it with 5.2.0-SNAPSHOT and 5.1.0.5, but to be 100% sure,
>>>> I'll clear out my local maven repo and test again this evening.
>>>>
>>>> Uli
>>>>
>>>> Am 27.11.2009 15:21 schrieb Joost Schouten (ml):
>>>>> Ulrich,
>>>>>
>>>>> So in your case the add row url has appended the t:componentformid
>>>>> and t:formid to the parameter list in the url with the already
>>>>> available activation context param t:ac? And are you using the
>>>>> 5.2.0-SNAPSHOT? This only started happening recently where before
>>>>> all worked fine.
>>>>>
>>>>> Thanks for you help,
>>>>> Joost
>>>>>
>>>>>
>>>>> Ulrich Stärk wrote:
>>>>>> I'm unable to reproduce any of the behaviour you are experiencing.
>>>>>> The URL for the Ajax update generated by tapestry is valid and I
>>>>>> don't get any errors.
>>>>>>
>>>>>> Uli
>>>>>>
>>>>>> Am 26.11.2009 17:15 schrieb Joost Schouten (ml):
>>>>>>> Hi,
>>>>>>>
>>>>>>> It's not just passivate, its an activate and passivate
>>>>>>> combination. The page [1] and tml [2] will reproduce the problem.
>>>>>>>
>>>>>>> Strangely enough when in this example you remove the activate and
>>>>>>> passivate you get another weird error which looks like it could
>>>>>>> be related to http://issues.apache.org/jira/browse/TAP5-733. My
>>>>>>> more complex production code does not have this problem. Since it
>>>>>>> is secondary to my current situation I'll just leave it for now.
>>>>>>>
>>>>>>> Cheers,
>>>>>>> Joost
>>>>>>>
>>>>>>> [1]
>>>>>>> public class TestPage {
>>>>>>> @Property
>>>>>>> private String loopValue;
>>>>>>> @Component(parameters = {"source=['one','two','three']",
>>>>>>> "value=loopValue"})
>>>>>>> private AjaxFormLoop myAjaxLoop;
>>>>>>>
>>>>>>> @OnEvent(value=EventConstants.ADD_ROW)
>>>>>>> private Object handleAddRow() {
>>>>>>> return "the new row";
>>>>>>> }
>>>>>>> @OnEvent(value = EventConstants.PASSIVATE)
>>>>>>> private Object passivate() {
>>>>>>> return "passivate_value";
>>>>>>> }
>>>>>>>
>>>>>>> @OnEvent(value = EventConstants.ACTIVATE)
>>>>>>> private void activate(String value) {
>>>>>>> }
>>>>>>> }
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> [2]
>>>>>>> <html
>>>>>>> xmlns:t="http://tapestry.apache.org/schema/tapestry_5_0_0.xsd">
>>>>>>> <head>
>>>>>>> <title>test page</title>
>>>>>>> </head>
>>>>>>> <body>
>>>>>>> <t:form>
>>>>>>> <div t:id="myAjaxLoop">
>>>>>>> <t:textfield value="loopValue"/>
>>>>>>> </div>
>>>>>>> </t:form>
>>>>>>> </body>
>>>>>>> </html>
>>>>>>>
>>>>>>>
>>>>>>> Ulrich Stärk wrote:
>>>>>>>> Could you provide a minimal example?
>>>>>>>>
>>>>>>>> Am 26.11.2009 15:53 schrieb Joost Schouten (ml):
>>>>>>>>> Hi,
>>>>>>>>>
>>>>>>>>> When using the AjaxFormLoop's AddRowLink in conjunction with
>>>>>>>>> onPassivate an invalid url is generated [1]. When I remove
>>>>>>>>> onPassivate from my page all works fine.
>>>>>>>>>
>>>>>>>>> Am I missing something, is this work in progress (I'm using the
>>>>>>>>> SNAPSHOT version) or shall I file a bug report?
>>>>>>>>>
>>>>>>>>> Cheers,
>>>>>>>>> Joost
>>>>>>>>>
>>>>>>>>> [1] (shortened version but notice the double question mark)
>>>>>>>>> http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> ---------------------------------------------------------------------
>>>>>>>>>
>>>>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>>>>
>>>>>>>>
>>>>>>>> ---------------------------------------------------------------------
>>>>>>>>
>>>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> ---------------------------------------------------------------------
>>>>>>>
>>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>>
>>>>>>
>>>>>> ---------------------------------------------------------------------
>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>
>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>
>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>> For additional commands, e-mail: users-help@tapestry.apache.org
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org
Re: [T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Posted by "Joost Schouten (ml)" <jo...@jsportal.com>.
Eureka! ;-) I figured it out (I think). The problem only exists when
using any UrlRewriteRule, activation context and additional parameters.
The t:ac seems to be added to the URL before the UrlRewriting happens,
where the AjaxFormLoop parameters seem to be added after the rewriting
which results in the buildURI being called twice, causing the double
question mark.
My provided patch in https://issues.apache.org/jira/browse/TAP5-937
solves the issue by making the LinkImpl handle baseUrl's with already
provided parameters use "&" in stead of "?" for the first parameter added.
I look forward to seeing the patch implemented.
Cheers,
Joost
Ulrich Stärk wrote:
> Joost,
>
> I still can't reproduce your issue. The requested URL when clicking
> the add row link according to firebug is
> http://localhost:8080/testpage.myajaxloop.rowinjector:inject?t:ac=passivate_value&t:formcomponentid=TestPage:form&t:formid=form
> which is perfectly valid and is the same as generated by Tapestry for
> the responsible javascript.
> I tried with 5.2.0-SNAPSHOT and even directly from trunk.
>
> Do you have some dispatcher or filter in place that might be messing
> with the URL?
>
> Uli
>
> Am 27.11.2009 15:53 schrieb Joost Schouten (ml):
>> I just tested it against 5.1.0.5 and it does *not* have the problem
>> with the double printing of the question mark. There seems to be a
>> problem on the snapshot. Has any work been done on it lately?
>>
>> Ulrich Stärk wrote:
>>> I tested it with 5.2.0-SNAPSHOT and 5.1.0.5, but to be 100% sure,
>>> I'll clear out my local maven repo and test again this evening.
>>>
>>> Uli
>>>
>>> Am 27.11.2009 15:21 schrieb Joost Schouten (ml):
>>>> Ulrich,
>>>>
>>>> So in your case the add row url has appended the t:componentformid
>>>> and t:formid to the parameter list in the url with the already
>>>> available activation context param t:ac? And are you using the
>>>> 5.2.0-SNAPSHOT? This only started happening recently where before
>>>> all worked fine.
>>>>
>>>> Thanks for you help,
>>>> Joost
>>>>
>>>>
>>>> Ulrich Stärk wrote:
>>>>> I'm unable to reproduce any of the behaviour you are experiencing.
>>>>> The URL for the Ajax update generated by tapestry is valid and I
>>>>> don't get any errors.
>>>>>
>>>>> Uli
>>>>>
>>>>> Am 26.11.2009 17:15 schrieb Joost Schouten (ml):
>>>>>> Hi,
>>>>>>
>>>>>> It's not just passivate, its an activate and passivate
>>>>>> combination. The page [1] and tml [2] will reproduce the problem.
>>>>>>
>>>>>> Strangely enough when in this example you remove the activate and
>>>>>> passivate you get another weird error which looks like it could
>>>>>> be related to http://issues.apache.org/jira/browse/TAP5-733. My
>>>>>> more complex production code does not have this problem. Since it
>>>>>> is secondary to my current situation I'll just leave it for now.
>>>>>>
>>>>>> Cheers,
>>>>>> Joost
>>>>>>
>>>>>> [1]
>>>>>> public class TestPage {
>>>>>> @Property
>>>>>> private String loopValue;
>>>>>> @Component(parameters = {"source=['one','two','three']",
>>>>>> "value=loopValue"})
>>>>>> private AjaxFormLoop myAjaxLoop;
>>>>>>
>>>>>> @OnEvent(value=EventConstants.ADD_ROW)
>>>>>> private Object handleAddRow() {
>>>>>> return "the new row";
>>>>>> }
>>>>>> @OnEvent(value = EventConstants.PASSIVATE)
>>>>>> private Object passivate() {
>>>>>> return "passivate_value";
>>>>>> }
>>>>>>
>>>>>> @OnEvent(value = EventConstants.ACTIVATE)
>>>>>> private void activate(String value) {
>>>>>> }
>>>>>> }
>>>>>>
>>>>>>
>>>>>>
>>>>>> [2]
>>>>>> <html
>>>>>> xmlns:t="http://tapestry.apache.org/schema/tapestry_5_0_0.xsd">
>>>>>> <head>
>>>>>> <title>test page</title>
>>>>>> </head>
>>>>>> <body>
>>>>>> <t:form>
>>>>>> <div t:id="myAjaxLoop">
>>>>>> <t:textfield value="loopValue"/>
>>>>>> </div>
>>>>>> </t:form>
>>>>>> </body>
>>>>>> </html>
>>>>>>
>>>>>>
>>>>>> Ulrich Stärk wrote:
>>>>>>> Could you provide a minimal example?
>>>>>>>
>>>>>>> Am 26.11.2009 15:53 schrieb Joost Schouten (ml):
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>> When using the AjaxFormLoop's AddRowLink in conjunction with
>>>>>>>> onPassivate an invalid url is generated [1]. When I remove
>>>>>>>> onPassivate from my page all works fine.
>>>>>>>>
>>>>>>>> Am I missing something, is this work in progress (I'm using the
>>>>>>>> SNAPSHOT version) or shall I file a bug report?
>>>>>>>>
>>>>>>>> Cheers,
>>>>>>>> Joost
>>>>>>>>
>>>>>>>> [1] (shortened version but notice the double question mark)
>>>>>>>> http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
>>>>>>>>
>>>>>>>>
>>>>>>>> ---------------------------------------------------------------------
>>>>>>>>
>>>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>>>
>>>>>>>
>>>>>>> ---------------------------------------------------------------------
>>>>>>>
>>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>>
>>>>>>
>>>>>>
>>>>>> ---------------------------------------------------------------------
>>>>>>
>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>
>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>> For additional commands, e-mail: users-help@tapestry.apache.org
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org
Re: [T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Posted by Ulrich Stärk <ul...@spielviel.de>.
Joost,
I still can't reproduce your issue. The requested URL when clicking the add row link according to
firebug is
http://localhost:8080/testpage.myajaxloop.rowinjector:inject?t:ac=passivate_value&t:formcomponentid=TestPage:form&t:formid=form
which is perfectly valid and is the same as generated by Tapestry for the responsible javascript.
I tried with 5.2.0-SNAPSHOT and even directly from trunk.
Do you have some dispatcher or filter in place that might be messing with the URL?
Uli
Am 27.11.2009 15:53 schrieb Joost Schouten (ml):
> I just tested it against 5.1.0.5 and it does *not* have the problem with
> the double printing of the question mark. There seems to be a problem on
> the snapshot. Has any work been done on it lately?
>
> Ulrich Stärk wrote:
>> I tested it with 5.2.0-SNAPSHOT and 5.1.0.5, but to be 100% sure, I'll
>> clear out my local maven repo and test again this evening.
>>
>> Uli
>>
>> Am 27.11.2009 15:21 schrieb Joost Schouten (ml):
>>> Ulrich,
>>>
>>> So in your case the add row url has appended the t:componentformid
>>> and t:formid to the parameter list in the url with the already
>>> available activation context param t:ac? And are you using the
>>> 5.2.0-SNAPSHOT? This only started happening recently where before all
>>> worked fine.
>>>
>>> Thanks for you help,
>>> Joost
>>>
>>>
>>> Ulrich Stärk wrote:
>>>> I'm unable to reproduce any of the behaviour you are experiencing.
>>>> The URL for the Ajax update generated by tapestry is valid and I
>>>> don't get any errors.
>>>>
>>>> Uli
>>>>
>>>> Am 26.11.2009 17:15 schrieb Joost Schouten (ml):
>>>>> Hi,
>>>>>
>>>>> It's not just passivate, its an activate and passivate combination.
>>>>> The page [1] and tml [2] will reproduce the problem.
>>>>>
>>>>> Strangely enough when in this example you remove the activate and
>>>>> passivate you get another weird error which looks like it could be
>>>>> related to http://issues.apache.org/jira/browse/TAP5-733. My more
>>>>> complex production code does not have this problem. Since it is
>>>>> secondary to my current situation I'll just leave it for now.
>>>>>
>>>>> Cheers,
>>>>> Joost
>>>>>
>>>>> [1]
>>>>> public class TestPage {
>>>>> @Property
>>>>> private String loopValue;
>>>>> @Component(parameters = {"source=['one','two','three']",
>>>>> "value=loopValue"})
>>>>> private AjaxFormLoop myAjaxLoop;
>>>>>
>>>>> @OnEvent(value=EventConstants.ADD_ROW)
>>>>> private Object handleAddRow() {
>>>>> return "the new row";
>>>>> }
>>>>> @OnEvent(value = EventConstants.PASSIVATE)
>>>>> private Object passivate() {
>>>>> return "passivate_value";
>>>>> }
>>>>>
>>>>> @OnEvent(value = EventConstants.ACTIVATE)
>>>>> private void activate(String value) {
>>>>> }
>>>>> }
>>>>>
>>>>>
>>>>>
>>>>> [2]
>>>>> <html xmlns:t="http://tapestry.apache.org/schema/tapestry_5_0_0.xsd">
>>>>> <head>
>>>>> <title>test page</title>
>>>>> </head>
>>>>> <body>
>>>>> <t:form>
>>>>> <div t:id="myAjaxLoop">
>>>>> <t:textfield value="loopValue"/>
>>>>> </div>
>>>>> </t:form>
>>>>> </body>
>>>>> </html>
>>>>>
>>>>>
>>>>> Ulrich Stärk wrote:
>>>>>> Could you provide a minimal example?
>>>>>>
>>>>>> Am 26.11.2009 15:53 schrieb Joost Schouten (ml):
>>>>>>> Hi,
>>>>>>>
>>>>>>> When using the AjaxFormLoop's AddRowLink in conjunction with
>>>>>>> onPassivate an invalid url is generated [1]. When I remove
>>>>>>> onPassivate from my page all works fine.
>>>>>>>
>>>>>>> Am I missing something, is this work in progress (I'm using the
>>>>>>> SNAPSHOT version) or shall I file a bug report?
>>>>>>>
>>>>>>> Cheers,
>>>>>>> Joost
>>>>>>>
>>>>>>> [1] (shortened version but notice the double question mark)
>>>>>>> http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
>>>>>>>
>>>>>>>
>>>>>>> ---------------------------------------------------------------------
>>>>>>>
>>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>>
>>>>>>
>>>>>> ---------------------------------------------------------------------
>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>
>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>
>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>> For additional commands, e-mail: users-help@tapestry.apache.org
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org
Re: [T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Posted by "Joost Schouten (ml)" <jo...@jsportal.com>.
I just tested it against 5.1.0.5 and it does *not* have the problem with
the double printing of the question mark. There seems to be a problem on
the snapshot. Has any work been done on it lately?
Ulrich Stärk wrote:
> I tested it with 5.2.0-SNAPSHOT and 5.1.0.5, but to be 100% sure, I'll
> clear out my local maven repo and test again this evening.
>
> Uli
>
> Am 27.11.2009 15:21 schrieb Joost Schouten (ml):
>> Ulrich,
>>
>> So in your case the add row url has appended the t:componentformid
>> and t:formid to the parameter list in the url with the already
>> available activation context param t:ac? And are you using the
>> 5.2.0-SNAPSHOT? This only started happening recently where before all
>> worked fine.
>>
>> Thanks for you help,
>> Joost
>>
>>
>> Ulrich Stärk wrote:
>>> I'm unable to reproduce any of the behaviour you are experiencing.
>>> The URL for the Ajax update generated by tapestry is valid and I
>>> don't get any errors.
>>>
>>> Uli
>>>
>>> Am 26.11.2009 17:15 schrieb Joost Schouten (ml):
>>>> Hi,
>>>>
>>>> It's not just passivate, its an activate and passivate combination.
>>>> The page [1] and tml [2] will reproduce the problem.
>>>>
>>>> Strangely enough when in this example you remove the activate and
>>>> passivate you get another weird error which looks like it could be
>>>> related to http://issues.apache.org/jira/browse/TAP5-733. My more
>>>> complex production code does not have this problem. Since it is
>>>> secondary to my current situation I'll just leave it for now.
>>>>
>>>> Cheers,
>>>> Joost
>>>>
>>>> [1]
>>>> public class TestPage {
>>>> @Property
>>>> private String loopValue;
>>>> @Component(parameters = {"source=['one','two','three']",
>>>> "value=loopValue"})
>>>> private AjaxFormLoop myAjaxLoop;
>>>>
>>>> @OnEvent(value=EventConstants.ADD_ROW)
>>>> private Object handleAddRow() {
>>>> return "the new row";
>>>> }
>>>> @OnEvent(value = EventConstants.PASSIVATE)
>>>> private Object passivate() {
>>>> return "passivate_value";
>>>> }
>>>>
>>>> @OnEvent(value = EventConstants.ACTIVATE)
>>>> private void activate(String value) {
>>>> }
>>>> }
>>>>
>>>>
>>>>
>>>> [2]
>>>> <html xmlns:t="http://tapestry.apache.org/schema/tapestry_5_0_0.xsd">
>>>> <head>
>>>> <title>test page</title>
>>>> </head>
>>>> <body>
>>>> <t:form>
>>>> <div t:id="myAjaxLoop">
>>>> <t:textfield value="loopValue"/>
>>>> </div>
>>>> </t:form>
>>>> </body>
>>>> </html>
>>>>
>>>>
>>>> Ulrich Stärk wrote:
>>>>> Could you provide a minimal example?
>>>>>
>>>>> Am 26.11.2009 15:53 schrieb Joost Schouten (ml):
>>>>>> Hi,
>>>>>>
>>>>>> When using the AjaxFormLoop's AddRowLink in conjunction with
>>>>>> onPassivate an invalid url is generated [1]. When I remove
>>>>>> onPassivate from my page all works fine.
>>>>>>
>>>>>> Am I missing something, is this work in progress (I'm using the
>>>>>> SNAPSHOT version) or shall I file a bug report?
>>>>>>
>>>>>> Cheers,
>>>>>> Joost
>>>>>>
>>>>>> [1] (shortened version but notice the double question mark)
>>>>>> http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
>>>>>>
>>>>>>
>>>>>> ---------------------------------------------------------------------
>>>>>>
>>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>
>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>> For additional commands, e-mail: users-help@tapestry.apache.org
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org
Re: [T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Posted by Ulrich Stärk <ul...@spielviel.de>.
I tested it with 5.2.0-SNAPSHOT and 5.1.0.5, but to be 100% sure, I'll clear out my local maven repo
and test again this evening.
Uli
Am 27.11.2009 15:21 schrieb Joost Schouten (ml):
> Ulrich,
>
> So in your case the add row url has appended the t:componentformid and
> t:formid to the parameter list in the url with the already available
> activation context param t:ac? And are you using the 5.2.0-SNAPSHOT?
> This only started happening recently where before all worked fine.
>
> Thanks for you help,
> Joost
>
>
> Ulrich Stärk wrote:
>> I'm unable to reproduce any of the behaviour you are experiencing. The
>> URL for the Ajax update generated by tapestry is valid and I don't get
>> any errors.
>>
>> Uli
>>
>> Am 26.11.2009 17:15 schrieb Joost Schouten (ml):
>>> Hi,
>>>
>>> It's not just passivate, its an activate and passivate combination.
>>> The page [1] and tml [2] will reproduce the problem.
>>>
>>> Strangely enough when in this example you remove the activate and
>>> passivate you get another weird error which looks like it could be
>>> related to http://issues.apache.org/jira/browse/TAP5-733. My more
>>> complex production code does not have this problem. Since it is
>>> secondary to my current situation I'll just leave it for now.
>>>
>>> Cheers,
>>> Joost
>>>
>>> [1]
>>> public class TestPage {
>>> @Property
>>> private String loopValue;
>>> @Component(parameters = {"source=['one','two','three']",
>>> "value=loopValue"})
>>> private AjaxFormLoop myAjaxLoop;
>>>
>>> @OnEvent(value=EventConstants.ADD_ROW)
>>> private Object handleAddRow() {
>>> return "the new row";
>>> }
>>> @OnEvent(value = EventConstants.PASSIVATE)
>>> private Object passivate() {
>>> return "passivate_value";
>>> }
>>>
>>> @OnEvent(value = EventConstants.ACTIVATE)
>>> private void activate(String value) {
>>> }
>>> }
>>>
>>>
>>>
>>> [2]
>>> <html xmlns:t="http://tapestry.apache.org/schema/tapestry_5_0_0.xsd">
>>> <head>
>>> <title>test page</title>
>>> </head>
>>> <body>
>>> <t:form>
>>> <div t:id="myAjaxLoop">
>>> <t:textfield value="loopValue"/>
>>> </div>
>>> </t:form>
>>> </body>
>>> </html>
>>>
>>>
>>> Ulrich Stärk wrote:
>>>> Could you provide a minimal example?
>>>>
>>>> Am 26.11.2009 15:53 schrieb Joost Schouten (ml):
>>>>> Hi,
>>>>>
>>>>> When using the AjaxFormLoop's AddRowLink in conjunction with
>>>>> onPassivate an invalid url is generated [1]. When I remove
>>>>> onPassivate from my page all works fine.
>>>>>
>>>>> Am I missing something, is this work in progress (I'm using the
>>>>> SNAPSHOT version) or shall I file a bug report?
>>>>>
>>>>> Cheers,
>>>>> Joost
>>>>>
>>>>> [1] (shortened version but notice the double question mark)
>>>>> http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>
>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>> For additional commands, e-mail: users-help@tapestry.apache.org
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org
Re: [T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Posted by "Joost Schouten (ml)" <jo...@jsportal.com>.
Ulrich,
So in your case the add row url has appended the t:componentformid and
t:formid to the parameter list in the url with the already available
activation context param t:ac? And are you using the 5.2.0-SNAPSHOT?
This only started happening recently where before all worked fine.
Thanks for you help,
Joost
Ulrich Stärk wrote:
> I'm unable to reproduce any of the behaviour you are experiencing. The
> URL for the Ajax update generated by tapestry is valid and I don't get
> any errors.
>
> Uli
>
> Am 26.11.2009 17:15 schrieb Joost Schouten (ml):
>> Hi,
>>
>> It's not just passivate, its an activate and passivate combination.
>> The page [1] and tml [2] will reproduce the problem.
>>
>> Strangely enough when in this example you remove the activate and
>> passivate you get another weird error which looks like it could be
>> related to http://issues.apache.org/jira/browse/TAP5-733. My more
>> complex production code does not have this problem. Since it is
>> secondary to my current situation I'll just leave it for now.
>>
>> Cheers,
>> Joost
>>
>> [1]
>> public class TestPage {
>> @Property
>> private String loopValue;
>> @Component(parameters = {"source=['one','two','three']",
>> "value=loopValue"})
>> private AjaxFormLoop myAjaxLoop;
>>
>> @OnEvent(value=EventConstants.ADD_ROW)
>> private Object handleAddRow() {
>> return "the new row";
>> }
>> @OnEvent(value = EventConstants.PASSIVATE)
>> private Object passivate() {
>> return "passivate_value";
>> }
>>
>> @OnEvent(value = EventConstants.ACTIVATE)
>> private void activate(String value) {
>> }
>> }
>>
>>
>>
>> [2]
>> <html xmlns:t="http://tapestry.apache.org/schema/tapestry_5_0_0.xsd">
>> <head>
>> <title>test page</title>
>> </head>
>> <body>
>> <t:form>
>> <div t:id="myAjaxLoop">
>> <t:textfield value="loopValue"/>
>> </div>
>> </t:form>
>> </body>
>> </html>
>>
>>
>> Ulrich Stärk wrote:
>>> Could you provide a minimal example?
>>>
>>> Am 26.11.2009 15:53 schrieb Joost Schouten (ml):
>>>> Hi,
>>>>
>>>> When using the AjaxFormLoop's AddRowLink in conjunction with
>>>> onPassivate an invalid url is generated [1]. When I remove
>>>> onPassivate from my page all works fine.
>>>>
>>>> Am I missing something, is this work in progress (I'm using the
>>>> SNAPSHOT version) or shall I file a bug report?
>>>>
>>>> Cheers,
>>>> Joost
>>>>
>>>> [1] (shortened version but notice the double question mark)
>>>> http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>> For additional commands, e-mail: users-help@tapestry.apache.org
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org
Re: [T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Posted by Ulrich Stärk <ul...@spielviel.de>.
I'm unable to reproduce any of the behaviour you are experiencing. The URL for the Ajax update
generated by tapestry is valid and I don't get any errors.
Uli
Am 26.11.2009 17:15 schrieb Joost Schouten (ml):
> Hi,
>
> It's not just passivate, its an activate and passivate combination. The
> page [1] and tml [2] will reproduce the problem.
>
> Strangely enough when in this example you remove the activate and
> passivate you get another weird error which looks like it could be
> related to http://issues.apache.org/jira/browse/TAP5-733. My more
> complex production code does not have this problem. Since it is
> secondary to my current situation I'll just leave it for now.
>
> Cheers,
> Joost
>
> [1]
> public class TestPage {
> @Property
> private String loopValue;
> @Component(parameters = {"source=['one','two','three']",
> "value=loopValue"})
> private AjaxFormLoop myAjaxLoop;
>
> @OnEvent(value=EventConstants.ADD_ROW)
> private Object handleAddRow() {
> return "the new row";
> }
> @OnEvent(value = EventConstants.PASSIVATE)
> private Object passivate() {
> return "passivate_value";
> }
>
> @OnEvent(value = EventConstants.ACTIVATE)
> private void activate(String value) {
> }
> }
>
>
>
> [2]
> <html xmlns:t="http://tapestry.apache.org/schema/tapestry_5_0_0.xsd">
> <head>
> <title>test page</title>
> </head>
> <body>
> <t:form>
> <div t:id="myAjaxLoop">
> <t:textfield value="loopValue"/>
> </div>
> </t:form>
> </body>
> </html>
>
>
> Ulrich Stärk wrote:
>> Could you provide a minimal example?
>>
>> Am 26.11.2009 15:53 schrieb Joost Schouten (ml):
>>> Hi,
>>>
>>> When using the AjaxFormLoop's AddRowLink in conjunction with
>>> onPassivate an invalid url is generated [1]. When I remove
>>> onPassivate from my page all works fine.
>>>
>>> Am I missing something, is this work in progress (I'm using the
>>> SNAPSHOT version) or shall I file a bug report?
>>>
>>> Cheers,
>>> Joost
>>>
>>> [1] (shortened version but notice the double question mark)
>>> http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>> For additional commands, e-mail: users-help@tapestry.apache.org
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org
Re: [T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Posted by "Joost Schouten (ml)" <jo...@jsportal.com>.
Hi,
It's not just passivate, its an activate and passivate combination. The
page [1] and tml [2] will reproduce the problem.
Strangely enough when in this example you remove the activate and
passivate you get another weird error which looks like it could be
related to http://issues.apache.org/jira/browse/TAP5-733. My more
complex production code does not have this problem. Since it is
secondary to my current situation I'll just leave it for now.
Cheers,
Joost
[1]
public class TestPage {
@Property
private String loopValue;
@Component(parameters = {"source=['one','two','three']",
"value=loopValue"})
private AjaxFormLoop myAjaxLoop;
@OnEvent(value=EventConstants.ADD_ROW)
private Object handleAddRow() {
return "the new row";
}
@OnEvent(value = EventConstants.PASSIVATE)
private Object passivate() {
return "passivate_value";
}
@OnEvent(value = EventConstants.ACTIVATE)
private void activate(String value) {
}
}
[2]
<html xmlns:t="http://tapestry.apache.org/schema/tapestry_5_0_0.xsd">
<head>
<title>test page</title>
</head>
<body>
<t:form>
<div t:id="myAjaxLoop">
<t:textfield value="loopValue"/>
</div>
</t:form>
</body>
</html>
Ulrich Stärk wrote:
> Could you provide a minimal example?
>
> Am 26.11.2009 15:53 schrieb Joost Schouten (ml):
>> Hi,
>>
>> When using the AjaxFormLoop's AddRowLink in conjunction with
>> onPassivate an invalid url is generated [1]. When I remove
>> onPassivate from my page all works fine.
>>
>> Am I missing something, is this work in progress (I'm using the
>> SNAPSHOT version) or shall I file a bug report?
>>
>> Cheers,
>> Joost
>>
>> [1] (shortened version but notice the double question mark)
>> http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>> For additional commands, e-mail: users-help@tapestry.apache.org
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org
Re: [T5.2.0] AddRowLink generates incorrect link in conjunction with
onPassivate
Posted by Ulrich Stärk <ul...@spielviel.de>.
Could you provide a minimal example?
Am 26.11.2009 15:53 schrieb Joost Schouten (ml):
> Hi,
>
> When using the AjaxFormLoop's AddRowLink in conjunction with onPassivate
> an invalid url is generated [1]. When I remove onPassivate from my page
> all works fine.
>
> Am I missing something, is this work in progress (I'm using the SNAPSHOT
> version) or shall I file a bug report?
>
> Cheers,
> Joost
>
> [1] (shortened version but notice the double question mark)
> http://domain.com/page.form.rowinjector:inject?t:ac=3?t:formcomponentid=page:form&t:formid=contactDetailForm-12530f31add
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org