Merge pull request #12066 from M393/vector-layer-get-feature-promise

Always return Promise from getFeatures

src/ol/layer/Layer.js

@@ -210,6 +210,9 @@ class Layer extends BaseLayer {
    * an array of features.
    */
   getFeatures(pixel) {
+    if (!this.renderer_) {
+      return new Promise((resolve) => resolve([]));
+    }
     return this.renderer_.getFeatures(pixel);
   }
 

src/ol/renderer/canvas/VectorImageLayer.js

@@ -63,18 +63,14 @@ class CanvasVectorImageLayerRenderer extends CanvasImageLayerRenderer {
    * @return {Promise<Array<import("../../Feature").default>>} Promise that resolves with an array of features.
    */
   getFeatures(pixel) {
-    if (this.vectorRenderer_) {
+    if (!this.vectorRenderer_) {
-      const vectorPixel = apply(
+      return new Promise((resolve) => resolve([]));
-        this.coordinateToVectorPixelTransform_,
-        apply(this.renderedPixelToCoordinateTransform_, pixel.slice())
-      );
-      return this.vectorRenderer_.getFeatures(vectorPixel);
-    } else {
-      const promise = new Promise(function (resolve, reject) {
-        resolve([]);
-      });
-      return promise;
     }
+    const vectorPixel = apply(
+      this.coordinateToVectorPixelTransform_,
+      apply(this.renderedPixelToCoordinateTransform_, pixel.slice())
+    );
+    return this.vectorRenderer_.getFeatures(vectorPixel);
   }
 
   /**