fix to not execute getAnyUserAndSetAsActive() until userRepository.getActiveUser() is empty

To make sure getAnyUserAndSetAsActive() is invoked lazily and avoid any side effects, it's explicitly wrapped it in a lambda.

Maybe.defer ensures that getAnyUserAndSetAsActive() is not invoked until switchIfEmpty decides it’s needed.

Signed-off-by: Marcel Hibbe <dev@mhibbe.de>

app/src/main/java/com/nextcloud/talk/users/UserManager.kt

@@ -31,7 +31,7 @@ class UserManager internal constructor(private val userRepository: UsersReposito
     val currentUser: Maybe<User>
         get() {
             return userRepository.getActiveUser()
-                .switchIfEmpty(getAnyUserAndSetAsActive())
+                .switchIfEmpty(Maybe.defer { getAnyUserAndSetAsActive() })
         }
 
     val currentUserObservable: Observable<User>