Pullup trunk for RC4.
Fixes: #694 Safari 1.3.2 doesn't work with OL 2.4 #695 GeoRSS serializer is broken #696 events need to fall through the overview map extent rectangle #697 Vector example to show how to use styles #698 add close box option to AnchoredBubble #701 SVG render does not always clear features when map extent changes #703 OpenLayers.Layer.Vector do not properly destroy its features #706 Full CSS support fails when Control.OverviewMap is loaded #708 change WKT format to deal in features instead of geometries #710 Install instructions unclear #711 OpenLayers.Layer.Image requires OpenLayers.Tile.Image #715 layer.js needs sanity check #718 WMS.Untiled Clone doesn't work #719 SVG renderer does not always redraw LineStrings and Polygons #720 remove console.log() from OpenLayers.Format.WKT git-svn-id: http://svn.openlayers.org/branches/openlayers/2.4@3177 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf
This commit is contained in:
@@ -97,8 +97,7 @@ OpenLayers.Layer.Vector.prototype =
|
||||
destroy: function() {
|
||||
OpenLayers.Layer.prototype.destroy.apply(this, arguments);
|
||||
|
||||
// HACK HACK -- I believe we should be iterating and
|
||||
// calling feature[i].destroy() here.
|
||||
this.destroyFeatures();
|
||||
this.features = null;
|
||||
this.selectedFeatures = null;
|
||||
if (this.renderer) {
|
||||
@@ -247,8 +246,10 @@ OpenLayers.Layer.Vector.prototype =
|
||||
var feature = features[i];
|
||||
this.features = OpenLayers.Util.removeItem(this.features, feature);
|
||||
|
||||
this.renderer.eraseGeometry(feature.geometry);
|
||||
|
||||
if (feature.geometry) {
|
||||
this.renderer.eraseGeometry(feature.geometry);
|
||||
}
|
||||
|
||||
//in the case that this feature is one of the selected features,
|
||||
// remove it from that array as well.
|
||||
if (OpenLayers.Util.indexOf(this.selectedFeatures, feature) != -1){
|
||||
|
||||
Reference in New Issue
Block a user