WktGenerator has a generate function for MultiPolygon, but lacks the build function.

CommentFileSizeAuthor
#7 3004633-7.patch1.55 KBitamair
#5 multipolygon.json_.zip2.2 KBducktape
#2 3004633-1.patch1.46 KBducktape
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ducktape created an issue. See original summary.

ducktape’s picture

ducktape CreditAttribution: ducktape commented
FileSize
3004633-1.patch1.46 KB

Adding patch that adds the build functions.

itamair’s picture

itamair CreditAttribution: itamair commented

tnx indeed @ducktape ... for this.
I would love to review and test this. Any advice on the use case to implement for reviewing ?

itamair’s picture

itamair CreditAttribution: itamair commented
Status: Active » Needs review
ducktape’s picture

ducktape CreditAttribution: ducktape commented
FileSize
multipolygon.json_.zip2.2 KB

The geodata I am using is coming from our internal geoserver, so I have attached one item here with multiple rings.

itamair’s picture

itamair CreditAttribution: itamair commented

thanks @ducktape ... will try to review with that asap

itamair’s picture

itamair CreditAttribution: itamair commented
FileSize
3004633-7.patch1.55 KB

Ok ... I don't have time to import and reproduce.
I had a look to the code, and seem to have sense. I just amended the original patch to better comply with type hinting ...
Committing into dev.

  • itamair committed 5f9dc01 on 8.x-1.x authored by ducktape 
    Issue #3004633 by ducktape, itamair: Add a build function for...
itamair’s picture

itamair CreditAttribution: itamair commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.