Closed (outdated)
Project:
Views Bonus Pack
Version:
6.x-1.x-dev
Component:
Views Export
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Mar 2011 at 19:33 UTC
Updated:
8 Mar 2023 at 20:28 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
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 commentedsame
Comment #3
okletsgo commentedsubscribed
Comment #4
4kant commentedsubscribed
Comment #5
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 commentedComment #7
Nick Fedchik commentedAnother way:
In first line I change ['separator'] to ['seperator'], and fix the problem
Comment #8
Max_Headroom commentedPatch for #7. hopefully this will now be committed.
Comment #9
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