#WAAbstractHtmlBuilder #list:do: Bug

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

#WAAbstractHtmlBuilder #list:do: Bug

Chiara-2
Hi, I'm new in seaside, I try to use the #list:do method but I figure
out that it has a bug.
The original code is:

#WAAbstractHtmlBuilder>>list: aCollection do: aBlock
        self unorderedList:
                [aCollection do:
                        [:item |
                        self listItem: [aBlock value: item]]]

I know that #listItem: expect a String but in the previos code, it is
receiving a Block (self listItem: [aBlock value: item]).
I change this method with:

#WAAbstractHtmlBuilder>>list: aCollection do: aBlock
        self unorderedList:
                [aCollection do:
                        [:item |
                        self listItem: (aBlock value: item)]]

and seems to work ok.
It is that ok? or I  just misunderstood the method? I realize of this
when a use it in:

html list: self persons do:[:person | person name].


--
Saludos Chiara

"Peace cannot be kept by force; it can only be achieved by understanding."
Albert Einstein
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: #WAAbstractHtmlBuilder #list:do: Bug

Damien Cassou-3
Chiara wrote:
> Hi, I'm new in seaside,

Hi Chiara,

welcome


> I try to use the #list:do method but I figure
> out that it has a bug.


I don't think so, but I might be wrong.


> The original code is:
>
> #WAAbstractHtmlBuilder>>list: aCollection do: aBlock
>     self unorderedList:
>         [aCollection do:
>             [:item |
>             self listItem: [aBlock value: item]]]
>
> I know that #listItem: expect a String


How do you know this ? I think listItem: expects a block.


> but in the previos code, it is
> receiving a Block (self listItem: [aBlock value: item]).
> I change this method with:
>
> #WAAbstractHtmlBuilder>>list: aCollection do: aBlock
>     self unorderedList:
>         [aCollection do:
>             [:item |
>             self listItem: (aBlock value: item)]]
>
> and seems to work ok.



You have broke a lot of code :-)


> It is that ok? or I  just misunderstood the method? I realize of this
> when a use it in:
>
> html list: self persons do:[:person | person name].


I think this code is wrong, you should have wrote:

html list: self persons do: [:person | html text: person name]

This is because #list:do: expects a block that will render something.
You could use also:

html list: (self persons collect: [:person | person name]). If you have
a look at how #list is implemented, you will see that it sends #text:
for each element.


Did you heard about the new Canvas API ? If you are new in Seaside, you
might to try it because it is the future. Just ask if you want to know
more about this.


Hope I'm not wrong

Bye
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: #WAAbstractHtmlBuilder #list:do: Bug

Chiara-2
> Hi Chiara,
>
> welcome

thanks

> > I know that #listItem: expect a String
>
> How do you know this ? I think listItem: expects a block.

actually, I suposse this, je my mistake :)

> You have broke a lot of code :-)

jeje, yes

> I think this code is wrong, you should have wrote:
>
> html list: self persons do: [:person | html text: person name]

OK thanks

> This is because #list:do: expects a block that will render something.
> You could use also:

:-D


> Did you heard about the new Canvas API ? If you are new in Seaside, you
> might to try it because it is the future. Just ask if you want to know
> more about this.

:-D ++ I'll want to know more about it.

> Hope I'm not wrong

I hope too

> Bye

--
Saludos Chiara

"Peace cannot be kept by force; it can only be achieved by understanding."
Albert Einstein
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: #WAAbstractHtmlBuilder #list:do: Bug

Damien Cassou-3
Chiara wrote:
>> Did you heard about the new Canvas API ? If you are new in Seaside, you
>> might to try it because it is the future. Just ask if you want to know
>> more about this.
>
> :-D ++ I'll want to know more about it.

Avi Bryant wrote this about the Canvas API:
http://lists.squeakfoundation.org/pipermail/seaside/2005-June/005260.html


Feel free to ask if you have questions
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

WABrowser for Seaside for Dolphin

BrunoBB
In reply to this post by Chiara-2
Hi,

I have implemented a Browser to browse Dolphin image from a web page.
It's same functions as in VW and Squeak.

The only thing that is not implemented is creation of a class, but it
will be finished in a couple of days.

If someone is merging the Seaside for Dolphin i can send you this packages.

Regards Bruno

_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: WABrowser for Seaside for Dolphin

Esteban A. Maringolo
Bruno:

Thanks for your effort.

Please send it to me, I'll integrate it ASAP.

I'm contacting you by private mail.

--
Esteban.

On 7/24/06, Bruno BB (st) <[hidden email]> wrote:
> Hi,
>
> I have implemented a Browser to browse Dolphin image from a web page.
> It's same functions as in VW and Squeak.
>
> The only thing that is not implemented is creation of a class, but it
> will be finished in a couple of days.
>
> If someone is merging the Seaside for Dolphin i can send you this packages.


--
Esteban A. Maringolo
[hidden email]
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: WABrowser for Seaside for Dolphin

marcelo Cortez
In reply to this post by BrunoBB

Bruno

  Great work !!!
  best regards
  MDC
 --- "Bruno BB (st)" <[hidden email]>
escribió:

> Hi,
>
> I have implemented a Browser to browse Dolphin image
> from a web page.
> It's same functions as in VW and Squeak.
>
> The only thing that is not implemented is creation
> of a class, but it
> will be finished in a couple of days.
>
> If someone is merging the Seaside for Dolphin i can
> send you this packages.
>
> Regards Bruno
>
> _______________________________________________
> Seaside mailing list
> [hidden email]
>
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>



       
       
               
__________________________________________________
Preguntá. Respondé. Descubrí.
Todo lo que querías saber, y lo que ni imaginabas,
está en Yahoo! Respuestas (Beta).
¡Probalo ya!
http://www.yahoo.com.ar/respuestas

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