Deprecate version attribute in xml.sax.handler

Author: hugovk

Doc/deprecations/pending-removal-in-3.20.rst

@@ -23,9 +23,10 @@ Pending removal in Python 3.20
   - :mod:`tabnanny`
   - :mod:`tkinter.font`
   - :mod:`tkinter.ttk`
+  - :mod:`wsgiref.simple_server`
   - :mod:`xml.etree.ElementTree`
   - :mod:`xml.sax.expatreader`
-  - :mod:`wsgiref.simple_server`
+  - :mod:`xml.sax.handler`
   - :mod:`zlib`
 
   (Contributed by Hugo van Kemenade and Stan Ulbrych in :gh:`76007`.)

Doc/whatsnew/3.15.rst

@@ -1040,9 +1040,10 @@ New deprecations
     - :mod:`tabnanny`
     - :mod:`tkinter.font`
     - :mod:`tkinter.ttk`
+    - :mod:`wsgiref.simple_server`
     - :mod:`xml.etree.ElementTree`
     - :mod:`xml.sax.expatreader`
-    - :mod:`wsgiref.simple_server`
+    - :mod:`xml.sax.handler`
     - :mod:`zlib`
 
     (Contributed by Hugo van Kemenade and Stan Ulbrych in :gh:`76007`.)

Lib/test/test_sax.py

@@ -1574,13 +1574,15 @@ def test_all(self):
 
 
 class TestModule(unittest.TestCase):
-    def test_deprecated_version(self):
-        with self.assertWarnsRegex(
-            DeprecationWarning,
-            "'version' is deprecated and slated for removal in Python 3.20",
-        ) as cm:
-                getattr(sax.expatreader, "version")
-        self.assertEqual(cm.filename, __file__)
+    def test_deprecated__version__and__date__(self):
+        for module in (sax.expatreader, sax.handler):
+            with self.subTest(module=module):
+                with self.assertWarnsRegex(
+                    DeprecationWarning,
+                    "'version' is deprecated and slated for removal in Python 3.20",
+                ) as cm:
+                    getattr(module, "version")
+                self.assertEqual(cm.filename, __file__)
 
 
 if __name__ == "__main__":

Lib/xml/sax/handler.py

@@ -9,8 +9,6 @@
 $Id$
 """
 
-version = '2.0beta'
-
 #============================================================================
 #
 # HANDLER INTERFACES
@@ -385,3 +383,12 @@ def startCDATA(self):
 
     def endCDATA(self):
         """Reports the end of a CDATA marked section."""
+
+
+def __getattr__(name):
+    if name == "version":
+        from warnings import _deprecated
+
+        _deprecated("version", remove=(3, 20))
+        return "2.0beta"  # Do not change
+    raise AttributeError(f"module {__name__!r} has no attribute {name!r}")