Merge pull request #218 from mapbox/deduplicate-result-event

Deduplicate result event emissions (#99)
mapbox · Mar 19, 2019 · 1c44b72 · 1c44b72
2 parents 6c6c98a + 5751660
commit 1c44b72
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,7 @@
 - Localize placeholder based on language set in constructor options [#150](https://github.com/mapbox/mapbox-gl-geocoder/issues/150)
 - `trackProximity` turned on by default [#195](https://github.com/mapbox/mapbox-gl-geocoder/issues/195)
 - Bump suggestions to v1.3.4
+- Fix duplicate event bug
 
 ## v3.1.4
 

diff --git a/lib/index.js b/lib/index.js
@@ -54,6 +54,7 @@ function MapboxGeocoder(options) {
   this.options = extend({}, this.options, options);
   this.inputString = '';
   this.fresh = true;
+  this.lastSelected = null;
 }
 
 MapboxGeocoder.prototype = {
@@ -190,8 +191,11 @@ MapboxGeocoder.prototype = {
           });
         }
       }
-      this._eventEmitter.emit('result', { result: selected });
-      this.eventManager.select(selected, this);
+      if (selected.id !== this.lastSelected){
+        this._eventEmitter.emit('result', { result: selected });
+        this.eventManager.select(selected, this);
+        this.lastSelected =  selected.id;
+      }
     }
   },