OpenLayers OpenLayers

Ticket #742 (closed feature: fixed)

Opened 1 year ago

Last modified 1 year ago

Need getControl() / removeControl() functions

Reported by: euzuro Assigned to:
Priority: minor Milestone: 2.5 Release
Component: Control Version: SVN
Keywords: Cc:
State:

Description

We have add/get/remove functions for Layers on the map, but we only have add for Controls.

Attachments

controlGetRemove.patch (4.2 kB) - added by euzuro on 06/07/07 11:07:48.
patch which adds getControl() and removeControl() functionality. Includes tests.
controlGetRemove.2.patch (1.8 kB) - added by euzuro on 06/12/07 13:35:51.
take two. removeControl() now takes a control instead of an id

Change History

06/07/07 11:07:48 changed by euzuro

  • attachment controlGetRemove.patch added.

patch which adds getControl() and removeControl() functionality. Includes tests.

06/07/07 11:08:14 changed by euzuro

  • keywords set to review.

06/07/07 14:01:07 changed by sderle

  • keywords changed from review to commit.

seems reasonable to me. go ahead and commit.

06/07/07 14:02:19 changed by euzuro

fixed with r3282. thanks sde!

06/07/07 14:02:25 changed by euzuro

  • status changed from new to closed.
  • resolution set to fixed.

06/12/07 13:34:21 changed by euzuro

  • status changed from closed to reopened.
  • resolution deleted.

as mentioned by cr5, removeLayer() takes a layer, not an id. removeControl() should do the same.

06/12/07 13:35:51 changed by euzuro

  • attachment controlGetRemove.2.patch added.

take two. removeControl() now takes a control instead of an id

06/14/07 15:14:29 changed by crschmidt

go ahead and commit

06/14/07 15:19:56 changed by euzuro

  • status changed from reopened to closed.
  • resolution set to fixed.

fixed with r3329

07/05/07 16:53:30 changed by euzuro

  • keywords deleted.