Make Wkt local. Edit arcgis samples to reflect that.

arthur-e / Wicket

A modest library for moving between Well-Known Text (WKT) and various framework geometries

https://arthur-e.github.io/Wicket/

Other

586 stars 227 forks source link

Make Wkt local. Edit arcgis samples to reflect that. #157

Closed cancastilho closed 3 years ago

cancastilho commented 3 years ago

Make Wkt local again. Possible fix to issue #156. Although I could not reproduce the error mentioned there.

arthur-e commented 3 years ago

@cancastilho Thanks! So, this doesn't break the ArcGIS extension again? I had assumed that moving Wkt to global scope was somehow essential to getting that working.

cancastilho commented 3 years ago

Hi @arthur-e. Fortunally, this won't break arcgis again. It turns out that wicked-arcgis-amd was missing a return statement in the end of the module. After changing that, we don't need Wkt global anymore for arcgis to work. You could check those working samples arcgis3.html and arcgis4.html to see them working.