I know this is nit-picking but I suggest fixing the spelling of this variable, and associated documentation.

CommentFileSizeAuthor
views_bonus.spelling.patch4.25 KBmvc
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

neclimdul’s picture

neclimdul
He/Him
CreditAttribution: neclimdul commented
Status: Active » Needs work

The variable name change will break existing settings.

neclimdul’s picture

neclimdul
He/Him
CreditAttribution: neclimdul commented
Status: Needs work » Fixed

Committed modified version supporting old saved options and templates.

arski’s picture

arski CreditAttribution: arski commented
Status: Fixed » Active

hmm, I tried out the dev version and my CSV files have no separators in them, no matter which one I pick. When I went back to 1.1 they worked fine.. assuming it has something to do with this? maybe you missed one?

michaelfavia’s picture

michaelfavia CreditAttribution: michaelfavia commented

Confirmed separator loss with this patch. Fix is here: #890544: [PATCH] Separator not showing on feed WFM