The CSV display has an option to convert to ASCII (via iconv), but options are limited to this.
It should be possible to use any destination charset and options that iconv supports.

CommentFileSizeAuthor
#5 0001-views_data_export-Issue-1842108-by-axel.rutz-Added-C.patch3.82 KBgeek-merlin
#4 0001-views_data_export-Issue-1842108-by-axel.rutz-Added-C.patch3.82 KBgeek-merlin
#3 0001-views_data_export-Issue-1842108-by-axel.rutz-Added-C.patch2.74 KBgeek-merlin
#2 0001-views_data_export-Issue-1842108-by-axel.rutz-Added-C.patch1.83 KBgeek-merlin
#1 0001-views_data_export-Issue-1842108-by-axel.rutz-Added-C.patch1.83 KBgeek-merlin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

geek-merlin’s picture

geek-merlin
they/omg
Primary language German
Location Freiburg, Germany
CreditAttribution: geek-merlin commented
Status: Active » Needs review
FileSize
0001-views_data_export-Issue-1842108-by-axel.rutz-Added-C.patch1.83 KB

Patch flying in. Does not break views.

geek-merlin’s picture

geek-merlin
they/omg
Primary language German
Location Freiburg, Germany
CreditAttribution: geek-merlin commented
FileSize
0001-views_data_export-Issue-1842108-by-axel.rutz-Added-C.patch1.83 KB

Patch flying in. Does not break existing views.

geek-merlin’s picture

geek-merlin
they/omg
Primary language German
Location Freiburg, Germany
CreditAttribution: geek-merlin commented
FileSize
0001-views_data_export-Issue-1842108-by-axel.rutz-Added-C.patch2.74 KB

URPS, here's the fixed fix.

geek-merlin’s picture

geek-merlin
they/omg
Primary language German
Location Freiburg, Germany
CreditAttribution: geek-merlin commented
FileSize
0001-views_data_export-Issue-1842108-by-axel.rutz-Added-C.patch3.82 KB

Again fixed and improved: As iconv has plenty of gotchas i added the much more robust "utf8_decode" as option.

geek-merlin’s picture

geek-merlin
they/omg
Primary language German
Location Freiburg, Germany
CreditAttribution: geek-merlin commented
FileSize
0001-views_data_export-Issue-1842108-by-axel.rutz-Added-C.patch3.82 KB

We're converging, honestly!
fixed a hopefully last typo in the docs url.

xamanu’s picture

xamanu CreditAttribution: xamanu commented
Status: Needs review » Reviewed & tested by the community

Works fine for me. Thank you!

Steven Jones’s picture

Steven Jones CreditAttribution: Steven Jones commented
Status: Reviewed & tested by the community » Fixed

Thanks so much for the patch, pushed to 7.x-3.x

Steven Jones’s picture

Steven Jones CreditAttribution: Steven Jones commented
Version: 7.x-3.x-dev » 6.x-2.x-dev
Status: Fixed » Patch (to be ported)

Now this needs backporting.

Steven Jones’s picture

Steven Jones CreditAttribution: Steven Jones commented
Status: Patch (to be ported) » Fixed

Backported to 6.x-2.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

link