Message6316
 
 
 
 
  | I wrote it as an extension (local_markdown).
For the translation from markdown to HTML I use the commonmark library.
A way to include 'local_markdown' in Roundup is to call it from
'local_replace'.
The call must come after the replacement of 'local_replace' because
'local_markdown' could disturb the regular expressions.
It should also be noted that the strings to be replaced by
'local_replace' can now start and end with '*' or '_' to highlight them
with Markdown.
diff -r 1025affaa63a local_replace.py
--- a/local_replace.py  Thu Jan 10 17:01:30 2019 +0100
+++ b/local_replace.py  Thu Jan 10 17:02:30 2019 +0100
@@ -1,4 +1,5 @@
 import re
+import local_markdown
 
 hg_url_base = r'http://sourceforge.net/p/roundup/code/ci/'
 
@@ -35,6 +36,8 @@
 def local_replace(message):
     for cre, replacement in substitutions:
         message = cre.sub(replacement, message)
+
+    message = local_markdown.local_markdown(message)
     return message
 
 def init(instance):
The final implementation should check if 'local_markdown' can be
imported or not to start Roundup without 'local_markdown'.
The advantage of this is that the templates do not have to be edited.
The disadvantage is that 'local_replace' must be edited and it only
works with 'local_replace'.
The other possibility is to call 'local_markdown' in the templates, like
'local_replace').
But for this all templates have to be edited and I haven't found a good
way to implement the call there yet.
Whether 'local_markdown' is technically safe (executable code) still has
to be checked. |  |
 
| Date | User | Action | Args |  | 2019-01-10 18:45:56 | mschieder | set | messageid: <1547145956.07.0.476198900875.issue2550856@roundup.psfhosted.org> |  | 2019-01-10 18:45:56 | mschieder | set | recipients:
  + mschieder, ber, rouilj, ThomasAH, ncoghlan |  | 2019-01-10 18:45:56 | mschieder | link | issue2550856 messages |  | 2019-01-10 18:45:55 | mschieder | create |  | 
 |