Ticket #34 (closed enhancement: fixed)

Opened 1 year ago

Last modified 1 year ago

feature and layer records should clone on copy

Reported by: tschaub Assigned to:
Priority: major Milestone: 0.5
Component: GeoExt.data Version:
Keywords: Cc:
State: Commit

Description

The FeatureRecord and LayerRecord prototypes need a custom copy so they clone their feature and layer values.

Attachments

copy.patch (3.9 kB) - added by tschaub on 03/27/09 20:56:23.
clone ol objects on record copy

Change History

03/27/09 20:56:23 changed by tschaub

  • attachment copy.patch added.

clone ol objects on record copy

03/27/09 20:57:11 changed by tschaub

  • state changed from None to Review.

Tests pass in FF3.

04/07/09 19:07:05 changed by ahocevar

  • state changed from Review to Commit.

This looks good. And I always admire the beauty of tschaub's unit tests. Not just in GeoExt, but also in OpenLayers. Please commit.

04/07/09 22:11:22 changed by tschaub

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

(In [357]) Records that contain references to OL objects should clone those objects on copy. Done here for the FeatureRecord and LayerRecord. r=ahocevar (closes #34)