Fix bean definitions and use strict equality check

Author: Isti01

backend/src/main/kotlin/hu/bme/sch/cmsch/component/pushnotification/FirebaseMessagingConfiguration.kt

@@ -13,6 +13,7 @@ import org.springframework.retry.policy.SimpleRetryPolicy
 import org.springframework.retry.support.RetryTemplate
 
 @Configuration
+@ConditionalOnBean(PushNotificationComponent::class)
 class FirebaseMessagingConfiguration {
     private val messagingScope = "https://www.googleapis.com/auth/firebase.messaging"
 

backend/src/main/kotlin/hu/bme/sch/cmsch/component/pushnotification/MessagingTokenEntity.kt

@@ -1,12 +1,14 @@
 package hu.bme.sch.cmsch.component.pushnotification
 
 import jakarta.persistence.*
+import org.springframework.boot.autoconfigure.condition.ConditionalOnBean
 
 @Entity
 @Table(
     name = "messaging_tokens",
     indexes = [Index(columnList = "userId"), Index(columnList = "userId,token", unique = true)]
 )
+@ConditionalOnBean(PushNotificationComponent::class)
 class MessagingTokenEntity(
     @Id
     @GeneratedValue(strategy = GenerationType.SEQUENCE)

backend/src/main/kotlin/hu/bme/sch/cmsch/component/pushnotification/MessagingTokenRepository.kt

@@ -1,9 +1,11 @@
 package hu.bme.sch.cmsch.component.pushnotification
 
 import hu.bme.sch.cmsch.model.RoleType
+import org.springframework.boot.autoconfigure.condition.ConditionalOnBean
 import org.springframework.data.jpa.repository.JpaRepository
 import org.springframework.data.jpa.repository.Query
 
+@ConditionalOnBean(PushNotificationComponent::class)
 interface MessagingTokenRepository : JpaRepository<MessagingTokenEntity, Long> {
 
     @Query("select m.token from MessagingTokenEntity m")

frontend/src/common-components/EnableNotifications.tsx

@@ -4,7 +4,7 @@ import { useConfigContext } from '../api/contexts/config/ConfigContext.tsx'
 export const EnableNotifications = () => {
   const { isOpen, onClose } = useDisclosure({ defaultIsOpen: true })
   const config = useConfigContext()
-  if (Notification.permission != 'default') return null // we cannot ask again
+  if (Notification.permission !== 'default') return null // we cannot ask again
 
   const component = config.components.pushnotification
   if (!component?.notificationsEnabled || !isOpen || !shouldShowAlert(component.permissionAllowNeverShowAgain)) return null

frontend/src/util/configs/firebase.config.ts

@@ -37,12 +37,10 @@ export async function initNotifications(messaging: Messaging, onNotification: (p
   await axios.post(ApiPaths.ADD_PUSH_NOTIFICATION_TOKEN, { token })
 }
 
-export async function disableNotifications() {
-  return unregisterServiceWorker()
-}
+export const disableNotifications = () => unregisterServiceWorker()
 
 export async function unsubscribeFromNotifications() {
-  if (Notification.permission != 'granted') return // There is nothing to unsubscribe from
+  if (Notification.permission !== 'granted') return // We cannot obtain a messaging token if the permission is not granted
 
   try {
     const messaging = getCloudMessaging()