[OpenLayers-Dev] DeleteFeature control

[Eric Lemoine]

On Tuesday, July 21, 2009, Alexandre Dube <adube at mapgears.com> wrote:
> Oh and by the way, here's what I did for the Control (attached).  Is it ok (too much, i.e. about the use of hasOwnProperty functions) ?

why not using the regular

var i, len, control;
for(i = 0, len = controls.length; i < len; i++) {
    control = controls[i];
}

?

(the OpenLayers way, which is safer than using for in I guess)
>
> Thanks,
>
> Alexandre
>
> Alexandre Dube wrote:
>
> Eric Lemoine wrote:
>
>
> so, to get to the delete control you've been working on, I see more
> value, in terms of flexibility, in adding "control composition" to the
> control base class.
>
>
>
>
>
> I also agree.  Originally, the control had a Feature handler instead of a SelectFeature control.  Looking at it now, it's too simple to be justified as a new control.
>
>
>
> About the 'controls' property and the act./dea. automatism : that's a good idea, but wouldn't it be good to have a 'handlers' property also and activate/deactivate all handlers as well ?  Currently, control has a 'handler' property only.  Would it make sense to have 'handlers' added a do the same kind of automatism as the new 'controls' property ?
>
>
> Agreed.
>
>
>
>
> Great.
>
>
>
>
>
>
> --
> Alexandre Dubé
> Mapgears
> www.mapgears.com
>
>


-- 
Eric Lemoine

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac, Cedex

Tel : 00 33 4 79 44 44 96
Mail : eric.lemoine at camptocamp.com
http://www.camptocamp.com