I reached out in slack #webform channel about this issue and was asked to report as a possible bug.

The issue is with exporting submissions to csv. Some of the characters are ansi encoded, like the apostrophe showing up ' in the csv.

Below is a simple Yaml snapshot of an example webform ( Make note of the "Master's Student" student level option.)
I attached an image of what I see in MS Excel.

This was generated using just the webform/webform ui module enabled aside from the standard modules via https://simplytest.me/

name:
  '#type': name
  '#title': Name
  '#title__access': false
  '#suffix__access': false
  '#degree__access': false
student_level:
  '#type': select
  '#title': 'Student Level'
  '#options':
    'Undergraduate Student': 'Undergraduate Student'
    'Post-Doctorate Student': 'Post-Doctorate Student'
    'Master''s Student': 'Master''s Student'
    'PhD Student': 'PhD Student'
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

asierrajr created an issue. See original summary.

jrockowitz’s picture

The attached form replicates the issue.

jrockowitz’s picture

Status: Active » Needs review
FileSize
5.36 KB

Attached patch should fix the CSV but may cause regressions elsewhere, which will hopefully be caught by the tests.

  • jrockowitz committed 27b3b6f on 2930089-csv-export
    Issue #2930089 by jrockowitz, asierrajr: CSV Export ASCII Formatted Data
    

Status: Needs review » Needs work

The last submitted patch, 3: 2930089-3.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

  • jrockowitz committed 6d2f00a on 2930089-csv-export
    Issue #2930089 by jrockowitz, asierrajr: CSV Export ASCII Formatted Data
    
jrockowitz’s picture

Status: Needs work » Needs review
FileSize
0 bytes

Status: Needs review » Needs work

The last submitted patch, 7: 2930089-6.patch, failed testing. View results

jrockowitz’s picture

Status: Needs work » Needs review
FileSize
10.16 KB

Status: Needs review » Needs work

The last submitted patch, 9: 2930089-9.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

jrockowitz’s picture

Status: Needs work » Needs review
FileSize
12.57 KB

  • jrockowitz committed 6f44c5c on 2930089-csv-export
    Issue #2930089 by jrockowitz, asierrajr: CSV Export ASCII Formatted Data
    

  • jrockowitz committed 76112df on 8.x-5.x
    Issue #2930089 by jrockowitz, asierrajr: CSV Export ASCII Formatted Data
    

  • jrockowitz committed 8a052b7 on 8.x-5.x
    Revert "Issue #2930089 by jrockowitz, asierrajr: CSV Export ASCII...

  • jrockowitz committed 7ca913c on 8.x-5.x
    Issue #2930089 by jrockowitz, asierrajr: CSV Export ASCII Formatted Data
    
jrockowitz’s picture

Status: Needs review » Fixed

I committed the patch. Please download the latest dev release to review.

Status: Fixed » Closed (fixed)

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