Skip to content

Commit 8cc0cdc

Browse files
author
Bart van den Eijnden
committed
incorporate suggestions by @ahocevar and @elemoine
1 parent a4df9a8 commit 8cc0cdc

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

lib/OpenLayers/Layer.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -86,13 +86,13 @@ OpenLayers.Layer = OpenLayers.Class({
8686
* element - {DOMElement} A reference to layer.events.element.
8787
*
8888
* Supported map event types:
89-
* loadstart - Triggered when layer loading starts. When using Fixed or
90-
* BBOX strategies, the event object includes a *filter* property
91-
* holding the OpenLayers.Filter used when calling read on the
92-
* protocol.
93-
* loadend - Triggered when layer loading ends. When using Fixed or BBOX
94-
* strategies, the event object includes a *response* property holding
95-
* an OpenLayers.Protocol.Response object.
89+
* loadstart - Triggered when layer loading starts. When using a Vector
90+
* layer with a Fixed or BBOX strategy, the event object includes
91+
* a *filter* property holding the OpenLayers.Filter used when
92+
* calling read on the protocol.
93+
* loadend - Triggered when layer loading ends. When using a Vector layer
94+
* with a Fixed or BBOX strategy, the event object includes a
95+
* *response* property holding an OpenLayers.Protocol.Response object.
9696
* visibilitychanged - Triggered when layer visibility is changed.
9797
* move - Triggered when layer moves (triggered with every mousemove
9898
* during a drag).

lib/OpenLayers/Strategy/BBOX.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -217,11 +217,11 @@ OpenLayers.Strategy.BBOX = OpenLayers.Class(OpenLayers.Strategy, {
217217
this.layer.protocol.abort(this.response);
218218
this.layer.events.triggerEvent("loadend");
219219
}
220-
var filter = this.createFilter();
221-
this.layer.events.triggerEvent("loadstart", {filter: filter});
220+
var evt = {filter: this.createFilter()};
221+
this.layer.events.triggerEvent("loadstart", evt);
222222
this.response = this.layer.protocol.read(
223223
OpenLayers.Util.applyDefaults({
224-
filter: filter,
224+
filter: evt.filter,
225225
callback: this.merge,
226226
scope: this
227227
}, options));

0 commit comments

Comments
 (0)