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

Consistency with tile loading events.

Browse Source 
Since loadstart is being fired here, it makes sense to fire loadend as well.  The isLoading flag also needs to be cleared.  Needs tests still.
This commit is contained in:
Tim Schaub
2012-02-25 20:45:18 -07:00
parent c18f6a23be
commit 12fa9b54ef
1 changed files with 13 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
lib/OpenLayers/Tile/UTFGrid.js
View File

@@ -95,6 +95,8 @@ OpenLayers.Tile.UTFGrid = OpenLayers.Class(OpenLayers.Tile, {
var ols = new OpenLayers.Protocol.Script({
url: this.url,
callback: function(response) {
this.isLoading = false;
this.events.triggerEvent("loadend");
this.json = response.data;
},
scope: this
@@ -104,7 +106,13 @@ OpenLayers.Tile.UTFGrid = OpenLayers.Class(OpenLayers.Tile, {
// Use standard XHR
OpenLayers.Request.GET({
url: this.url,
callback: this.parseData,
callback: function(response) {
this.isLoading = false;
this.events.triggerEvent("loadend");
if (response.status === 200) {
this.parseData(response);
}
},
scope: this
});
}
@@ -122,12 +130,10 @@ OpenLayers.Tile.UTFGrid = OpenLayers.Class(OpenLayers.Tile, {
* {Object} parsed javascript data
*/
parseData: function(req) {
if (req.status == 200) {
if (!this.format) {
this.format = new OpenLayers.Format.JSON();
}
this.json = this.format.read(req.responseText);
}
if (!this.format) {
this.format = new OpenLayers.Format.JSON();
}
this.json = this.format.read(req.responseText);
},
/**