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.
It would be nice to have a comma separator between multiple values.
Comment | File | Size | Author |
---|---|---|---|
#4 | views_bonus_pack-1829634-add-comma-separator.patch | 921 bytes | texas-bronius |
#1 | separator-1829634-1.patch | 757 bytes | langworthy |
Comments
Comment #1
langworthy CreditAttribution: langworthy commentedComment #2
texas-bronius CreditAttribution: texas-bronius commented+1
Works for me!
Comment #3
texas-bronius CreditAttribution: texas-bronius commentedAlthough that said, it wouldn't be bad to refactor the code a little like:
in order to write out the html out only 1 time and to reduce the call to
count($items)
from 1s per iteration to 1s ever.I've slapped this back from @langworthy's rtbc to needs review, bc I am the only one who has tested my code above.
Comment #4
texas-bronius CreditAttribution: texas-bronius commentedPatch attached that does the above.
Comment #5
Rob T CreditAttribution: Rob T commentedI approached a similar issue using a patch from here for dealing with multiple value fields exported to XLS via Views Bonus Pack. https://drupal.org/node/758692
While it works for my particular need, this approach here as well as the approach provided here https://drupal.org/node/743200 are alternatives of interest.