$mol_service_worker simple notify

hyoo-ru · Nov 3, 2024 · 2169769 · 2169769
1 parent 7f008e6
commit 2169769
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 9 deletions.
diff --git a/service/registration/registration.web.ts b/service/registration/registration.web.ts
@@ -32,10 +32,6 @@ namespace $ {
 			}))
 		}
 
-		protected container() {
-			return this.$.$mol_dom_context.navigator?.serviceWorker ?? $mol_fail(new Error('Service worker not found'))
-		}
-
 		@ $mol_mem
 		state(reset?: null) { return this.worker().state ?? null }
 

diff --git a/service/worker/worker.web.ts b/service/worker/worker.web.ts
@@ -81,8 +81,8 @@ namespace $ {
 			scope.addEventListener( 'message', this.handle(this.message.bind(this)))
 			scope.addEventListener( 'messageerror', this.message_error.bind(this))
 
-			scope.addEventListener( 'notificationclick', this.handle(this.notify.bind(this, false)))
-			scope.addEventListener( 'notificationclose', this.handle(this.notify.bind(this, true)))
+			scope.addEventListener( 'notificationclick', this.handle(this.notify.bind(this, 'notification')))
+			scope.addEventListener( 'notificationclose', this.handle(this.notify.bind(this, 'notification_close')))
 
 			scope.addEventListener( 'push', this.handle(this.push.bind(this)))
 			scope.addEventListener( 'fetch', event => {
@@ -220,13 +220,13 @@ namespace $ {
 			}
 		}
 
-		static notify(closing: boolean, event: NotificationEvent) {
+		static notify(method: 'notification' | 'notification_close', event: NotificationEvent) {
 			for (const plugin of this.plugins_notify()) {
 				try {
-					closing ? plugin.notification_close(event.notification) : plugin.notification(event.notification)
+					plugin[method](event.notification)
 				} catch (error) {
 					if ( ! $mol_fail_catch(error) ) continue
-					this.plugin_error(error, `${plugin}.${closing ? 'notification_close' : 'notification'}()`)
+					this.plugin_error(error, `${plugin}.${method}()`)
 				}
 			}
 		}