[AndyScherzinger]

Opening a Nextcloud contact from the Android Contacts app ("Chat via Talk") reported "account not found" for contacts that belong to an existing account other than the current default one.

Cause: the phone book integration (ContactAddressBookWorker) syncs for the account that is the default at the time, and stores the contact's cloud ID (user@host[/path]). After switching the default account, entries synced for the previous one remain. MainActivity.openChatOfContact() only compared the contact's server with the current default account, so these contacts could not be opened although their account exists.

Fix: choose the account by the server in the contact's cloud ID: the default account if it is on that server, otherwise another account on it (MainActivity.accountForContactServer()). The error is only shown if no account is on that server.

The server is now compared exactly (scheme stripped, trailing slash and case ignored) instead of with currentUser.baseUrl.endsWith(server), which also matched e.g. https://mycloud.example.com for a contact on cloud.example.com.

The snackbar for the "account not found" case is left as is here; #6937 turns it into a toast, as it was shown underneath the conversation list.

Tests: MainActivityContactAccountTest covers default account, another account on the contact's server, no default account, no matching account, suffix-only match, path, case/trailing slash and invalid servers. ktlint clean; detekt only reports the existing CyclomaticComplexMethod on handleIntent (same on master).

🖼️ Screenshots

🏚️ Before	🏡 After
B	A

🚧 TODO

• Test on a device with two accounts on different servers: sync the phone book with account A as default, switch the default to B, open a contact of A from the Contacts app

🏁 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?)

🤖 AI (if applicable)

• The content of this PR was partly or fully generated using AI

[github-actions]

Codacy

Lint

Type	master	PR
Warnings	138	138
Errors	15	15

SpotBugs

Category	Base	New
Bad practice	7	7
Correctness	8	8
Dodgy code	40	40
Internationalization	3	3
Malicious code vulnerability	3	3
Performance	8	8
Security	11	11
Total	80	80

[github-actions]

📱 QA build

Download	app-qa-debug.apk
QR code	Open the QR code for this download
Commit	26892c8
Version	6938
Available until	7 days after this build

The QA build installs alongside a released Nextcloud app, so you can keep
using your existing install while testing.

Downloading the file requires a GitHub account, so open this link on the
device you want to test on, or transfer the APK to it.

[mahibi]

tested and works fine