[fix]ol高效率点击图层缩放出现多次 review by qiw

Author: luoxiao-supermap

src/openlayers/overlay/Graphic.js

@@ -139,7 +139,6 @@ export class Graphic extends ImageCanvasSource {
             me.renderer.selected = this.selected;
             me.renderer.drawGraphics(graphics);
             me.isDeckGLRender = me.renderer instanceof GraphicWebGLRenderer;
-            return me.renderer.getCanvas();
         }
 
         function createRenderer(size, pixelRatio) {
@@ -543,6 +542,7 @@ export class Graphic extends ImageCanvasSource {
      * @private
      */
     _highLight(center, image, selectGraphic, evtPixel) {
+        console.log('highlight')
         if (selectGraphic.getStyle() instanceof CloverShape) {
             if (this.hitGraphicLayer) {
                 this.map.removeLayer(this.hitGraphicLayer);

test/openlayers/overlay/GraphSpec.js

@@ -117,6 +117,7 @@ describe('openlayers_Graph', () => {
       });
 
       var key = layer.on('postrender', function () {
+        console.log('++++++++++++++++')
         unByKey(key);
         expect(graphThemeSource.chartsType).toBe('Bar');
         graphThemeSource.setChartsType('Line');

test/openlayers/overlay/GraphicSpec.js

@@ -637,7 +637,7 @@ describe('openlayers_GraphicLayer', () => {
     });
   });
   it('onCLick', (done) => {
-    map = new Map({
+    const map = new Map({
       target: 'map',
       view: new View({
         center: [0, 0],
@@ -669,21 +669,24 @@ describe('openlayers_GraphicLayer', () => {
         })
       });
       map.addLayer(graphicLayer);
-      const source = graphicLayer.getSource();
-      source.renderer = { _clearBuffer: () => {}, drawGraphics: () => {}, getCanvas: () => {} };
-      source._forEachFeatureAtCoordinate([0, 0], 1, (result) => {
-        expect(result).not.toBeNull();
-      });
-      const res = source.findGraphicByPixel({ pixel: [0, 0] }, source);
-      expect(res).toBeUndefined();
-      const res1 = source.getDeckglArguments(source, { pixel: [0, 0] }, graphics[0]);
-      expect(res1).not.toBeNull();
-      let pixel = map.getPixelFromCoordinate([0, 0]);
-      map.forEachFeatureAtPixel(pixel, (graphic, layer) => {
-        expect(graphic).toBe(graphics[0]);
-        expect(layer).toBe(graphicLayer);
+      graphicLayer.on('postrender', function () {
+        const source = graphicLayer.getSource();
+        if (source.renderer) {
+          source._forEachFeatureAtCoordinate([0, 0], 1, (result) => {
+            expect(result).not.toBeNull();
+          });
+          const res = source.findGraphicByPixel({ pixel: [0, 0] }, source);
+          expect(res).toBeUndefined();
+          const res1 = source.getDeckglArguments(source, { pixel: [0, 0] }, graphics[0]);
+          expect(res1).not.toBeNull();
+          let pixel = map.getPixelFromCoordinate([0, 0]);
+          map.forEachFeatureAtPixel(pixel, (graphic, layer) => {
+            expect(graphic).toBe(graphics[0]);
+            expect(layer).toBe(graphicLayer);
+          });
+          done();
+        }
       });
-      done();
     });
   });
 });