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.
This is odd - as stated in the title.
I'm displaying a simple CCK type, in this case a title with an external link as an HTML list with both fields inline.
The separator doesn't display. I was able to get around it easily (Views is so cool) by rewriting the output of the URL to be "- [token]" (I only wanted to put a dash between the title and the URL), but this looks like a bug to me. Weird.
Thought I might as well say something - carry on!
Comment | File | Size | Author |
---|---|---|---|
#3 | views-6-2-inline-separator-1101384-3.patch | 638 bytes | osopolar |
#3 | views-6-3-inline-separator-1101384-3.patch | 655 bytes | osopolar |
Comments
Comment #1
dawehnerSadly i can't find the other issue but i'm 100% convinced that there was an issue about this bug already.
Comment #2
franzI also confirm, not related to CCK. I had 4 fields, and made the 2 in the middle inline fields with a '|' as a separator. It doesn't get rendered. In some of the rows, the first field is empty (might be causing this?). Since the second field is always there, I'll put the separator inside the first, as a workaround.
Issue names on views issue queue are not representative, so it's really hard to find stuff. =/
Comment #3
osopolarIn theme.inc's function template_preprocess_views_view_fields() is responsible for setting the separator:
But if the object's content is empty and also if it is the string '0' the separator won't be set. I am not sure, why $object->content is checket here but I guess it should be enough to check isset($object->content). Because also if it is an empty string there should be a separator, because an empty field could for example have a label. If you hide empty fields, than there won't be an $object variable and so no separator. Therefore I guess we can also omit the check for isset($object->content). But on the last assumption I am not sure, so I created the patch with isset - for views-6.x-2.x-dev and views-6.x-3.x-dev.
Comment #4
MustangGB CreditAttribution: MustangGB commented