From 1685913a55a25cbd5afb550c934e7887059592e7 Mon Sep 17 00:00:00 2001 From: mike-000 <49240900+mike-000@users.noreply.github.com> Date: Wed, 17 Jun 2020 16:46:24 +0100 Subject: [PATCH] getExtent() return getEmpty() for invalid extents --- src/ol/geom/Geometry.js | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/src/ol/geom/Geometry.js b/src/ol/geom/Geometry.js index 2c1f4dff33..16e1a48a18 100644 --- a/src/ol/geom/Geometry.js +++ b/src/ol/geom/Geometry.js @@ -8,7 +8,12 @@ import { compose as composeTransform, create as createTransform, } from '../transform.js'; -import {createEmpty, getHeight, returnOrUpdate} from '../extent.js'; +import { + containsExtent, + createEmpty, + getHeight, + returnOrUpdate, +} from '../extent.js'; import {get as getProjection, getTransform} from '../proj.js'; import {memoizeOne} from '../functions.js'; import {transform2D} from './flat/transform.js'; @@ -170,6 +175,11 @@ class Geometry extends BaseObject { getExtent(opt_extent) { if (this.extentRevision_ != this.getRevision()) { this.extent_ = this.computeExtent(this.extent_); + if ( + !containsExtent([-Infinity, -Infinity, Infinity, Infinity], this.extent_) + ) { + this.extent_ = createEmpty(); + } this.extentRevision_ = this.getRevision(); } return returnOrUpdate(this.extent_, opt_extent);