Download & Extend

Markup escaped in modules description on feature overview page

Project:Features
Version:7.x-1.x-dev
Component:User interface
Category:bug report
Priority:minor
Assigned:Unassigned
Status:needs review

Issue Summary

In features_admin_form() some markup is actually injected into the description to add information on conflicts, dependencies and requirements, however the description is later being outputted using check_plain() which escapes the markup and just looks weird. Simple patch on its way.

AttachmentSizeStatusTest resultOperations
Screen Shot 2013-01-10 at 5.08.42 PM.png109.92 KBIgnoredNoneNone

Comments

#1

Status:active» needs review

Since the information comes from a module's .info file its safe to avoid running check_plain() on it IMO.

AttachmentSizeStatusTest resultOperations
features-escape-markup-1885482-1.patch1004 bytesIgnoredNoneNone

#2

After looking at the 2.x branch I realised I had missed two more unneeded check_plains() in the string substitutions from l()

AttachmentSizeStatusTest resultOperations
features-escape-markup-1885482-2.patch1.86 KBIgnoredNoneNone