Closed (duplicate)
Project:
Legal
Version:
7.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
19 Dec 2011 at 12:02 UTC
Updated:
2 Jun 2015 at 03:55 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jim005 commented+1
Comment #2
mcfilms commentedYes please.
Comment #3
marty.true commentedI was able to manipulate this but creating a separate page, using a Full HTML input filter, and after saving it, I went back to edit screen of that page, clicked "Source" and copied the HTML output and then went to the Legal module and pasted that in the textarea... It does render in HTML if the tags are in there.
But I agree, the module should utilize Drupal's input filters hook for that textarea...
Comment #4
lindsayo commented@xstatic, I did the same thing which worked for authenticated viewing, but still am finding that the anonymous rendering of /legal is completely unformatted. I'm using 6.x-8.5, but this seems to be a universal issue?
Comment #5
anybodyThe problem still exists and is a clear bug, because the output is different from other displays. All linebreaks are missing, etc. that should NOT be the case.
Clear bug which makes this display style nearly impossible to use without hacking!
Comment #6
anybodyThe responsible code can be found in "legal.pages.inc" in these lines:
If the only problem are missing linebreaks, this could be fixed by simply adding an "nl2br()".
Comment #7
anybodyThis should be definitely fixed in the next stable release from my point of view, because it's a clear bug and can't create more problems than already existing. The output is now identical to the Scroll box. Who's able to make that change?
Comment #8
anybodyIs there still an active maintainer in this module? Otherwise I'd like to offer my help to bring the changes in...
Comment #9
robert castelo commentedYes, actively maintained. Please provide a patch for review.
Comment #10
anybodyPatch attached :)
Comment #11
anybodyComment #12
geek-merlinAfaict this code is already included in #445308: Add support of text formats and WYSIWYG.
Comment #13
geek-merlin