Move check_anchors test first; combine two if-statements

jayaddison · Aug 1, 2023 · 6f17899 · 6f17899
1 parent 1c34aef
commit 6f17899
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/sphinx/builders/linkcheck.py b/sphinx/builders/linkcheck.py
@@ -407,8 +407,8 @@ def _check_uri(self, uri: str, hyperlink: Hyperlink) -> tuple[str, str, int]:
                     _user_agent=self.user_agent,
                     _tls_info=(self.tls_verify, self.tls_cacerts),
                 ) as response:
-                    if response.ok and self.check_anchors:
-                        if anchor and not contains_anchor(response, anchor):
+                    if (self.check_anchors and response.ok and anchor
+                        and not contains_anchor(response, anchor)):
                             raise Exception(__(f'Anchor {anchor!r} not found'))
 
                 # Copy data we need from the (closed) response