Fix #7812: autosummary: generates broken stub files

CHANGES

@@ -16,6 +16,9 @@ Features added
 Bugs fixed
 ----------
 
+* #7812: autosummary: generates broken stub files if the target code contains
+  an attribute and module that are same name
+
 Testing
 --------
 

sphinx/util/__init__.py

@@ -33,6 +33,7 @@
     PycodeError, SphinxParallelError, ExtensionError, FiletypeNotFoundError
 )
 from sphinx.locale import __
+from sphinx.util import inspect
 from sphinx.util import logging
 from sphinx.util.console import strip_colors, colorize, bold, term_width_line  # type: ignore
 from sphinx.util.typing import PathMatcher
@@ -619,9 +620,22 @@ def split_full_qualified_name(name: str) -> Tuple[str, str]:
     for i, part in enumerate(parts, 1):
         try:
             modname = ".".join(parts[:i])
-            import_module(modname)
+            module = import_module(modname)
+
+            # check the module has a member named as attrname
+            #
+            # Note: This is needed to detect the attribute having the same name
+            #       as the module.
+            #       ref: https://github.com/sphinx-doc/sphinx/issues/7812
+            attrname = parts[i]
+            if hasattr(module, attrname):
+                value = inspect.safe_getattr(module, attrname)
+                if not inspect.ismodule(value):
+                    return ".".join(parts[:i]), ".".join(parts[i:])
         except ImportError:
             return ".".join(parts[:i - 1]), ".".join(parts[i - 1:])
+        except IndexError:
+            pass
 
     return name, ""
 

sphinx/util/inspect.py

@@ -18,7 +18,7 @@
 import warnings
 from functools import partial, partialmethod
 from inspect import (  # NOQA
-    Parameter, isclass, ismethod, ismethoddescriptor
+    Parameter, isclass, ismethod, ismethoddescriptor, ismodule
 )
 from io import StringIO
 from typing import Any, Callable, Mapping, List, Optional, Tuple