better handling of coordinate parsing

Author: joshday

Project.toml

@@ -1,7 +1,7 @@
 name = "KML"
 uuid = "1284bf3a-1e3d-4f4e-a7a9-b9d235a28f35"
 authors = ["Josh Day <emailjoshday@gmail.com> and contributors"]
-version = "0.2.4"
+version = "0.2.5"
 
 [deps]
 GeoInterface = "cf35fbd7-0cd7-5166-be24-54bfbe79505f"

src/parsing.jl

@@ -104,15 +104,10 @@ function autosetfield!(o::Union{Object,KMLElement}, sym::Symbol, x::String)
     T <: Number && return setfield!(o, sym, parse(T, x))
     T <: AbstractString && return setfield!(o, sym, x)
     T <: Enums.AbstractKMLEnum && return setfield!(o, sym, T(x))
-    # if sym == :coordinates
-    #     val = occursin('\n', x) ?
-    #         [Tuple(parse.(Float64, split(s, ','))) for s in split(x, '\n')] :
-    #         Tuple(parse.(Float64, split(x, ',')))
-    #     return setfield!(o, sym, val)
-    # end
     if sym == :coordinates
         val = [Tuple(parse.(Float64, split(v, ','))) for v in split(x)]
-        if length(val) == 1
+        # coordinates can be a tuple or a vector of tuples, so we need to do this:
+        if fieldtype(typeof(o), sym) <: Union{Nothing, Tuple}
             val = val[1]
         end
         return setfield!(o, sym, val)

test/runtests.jl

@@ -55,3 +55,13 @@ end
     file2 = read(temp, KMLFile)
     @test file == file2
 end
+
+@testset "coordinates" begin
+    # `coordinates` are tuple
+    s = "<Point><coordinates>1,2,3</coordinates></Point>"
+    @test KML.object(XML.parse(s, XML.Node)[1]) isa Point
+
+    # `coordinates` are vector of tuples
+    s = "<LineString><coordinates>1,2,3</coordinates></LineString>"
+    @test KML.object(XML.parse(s, XML.Node)[1]) isa LineString
+end