-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Fix com_contact > created vcf in contactform can´t handle german umlauts (issue #7937) #7938
Fix com_contact > created vcf in contactform can´t handle german umlauts (issue #7937) #7938
Conversation
Not bad for a non developer :-) Indentation gets lost somewhere and the comment is in German, fine for me but not the usual way we are doing it. Some code style issues on top. Pretty normal for the first PR :-) |
This breaks UTF8 characters here. so it may work for some latin characters (for example with umlauts) but not for Greek or Chinese, etc. |
Let Travis be a happy guy
@infograf768 does it work with Greek/Chinese without this patch? |
the seccond CS one ;)
It does on Macintosh:
This is what we get after this patch...
Each non-latin glyph is replaced by a ? |
@infograf768 on a apple it works with german umlauts also, but not in win outlook |
Yes, as stated in the link I gave above. We can't though provide wrong vcf because Windows is a piece of s... 😄 In any case the solution provided here does not work. |
You should add the charset to the lines containing the value. |
all what i have read on the net says it will not help for microsoft apps. |
You're right. I just tested it on the latest outlook with win 10. Doesn't work. Seems that microsoft doesn't obey standards again. Vcard version 3.0 requires the values to be in UTF-8. |
Maybe it would be useful to build a switch that allows to export in different codesets depending on the browser's OS. |
Not so simple:
|
In any case the view |
Sorry, but based on the feedback by @infograf768 and that this seems to be a bug in Outlook, I think this PR should be closed. I'm not using Outlook on my Windows system and I'd rather have standards-compliant output than account for one program. (Even though it is admittedly the one with the largest distribution) |
From what I understand it now doesn't work anywhere with UTF-8 characters, at least with the patch it works for people using a Mac? Is that correct? If so, I believe we should more this forward otherwise we may as well close it. This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7938. |
No. @infograf768 said it worked on a Mac without this patch This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7938. |
Closing this PR because it can't solve the issue. Thank you for your contribution. This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7938. |
Steps to reproduce the issue
Put in some contact information with ß ä ö ü inside and enable the Option .vcf for the contact view.
Download the vcf
Expected result
The vcf should contain the umlauts.
Actual result
On Apple devices the umlauts are shown correctly but but not in outlook on windows.
System information (as much as possible)
Additional comments
I added utf8decoding in the code, that fixes the problem. As i´m not a developer please review also the way how i fixed it.
This fixes issue: #7937