OpenLayers OpenLayers

Ticket #1536 (closed feature: fixed)

Opened 4 months ago

Last modified 3 months ago

Layer.Vector should destroy itself before its parent

Reported by: elemoine Assigned to: elemoine
Priority: minor Milestone: 2.7 Release
Component: general Version: 2.6
Keywords: Cc:
State: Complete

Description

Currently, Layer.Vector.destroy() calls its parent's destroy() method first and then does the rest of the cleanup. It should do the opposite - do its own cleanup and then calls its parent's destroy() method, in case the operations for its own cleanup relies on its parent's properties not being null. I have a patch attached to ticket #1535 that depends on that behavior.

Attachments

patch-1536-r7059-A0.diff (0.7 kB) - added by elemoine on 05/03/08 07:31:43.

Change History

05/03/08 07:31:43 changed by elemoine

  • attachment patch-1536-r7059-A0.diff added.

05/03/08 07:33:05 changed by elemoine

Unit tests pass on FF2. Please review.

05/26/08 02:53:50 changed by fredj

  • state set to Commit.

Tests pass on FF3, opera, IE6 & 7 and Safari Windows. Please commit.

05/27/08 03:53:48 changed by elemoine

  • status changed from new to closed.
  • state changed from Commit to Complete.
  • resolution set to fixed.

(In [7276]) Layer.Vector should destroy itself before its parent, r=fredj (closes #1536)