Resolved issue GEOT-5021;

In readGeometryTaggedText in WKTReader2, made case for "POINT" case insensitive; Added a simple test case for case insensitivity in WKTReader2Test;
gerson721 · Feb 28, 2015 · 3a307ff · 3a307ff
1 parent 1077e07
commit 3a307ff
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/modules/library/main/src/main/java/org/geotools/geometry/jts/WKTReader2.java b/modules/library/main/src/main/java/org/geotools/geometry/jts/WKTReader2.java
@@ -397,7 +397,7 @@ private Geometry readGeometryTaggedText() throws IOException, ParseException {
             return null;
         }
 
-        if (type.equals("POINT")) {
+        if (type.equalsIgnoreCase("POINT")) {
             return readPointText();
         } else if (type.equalsIgnoreCase("LINESTRING")) {
             return readLineStringText();

diff --git a/modules/library/main/src/test/java/org/geotools/geometry/jts/WKTReader2Test.java b/modules/library/main/src/test/java/org/geotools/geometry/jts/WKTReader2Test.java
@@ -187,4 +187,14 @@ public void testParseMulticurve() throws Exception {
         assertTrue(ml.getGeometryN(0).getClass() == LineString.class);
         assertTrue(ml.getGeometryN(1) instanceof CompoundRing);
     }
+
+    @Test
+    public void testCaseInsensitive() throws Exception {
+        WKTReader reader = new WKTReader2();
+        assertNotNull( reader.read("POINT(1 2)") );
+        assertNotNull( reader.read("Point(1 2)") );
+
+        assertNotNull( reader.read("LINESTRING(0 2, 2 0, 8 6)") );
+        assertNotNull( reader.read("LineString(0 2, 2 0, 8 6)") );
+    }
 }