@wemap/providers 9.0.5 → 9.0.6

package/package.json

@@ -9,13 +9,13 @@
   ],
   "dependencies": {
     "@wemap/camera": "^9.0.0",
-    "@wemap/geo": "^9.0.4",
+    "@wemap/geo": "^9.0.6",
     "@wemap/geomagnetism": "^0.1.1",
     "@wemap/logger": "^9.0.0",
-    "@wemap/map": "^9.0.4",
+    "@wemap/map": "^9.0.6",
     "@wemap/maths": "^9.0.0",
-    "@wemap/osm": "^9.0.4",
-    "@wemap/routers": "^9.0.4",
+    "@wemap/osm": "^9.0.6",
+    "@wemap/routers": "^9.0.6",
     "@wemap/utils": "^9.0.0"
   },
   "description": "A package using different geoloc systems",
@@ -42,6 +42,6 @@
     "url": "git+https://github.com/wemap/wemap-modules-js.git"
   },
   "type": "module",
-  "version": "9.0.5",
-  "gitHead": "a383d28828b66c60339a021c53ed55be62da96c0"
+  "version": "9.0.6",
+  "gitHead": "a19cb8d019e18a47cd5fd1ee1b6c02d6bae5acad"
 }

package/src/providers/Provider.js

@@ -23,7 +23,7 @@ class Provider {
     id;
 
     /** @type {number} */
-    state = ProviderState.STOPPPED;
+    state = ProviderState.STOPPED;
 
     /** @type {ProviderEvent[]} */
     _lastEvents = null;
@@ -170,7 +170,7 @@ class Provider {
 
 
         // If the provider is already started do not go further
-        if (this.state !== ProviderState.STOPPPED) {
+        if (this.state !== ProviderState.STOPPED) {
             return id;
         }
         this.state = ProviderState.STARTING;
@@ -181,23 +181,24 @@ class Provider {
             availabilityPromise = this.availability;
         }
 
-        availabilityPromise
-            .then(() => {
+        (async () => {
+            try {
+                await availabilityPromise;
+            } catch (e) {
+                this.state = ProviderState.STOPPED;
+                this.notifyError(e);
+                return;
+            }
 
-                ProvidersLoggerOld.addEvent(this, 'start');
-                // Call the child start function
-                this.start();
+            ProvidersLoggerOld.addEvent(this, 'start');
 
-                this.state = ProviderState.STARTED;
+            // Call the child start function
+            this.start();
 
-                this._monitoringCallbacks.forEach(({ onStarted }) => onStarted());
-            })
-            .catch(e => {
-                this.state = ProviderState.STOPPPED;
-                this.notifyError(e);
-            })
-            // notifyError can throw an error if onStop is not defined
-            .catch(() => { });
+            this.state = ProviderState.STARTED;
+
+            this._monitoringCallbacks.forEach(({ onStarted }) => onStarted());
+        })();
 
         return id;
     }
@@ -230,7 +231,7 @@ class Provider {
 
         // If the provider is already stopped, do not stop it again.
         // This condition can be true if checkAvailabilityOnStart is true and returns an error
-        if (this.state === ProviderState.STOPPPED) {
+        if (this.state === ProviderState.STOPPED) {
             return;
         }
 
@@ -238,7 +239,7 @@ class Provider {
         // Call the child stop function
         this.stop();
 
-        this.state = ProviderState.STOPPPED;
+        this.state = ProviderState.STOPPED;
 
         this._monitoringCallbacks.forEach(({ onStopped }) => onStopped());
     }

package/src/providers/Provider.spec.js

@@ -59,7 +59,7 @@ describe('Provider', () => {
         await sleep(2);
 
         expect(errored).is.true;
-        expect(FakeProvider2.state).equals(ProviderState.STOPPPED);
+        expect(FakeProvider2.state).equals(ProviderState.STOPPED);
         expect(FakeProvider2._eventsCallbacks.length).equals(0);
 
         /**
@@ -76,7 +76,7 @@ describe('Provider', () => {
         expect(FakeProvider2.state).equals(ProviderState.STARTED);
 
         FakeProvider2.removeEventListener(id);
-        expect(FakeProvider2.state).equals(ProviderState.STOPPPED);
+        expect(FakeProvider2.state).equals(ProviderState.STOPPED);
         expect(FakeProvider2._eventsCallbacks.length).equals(0);
 
         ProvidersOptions.checkAvailabilityOnStart = true;

package/src/providers/ProviderState.js

@@ -1,5 +1,5 @@
 export default {
     STARTING: 0,
     STARTED: 1,
-    STOPPPED: 2
+    STOPPED: 2
 };

package/src/providers/others/CameraNative.js

@@ -35,7 +35,7 @@ class CameraNative extends Provider {
      */
     start() {
 
-        if (ArCore.state !== ProviderState.STOPPPED) {
+        if (ArCore.state !== ProviderState.STOPPED) {
             this.notify(this.createEvent(EventType.CameraNative, 'started'));
         }
         this.providerId = ArCore.addMonitoringListener(

package/src/providers/vision/ArCore.js

@@ -76,7 +76,7 @@ class ArCore extends Provider {
 
     pullDataLoop = () => {
 
-        if (this.state === ProviderState.STOPPPED) {
+        if (this.state === ProviderState.STOPPED) {
             return;
         }
 

package/src/providers/vision/Vps.js

@@ -126,7 +126,7 @@ class Vps extends Provider {
             return;
         }
 
-        while (this.state !== ProviderState.STOPPPED) {
+        while (this.state !== ProviderState.STOPPED) {
 
             // 1. Handle the time to wait between two requests (MIN_TIME_BETWEEN_TWO_REQUESTS)
             if (this.lastEvent) {
@@ -136,7 +136,7 @@ class Vps extends Provider {
             }
 
             // 2. Check if Vps is not stopped and camera is still started
-            if (this.state === ProviderState.STOPPPED
+            if (this.state === ProviderState.STOPPED
                 || !this._camera
                 || this._camera.state !== Camera.State.STARTED) {
                 break;
@@ -164,7 +164,7 @@ class Vps extends Provider {
             const attitude = new Attitude(deviceQuaternion, res.attitude.time, res.attitude.accuracy);
 
             // 5. Finally, notify the listeners if the VPS is not stopped
-            if (this.state === ProviderState.STOPPPED) {
+            if (this.state === ProviderState.STOPPED) {
                 break;
             }