fix(performance): add cache on nominatim,osrm,taginfo

Author: toutpt

src/nominatim/nominatim.service.js

@@ -12,6 +12,10 @@ class NominatimAPI{
     constructor($http, options) {
         this.url = options.url;
         this.$http = $http;
+        this.cache = true;
+        if (options.cache === false) {
+            this.cache = false;
+        }
     }
 
     /**
@@ -34,7 +38,8 @@ class NominatimAPI{
             params.format = 'json';
         }
         var config = {
-            params: params
+            params: params,
+            cache: this.cache
         };
         let url = this.url + '/search';
         return this.$http.get(url, config);
@@ -60,7 +65,8 @@ class NominatimAPI{
             params.format = 'json';
         }
         var config = {
-            params: params
+            params: params,
+            cache: this.cache
         };
         let url = this.url + '/reverse';
         return this.$http.get(url, config);
@@ -85,7 +91,8 @@ class NominatimAPI{
             params.format = 'json';
         }
         var config = {
-            params: params
+            params: params,
+            cache: this.cache
         };
         let url = this.url + '/lookup';
         return this.$http.get(url, config);

src/osrm/osrm.service.js

@@ -13,14 +13,19 @@ class OSRMAPI{
         this.url = options.url;
         this.$http = $http;
         this.$q = $q;
+        this.cache = true;
+        if (options.cache === false) {
+            this.cache = false;
+        }
     }
     /**
      * internal get request to the remote API
      * @param {string} service
      * @param {string} version
      * @param {string} profile
      * @param {string|Object} coordinates
-     * the string format is {longitude},{latitude};{longitude},{latitude}[;{longitude},{latitude} ...]
+     * the string format is
+     * {longitude},{latitude};{longitude},{latitude}[;{longitude},{latitude} ...]
      * @param {Object} options
      */
     get(service, version, profile, coordinates, options) {
@@ -29,7 +34,7 @@ class OSRMAPI{
             _coordinates = coordinates.join(';');
         }
         var url = `${this.url}/${service}/${version}/${profile}/${_coordinates}`;
-        return this.$http.get(url, {params: options});
+        return this.$http.get(url, {params: options, cache: this.cache});
     }
     /**
      * neareset service

src/taginfo/taginfo.js

@@ -4,6 +4,14 @@
 import osmTagInfoAPI from './taginfo.service';
 
 var osmTagInfoModule = angular.module('osm.taginfo', [])
-.service('osmTagInfoAPI', osmTagInfoAPI);
+.provider('osmTagInfoAPI', function osmTagInfoAPIProvider () {
+    this.options = {
+        url: 'https://taginfo.openstreetmap.org/api/4'
+    };
+    this.$get = function osmTagInfoAPIFactory($http, $q) {
+        return new osmTagInfoAPI($http, $q, this.options);
+    };
+    this.$get.$inject = ['$http', '$q'];
+});
 
 export default osmTagInfoModule;

src/taginfo/taginfo.service.js

@@ -9,10 +9,14 @@ class TagInfoAPI{
      * @param {Object} $http angular $http service
      * @param {Object} $q angular $q service
      */
-    constructor($http, $q) {
+    constructor($http, $q, options) {
         this.$http = $http;
         this.$q = $q;
-        this.url = 'https://taginfo.openstreetmap.org/api/4';
+        this.url = options.url;
+        this.cache = true;
+        if (options.cache === false) {
+            this.cache = false;
+        }
     }
     /**
      * internal get request to the remote API
@@ -22,6 +26,12 @@ class TagInfoAPI{
      */
     get(method, config) {
         var deferred = this.$q.defer();
+        if (!config) {
+            config = {};
+        }
+        if (config.cache === undefined) {
+            config.cache = this.cache;
+        }
         this.$http.get(this.url + method, config).then(
             function (data) {
                 deferred.resolve(data.data);