[OpenLayers-Dev] [OpenLayers-Trac] [OpenLayers] #1324: OpenLayers.Feature.Vector.clone() must clone the fid property.

[Tim Schaub]

Hey-

For what it's worth, I agree with Chris on not cloning the fid.  The fid 
is our key to the "real" feature.  Cloning with the fid would be 
equivalent to just creating another reference to the same feature - 
which is not what clone is about.

As mentioned below, I am interested to hear the use case where you'd 
want a clone with the same fid.

Tim

Christopher Schmidt wrote:
> On Mon, Feb 11, 2008 at 05:51:01PM -0500, Tim Schaub wrote:
>> Christopher Schmidt wrote:
>>> On Thu, Feb 07, 2008 at 03:18:55PM -0000, OpenLayers wrote:
>>>> #1324: OpenLayers.Feature.Vector.clone() must clone the fid property.
>>> Anyone have thoughts on this? I'm not sure if we should be cloning this,
>>> since it feels to me like a clone is a different object, and fids should
>>> be unique...
>>>
>>> Regards,
>> Yeah, I agree.  Interested to hear the use case for it.
> 
> You replied just to me...
>