Fix type check errors in ol/interaction/Modify

This commit is contained in:
Kevin Schmidt
2018-10-02 08:31:03 -06:00
parent 023ad3c531
commit 25cf54193a
+9 -8
View File
@@ -63,7 +63,7 @@ const ModifyEventType = {
* @property {Array<number>} [depth] * @property {Array<number>} [depth]
* @property {Feature} feature * @property {Feature} feature
* @property {import("../geom/SimpleGeometry.js").default} geometry * @property {import("../geom/SimpleGeometry.js").default} geometry
* @property {number} index * @property {number} [index]
* @property {Array<import("../extent.js").Extent>} segment * @property {Array<import("../extent.js").Extent>} segment
* @property {Array<SegmentData>} [featureSegments] * @property {Array<SegmentData>} [featureSegments]
*/ */
@@ -375,7 +375,7 @@ class Modify extends PointerInteraction {
// Remove the vertex feature if the collection of canditate features // Remove the vertex feature if the collection of canditate features
// is empty. // is empty.
if (this.vertexFeature_ && this.features_.getLength() === 0) { if (this.vertexFeature_ && this.features_.getLength() === 0) {
this.overlay_.getSource().removeFeature(this.vertexFeature_); /** @type {VectorSource} */ (this.overlay_.getSource()).removeFeature(this.vertexFeature_);
this.vertexFeature_ = null; this.vertexFeature_ = null;
} }
unlisten(feature, EventType.CHANGE, unlisten(feature, EventType.CHANGE,
@@ -408,7 +408,7 @@ class Modify extends PointerInteraction {
*/ */
setActive(active) { setActive(active) {
if (this.vertexFeature_ && !active) { if (this.vertexFeature_ && !active) {
this.overlay_.getSource().removeFeature(this.vertexFeature_); /** @type {VectorSource} */ (this.overlay_.getSource()).removeFeature(this.vertexFeature_);
this.vertexFeature_ = null; this.vertexFeature_ = null;
} }
super.setActive(active); super.setActive(active);
@@ -659,7 +659,7 @@ class Modify extends PointerInteraction {
if (!vertexFeature) { if (!vertexFeature) {
vertexFeature = new Feature(new Point(coordinates)); vertexFeature = new Feature(new Point(coordinates));
this.vertexFeature_ = vertexFeature; this.vertexFeature_ = vertexFeature;
this.overlay_.getSource().addFeature(vertexFeature); /** @type {VectorSource} */ (this.overlay_.getSource()).addFeature(vertexFeature);
} else { } else {
const geometry = /** @type {Point} */ (vertexFeature.getGeometry()); const geometry = /** @type {Point} */ (vertexFeature.getGeometry());
geometry.setCoordinates(coordinates); geometry.setCoordinates(coordinates);
@@ -701,6 +701,7 @@ class Modify extends PointerInteraction {
const vertexPixel = map.getPixelFromCoordinate(vertex); const vertexPixel = map.getPixelFromCoordinate(vertex);
let dist = coordinateDistance(pixel, vertexPixel); let dist = coordinateDistance(pixel, vertexPixel);
if (dist <= this.pixelTolerance_) { if (dist <= this.pixelTolerance_) {
/** @type {Object<string, boolean>} */
const vertexSegments = {}; const vertexSegments = {};
if (node.geometry.getType() === GeometryType.CIRCLE && if (node.geometry.getType() === GeometryType.CIRCLE &&
@@ -738,7 +739,7 @@ class Modify extends PointerInteraction {
} }
} }
if (this.vertexFeature_) { if (this.vertexFeature_) {
this.overlay_.getSource().removeFeature(this.vertexFeature_); /** @type {VectorSource} */ (this.overlay_.getSource()).removeFeature(this.vertexFeature_);
this.vertexFeature_ = null; this.vertexFeature_ = null;
} }
} }
@@ -839,7 +840,7 @@ class Modify extends PointerInteraction {
for (i = dragSegments.length - 1; i >= 0; --i) { for (i = dragSegments.length - 1; i >= 0; --i) {
dragSegment = dragSegments[i]; dragSegment = dragSegments[i];
segmentData = dragSegment[0]; segmentData = dragSegment[0];
uid = getUid(segmentData.feature); uid = String(getUid(segmentData.feature));
if (segmentData.depth) { if (segmentData.depth) {
// separate feature components // separate feature components
uid += '-' + segmentData.depth.join('-'); uid += '-' + segmentData.depth.join('-');
@@ -933,7 +934,7 @@ class Modify extends PointerInteraction {
} }
this.updateSegmentIndices_(geometry, index, segmentData.depth, -1); this.updateSegmentIndices_(geometry, index, segmentData.depth, -1);
if (this.vertexFeature_) { if (this.vertexFeature_) {
this.overlay_.getSource().removeFeature(this.vertexFeature_); /** @type {VectorSource} */ (this.overlay_.getSource()).removeFeature(this.vertexFeature_);
this.vertexFeature_ = null; this.vertexFeature_ = null;
} }
dragSegments.length = 0; dragSegments.length = 0;
@@ -1009,7 +1010,7 @@ function handleDownEvent(evt) {
for (let i = 0, ii = segmentDataMatches.length; i < ii; ++i) { for (let i = 0, ii = segmentDataMatches.length; i < ii; ++i) {
const segmentDataMatch = segmentDataMatches[i]; const segmentDataMatch = segmentDataMatches[i];
const segment = segmentDataMatch.segment; const segment = segmentDataMatch.segment;
let uid = getUid(segmentDataMatch.feature); let uid = String(getUid(segmentDataMatch.feature));
const depth = segmentDataMatch.depth; const depth = segmentDataMatch.depth;
if (depth) { if (depth) {
uid += '-' + depth.join('-'); // separate feature components uid += '-' + depth.join('-'); // separate feature components