removing unused functions and unused latex handling

Author: AakashGfude

sphinx_exercise/nodes.py

@@ -12,7 +12,6 @@
 from docutils import nodes as docutil_nodes
 from sphinx import addnodes as sphinx_nodes
 from sphinx.writers.latex import LaTeXTranslator
-from .utils import get_node_number, list_rindex
 from .latex import LaTeXMarkup
 
 logger = logging.getLogger(__name__)
@@ -89,17 +88,6 @@ def is_extension_node(node):
 
 # Visit and Depart Functions
 
-# TODO: Review _depart_nodes_latex to see if these pop methods
-# are really needed.
-
-
-def _depart_nodes_latex(self, node, title, pop_index=False):
-    """ Function to handle depart_node for latex. """
-    idx = list_rindex(self.body, LaTeX.visit_admonition()) + 2
-    if pop_index:
-        self.body.pop(idx)
-    self.body.append(LaTeX.depart_admonition())
-
 
 def visit_exercise_node(self, node: Node) -> None:
     if isinstance(self, LaTeXTranslator):
@@ -114,10 +102,8 @@ def visit_exercise_node(self, node: Node) -> None:
 
 
 def depart_exercise_node(self, node: Node) -> None:
-    typ = node.attributes.get("type", "")
     if isinstance(self, LaTeXTranslator):
-
-        _depart_nodes_latex(self, node, f"{typ.title()} ")
+        self.body.append(LaTeX.depart_admonition())
     else:
         self.body.append("</div>")
 
@@ -139,10 +125,8 @@ def visit_exercise_enumerable_node(self, node: Node) -> None:
 
 
 def depart_exercise_enumerable_node(self, node: Node) -> None:
-    typ = node.attributes.get("type", "")
     if isinstance(self, LaTeXTranslator):
-        number = get_node_number(self, node, typ)
-        _depart_nodes_latex(self, node, f"{typ.title()} {number} ")
+        self.body.append(LaTeX.depart_admonition())
     else:
         self.body.append("</div>")
         self.body.append("\n")
@@ -169,10 +153,8 @@ def visit_solution_node(self, node: Node) -> None:
 
 
 def depart_solution_node(self, node: Node) -> None:
-    typ = node.attributes.get("type", "")
     if isinstance(self, LaTeXTranslator):
-        number = get_node_number(self, node, typ)
-        _depart_nodes_latex(self, node, f"{typ.title()} {number} ")
+        self.body.append(LaTeX.depart_admonition())
     else:
         self.body.append("</div>")
         self.body.append("\n")

sphinx_exercise/utils.py

@@ -1,7 +1,6 @@
 # Utility functions
 
 from sphinx.writers.latex import LaTeXTranslator
-from docutils import nodes as docutil_nodes
 
 
 def find_parent(env, node, parent_tag):
@@ -37,34 +36,3 @@ def get_node_number(self, node, typ) -> str:
     fignumbers = self.builder.env.toc_fignumbers.get(docname, {})
     number = fignumbers.get(typ, {}).get(ids, ())
     return ".".join(map(str, number))
-
-
-def has_math_child(node):
-    """ Check if a parent node as a math child node. """
-
-    for item in node:
-        if isinstance(item, docutil_nodes.math):
-            return True
-    return False
-
-
-def get_refuri(node):
-    """ Check both refuri and refid, to see which one is available. """
-
-    id_ = ""
-    if node.get("refuri", ""):
-        id_ = node.get("refuri", "")
-
-    if node.get("refid", ""):
-        id_ = node.get("refid", "")
-
-    return id_.split("#")[-1]
-
-
-def list_rindex(li, x) -> int:
-    """Getting the last occurence of an item in a list."""
-
-    for i in reversed(range(len(li))):
-        if li[i] == x:
-            return i
-    raise ValueError("{} is not in list".format(x))