This has been an "issue" in past version(s). I do believe
use to work but I have tried it here in v7.x-3.0-beta3 with no success. Am I overlooking the simple way to achieve this or is it really a bug?

CommentFileSizeAuthor
#6 1182800-invalid-check-plain-on-separator.patch597 bytesmerlinofchaos

Comments

dawehner’s picture

dawehner
Primary language German
commented

Doesn't <br /> works for you?

dawehner’s picture

dawehner
Primary language German
commented
Status: Active » Postponed (maintainer needs more info)

Update status.

cbergman’s picture

cbergman commented

Nope, <br /> doesn't work for me either. It just places a <br /> as plain text between each of my items :/
Is there any other module(s) I need to ensure I have updated that would affect this at all?

merlinofchaos’s picture

merlinofchaos commented

Separator in what context?

Please be specific and provide details. We can't read your mind or look over your shoulder.

cbergman’s picture

cbergman commented

Oh sorry, it's too early in the morning yet :/

I have a Table of people - their Names, Titles, Phone numbers, etc. I have Views pulling the data from a content type I created with a Name field, a Phone number field, etc. Now in Views, under Fields, I can click on each of the fields (Name, Title, phone number, etc.) and it opens the Configure Field window. Under 'Multiple Field Settings' I have 'Simple Separator' selected and want the separator to be a newline character, opposed to the default comma.

That help?

merlinofchaos’s picture

merlinofchaos commented
Status: Postponed (maintainer needs more info) » Fixed
StatusFileSize
new 1182800-invalid-check-plain-on-separator.patch597 bytes

Yeah, it helps. The separator is being check_plain()ed when it should be filter_xss_admin()'d. This patch will fix it. Which I've committed.

cbergman’s picture

cbergman commented

Thanks! Now I just got to read up on how to apply module patches and I should (hopefully) be good to go. If nothing else, hopefully this fix will be in the next Views update ;)

merlinofchaos’s picture

merlinofchaos commented

Yeah, it'll turn up on the midnight GMT roll of 7.x-dev

cbergman’s picture

cbergman commented

Oh, okay cool. I'll just apply that tomorrow then. Thanks again :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.