random bug with textInput

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

random bug with textInput

Yann Monclair-2
Hi everyone,

I am working on my web iCalendar for the SummerTalk. a Demo is  
running at http://icalendar.seasidehosting.st/seaside/icalendar
I am experiencing a random bug with the editor (when you click on an  
event or a calendar you can edit it).
The code seems to work, since it sometimes works and sometimes  
doesn't. The bug only occurs with the textInput forms for me.

Here is how it is built :

The editor holds different forms, one for each editable field (text  
or selection). I submit all forms when the user hits the save button.
To do that I added this method:

SUAjax>>triggerForms: aCollection
aCollection do: [ :elt | self triggerForm: elt]

the collection holds all the ids of the forms. it is called like this :

ICWebEditor>>renderSaveButtonOn: html
"formIDs is an instance variable holding a collection of all the ids  
used in this editor"

html form id: html nextId;
        with: [html submitButton callback: [self save];
                        onClick: (html update id: html nextId;
                                         triggerForms: formIDs;
                                         callback: [:r|r]);
                         text: 'save'
                ]

IIRC this used to work fine a few weeks back. Was there some change  
in Seaside or Scriptaculous that could have broken this ?

Thanks,

Yann

PS: you can load the code from Squeak Source : http://
www.squeaksource.com/iCalSummerTalk.html
        You need to load first LogIn then SummerTime
        You'll need to have a working seaside install with seaside 2.7  
scriptaculous.
        You'll need iCal http://www.squeaksource.com/ical.html
       
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: random bug with textInput

Philippe Marschall
2006/12/5, Yann Monclair <[hidden email]>:

> Hi everyone,
>
> I am working on my web iCalendar for the SummerTalk. a Demo is
> running at http://icalendar.seasidehosting.st/seaside/icalendar
> I am experiencing a random bug with the editor (when you click on an
> event or a calendar you can edit it).
> The code seems to work, since it sometimes works and sometimes
> doesn't. The bug only occurs with the textInput forms for me.
>
> Here is how it is built :
>
> The editor holds different forms, one for each editable field (text
> or selection). I submit all forms when the user hits the save button.
> To do that I added this method:
>
> SUAjax>>triggerForms: aCollection
> aCollection do: [ :elt | self triggerForm: elt]
>
> the collection holds all the ids of the forms. it is called like this :
>
> ICWebEditor>>renderSaveButtonOn: html
> "formIDs is an instance variable holding a collection of all the ids
> used in this editor"
>
> html form id: html nextId;
>         with: [html submitButton callback: [self save];
>                         onClick: (html update id: html nextId;
>                                          triggerForms: formIDs;
>                                          callback: [:r|r]);
>                          text: 'save'
>                 ]
>
> IIRC this used to work fine a few weeks back. Was there some change
> in Seaside or Scriptaculous that could have broken this ?

There was the collection tag hierarchy refactoring. This broke one of
our applications. I don't know if this is related to your problem.

Cheers
Philippe

> Thanks,
>
> Yann
>
> PS: you can load the code from Squeak Source : http://
> www.squeaksource.com/iCalSummerTalk.html
>         You need to load first LogIn then SummerTime
>         You'll need to have a working seaside install with seaside 2.7
> scriptaculous.
>         You'll need iCal http://www.squeaksource.com/ical.html
>
> _______________________________________________
> Seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: random bug with textInput

Yann Monclair-2

On 5 déc. 06, at 17:42, Philippe Marschall wrote:

> 2006/12/5, Yann Monclair <[hidden email]>:
>> Hi everyone,
>>
>> I am working on my web iCalendar for the SummerTalk. a Demo is
>> running at http://icalendar.seasidehosting.st/seaside/icalendar
>> I am experiencing a random bug with the editor (when you click on an
>> event or a calendar you can edit it).
>> The code seems to work, since it sometimes works and sometimes
>> doesn't. The bug only occurs with the textInput forms for me.
>>
>> Here is how it is built :
>>
>> The editor holds different forms, one for each editable field (text
>> or selection). I submit all forms when the user hits the save button.
>> To do that I added this method:
>>
>> SUAjax>>triggerForms: aCollection
>> aCollection do: [ :elt | self triggerForm: elt]
>>
>> the collection holds all the ids of the forms. it is called like  
>> this :
>>
>> ICWebEditor>>renderSaveButtonOn: html
>> "formIDs is an instance variable holding a collection of all the ids
>> used in this editor"
>>
>> html form id: html nextId;
>>         with: [html submitButton callback: [self save];
>>                         onClick: (html update id: html nextId;
>>                                          triggerForms: formIDs;
>>                                          callback: [:r|r]);
>>                          text: 'save'
>>                 ]
>>
>> IIRC this used to work fine a few weeks back. Was there some change
>> in Seaside or Scriptaculous that could have broken this ?
>
> There was the collection tag hierarchy refactoring. This broke one of
> our applications. I don't know if this is related to your problem.
>

It doesn't look like this is the reason of the problem. i'll take  
more time to look into the depths of Seaside ;) and the textInput tag

Thanks,

Yann

> Cheers
> Philippe
>
>> Thanks,
>>
>> Yann
>>
>> PS: you can load the code from Squeak Source : http://
>> www.squeaksource.com/iCalSummerTalk.html
>>         You need to load first LogIn then SummerTime
>>         You'll need to have a working seaside install with seaside  
>> 2.7
>> scriptaculous.
>>         You'll need iCal http://www.squeaksource.com/ical.html
>>
>> _______________________________________________
>> Seaside mailing list
>> [hidden email]
>> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>>
> _______________________________________________
> Seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside

_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside