Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Settings/Contacts/Sorting column in Roundcube does not work with CardDAV Addressbook #325

Closed
JMA-Github opened this issue Nov 30, 2020 · 4 comments
Assignees
Labels
Milestone

Comments

@JMA-Github
Copy link

When I change the sorting column i.E. to Last Name, it will have the expected effect in my Personal Addressbook.
All contacts are sorted ba "Last Name".
But, contacts in my CardDAV Addressbook (connected to a Nextcloud based CardDAV-Server) will always be sorted by "Display-Name". The change of "Sorting Column" in the settings has no effect.

@blind-coder
Copy link
Collaborator

Please check if you actually HAVE a lastname / firstname set in your contacts. NextCloud does not automatically extract those from the Display Name, contrary to how pretty much every other service does it.

@JMA-Github
Copy link
Author

Lastname and Firstname are set . I've checked it in the contact edit window of RC and in the database-table carddav_contacts.
I figured out, that 3 columns are filled (name, firstname, surname)
name seems to be build from firstname + surname
When I enter a "display name " in RC, column "name" is filled with the "display name"
The column "showas" is always empty
The order in the contact list is always by coulumn "name"

@mstilkerich mstilkerich self-assigned this Nov 30, 2020
@mstilkerich mstilkerich added this to the 4.1.0 milestone Nov 30, 2020
@mstilkerich
Copy link
Owner

Thanks for reporting this, this broke with 968fe9f.

I think this should be fixed in roundcube and created a pull request roundcube/roundcubemail#7758, but I'll also fix it in rcmcarddav to restore the functionality.

mstilkerich added a commit that referenced this issue Nov 30, 2020
Plus some other tweaks to fix psalm analysis with current roundcube
master.
@mstilkerich
Copy link
Owner

Fixed in v4.1. If you feel adventurous you can try that branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants