Opened 5 years ago
Closed 5 years ago
#13961 closed defect (fixed)
Incompatibility with Trac 1.4.2
| Reported by: | Owned by: | Ryan J Ollos | |
|---|---|---|---|
| Priority: | normal | Component: | MarkdownMacro |
| Severity: | normal | Keywords: | |
| Cc: | Trac Release: | 1.4 |
Description (last modified by )
I recently installed Trac 1.4.2 and MarkdownMacro from https://trac-hacks.org/browser/markdownmacro?rev=18008&format=zip but MarkdownMacro didn't work. I traced the failures to two issues with the plugin installed to plugins of my project:
1) The installation instructions refer to component tracmarkdown.macro.markdowneverywhere, but that did nothing for me. What worked for me was tracmarkdown.macro.markdownformatter. With the latter I was able to disable processing of markdown everywhere, leaving just the macro processor.
2) Processing markdown everywhere failed, resulting in blank pages, when markdown processing everywhere was enabled. I traced this to the tab_length argument to the markdown function. It was u'4', which was an incompatible type. Changing it to 4 worked and changing the call to set tab_length in format_to_markdown from env.config.get to env.config.getint returned 4 instead of u'4', which also worked.
I was working with a fresh install of Python 2.7.18 installed by asdf, in case that makes a difference, though it appears that env.config is Trac code and the get method of the Configuration class appears to be documented to return a string, though I don't know the code well and might be misunderstanding.
Attachments (0)
Change History (5)
comment:1 Changed 5 years ago by
| Component: | SELECT A HACK → MarkdownMacro |
|---|---|
| Trac Release: | → 1.4 |
comment:2 Changed 5 years ago by
| Owner: | set to Ryan J Ollos |
|---|---|
| Status: | new → accepted |
comment:3 Changed 5 years ago by
| Description: | modified (diff) |
|---|
- See revisions in MarkdownMacro@29.
- Good find. I will fix that.
comment:5 Changed 5 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | accepted → closed |


