windrobin / winforms-geplugin-control-library

Automatically exported from code.google.com/p/winforms-geplugin-control-library
GNU General Public License v3.0
0 stars 1 forks source link

KmlHelpers.CreateLineString IList overload doesn't set ID correctly #100

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Call CreateLineString using the IList<Coordinate> overload, passing in a 
valid Id
2. Call GEHelpers.GetAllFeaturesKml
3. Compare the output.

What is the expected output? What do you see instead?
The KML output should have the ID of the input to CreateLineString.  The output 
has no id.

What version of the product are you using? On what operating system?
Code from Test_Application_7, downloaded today (14/3/2013)

Please provide any additional information below.
Looked at the GEHelpers in reflector and I can see that the IList variant seems 
to call ge.createLineString with String.Empty, rather than the varianble "id".

Original issue reported on code.google.com by imagir...@gmail.com on 14 Mar 2013 at 5:51

GoogleCodeExporter commented 9 years ago
Hi, thanks for this.

The CreateLineString method actually creates a linestring and a placemark to 
hold it. The ID is for the container placemark not the linestring. I realize 
this is slightly confusing so I will make clear in the documentation or 
possibly change the behavior so that the ID is attached to the actual 
linestring element. 

Original comment by fraser.c...@gmail.com on 25 Mar 2013 at 10:08

GoogleCodeExporter commented 9 years ago
Ok this is fixed now - the ID is correctly applied to the LineString rather 
than the containing placemark.

Thanks again for bringing this up.

Original comment by fraser.c...@gmail.com on 17 May 2013 at 10:51