ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Improved extension example docs and destroy method.

Browse Source
This commit is contained in:
ahocevar
2012-01-22 21:19:00 +01:00
parent ed88941280
commit 5a1378cdab
2 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/OpenLayers/Events.js
+5 -3
View File
@@ -486,10 +486,12 @@ OpenLayers.Events = OpenLayers.Class({
* this.target.extensions["fooend"] = true; * this.target.extensions["fooend"] = true;
* }, * },
* destroy: function() { * destroy: function() {
* this.target.unregister("click", this, this.doStuff); * var target = this.target;
* target.unregister("click", this, this.doStuff);
* delete this.target;
* // only required if extension provides more than one event type * // only required if extension provides more than one event type
* delete this.target.extensions["foostart"]; * delete target.extensions["foostart"];
* delete this.target.extensions["fooend"]; * delete target.extensions["fooend"];
* }, * },
* doStuff: function(evt) { * doStuff: function(evt) {
* var propagate = true; * var propagate = true;
lib/OpenLayers/Events/buttonclick.js
+1
View File
@@ -94,6 +94,7 @@ OpenLayers.Events.buttonclick = OpenLayers.Class({
for (var i=this.events.length-1; i>=0; --i) { for (var i=this.events.length-1; i>=0; --i) {
this.target.unregister(this.events[i], this, this.buttonClick); this.target.unregister(this.events[i], this, this.buttonClick);
} }
delete this.target;
}, },
/** /**