drawing features while the layer is not in the map may cause problem, r=pgiraud (closes #2546)
git-svn-id: http://svn.openlayers.org/trunk/openlayers@10151 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf
This commit is contained in:
@@ -421,6 +421,7 @@ OpenLayers.Layer.Vector = OpenLayers.Class(OpenLayers.Layer, {
|
|||||||
* map - {<OpenLayers.Map>}
|
* map - {<OpenLayers.Map>}
|
||||||
*/
|
*/
|
||||||
removeMap: function(map) {
|
removeMap: function(map) {
|
||||||
|
this.drawn = false;
|
||||||
if(this.strategies) {
|
if(this.strategies) {
|
||||||
var strategy, i, len;
|
var strategy, i, len;
|
||||||
for(i=0, len=this.strategies.length; i<len; i++) {
|
for(i=0, len=this.strategies.length; i<len; i++) {
|
||||||
|
|||||||
@@ -601,6 +601,48 @@
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
function test_removeLayer_drawFeature(t) {
|
||||||
|
// test behaviour when features are redrawn while
|
||||||
|
// the layer has been removed from the map
|
||||||
|
|
||||||
|
t.plan(1);
|
||||||
|
|
||||||
|
// set up
|
||||||
|
|
||||||
|
var map, layer, feature;
|
||||||
|
|
||||||
|
map = new OpenLayers.Map("map");
|
||||||
|
map.addLayer(new OpenLayers.Layer("base", {isBaseLayer: true}));
|
||||||
|
|
||||||
|
layer = new OpenLayers.Layer.Vector("vector");
|
||||||
|
map.addLayer(layer);
|
||||||
|
|
||||||
|
map.zoomToMaxExtent();
|
||||||
|
|
||||||
|
feature = new OpenLayers.Feature.Vector(
|
||||||
|
new OpenLayers.Geometry.Point(1.0, 1.0));
|
||||||
|
layer.addFeatures(feature);
|
||||||
|
|
||||||
|
// test
|
||||||
|
|
||||||
|
map.removeLayer(layer);
|
||||||
|
layer.drawFeature(feature);
|
||||||
|
layer.drawFeature(feature);
|
||||||
|
map.addLayer(layer);
|
||||||
|
|
||||||
|
var count = 0, node;
|
||||||
|
while(node = document.getElementById(feature.geometry.id)) {
|
||||||
|
node.parentNode.removeChild(node);
|
||||||
|
count++;
|
||||||
|
}
|
||||||
|
|
||||||
|
t.eq(count, 1, "one geometry added, one geometry removed");
|
||||||
|
|
||||||
|
// tear down
|
||||||
|
|
||||||
|
map.destroy();
|
||||||
|
}
|
||||||
|
|
||||||
</script>
|
</script>
|
||||||
</head>
|
</head>
|
||||||
<body>
|
<body>
|
||||||
|
|||||||
Reference in New Issue
Block a user