Added json_render method to separate filtering from view (#14024)

airflow/www/utils.py

@@ -321,6 +321,17 @@ def render(obj, lexer):
     return out
 
 
+def json_render(obj, lexer):
+    """Render a given Python object with json lexer"""
+    out = ""
+    if isinstance(obj, str):
+        out = Markup(pygment_html_render(obj, lexer))
+    elif isinstance(obj, (dict, list)):
+        content = json.dumps(obj, sort_keys=True, indent=4)
+        out = Markup(pygment_html_render(content, lexer))
+    return out
+
+
 def wrapped_markdown(s, css_class=None):
     """Convert a Markdown string to HTML."""
     if s is None:
@@ -343,7 +354,7 @@ def get_attr_renderer():
         'doc_rst': lambda x: render(x, lexers.RstLexer),
         'doc_yaml': lambda x: render(x, lexers.YamlLexer),
         'doc_md': wrapped_markdown,
-        'json': lambda x: render(x, lexers.JsonLexer),
+        'json': lambda x: json_render(x, lexers.JsonLexer),
         'md': wrapped_markdown,
         'py': lambda x: render(get_python_source(x), lexers.PythonLexer),
         'python_callable': lambda x: render(get_python_source(x), lexers.PythonLexer),

airflow/www/views.py

@@ -910,8 +910,6 @@ def rendered_templates(self):
             content = getattr(task, template_field)
             renderer = task.template_fields_renderers.get(template_field, template_field)
             if renderer in renderers:
-                if isinstance(content, (dict, list)):
-                    content = json.dumps(content, sort_keys=True, indent=4)
                 html_dict[template_field] = renderers[renderer](content)
             else:
                 html_dict[template_field] = Markup("<pre><code>{}</pre></code>").format(