The content of the README that Advanced Help shows for this module is in italic, starting from Italic: Encloses the selected text with the tag.

CommentFileSizeAuthor
#6 readme_10.patch1.58 KBfgm
#4 readme_7.patch1.63 KBfgm
#3 readme.patch1.58 KBfgm

Comments

ufku’s picture

ufku commented
Project: BUEditor » Advanced Help
Version: 6.x-1.3 » 6.x-1.x-dev
Component: Documentation » Code

I think this issue belongs to advanced help module. Html tags in readme files should be escaped before printing.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented

For reference, there is currently a @todo for this at line 527 in the module.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Active » Needs review
StatusFileSize
new readme.patch1.58 KB

The problem is the although AH identifies the specific case of README files earlier on in the processing, it does not keep the information when building the cache.

Suggested patch adds this information and check_plains such entries.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
StatusFileSize
new readme_7.patch1.63 KB

And matching patch for D7

dmitrig01’s picture

dmitrig01 commented
Version: 7.x-1.x-dev » 6.x-1.x-dev

Committed the d7 version, thanks.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
StatusFileSize
new readme_10.patch1.58 KB

D6 patch. I just tested it again on the latest version.

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Issue summary: View changes
Status: Needs review » Closed (cannot reproduce)

I am unable to reproduce the problem with the latest release.

Patch is four years old and do not longer apply. Closing for now.

Feel free to repopen if you still think problem persists in latest version.