fix #12

Author: joshday

src/parsing.jl

@@ -104,10 +104,17 @@ 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 = occursin('\n', x) ?
-            [Tuple(parse.(Float64, split(s, ','))) for s in split(x, '\n')] :
-            Tuple(parse.(Float64, split(x, ',')))
+        val = [Tuple(parse.(Float64, split(v, ','))) for v in split(x)]
+        if length(val) == 1
+            val = val[1]
+        end
         return setfield!(o, sym, val)
     end
     setfield!(o, sym, x)

test/issue12.kml

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<kml xmlns:kml="http://www.opengis.net/kml/2.2">
+<Document>
+	<name>tmp.kml</name>
+	<Placemark>
+		<name>tmp</name>
+		<styleUrl>#m_ylw-pushpin</styleUrl>
+		<Polygon>
+			<tessellate>1</tessellate>
+			<outerBoundaryIs>
+				<LinearRing>
+					<coordinates>
+						-50.80770360421555,27.50794679203254,0 -46.63827492868646,32.0230059721323,0 -54.85996847570588,35.7028706316008,0 -50.80770360421555,27.50794679203254,0
+					</coordinates>
+				</LinearRing>
+			</outerBoundaryIs>
+		</Polygon>
+	</Placemark>
+</Document>
+</kml>

test/runtests.jl

@@ -3,6 +3,14 @@ using GeoInterface
 using Test
 using XML
 
+@testset "Issue Coverage" begin
+    # https://github.com/JuliaComputing/KML.jl/issues/8
+    @test_warn "Unhandled case" read(joinpath(@__DIR__, "outside_spec.kml"), KMLFile)
+
+    # https://github.com/JuliaComputing/KML.jl/issues/12
+    @test read(joinpath(@__DIR__, "issue12.kml"), KMLFile) isa KMLFile
+end
+
 @testset "Empty Constructors" begin
     for T in KML.all_concrete_subtypes(KML.Object)
         @test T() isa T
@@ -47,8 +55,3 @@ end
     file2 = read(temp, KMLFile)
     @test file == file2
 end
-
-@testset "Issue Coverage" begin
-    # https://github.com/JuliaComputing/KML.jl/issues/8
-    @test_warn "Unhandled case" read(joinpath(@__DIR__, "outside_spec.kml"), KMLFile)
-end