correctly calculate total years in Duration constructor.

RDFLib · Oct 15, 2024 · 5531b46 · 5531b46
1 parent 52eee4c
commit 5531b46
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/rdflib/xsd_datetime.py b/rdflib/xsd_datetime.py
@@ -129,7 +129,7 @@ def __init__(
             years = Decimal(str(years))
         new_years, months = fquotmod(months, 0, 12)
         self.months = months
-        self.years = Decimal(new_years)
+        self.years = Decimal(years+new_years)
         self.tdelta = timedelta(
             days, seconds, microseconds, milliseconds, minutes, hours, weeks
         )

diff --git a/test/test_sparql/test_datetime_processing.py b/test/test_sparql/test_datetime_processing.py
@@ -94,8 +94,8 @@ def test_datetime_duration_subs():
     SELECT (?d - ?duration AS ?next_year)
     WHERE {
         VALUES (?duration ?d) {
-            ("P1Y"^^xsd:yearMonthDuration"2019-05-28T12:14:45Z"^^xsd:dateTime)
-            ("P1Y"^^xsd:yearMonthDuration"2019-05-28"^^xsd:date)
+            ("P1Y"^^xsd:yearMonthDuration "2019-05-28T12:14:45Z"^^xsd:dateTime)
+            ("P1Y"^^xsd:yearMonthDuration "2019-05-28"^^xsd:date)
         }
     }
     """