Moved OneSignal.context access down a level

* Clean up - Moved access to OneSignal.context.serviceWorkerManager down one call stack from getRawPushSubscriptionFromServiceWorkerRegistration to getRawPushSubscription
OneSignal · Feb 19, 2021 · 0b6b420 · 0b6b420
1 parent b3de742
commit 0b6b420
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/helpers/SubscriptionHelper.ts b/src/helpers/SubscriptionHelper.ts
@@ -161,8 +161,7 @@ export default class SubscriptionHelper {
     return subscription;
   }
 
-  static async getRawPushSubscriptionFromServiceWorkerRegistration(): Promise<RawPushSubscription | null> {
-    const registration = await OneSignal.context.serviceWorkerManager.getRegistration();
+  static async getRawPushSubscriptionFromServiceWorkerRegistration(registration?: ServiceWorkerRegistration): Promise<RawPushSubscription | null> {
     if (!registration) {
       return null;
     }
@@ -190,7 +189,8 @@ export default class SubscriptionHelper {
     }
 
     if (environmentInfo.isBrowserAndSupportsServiceWorkers) {
-      return await SubscriptionHelper.getRawPushSubscriptionFromServiceWorkerRegistration();
+      const registration = await OneSignal.context.serviceWorkerManager.getRegistration();
+      return await SubscriptionHelper.getRawPushSubscriptionFromServiceWorkerRegistration(registration);
     }
 
     return null;