WKT parser: do not raise warning when parsing a WKT2:2015 TIMECRS who…

…se TIMEUNIT is at the CS level, and not inside (#2281) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
OSGeo · Jul 2, 2020 · 1c1585c · 1c1585c
1 parent ed32eb0
commit 1c1585c
Show file tree

Hide file tree

Showing 4 changed files with 1,250 additions and 1,194 deletions.
diff --git a/src/iso19111/io.cpp b/src/iso19111/io.cpp
@@ -2642,10 +2642,15 @@ WKTParser::Private::buildCS(const WKTNodeNNPtr &node, /* maybe null */
         }
     } else if (ci_equal(csType, "temporal")) { // WKT2-2015
         if (axisCount == 1) {
-            return DateTimeTemporalCS::create(
-                csMap,
-                axisList[0]); // FIXME: there are 3 possible subtypes of
-                              // TemporalCS
+            if (isNull(
+                    parentNode->GP()->lookForChild(WKTConstants::TIMEUNIT)) &&
+                isNull(parentNode->GP()->lookForChild(WKTConstants::UNIT))) {
+                return DateTimeTemporalCS::create(csMap, axisList[0]);
+            } else {
+                // Default to TemporalMeasureCS
+                // TemporalCount could also be possible
+                return TemporalMeasureCS::create(csMap, axisList[0]);
+            }
         }
     } else if (ci_equal(csType, "TemporalDateTime")) { // WKT2-2019
         if (axisCount == 1) {