refactor: allow datetimes without +, defaulting to + (#435)

datetimes without plus at the start
should be mapped to datetimes with +

fixes #434

Author: arcstur

src/core/parsers/base.py

@@ -314,6 +314,9 @@ def parse_value_time(self, v):
         else:
             calendar_model = "http://www.wikidata.org/entity/Q1985727"  # Gregorian
 
+        if not match.group("sign"):
+            v = "+" + v
+
         # Remove trailing precision if present (e.g. /11)
         v_clean = re.sub(r"/\d+$", "", v)
 

src/core/tests/test_base_parser.py

@@ -389,6 +389,45 @@ def test_parse_value_time_custom_calendar(self):
         }
         self.assertEqual(parser.parse_value("+1967-00-00T00:00:00Z/9/C999999"), ret)
 
+    def test_parse_value_time_without_plus_adds_plus(self):
+        parser = BaseParser()
+        ret = {
+            "type": "time",
+            "value": {
+                "time": "+1967-01-17T00:00:00Z",
+                "precision": 11,
+                "calendarmodel": "http://www.wikidata.org/entity/Q1985727",
+            },
+        }
+        self.assertEqual(parser.parse_value("1967-01-17T00:00:00Z/11"), ret)
+        ret = {
+            "type": "time",
+            "value": {
+                "time": "+2026-01-01T00:00:00Z",
+                "precision": 9,
+                "calendarmodel": "http://www.wikidata.org/entity/Q1985727",
+            },
+        }
+        self.assertEqual(parser.parse_value("2026-01-01T00:00:00Z/9"), ret)
+        ret = {
+            "type": "time",
+            "value": {
+                "time": "-1967-01-00T00:00:00Z",
+                "precision": 10,
+                "calendarmodel": "http://www.wikidata.org/entity/Q1985727",
+            },
+        }
+        self.assertEqual(parser.parse_value("-1967-01-00T00:00:00Z/10"), ret)
+
+    def test_parse_value_time_other_signs_are_none(self):
+        parser = BaseParser()
+        self.assertIsNone(parser.parse_value("x1967-01-01T00:00:00Z/10"))
+        self.assertIsNone(parser.parse_value("_1967-01-01T00:00:00Z/10"))
+        self.assertIsNone(parser.parse_value(":1967-01-01T00:00:00Z/10"))
+        self.assertIsNone(parser.parse_value(";1967-01-01T00:00:00Z/10"))
+        self.assertIsNone(parser.parse_value("=1967-01-01T00:00:00Z/10"))
+        self.assertIsNone(parser.parse_value("s1967-01-01T00:00:00Z/10"))
+
     def test_parse_value_location_earth(self):
         parser = BaseParser()
         ret = {