diff --git a/src/ol/layer/TileLayer.js b/src/ol/layer/TileLayer.js index cbf98ec5fd..fb34627a21 100644 --- a/src/ol/layer/TileLayer.js +++ b/src/ol/layer/TileLayer.js @@ -1,5 +1,6 @@ goog.provide('ol.layer.TileLayer'); +goog.require('ol.error'); goog.require('ol.layer.Layer'); goog.require('ol.Tile'); goog.require('ol.TileCache'); @@ -182,7 +183,7 @@ ol.layer.TileLayer.prototype.getTileOrigin = function() { goog.isDef(this.tileOriginY_)) { return [this.tileOriginX_, this.tileOriginY_]; } - var errmsg; + var errmsg = 'Cannot calculate tile origin; '; if (goog.isDef(this.tileOriginCorner_)) { var extent = this.getExtent(); if (!goog.isNull(extent)) { @@ -205,16 +206,16 @@ ol.layer.TileLayer.prototype.getTileOrigin = function() { tileOriginY = extent.getMinY(); break; default: - // FIXME user error - goog.asserts.assert(false); + errmsg += 'tileOriginCorner value is incorrect.'; + ol.error(errmsg); } return [tileOriginX, tileOriginY]; } - // FIXME user error - goog.asserts.assert(false); + errmsg += 'layer has no extent.'; + ol.error(errmsg); } - // FIXME user error - goog.asserts.assert(false); + errmsg += 'layer has no tileOriginCorner.'; + ol.error(errmsg); return null; }; diff --git a/test/spec/ol/layer/TileLayer.test.js b/test/spec/ol/layer/TileLayer.test.js index dab1be98eb..4c21263eda 100644 --- a/test/spec/ol/layer/TileLayer.test.js +++ b/test/spec/ol/layer/TileLayer.test.js @@ -97,7 +97,7 @@ describe('ol.layer.TileLayer', function() { it('throws an error or return null', function() { var origin; - if (goog.DEBUG) { + if (ol.error.VERBOSE_ERRORS) { expect(function() { origin = layer.getTileOrigin(); }).toThrow(); @@ -118,7 +118,7 @@ describe('ol.layer.TileLayer', function() { }); it('returns the expected origin', function() { - if (goog.DEBUG) { + if (ol.error.VERBOSE_ERRORS) { expect(function() { var origin = layer.getTileOrigin(); }).toThrow();