LaTeX: revisit and trim visit_target()

Refs: sphinx-doc#13609, sphinx-doc#11093 (sphinx-doc#11333), sphinx-doc#6026 (sphinx-doc#6051).

- Fix misnomer in test-latex-labels/index.rst text.

  Refs:
  sphinx-doc#13609 (comment)

- Patch writers/latex.py as per the proposal at end of:
  sphinx-doc#13609 (comment)

- Update test_latex_build.py::test_latex_labels to detect non
  desired extra label in LaTeX output.

Nota bene: this commit does not revert "Tests: update LaTeX label
test expectations from Docutils r10151 (sphinx-doc#13610)" (commit
68d5610) as it is still
necessary for compatibility with both Docutils 0.21.2 and
Docutils HEAD at their r10151.

Author: jfbu

sphinx/writers/latex.py

@@ -1843,30 +1843,10 @@ def add_target(id: str) -> None:
                 and node['refid'] == prev_node['refid']
             ):
                 # a target for a hyperlink reference having alias
-                pass
+                return
             else:
                 add_target(node['refid'])
-        # Temporary fix for https://github.com/sphinx-doc/sphinx/issues/11093
-        # TODO: investigate if a more elegant solution exists
-        # (see comments of https://github.com/sphinx-doc/sphinx/issues/11093)
-        if node.get('ismod', False):
-            # Detect if the previous nodes are label targets. If so, remove
-            # the refid thereof from node['ids'] to avoid duplicated ids.
-            def has_dup_label(sib: Node | None) -> bool:
-                return isinstance(sib, nodes.target) and sib.get('refid') in node['ids']
-
-            prev = get_prev_node(node)
-            if has_dup_label(prev):
-                ids = node['ids'][:]  # copy to avoid side-effects
-                while has_dup_label(prev):
-                    ids.remove(prev['refid'])  # type: ignore[index]
-                    prev = get_prev_node(prev)  # type: ignore[arg-type]
-            else:
-                ids = iter(node['ids'])  # read-only iterator
-        else:
-            ids = iter(node['ids'])  # read-only iterator
-
-        for id in ids:
+        for id in node['ids']:
             add_target(id)
 
     def depart_target(self, node: Element) -> None:

tests/roots/test-latex-labels/index.rst

@@ -69,6 +69,6 @@ subsubsection
 
    otherdoc
 
-* Embedded standalone hyperlink reference: `subsection <section1_>`_.
+* Named hyperlink reference with embedded alias reference: `subsection <section1_>`_.
 
   .. See: https://github.com/sphinx-doc/sphinx/issues/5948

tests/test_builders/test_build_latex.py

@@ -2019,9 +2019,11 @@ def test_latex_labels(app: SphinxTestApp) -> None:
         r'\label{\detokenize{otherdoc::doc}}'
     ) in result
 
-    # Embedded standalone hyperlink reference
+    # Named hyperlink reference with embedded alias reference
     # See: https://github.com/sphinx-doc/sphinx/issues/5948
     assert result.count(r'\label{\detokenize{index:section1}}') == 1
+    # https://github.com/sphinx-doc/sphinx/issues/13609
+    assert r'\phantomsection\label{\detokenize{index:id' not in result
 
 
 @pytest.mark.sphinx('latex', testroot='latex-figure-in-admonition')