fix: close KML polygons

[javisantana]

hey @davethau, this pull request should fix the problem with open polygons when they're saved to fusion tables.

I didn't tested it

[davethau]

Hi Javi!
Can you check if the polygons are closed in the Area object too? I'd prefer it they were in sync.
Best,
Thau

[javisantana]

Hi Dave, I don't remember exactly what you are referring to but fixing that function should fix all the places where polygons are encoded to kml

[javisantana]

Hey @davethau, i've just commited a fix that closes polygons client side so they are stored closed in the database.

I tested it only using unit tests, if you have any problem when you try in real environment let me know.

The way I fixed it is pretty easy, just when polygon mode is serialized (toJSON method) I close polygons. In the deserialization (parse method) I check if it's closed and remove the old one in order to the data model is the same for the app.

I also removed the code for close polygons in KML conversion code since the paths are closed in the database.

Cheers