Maybe it is a german excel problem - but in order to make excel 2007 open this right away, i need to use ; instead of ,

regards
Christian

CommentFileSizeAuthor
#1 views_bonus_export_comma_semicolon.module.patch1.57 KBrobloach

Comments

robloach’s picture

robloach
he/him
commented
Status: Active » Needs review
StatusFileSize
new views_bonus_export_comma_semicolon.module.patch1.57 KB

Would sticking the comma in a t() help? Then you can translate it to a semi-colon instead.

chriszz’s picture

chriszz commented

Well - for me it was easy to just hardcode the problem away - and as i am changing other things as well (utf8-Problem) it can stay this way.

But on the long run - as i do not expect everyone to understand the problem when it is occuring, it should be a well explained management option - maybe there are other clients which prefer different separators.

robloach’s picture

robloach
he/him
commented

An option would be a good idea, but I'm not entirely sure how that works when it comes to Views. dmitrig01 would know more.

dmitrig01’s picture

dmitrig01 commented

Rob: patch looks good enough for a commit.

robloach’s picture

robloach
he/him
commented
Status: Needs review » Fixed

Hopefully that doesn't break anything. Now if you want to use something other than a comma, just use locale to translate it. Does that help?

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)