Extensions cleanup (and resurrection of html_domain)

conf.py

@@ -99,6 +99,12 @@ extensions = [
 
     # Redirection generator
     'redirects',
+
+    # Code switcher (switcher and case directives)
+    'switcher',
+
+    # Strange html domain logic used in memento pages
+    'html_domain',
 ]
 
 todo_include_todos = False

extensions/html_domain/__init__.py

@@ -0,0 +1,180 @@
+# -*- coding: utf-8 -*-
+
+"""
+Defines a "raw HTML" domain with a ``div[classes]`` and a number of roles
+rendered more or less directly to HTML.
+
+.. warning::
+
+    the purpose of this domain is *not* to document HTML or components
+
+NOTE: this extension is only used by special accounting pages (memento, valuation, ...)
+for directives likes .. h:div::
+"""
+
+from docutils import nodes, utils
+from docutils.parsers.rst import Directive, directives
+from docutils.parsers.rst.directives.body import LineBlock
+
+import sphinx
+import sphinx.roles
+from sphinx.domains import Domain
+
+
+def setup(app):
+    app.add_domain(HtmlDomain)
+    app.add_node(div, html=(
+        lambda self, node: self.body.append(self.starttag(node, 'div')),
+        lambda self, node: self.body.append('</div>\n')))
+    # override parameter was added for version sphinx >= 1.4, TypeError before
+    kw = {'override': True} if sphinx.version_info >= (1, 4) else {}
+    app.add_node(address, html=(
+        lambda self, node: self.body.append(self.starttag(node, 'address')),
+        lambda self, node: self.body.append('</address>\n')
+    ), **kw)  # docutils.nodes.address exists and is a bibliographic element
+    app.add_node(cite, html=(visit_cite, depart_cite))
+    for name, node in [('mark', mark), ('ins', insert), ('del', delete),
+                       ('s', strikethrough), ('u', underline), ('small', small),
+                       ('kbd', kbd), ('var', var), ('samp', samp)]:
+        addnode(app, node, name)
+
+
+class div(nodes.General, nodes.Element):
+    pass
+
+
+class Div(Directive):
+    optional_arguments = 1
+    final_argument_whitespace = 1
+    has_content = True
+
+    def run(self):
+        self.assert_has_content()
+        text = '\n'.join(self.content)
+        try:
+            if self.arguments:
+                classes = directives.class_option(self.arguments[0])
+            else:
+                classes = []
+        except ValueError:
+            raise self.error(
+                'Invalid class attribute value for "%s" directive: "%s".'
+                % (self.name, self.arguments[0]))
+        node = div(text)
+        node['classes'].extend(classes)
+        self.add_name(node)
+        self.state.nested_parse(self.content, self.content_offset, node)
+        return [node]
+
+
+class address(nodes.General, nodes.Element):
+    pass
+
+
+class Address(LineBlock):
+    def run(self):
+        [node] = super(Address, self).run()
+        ad = address(node.rawsource, *node.children)
+        return [ad]
+
+
+class mark(nodes.Inline, nodes.TextElement):
+    pass
+
+
+class insert(nodes.Inline, nodes.TextElement):
+    pass
+
+
+class delete(nodes.Inline, nodes.TextElement):
+    pass
+
+
+class strikethrough(nodes.Inline, nodes.TextElement):
+    pass
+
+
+class underline(nodes.Inline, nodes.TextElement):
+    pass
+
+
+class small(nodes.Inline, nodes.TextElement):
+    pass
+
+
+class kbd(nodes.Inline, nodes.FixedTextElement):
+    pass
+
+
+class var(nodes.Inline, nodes.FixedTextElement):
+    pass
+
+
+class samp(nodes.Inline, nodes.FixedTextElement):
+    pass
+
+
+def makerole(node):
+    return lambda name, rawtext, text, lineno, inliner, options=None, content=None:\
+        ([node(rawtext.strip(), text.strip())], [])
+
+
+def addnode(app, node, nodename):
+    app.add_node(node, html=(
+        lambda self, n: self.body.append(self.starttag(n, nodename)),
+        lambda self, n: self.body.append('</%s>' % nodename)
+    ))
+
+
+def initialism(*args, **kwargs):
+    nodes, _ = sphinx.roles.abbr_role(*args, **kwargs)
+    [abbr] = nodes
+    abbr.attributes.setdefault('classes', []).append('initialism')
+    return [abbr], []
+
+
+def cite_role(typ, rawtext, text, lineno, inliner, options=None, content=None):
+    text = utils.unescape(text)
+    m = sphinx.roles._abbr_re.search(text)
+    if m is None:
+        return [cite(text, text, **(options or {}))], []
+    content = text[:m.start()].strip()
+    source = m.group(1)
+    return [cite(content, content, source=source)], []
+
+
+class cite(nodes.Inline, nodes.TextElement):
+    pass
+
+
+def visit_cite(self, node):
+    attrs = {}
+    if node.hasattr('source'):
+        attrs['title'] = node['source']
+    self.body.append(self.starttag(node, 'cite', '', **attrs))
+
+
+def depart_cite(self, node):
+    self.body.append('</abbr>')
+
+
+class HtmlDomain(Domain):
+    name = 'h'
+    label = 'HTML'
+    directives = {
+        'div': Div,
+        'address': Address,
+    }
+    roles = {
+        'mark': makerole(mark),
+        'ins': makerole(insert),
+        'del': makerole(delete),
+        's': makerole(strikethrough),
+        'u': makerole(underline),
+        'small': makerole(small),
+        'initialism': initialism,
+        'cite': cite_role,
+        'kbd': makerole(kbd),
+        'var': makerole(var),
+        'samp': makerole(samp),
+    }

