SUForm>>inputs - Enumerator & Optional Arguments

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

SUForm>>inputs - Enumerator & Optional Arguments

Ken Treis
I was a little surprised to see that SUForm>>inputs doesn't return an  
enumerator, like SUElement>>childElements does. SUForm>>elements  
doesn't either. Do I need to specify the enumerable explicitly?

        (SUForm new
                id: domId;
                enumerator) do: ...

Is there a convention as to whether an enumerator is returned or not?

--
Ken Treis
Miriam Technologies, Inc.

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

Re: SUForm>>inputs - Enumerator & Optional Arguments

Lukas Renggli
> I was a little surprised to see that SUForm>>inputs doesn't return an
> enumerator, like SUElement>>childElements does. SUForm>>elements
> doesn't either. Do I need to specify the enumerable explicitly?
>
>         (SUForm new
>                 id: domId;
>                 enumerator) do: ...
>
> Is there a convention as to whether an enumerator is returned or not?

This is certainly a bug. I simply forgot to refactor its subclasses
after having introduced the enumerables in SUElement.

Fixed and tested in:

Name: Scriptaculous-lr.217
Author: lr
Time: 12 September 2007, 7:40:45 am
UUID: d316f3e9-ee58-4ffc-a07e-e0f9b41444c6
Ancestors: Scriptaculous-lr.216

- SUForm>>#inputs and SUForm>>#elements return an enumerator object
(thanks to Ken Treis)

Lukas

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

Re: SUForm>>inputs - Enumerator & Optional Arguments

Ken Treis
On Sep 11, 2007, at 10:41 PM, Lukas Renggli wrote:

> This is certainly a bug. I simply forgot to refactor its subclasses
> after having introduced the enumerables in SUElement.
>
> Fixed and tested in:
>
> Name: Scriptaculous-lr.217

Great, thanks Lukas!

--
Ken Treis
Miriam Technologies, Inc.

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