Opened 13 years ago
Closed 12 years ago
#17714 closed Bug (wontfix)
Markdown filter does not support multiple configuration options for extensions
Reported by: | Scott McGinness | Owned by: | nobody |
---|---|---|---|
Component: | contrib.markup | Version: | dev |
Severity: | Normal | Keywords: | markup, markdown |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | yes | UI/UX: | no |
Description
Attempting to use a markdown extension with multiple options does not work. For example, using
{{ my_value|markdown:"safe,headerid(forceid=False,level=3),def_list" }}
within a template results in an exception with message
Extension "__builtin__.NoneType" must be of type: "markdown.Extension".
A patch for this is attached
Attachments (2)
Change History (6)
by , 13 years ago
Attachment: | 17714_markdown_extensions_multiple_options.diff added |
---|
comment:1 by , 13 years ago
Needs tests: | set |
---|---|
Triage Stage: | Unreviewed → Accepted |
by , 13 years ago
Attachment: | 17714_markdown_extensions_multiple_options_with_tests.diff added |
---|
Updated original patch with Markdown extension tests
comment:2 by , 13 years ago
I have added an updated patch to test markdown extensions.
I was not sure which markdown extensions, if any, were supported. However I have used def_list and headerid as a starting point.
comment:3 by , 13 years ago
Needs tests: | unset |
---|---|
Triage Stage: | Accepted → Ready for checkin |
Looks good to me
comment:4 by , 12 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Marking as won't fix since markup has been deprecated.
Regression tests are needed.