extensions/odoo_theme/__init__.py

@@ -1,5 +1,4 @@
 from . import pygments_override
-from . import switcher
 from . import translator
 
 import sphinx.builders.html
@@ -11,8 +10,6 @@ from docutils import nodes
 def setup(app):
     app.set_translator('html', translator.BootstrapTranslator)
 
-    # FIXME ANVFE Separate extensions and clean import in conf.py ???
-    switcher.setup(app)
     # VFE TODO check if default meta initialization is necessary.
     # If not, remove update_meta method
     app.connect('html-page-context', update_meta)
@@ -23,6 +20,8 @@ def update_meta(app, pagename, templatename, context, doctree):
     if meta is None:
         meta = context['meta'] = {}
 
+# TODO VFE detailed explanation of the patch logic and use.
+
 class monkey(object):
     def __init__(self, obj):
         self.obj = obj

extensions/switcher/__init__.py

@@ -3,11 +3,11 @@ from docutils.parsers.rst import Directive
 
 from pygments.lexers import get_lexer_by_name
 
+
 def setup(app):
     app.add_directive('switcher', SwitcherDirective)
     app.add_directive('case', CaseDirective)
 
-# VFE FIXME separate this extension from odoo_theme (and clean?)
 
 class SwitcherDirective(Directive):
     has_content = True
@@ -23,7 +23,7 @@ class SwitcherDirective(Directive):
             if isinstance(child, nodes.literal_block):
                 titles.append(get_lexer_by_name(child['language']).name)
             else:
-                assert child['names'], ("A switcher case must be either a "\
+                assert child['names'], ("A switcher case must be either a "
                                         "code block or a compound with a name")
                 titles.append(' '.join(child['names']))
         tabs = nodes.bullet_list('', *[
@@ -33,6 +33,7 @@ class SwitcherDirective(Directive):
         node = nodes.compound('', tabs, body, classes=['content-switcher'])
         return [node]
 
+
 class CaseDirective(Directive):
     required_arguments = 1
     final_argument_whitespace = True
@@ -41,6 +42,7 @@ class CaseDirective(Directive):
     def run(self):
         self.assert_has_content()
 
-        node = nodes.compound('\n'.join(self.content), names=[self.arguments[0]])
+        node = nodes.compound('\n'.join(self.content),
+                              names=[self.arguments[0]])
         self.state.nested_parse(self.content, self.content_offset, node)
         return [node]