Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I'm not getting Commas out in the CSV reports of the Dev version. I've checked a few different configurations and it happens for all cases.
Comment | File | Size | Author |
---|---|---|---|
#8 | views_bonus_export_1.patch | 663 bytes | Max_Headroom |
Comments
Comment #1
scarr CreditAttribution: scarr commentedThis is happening to me as well - if I choose other delimiters, none get added. I've even made copies of existing, working views feeds, and they don't show the delimiter.
Comment #2
mcaden CreditAttribution: mcaden commentedsame
Comment #3
okletsgo CreditAttribution: okletsgo commentedsubscribed
Comment #4
4kant CreditAttribution: 4kant commentedsubscribed
Comment #5
mcaden CreditAttribution: mcaden commentedviews_bonus_export.theme.inc: line 17
change from "if(isset..." to "if(!isset...".
It should look like this:
Without this the logic reads "If the variable spelled correctly is set, change it to be the old, incorrectly spelled variable".
Thus, it was taking a perfectly good value and setting it to null before sending it to the theming function which was calling implode with null as a field delimiter.
Comment #6
mcaden CreditAttribution: mcaden commentedComment #7
Nick Fedchik CreditAttribution: Nick Fedchik commentedAnother way:
In first line I change ['separator'] to ['seperator'], and fix the problem
Comment #8
Max_Headroom CreditAttribution: Max_Headroom commentedPatch for #7. hopefully this will now be committed.
Comment #9
mcaden CreditAttribution: mcaden commentedMinor detail - this gives priority to the old, misspelled value. If for some reason both values are set, the misspelled one will overwrite the newer, correctly spelled variable. This is, I think, contrary to what is intended by those code which is to mean if the new variable isn't set appropriately, fall back to the old.
Just a minor detail that probably shouldn't affect anything (hmm...famous last words?)
Comment #10
neclimdul