importer UI for CSVs should quote column names that contain commas

Sounds like it's easy to solve. Tagging as a novice issue.

Patch adds Quotes for target column name that contains comma.

When I have a mapping source called "eats, shoots, and leaves" (manually entered on the mapping form) I still see the following when I go to /import/%importer_id:

foo, eats, shoots, and leaves, bar

I think the change need to be made in /plugins/FeedsCSVParser.inc.

Sorry I miss understood the OP. Thanks for guiding me.

Tested and works okay. Tiny coding standard issues, though:

• "false" should be in uppercase:
  

  Constants should always be all-uppercase, with underscores to separate words. (This includes pre-defined PHP constants like TRUE, FALSE, and NULL.)

  https://www.drupal.org/coding-standards#naming

• There should always be spaces between commas and each parameter in function calls.
  https://www.drupal.org/coding-standards#functcall

Otherwise good.

@MegaChriz: Thanks for the review. I made the recommended changes

Good. I see no more issues with this patch. RTBC.

This should check for $this->config['delimiter'];, rather than hard code the delimiter.

I think I am not hard coding delimiter here. Just adding quotes to the column names which are comma separated.

@twistor I thought at first too that something with the delimiter needed to be done, but that isn't the case. It is only the text in the UI that tells which columns should be used in the CSV and the UI text always separates the column names with comma's, regardless of the delimiter. This issue is about fixing the UI text in case comma's are used in column names.
See also the attached image:

Back to RTBC.

Works for me, thanks!

There's still going to be some oddness, if the column contains a quote character, but that should be rare.