New approach to check webpush worker update on all pages. See

https://developer.mozilla.org/en-US/docs/Web/API/ServiceWorkerRegistration/update

Author: rxu

event/listener.php

@@ -30,6 +30,7 @@ public static function getSubscribedEvents()
 			'core.ucp_notifications_output_notification_types_modify_template_vars'	=> 'load_template_data',
 			'core.ucp_display_module_before'	=> 'load_language',
 			'core.acp_main_notice'				=> 'compatibility_notice',
+			'core.page_header_after'			=> 'service_worker_url',
 		];
 	}
 
@@ -90,4 +91,12 @@ public function compatibility_notice()
 	{
 		$this->template->assign_var('S_WPN_COMPATIBILITY_NOTICE', phpbb_version_compare(PHPBB_VERSION, '4.0.0-dev', '>='));
 	}
+
+	/**
+	 * Generate service worker URL globally for update
+	 */
+	public function service_worker_url()
+	{
+		$this->template->assign_var('U_WEBPUSH_WORKER_URL', $this->controller_helper->route('phpbb_webpushnotifications_ucp_push_worker_controller'));
+	}
 }

notification/method/webpush.php

@@ -445,7 +445,7 @@ protected function clean_expired_subscriptions(array $user_subscription_map, arr
 	 * into https://myboard.url/path/to/avatar=123456789.gif
 	 *
 	 * @param string $avatar
-	 * @return string Absolute path to avatar image
+	 * @return array 'src' => Absolute path to avatar image
 	 */
 	protected function prepare_avatar($avatar)
 	{

styles/all/template/update_worker.js

@@ -0,0 +1,27 @@
+'use strict';
+
+function webpushWorkerUpdate() {
+	if ('serviceWorker' in navigator) {
+		navigator.serviceWorker.register(serviceWorkerUrl)
+			.then((registration) => {
+				registration.update();
+			})
+			.catch(error => {
+				// Service worker could not be updated
+				console.info(error);
+			});
+	}
+}
+
+function domReady(callBack) {
+	if (document.readyState === 'loading') {
+		document.addEventListener('DOMContentLoaded', callBack);
+	} else {
+		callBack();
+	}
+}
+
+domReady(() => {
+	/* global serviceWorkerUrl */
+	webpushWorkerUpdate();
+});

styles/all/template/webpush.js

@@ -57,10 +57,9 @@ function PhpbbWebpush() {
 
 		if ('serviceWorker' in navigator && 'PushManager' in window) {
 			navigator.serviceWorker.register(serviceWorkerUrl)
-				.then((registration) => {
+				.then(() => {
 					subscribeButton.addEventListener('click', subscribeButtonHandler);
 					unsubscribeButton.addEventListener('click', unsubscribeButtonHandler);
-					registration.update();
 
 					updateButtonState();
 				})

styles/prosilver/template/event/overall_footer_after.html

@@ -0,0 +1,4 @@
+<script>
+	var serviceWorkerUrl = '{{ U_WEBPUSH_WORKER_URL }}';
+</script>
+{% INCLUDEJS('@phpbb_webpushnotifications/update_worker.js') %}

tests/event/listener_test.php

@@ -117,6 +117,7 @@ public function test_getSubscribedEvents()
 			'core.ucp_notifications_output_notification_types_modify_template_vars',
 			'core.ucp_display_module_before',
 			'core.acp_main_notice',
+			'core.page_header_after',
 		], array_keys(\phpbb\webpushnotifications\event\listener::getSubscribedEvents()));
 	}