Merge pull request #3369 from element-hq/feature/bma/fixPinLockCrash

Ensure starting PinUnlockActivity does not crash the application.

features/lockscreen/impl/src/main/kotlin/io/element/android/features/lockscreen/impl/unlock/di/PinUnlockBindings.kt

@@ -18,9 +18,9 @@ package io.element.android.features.lockscreen.impl.unlock.di
 
 import com.squareup.anvil.annotations.ContributesTo
 import io.element.android.features.lockscreen.impl.unlock.activity.PinUnlockActivity
-import io.element.android.libraries.di.SessionScope
+import io.element.android.libraries.di.AppScope
 
-@ContributesTo(SessionScope::class)
+@ContributesTo(AppScope::class)
 interface PinUnlockBindings {
     fun inject(activity: PinUnlockActivity)
 }

features/logout/impl/src/main/kotlin/io/element/android/features/logout/impl/DefaultLogoutUseCase.kt

@@ -17,27 +17,25 @@
 package io.element.android.features.logout.impl
 
 import com.squareup.anvil.annotations.ContributesBinding
-import dagger.assisted.Assisted
-import dagger.assisted.AssistedFactory
-import dagger.assisted.AssistedInject
 import io.element.android.features.logout.api.LogoutUseCase
 import io.element.android.libraries.di.AppScope
 import io.element.android.libraries.matrix.api.MatrixClientProvider
-import io.element.android.libraries.matrix.api.core.SessionId
+import io.element.android.libraries.matrix.api.auth.MatrixAuthenticationService
+import javax.inject.Inject
 
-class DefaultLogoutUseCase @AssistedInject constructor(
+@ContributesBinding(AppScope::class)
-    @Assisted private val sessionId: String,
+class DefaultLogoutUseCase @Inject constructor(
+    private val authenticationService: MatrixAuthenticationService,
     private val matrixClientProvider: MatrixClientProvider,
 ) : LogoutUseCase {
-    @ContributesBinding(AppScope::class)
-    @AssistedFactory
-    interface Factory : LogoutUseCase.Factory {
-        override fun create(sessionId: String): DefaultLogoutUseCase
-    }
-
     override suspend fun logout(ignoreSdkError: Boolean): String? {
-        val matrixClient = matrixClientProvider.getOrRestore(SessionId(sessionId)).getOrThrow()
+        val currentSession = authenticationService.getLatestSessionId()
-        val result = matrixClient.logout(ignoreSdkError = ignoreSdkError)
+        return if (currentSession != null) {
-        return result
+            matrixClientProvider.getOrRestore(currentSession)
+                .getOrThrow()
+                .logout(ignoreSdkError = true)
+        } else {
+            error("No session to sign out")
+        }
     }
 }

features/logout/impl/src/main/kotlin/io/element/android/features/logout/impl/SessionLogoutModule.kt

@@ -1,36 +0,0 @@
-/*
- * Copyright (c) 2024 New Vector Ltd
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     https://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package io.element.android.features.logout.impl
-
-import com.squareup.anvil.annotations.ContributesTo
-import dagger.Module
-import dagger.Provides
-import io.element.android.features.logout.api.LogoutUseCase
-import io.element.android.libraries.di.SessionScope
-import io.element.android.libraries.matrix.api.user.CurrentSessionIdHolder
-
-@Module
-@ContributesTo(SessionScope::class)
-object SessionLogoutModule {
-     @Provides
-     fun provideLogoutUseCase(
-         currentSessionIdHolder: CurrentSessionIdHolder,
-         factory: DefaultLogoutUseCase.Factory,
-     ): LogoutUseCase {
-         return factory.create(currentSessionIdHolder.current.value)
-     }
-}