Remove unused hasFeatureAtCoordinate from ol/renderer/Layer

This commit is contained in:
Frederic Junod
2019-06-28 10:58:21 +02:00
committed by Olivier Guyot
parent 3bca9b5297
commit 933a6297bb
3 changed files with 2 additions and 21 deletions

View File

@@ -132,15 +132,6 @@ class LayerRenderer extends Observable {
}
}
/**
* @param {import("../coordinate.js").Coordinate} coordinate Coordinate.
* @param {import("../PluggableMap.js").FrameState} frameState Frame state.
* @return {boolean} Is there a feature at the given coordinate?
*/
hasFeatureAtCoordinate(coordinate, frameState) {
return false;
}
/**
* Load the image if not already loaded, and register the image change
* listener if needed.

View File

@@ -3,7 +3,6 @@
*/
import LayerRenderer from '../Layer.js';
import WebGLHelper from '../../webgl/Helper.js';
import {TRUE} from '../../functions.js';
/**
@@ -83,13 +82,6 @@ class WebGLLayerRenderer extends LayerRenderer {
return this.helper.getShaderCompileErrors();
}
/**
* @inheritDoc
*/
hasFeatureAtCoordinate(coordinate, frameState) {
const feature = this.forEachFeatureAtCoordinate(coordinate, frameState, 0, TRUE, null);
return feature !== undefined;
}
}

View File

@@ -147,7 +147,7 @@ describe('ol.renderer.webgl.PointsLayer', function() {
});
});
describe('#forEachFeatureAtCoordinate and #hasFeatureAtCoordinate', function() {
describe('#forEachFeatureAtCoordinate', function() {
let layer, renderer, feature;
beforeEach(function() {
@@ -181,7 +181,7 @@ describe('ol.renderer.webgl.PointsLayer', function() {
viewHints: [],
coordinateToPixelTransform: transform
};
let found, hit;
let found;
const cb = function(feature) {
found = feature;
};
@@ -196,9 +196,7 @@ describe('ol.renderer.webgl.PointsLayer', function() {
function checkHit(x, y, expected) {
found = null;
renderer.forEachFeatureAtCoordinate([x, y], frameState, 0, cb, null);
hit = renderer.hasFeatureAtCoordinate([x, y], frameState);
expect(found).to.be(expected ? feature : null);
expect(hit).to.eql(expected);
}
checkHit(0, 0, true);