B/fix airport layer bug (#50)

* fix a vt layer bug * 4.0.2 * safer way * also run lint at CI
haoliangyu · Feb 12, 2018 · 1f438a2 · 1f438a2
1 parent 17cc8d1
commit 1f438a2
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 5 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -4,6 +4,7 @@ node_js:
 install:
   - npm install
 script:
+  - npm run lint
   - npm run build
 deploy:
   provider: pages

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "ngx-leaflet-starter",
-  "version": "4.0.1",
+  "version": "4.0.2",
   "description": "A minimal Angular project with leaflet",
   "main": "index.js",
   "repository": {

diff --git a/src/app/map.service.ts b/src/app/map.service.ts
@@ -54,13 +54,13 @@ export class MapService {
 
   toggleAirPortLayer(on: boolean) {
     if (on) {
-      this.map.removeLayer(this.vtLayer);
-      delete this.vtLayer;
-    } else {
       this.http.get("assets/airports.min.geojson").subscribe(result => {
         this.vtLayer = L.vectorGrid.slicer(result);
         this.vtLayer.addTo(this.map);
       });
+    } else if (this.vtLayer) {
+      this.map.removeLayer(this.vtLayer);
+      delete this.vtLayer;
     }
   }