std domain: Generate node_id for productionlists in the right way

sphinx-doc · Mar 3, 2020 · f58c154 · f58c154
1 parent fa40ea0
commit f58c154
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 28 deletions.
diff --git a/sphinx/domains/std.py b/sphinx/domains/std.py
@@ -488,13 +488,9 @@ def run(self) -> List[Node]:
             subnode = addnodes.production(rule)
             subnode['tokenname'] = name.strip()
             if subnode['tokenname']:
-                # nodes.make_id converts '_' to '-',
-                # so we can use '_' to delimit group from name,
-                # and make sure we don't clash with other IDs.
-                idname = 'grammar-token-%s_%s' \
-                         % (nodes.make_id(productionGroup), nodes.make_id(name))
-                if idname not in self.state.document.ids:
-                    subnode['ids'].append(idname)
+                prefix = 'grammar-token-%s' % productionGroup
+                node_id = make_id(self.env, self.state.document, prefix, name)
+                subnode['ids'].append(node_id)
 
                 idnameOld = nodes.make_id('grammar-token-' + name)
                 if idnameOld not in self.state.document.ids:
@@ -504,8 +500,7 @@ def run(self) -> List[Node]:
                     objName = "%s:%s" % (productionGroup, name)
                 else:
                     objName = name
-                domain.note_object(objtype='token', name=objName, labelid=idname,
-                                   location=node)
+                domain.note_object('token', objName, node_id, location=node)
             subnode.extend(token_xrefs(tokens, productionGroup))
             node.append(subnode)
         return [node]

diff --git a/tests/test_build_html.py b/tests/test_build_html.py
@@ -222,7 +222,7 @@ def test_html4_output(app, status, warning):
          "[@class='reference internal']/code/span[@class='pre']", 'HOME'),
         (".//a[@href='#with']"
          "[@class='reference internal']/code/span[@class='pre']", '^with$'),
-        (".//a[@href='#grammar-token-_try-stmt']"
+        (".//a[@href='#grammar-token-try-stmt']"
          "[@class='reference internal']/code/span", '^statement$'),
         (".//a[@href='#some-label'][@class='reference internal']/span", '^here$'),
         (".//a[@href='#some-label'][@class='reference internal']/span", '^there$'),
@@ -254,7 +254,7 @@ def test_html4_output(app, status, warning):
         (".//dl/dt[@id='term-boson']", 'boson'),
         # a production list
         (".//pre/strong", 'try_stmt'),
-        (".//pre/a[@href='#grammar-token-_try1-stmt']/code/span", 'try1_stmt'),
+        (".//pre/a[@href='#grammar-token-try1-stmt']/code/span", 'try1_stmt'),
         # tests for ``only`` directive
         (".//p", 'A global substitution.'),
         (".//p", 'In HTML.'),

diff --git a/tests/test_domain_std.py b/tests/test_domain_std.py
@@ -352,23 +352,23 @@ def test_productionlist(app, status, warning):
         linkText = span.text.strip()
         cases.append((text, link, linkText))
     assert cases == [
-        ('A', 'Bare.html#grammar-token-_a', 'A'),
-        ('B', 'Bare.html#grammar-token-_b', 'B'),
-        ('P1:A', 'P1.html#grammar-token-p1_a', 'P1:A'),
-        ('P1:B', 'P1.html#grammar-token-p1_b', 'P1:B'),
-        ('P2:A', 'P1.html#grammar-token-p1_a', 'P1:A'),
-        ('P2:B', 'P2.html#grammar-token-p2_b', 'P2:B'),
-        ('Explicit title A, plain', 'Bare.html#grammar-token-_a', 'MyTitle'),
-        ('Explicit title A, colon', 'Bare.html#grammar-token-_a', 'My:Title'),
-        ('Explicit title P1:A, plain', 'P1.html#grammar-token-p1_a', 'MyTitle'),
-        ('Explicit title P1:A, colon', 'P1.html#grammar-token-p1_a', 'My:Title'),
-        ('Tilde A', 'Bare.html#grammar-token-_a', 'A'),
-        ('Tilde P1:A', 'P1.html#grammar-token-p1_a', 'A'),
-        ('Tilde explicit title P1:A', 'P1.html#grammar-token-p1_a', '~MyTitle'),
-        ('Tilde, explicit title P1:A', 'P1.html#grammar-token-p1_a', 'MyTitle'),
-        ('Dup', 'Dup2.html#grammar-token-_dup', 'Dup'),
-        ('FirstLine', 'firstLineRule.html#grammar-token-_firstline', 'FirstLine'),
-        ('SecondLine', 'firstLineRule.html#grammar-token-_secondline', 'SecondLine'),
+        ('A', 'Bare.html#grammar-token-a', 'A'),
+        ('B', 'Bare.html#grammar-token-b', 'B'),
+        ('P1:A', 'P1.html#grammar-token-p1-a', 'P1:A'),
+        ('P1:B', 'P1.html#grammar-token-p1-b', 'P1:B'),
+        ('P2:A', 'P1.html#grammar-token-p1-a', 'P1:A'),
+        ('P2:B', 'P2.html#grammar-token-p2-b', 'P2:B'),
+        ('Explicit title A, plain', 'Bare.html#grammar-token-a', 'MyTitle'),
+        ('Explicit title A, colon', 'Bare.html#grammar-token-a', 'My:Title'),
+        ('Explicit title P1:A, plain', 'P1.html#grammar-token-p1-a', 'MyTitle'),
+        ('Explicit title P1:A, colon', 'P1.html#grammar-token-p1-a', 'My:Title'),
+        ('Tilde A', 'Bare.html#grammar-token-a', 'A'),
+        ('Tilde P1:A', 'P1.html#grammar-token-p1-a', 'A'),
+        ('Tilde explicit title P1:A', 'P1.html#grammar-token-p1-a', '~MyTitle'),
+        ('Tilde, explicit title P1:A', 'P1.html#grammar-token-p1-a', 'MyTitle'),
+        ('Dup', 'Dup2.html#grammar-token-dup', 'Dup'),
+        ('FirstLine', 'firstLineRule.html#grammar-token-firstline', 'FirstLine'),
+        ('SecondLine', 'firstLineRule.html#grammar-token-secondline', 'SecondLine'),
     ]
 
     text = (app.outdir / 'LineContinuation.html').read_text()