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

GroupNavigationItem: encode exported contact groups as UTF-8 #4107

Merged
merged 1 commit into from
Sep 2, 2024

Conversation

GVodyanov
Copy link
Contributor

Fix #2365
Fix #946

Previously exported contact groups would be encoded either as us-ascii, or iso-8859-1 if they had umlauts or other special characters. I made all of them be exported as utf-8, as singular contacts are.

The main issues with this were that when contact groups were encoded as iso-8859-1 they would have a weird character instead of umlauts and would bug out while importing.

Signed-off-by: Grigory Vodyanov <scratchx@gmx.com>
@GVodyanov GVodyanov added the 3. to review Waiting for reviews label Sep 2, 2024
@GVodyanov GVodyanov self-assigned this Sep 2, 2024
Copy link
Member

@st3iny st3iny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very weird API. Especially that it doesn't assume utf-8 strings by default.

Tested and works!

@st3iny
Copy link
Member

st3iny commented Sep 2, 2024

/backport to stable6.1

@st3iny
Copy link
Member

st3iny commented Sep 2, 2024

/backport to stable6.0

@st3iny
Copy link
Member

st3iny commented Sep 2, 2024

/backport to stable5.5

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

Successfully merging this pull request may close these issues.

Add a contact (with umlauts) to a group (with umlauts) breaks encoding umlauts make name disappear
2 participants