Issue #618 - fix reference to re_match attribute in copy of Word object to first check with hasattr

ptmcg · ptmcg · commit f5bdb2deff92 · 2025-09-16T18:28:40.000-05:00
diff --git a/CHANGES b/CHANGES
@@ -44,6 +44,11 @@ Required Python versions by pyparsing version
 
 Version 3.2.5 - September, 2025
 -------------------------------
+- JINX! Well, 3.2.4 had a bug for `Word` expressions that include a space
+  character, if that expression was then copied, either directly with .copy() or
+  by adding a results name, or including in another construct (like `DelimitedList`)
+  that makes a copy internally. Issue #618, reported by mstinberg, among others -
+  thanks, and sorry for the inconvenience.
 
 
 Version 3.2.4 - September, 2025
diff --git a/pyparsing/__init__.py b/pyparsing/__init__.py
@@ -125,7 +125,7 @@ def __repr__(self):
 
 
 __version_info__ = version_info(3, 2, 5, "final", 1)
-__version_time__ = "16 Sep 2025 19:39 UTC"
+__version_time__ = "16 Sep 2025 22:24 UTC"
 __version__ = __version_info__.__version__
 __versionTime__ = __version_time__
 __author__ = "Paul McGuire <ptmcg.gm+pyparsing@gmail.com>"
diff --git a/pyparsing/core.py b/pyparsing/core.py
@@ -3226,7 +3226,9 @@ def __init__(
 
     def copy(self) -> Word:
         ret: Word = cast(Word, super().copy())
-        ret.parseImpl = ret.parseImpl_regex  # type: ignore[method-assign]
+        if hasattr(self, "re_match"):
+            ret.re_match = self.re_match
+            ret.parseImpl = ret.parseImpl_regex  # type: ignore[method-assign]
         return ret
 
     def _generateDefaultName(self) -> str:
diff --git a/tests/test_unit.py b/tests/test_unit.py
@@ -7892,6 +7892,17 @@ def testWordParseImpl(self):
             with self.assertRaisesParseException(expected_msg="Expected custom integer"):
                 custom_int.parse_string("z", parse_all=True)
 
+    def testWordCopyWhenWordCharsIncludeSpace(self):
+        word_with_space = pp.Word(pp.alphas + " ")
+        word_with_space.parse_string("ABC")
+        word_with_space.copy().parse_string("ABC")
+
+    def testWordCopyWhenWordCharsIncludeSpace2(self):
+        element = pp.QuotedString('"') | pp.Combine(pp.Word(' abcdefghijklmnopqrstuvwxyz'))
+        element.parse_string("abc")
+        element_list = pp.DelimitedList(element)
+        element_list.parse_string("abc")
+
     def testLiteralVsKeyword(self):
         integer = ppc.integer
         literal_expr = integer + pp.Literal("start") + integer
