Disable push subscribing functionality from guests

Signed-off-by: Matt Friedman <maf675@gmail.com>

fix

Signed-off-by: Matt Friedman <maf675@gmail.com>

Is you is or is you ain’t

Signed-off-by: Matt Friedman <maf675@gmail.com>

Author: iMattPro

notification/method/webpush.php

@@ -379,7 +379,7 @@ public function get_ucp_template_data(helper $controller_helper, form_helper $fo
 		}
 
 		return [
-			'NOTIFICATIONS_WEBPUSH_ENABLE'	=> $this->config['wpn_webpush_dropdown_subscribe'] || stripos($this->user->page['page'], 'notification_options'),
+			'NOTIFICATIONS_WEBPUSH_ENABLE'	=> $this->allowed_user() && ($this->config['wpn_webpush_dropdown_subscribe'] || stripos($this->user->page['page'], 'notification_options')),
 			'U_WEBPUSH_SUBSCRIBE'			=> $controller_helper->route('phpbb_webpushnotifications_ucp_push_subscribe_controller'),
 			'U_WEBPUSH_UNSUBSCRIBE'			=> $controller_helper->route('phpbb_webpushnotifications_ucp_push_unsubscribe_controller'),
 			'VAPID_PUBLIC_KEY'				=> $this->config['wpn_webpush_vapid_public'],
@@ -553,4 +553,17 @@ protected function load_recipients_data(): array
 
 		return $notify_users;
 	}
+
+	/**
+	 * User is allowed to use web push notifications
+	 *
+	 * @return bool
+	 */
+	protected function allowed_user()
+	{
+		return $this->user->id() !== ANONYMOUS
+			&& !$this->user->data['is_bot']
+			&& (int) $this->user->data['user_type'] !== USER_IGNORE
+			&& (int) $this->user->data['user_type'] !== USER_INACTIVE;
+	}
 }