Ticket #2502 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

SelectFeature: When control destroy the layers disappear.

Reported by: jorix Owned by: tschaub
Priority: minor Milestone: 2.9 Release
Component: Control.SelectFeature Version: SVN
Keywords: Cc:
State: Complete

Description

When control destroy the layers disappear if layers parameter constructor was array.

This occurs because the root container, when layers are array parameter, not are moved correctly when destroy.

The attached patch fixes this and extensive testing.

Test work ok in IE8, FF35 and CH4.

SVN Revision Number: 10094

Attachments

ControlSelectFeature-2502.patch Download (1.4 KB) - added by jorix 3 years ago.
Patch to fix the bug

Change History

Changed 3 years ago by jorix

Patch to fix the bug

  Changed 3 years ago by jorix

  • keywords review added
  • state set to Review

please review this ticket, and the others who have opened today.

follow-up: ↓ 3   Changed 3 years ago by ahocevar

  • state changed from Review to Awaiting User Feedback

This looks good. @jorix: Do we have a CLA from you? If not, please follow the steps explained in HowToContribute to submit a CLA. Please set ticket state to Review again when done.

in reply to: ↑ 2   Changed 3 years ago by jorix

  • state changed from Awaiting User Feedback to Review

Replying to ahocevar:

This looks good. @jorix: Do we have a CLA from you? If not, please follow the steps explained in HowToContribute to submit a CLA. Please set ticket state to Review again when done.

Yes, I sent the my ICLA on January 18, 2010 to psc_at_openlayers.org.

  Changed 3 years ago by bartvde

  • milestone changed from 2.10 Release to 2.9 Release

  Changed 3 years ago by bartvde

  • keywords review removed
  • status changed from new to closed
  • state changed from Review to Complete
  • resolution set to fixed

(In [10115]) when select feature control is destroyed and layers parameter was an array, the root container is not moved correctly, p=jorix, r=ahocevar,me (closes #2502)

Note: See TracTickets for help on using tickets.