Merge pull request #1149 from twpayne/bing-maps-errors
Improve handling of error responses from Bing
This commit is contained in:
@@ -54,17 +54,18 @@ goog.inherits(ol.source.BingMaps, ol.source.TileImage);
|
|||||||
ol.source.BingMaps.prototype.handleImageryMetadataResponse =
|
ol.source.BingMaps.prototype.handleImageryMetadataResponse =
|
||||||
function(response) {
|
function(response) {
|
||||||
|
|
||||||
goog.asserts.assert(
|
if (response.statusCode != 200 ||
|
||||||
response.authenticationResultCode == 'ValidCredentials');
|
response.statusDescription != 'OK' ||
|
||||||
goog.asserts.assert(response.statusCode == 200);
|
response.authenticationResultCode != 'ValidCredentials' ||
|
||||||
goog.asserts.assert(response.statusDescription == 'OK');
|
response.resourceSets.length != 1 ||
|
||||||
|
response.resourceSets[0].length != 1) {
|
||||||
|
this.setState(ol.source.State.ERROR);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
var brandLogoUri = response.brandLogoUri;
|
var brandLogoUri = response.brandLogoUri;
|
||||||
//var copyright = response.copyright; // FIXME do we need to display this?
|
//var copyright = response.copyright; // FIXME do we need to display this?
|
||||||
goog.asserts.assert(response.resourceSets.length == 1);
|
var resource = response.resourceSets[0].resources[0];
|
||||||
var resourceSet = response.resourceSets[0];
|
|
||||||
goog.asserts.assert(resourceSet.resources.length == 1);
|
|
||||||
var resource = resourceSet.resources[0];
|
|
||||||
|
|
||||||
var tileGrid = new ol.tilegrid.XYZ({
|
var tileGrid = new ol.tilegrid.XYZ({
|
||||||
minZoom: resource.zoomMin,
|
minZoom: resource.zoomMin,
|
||||||
|
|||||||
Reference in New Issue
Block a user