Backport PR #39355: BUG: read_excel failing to check older xlrd versions properly (#39535)

Co-authored-by: Thomas Li <[email protected]>

Author: simonjayhawkins

doc/source/whatsnew/v1.2.2.rst

@@ -14,6 +14,8 @@ including other versions of pandas.
 
 Fixed regressions
 ~~~~~~~~~~~~~~~~~
+
+- Fixed regression in :func:`read_excel` that caused it to raise ``AttributeError`` when checking version of older xlrd versions (:issue:`38955`)
 - Fixed regression in :class:`DataFrame` constructor reordering element when construction from datetime ndarray with dtype not ``"datetime64[ns]"`` (:issue:`39422`)
 - Fixed regression in :meth:`~DataFrame.to_pickle` failing to create bz2/xz compressed pickle files with ``protocol=5`` (:issue:`39002`)
 - Fixed regression in :func:`pandas.testing.assert_series_equal` and :func:`pandas.testing.assert_frame_equal` always raising ``AssertionError`` when comparing extension dtypes (:issue:`39410`)

pandas/compat/_optional.py

@@ -46,7 +46,7 @@
 }
 
 
-def _get_version(module: types.ModuleType) -> str:
+def get_version(module: types.ModuleType) -> str:
     version = getattr(module, "__version__", None)
     if version is None:
         # xlrd uses a capitalized attribute name
@@ -112,7 +112,7 @@ def import_optional_dependency(
 
     minimum_version = VERSIONS.get(name)
     if minimum_version:
-        version = _get_version(module)
+        version = get_version(module)
         if distutils.version.LooseVersion(version) < minimum_version:
             assert on_version in {"warn", "raise", "ignore"}
             msg = (

pandas/io/excel/_base.py

@@ -13,7 +13,7 @@
 
 from pandas._libs.parsers import STR_NA_VALUES
 from pandas._typing import Buffer, FilePathOrBuffer, StorageOptions
-from pandas.compat._optional import import_optional_dependency
+from pandas.compat._optional import get_version, import_optional_dependency
 from pandas.errors import EmptyDataError
 from pandas.util._decorators import Appender, deprecate_nonkeyword_arguments, doc
 
@@ -1049,7 +1049,7 @@ def __init__(
         else:
             import xlrd
 
-            xlrd_version = LooseVersion(xlrd.__version__)
+            xlrd_version = LooseVersion(get_version(xlrd))
 
         if xlrd_version is not None and isinstance(path_or_buffer, xlrd.Book):
             ext = "xls"

pandas/tests/io/excel/__init__.py

@@ -2,7 +2,7 @@
 
 import pytest
 
-from pandas.compat._optional import import_optional_dependency
+from pandas.compat._optional import get_version, import_optional_dependency
 
 pytestmark = [
     pytest.mark.filterwarnings(
@@ -32,4 +32,4 @@
 else:
     import xlrd
 
-    xlrd_version = LooseVersion(xlrd.__version__)
+    xlrd_version = LooseVersion(get_version(xlrd))

pandas/util/_print_versions.py

@@ -8,7 +8,7 @@
 from typing import Dict, Optional, Union
 
 from pandas._typing import JSONSerializable
-from pandas.compat._optional import VERSIONS, _get_version, import_optional_dependency
+from pandas.compat._optional import VERSIONS, get_version, import_optional_dependency
 
 
 def _get_commit_hash() -> Optional[str]:
@@ -83,7 +83,7 @@ def _get_dependency_info() -> Dict[str, JSONSerializable]:
         mod = import_optional_dependency(
             modname, raise_on_missing=False, on_version="ignore"
         )
-        result[modname] = _get_version(mod) if mod else None
+        result[modname] = get_version(mod) if mod else None
     return result
 
 

scripts/validate_unwanted_patterns.py

@@ -29,7 +29,6 @@
     "_doc_template",
     "_agg_template",
     "_pipe_template",
-    "_get_version",
     "__main__",
     "_transform_template",
     "_flex_comp_doc_FRAME",