filter out user only if userId and baseUrl are the same

[tobiasKaminsky]

🖼️ Screenshots

🏚️ Before	🏡 After
B	A

🚧 TODO

• ...

🏁 Checklist

• ⛑️ Tests (unit and/or integration) are included or not needed
• 🔖 Capability is checked or not needed
• 🔙 Backport requests are created or not needed: /backport to stable-xx.x
• 📅 Milestone is set
• 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

[github-actions]

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/5567.apk



To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

[mahibi]

Thanks Tobias!
I did not test but this should solve it.

However do we need

val userId: String?

inside AccountItem at all?

It's confusing that the userId it saved twice so it can be accessed via:

user.userId
and
user.user.userId

?

The PR should fix the issue caused by

talk-android/app/src/main/java/com/nextcloud/talk/ui/dialog/ChooseAccountDialogCompose.kt

Line 184 in fd3a4aa

addAccountToList(user, user.userId ?: user.username, pendingCount)

but it feels more like a workaround while the main issue is not solved: why can userId be null?