Enable detekt rules DataClassShouldBeImmutable and fix existing issues.

libraries/push/impl/src/main/kotlin/io/element/android/libraries/push/impl/notifications/RoomEventGroupInfo.kt

@@ -23,17 +23,15 @@ import io.element.android.libraries.matrix.api.core.SessionId
  * Data class to hold information about a group of notifications for a room.
  */
 data class RoomEventGroupInfo(
-        val sessionId: SessionId,
+    val sessionId: SessionId,
-        val roomId: RoomId,
+    val roomId: RoomId,
-        val roomDisplayName: String,
+    val roomDisplayName: String,
-        val isDirect: Boolean = false
+    val isDirect: Boolean = false,
-) {
     // An event in the list has not yet been display
-    var hasNewEvent: Boolean = false
+    val hasNewEvent: Boolean = false,
-
     // true if at least one on the not yet displayed event is noisy
-    var shouldBing: Boolean = false
+    val shouldBing: Boolean = false,
-    var customSound: String? = null
+    val customSound: String? = null,
-    var hasSmartReplyError: Boolean = false
+    val hasSmartReplyError: Boolean = false,
-    var isUpdated: Boolean = false
+    val isUpdated: Boolean = false,
-}
+)

libraries/push/impl/src/main/kotlin/io/element/android/libraries/push/impl/notifications/RoomGroupMessageCreator.kt

@@ -85,12 +85,11 @@ class RoomGroupMessageCreator @Inject constructor(
                     roomId = roomId,
                     roomDisplayName = roomName,
                     isDirect = !roomIsGroup,
-                ).also {
+                    hasSmartReplyError = smartReplyErrors.isNotEmpty(),
-                    it.hasSmartReplyError = smartReplyErrors.isNotEmpty()
+                    shouldBing = meta.shouldBing,
-                    it.shouldBing = meta.shouldBing
+                    customSound = events.last().soundName,
-                    it.customSound = events.last().soundName
+                    isUpdated = events.last().isUpdated,
-                    it.isUpdated = events.last().isUpdated
+                ),
-                },
                 threadId = lastKnownRoomEvent.threadId,
                 largeIcon = largeBitmap,
                 lastMessageTimestamp,

libraries/push/impl/src/main/kotlin/io/element/android/libraries/push/impl/notifications/model/SimpleNotifiableEvent.kt

@@ -30,7 +30,7 @@ data class SimpleNotifiableEvent(
     val type: String?,
     val timestamp: Long,
     val soundName: String?,
-    override var canBeReplaced: Boolean,
+    override val canBeReplaced: Boolean,
     override val isRedacted: Boolean = false,
     override val isUpdated: Boolean = false
 ) : NotifiableEvent

libraries/pushproviders/firebase/src/main/kotlin/io/element/android/libraries/pushproviders/firebase/PushDataFirebase.kt

@@ -36,7 +36,7 @@ import io.element.android.libraries.pushproviders.api.PushData
 data class PushDataFirebase(
     val eventId: String?,
     val roomId: String?,
-    var unread: Int?,
+    val unread: Int?,
     val clientSecret: String?
 )
 

libraries/pushproviders/unifiedpush/src/main/kotlin/io/element/android/libraries/pushproviders/unifiedpush/PushDataUnifiedPush.kt

@@ -47,7 +47,7 @@ data class PushDataUnifiedPush(
 data class PushDataUnifiedPushNotification(
     @SerialName("event_id") val eventId: String? = null,
     @SerialName("room_id") val roomId: String? = null,
-    @SerialName("counts") var counts: PushDataUnifiedPushCounts? = null,
+    @SerialName("counts") val counts: PushDataUnifiedPushCounts? = null,
 )
 
 @Serializable

tools/detekt/detekt.yml

@@ -6,6 +6,8 @@ style:
   CascadingCallWrapping:
     active: true
     includeElvis: true
+  DataClassShouldBeImmutable:
+    active: true
   MaxLineLength:
     # Default is 120
     maxLineLength: 160