diff --git a/src/ol/interaction/Snap.js b/src/ol/interaction/Snap.js index 8ed63e2cee..8ed90efe25 100644 --- a/src/ol/interaction/Snap.js +++ b/src/ol/interaction/Snap.js @@ -469,9 +469,7 @@ class Snap extends PointerInteraction { if (this.vertex_) { for (let i = 0; i < segmentsLength; ++i) { const segmentData = segments[i]; - if ( - segmentData.feature.getGeometry().getType() !== GeometryType.CIRCLE - ) { + if (segmentData.feature.getGeometry().getType() !== 'Circle') { segmentData.segment.forEach((vertex) => { const tempVertexCoord = fromUserCoordinate(vertex, projection); const delta = squaredDistance(projectedCoordinate, tempVertexCoord); @@ -492,9 +490,7 @@ class Snap extends PointerInteraction { for (let i = 0; i < segmentsLength; ++i) { let vertex = null; const segmentData = segments[i]; - if ( - segmentData.feature.getGeometry().getType() === GeometryType.CIRCLE - ) { + if (segmentData.feature.getGeometry().getType() === 'Circle') { let circleGeometry = segmentData.feature.getGeometry(); const userProjection = getUserProjection(); if (userProjection) { diff --git a/src/ol/layer/Layer.js b/src/ol/layer/Layer.js index 1eb163f4a3..d481d90c6a 100644 --- a/src/ol/layer/Layer.js +++ b/src/ol/layer/Layer.js @@ -208,7 +208,7 @@ class Layer extends BaseLayer { } /** - * @return {import("../source/State.js").default} Source state. + * @return {import("../source/Source.js").State} Source state. */ getSourceState() { const source = this.getSource(); diff --git a/src/ol/layer/WebGLTile.js b/src/ol/layer/WebGLTile.js index 65c3038337..1dcb919e84 100644 --- a/src/ol/layer/WebGLTile.js +++ b/src/ol/layer/WebGLTile.js @@ -3,7 +3,6 @@ */ import BaseTileLayer from './BaseTile.js'; import LayerProperty from '../layer/Property.js'; -import SourceState from '../source/State.js'; import WebGLTileLayerRenderer, { Attributes, Uniforms, @@ -383,11 +382,11 @@ class WebGLTileLayer extends BaseTileLayer { } /** - * @return {import("../source/State.js").default} Source state. + * @return {import("../source/Source.js").State} Source state. */ getSourceState() { const source = this.getRenderSource(); - return source ? source.getState() : SourceState.UNDEFINED; + return source ? source.getState() : 'undefined'; } /** @@ -454,16 +453,16 @@ class WebGLTileLayer extends BaseTileLayer { for (let i = 0, ii = sources.length; i < ii; ++i) { const source = sources[i]; const sourceState = source.getState(); - if (sourceState == SourceState.LOADING) { + if (sourceState == 'loading') { const onChange = () => { - if (source.getState() == SourceState.READY) { + if (source.getState() == 'ready') { source.removeEventListener('change', onChange); this.changed(); } }; source.addEventListener('change', onChange); } - ready = ready && sourceState == SourceState.READY; + ready = ready && sourceState == 'ready'; } const canvas = this.renderSources(frameState, sources); if (this.getRenderer().renderComplete && ready) { diff --git a/src/ol/render/Feature.js b/src/ol/render/Feature.js index 6d483ef65c..9a6f9d2e7c 100644 --- a/src/ol/render/Feature.js +++ b/src/ol/render/Feature.js @@ -347,25 +347,25 @@ RenderFeature.prototype.getFlatCoordinates = export function toGeometry(renderFeature) { const geometryType = renderFeature.getType(); switch (geometryType) { - case GeometryType.POINT: + case 'Point': return new Point(renderFeature.getFlatCoordinates()); - case GeometryType.MULTI_POINT: + case 'MultiPoint': return new MultiPoint( renderFeature.getFlatCoordinates(), GeometryLayout.XY ); - case GeometryType.LINE_STRING: + case 'LineString': return new LineString( renderFeature.getFlatCoordinates(), GeometryLayout.XY ); - case GeometryType.MULTI_LINE_STRING: + case 'MultiLineString': return new MultiLineString( renderFeature.getFlatCoordinates(), GeometryLayout.XY, /** @type {Array} */ (renderFeature.getEnds()) ); - case GeometryType.POLYGON: + case 'Polygon': const flatCoordinates = renderFeature.getFlatCoordinates(); const ends = /** @type {Array} */ (renderFeature.getEnds()); const endss = inflateEnds(flatCoordinates, ends); diff --git a/src/ol/renderer/Composite.js b/src/ol/renderer/Composite.js index bcb6df4d4c..3ef0b77a16 100644 --- a/src/ol/renderer/Composite.js +++ b/src/ol/renderer/Composite.js @@ -114,8 +114,7 @@ class CompositeMapRenderer extends MapRenderer { const sourceState = layer.getSourceState(); if ( !inView(layerState, viewState) || - (layerState.sourceState != 'ready' && - layerState.sourceState != 'undefined') + (sourceState != 'ready' && sourceState != 'undefined') ) { layer.unrender(); continue; diff --git a/test/browser/spec/ol/interaction/draw.test.js b/test/browser/spec/ol/interaction/draw.test.js index 3028b72555..5b80d01246 100644 --- a/test/browser/spec/ol/interaction/draw.test.js +++ b/test/browser/spec/ol/interaction/draw.test.js @@ -1968,14 +1968,14 @@ describe('ol.interaction.Draw', function () { } function drawPoint(geometryLayout) { - createDrawInteraction(GeometryType.POINT, geometryLayout); + createDrawInteraction('Point', geometryLayout); simulateEvent('pointermove', 10, 20); simulateEvent('pointerdown', 10, 20); simulateEvent('pointerup', 10, 20); } function drawLineString(geometryLayout) { - createDrawInteraction(GeometryType.LINE_STRING, geometryLayout); + createDrawInteraction('LineString', geometryLayout); // first point simulateEvent('pointermove', 10, 20); simulateEvent('pointerdown', 10, 20); @@ -1992,7 +1992,7 @@ describe('ol.interaction.Draw', function () { } function drawPolygon(geometryLayout) { - createDrawInteraction(GeometryType.POLYGON, geometryLayout); + createDrawInteraction('Polygon', geometryLayout); // first point simulateEvent('pointermove', 10, 20); simulateEvent('pointerdown', 10, 20); @@ -2014,7 +2014,7 @@ describe('ol.interaction.Draw', function () { } function drawCircle(geometryLayout) { - createDrawInteraction(GeometryType.CIRCLE, geometryLayout); + createDrawInteraction('Circle', geometryLayout); // first point simulateEvent('pointermove', 10, 20); simulateEvent('pointerdown', 10, 20); diff --git a/test/node/ol/render/Feature.test.js b/test/node/ol/render/Feature.test.js index 3c432e6215..367c15c222 100644 --- a/test/node/ol/render/Feature.test.js +++ b/test/node/ol/render/Feature.test.js @@ -1,4 +1,3 @@ -import GeometryType from '../../../../src/ol/geom/GeometryType.js'; import RenderFeature, { toFeature, toGeometry, @@ -141,7 +140,7 @@ describe('ol/render/Feature', function () { ], ]); const renderFeature = new RenderFeature( - GeometryType.POLYGON, + 'Polygon', geometry.getFlatCoordinates().slice(), geometry.getEndss().flat(1) );