Fix annotations for transforms

1 files changed, 21 insertions, 20 deletions

diff --git a/sphinx/transforms/post_transforms/compat.py b/sphinx/transforms/post_transforms/compat.py
index 144b80e85..4f387449c 100644
--- a/sphinx/transforms/post_transforms/compat.py
+++ b/sphinx/transforms/post_transforms/compat.py
@@ -43,14 +43,14 @@ class MathNodeMigrator(SphinxTransform):
 
     def apply(self, **kwargs):
         # type: (Any) -> None
-        for node in self.document.traverse(nodes.math):
+        for math_node in self.document.traverse(nodes.math):
             # case: old styled ``math`` node generated by old extensions
-            if len(node) == 0:
+            if len(math_node) == 0:
                 warnings.warn("math node for Sphinx was replaced by docutils'. "
                               "Please use ``docutils.nodes.math`` instead.",
                               RemovedInSphinx30Warning)
-                equation = node['latex']
-                node += nodes.Text(equation, equation)
+                equation = math_node['latex']
+                math_node += nodes.Text(equation, equation)
 
         translator = self.app.builder.get_translator_class()
         if hasattr(translator, 'visit_displaymath') and translator != XMLTranslator:
@@ -58,30 +58,31 @@ class MathNodeMigrator(SphinxTransform):
             warnings.warn("Translator for %s does not support math_block node'. "
                           "Please update your extension." % translator,
                           RemovedInSphinx30Warning)
-            for node in self.document.traverse(math_block):
-                alt = displaymath(latex=node.astext(),
-                                  number=node.get('number'),
-                                  label=node.get('label'),
-                                  nowrap=node.get('nowrap'),
-                                  docname=node.get('docname'))
-                node.replace(alt)
+            for math_block_node in self.document.traverse(math_block):
+                alt = displaymath(latex=math_block_node.astext(),
+                                  number=math_block_node.get('number'),
+                                  label=math_block_node.get('label'),
+                                  nowrap=math_block_node.get('nowrap'),
+                                  docname=math_block_node.get('docname'))
+                math_block_node.replace(alt)
         elif getattr(self.app.builder, 'math_renderer_name', None) == 'unknown':
             # case: math extension provides old styled math renderer
-            for node in self.document.traverse(nodes.math_block):
-                node['latex'] = node.astext()
+            for math_block_node in self.document.traverse(nodes.math_block):
+                math_block_node['latex'] = math_block_node.astext()
 
         # case: old styled ``displaymath`` node generated by old extensions
-        for node in self.document.traverse(math_block):
-            if len(node) == 0:
+        for math_block_node in self.document.traverse(math_block):
+            if len(math_block_node) == 0:
                 warnings.warn("math node for Sphinx was replaced by docutils'. "
                               "Please use ``docutils.nodes.math_block`` instead.",
                               RemovedInSphinx30Warning)
-                if isinstance(node, displaymath):
-                    newnode = nodes.math_block('', node['latex'], **node.attributes)
-                    node.replace_self(newnode)
+                if isinstance(math_block_node, displaymath):
+                    newnode = nodes.math_block('', math_block_node['latex'],
+                                               **math_block_node.attributes)
+                    math_block_node.replace_self(newnode)
                 else:
-                    latex = node['latex']
-                    node += nodes.Text(latex, latex)
+                    latex = math_block_node['latex']
+                    math_block_node += nodes.Text(latex, latex)
 
 
 def setup(app):