Opened 12 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 , 12 years ago
| Attachment: | 17714_markdown_extensions_multiple_options.diff added |
|---|
comment:1 by , 12 years ago
| Needs tests: | set |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
by , 12 years ago
| Attachment: | 17714_markdown_extensions_multiple_options_with_tests.diff added |
|---|
Updated original patch with Markdown extension tests
comment:2 by , 12 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 , 12 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.
Note:
See TracTickets
for help on using tickets.
Regression tests are needed.