arcgis-rest-js icon indicating copy to clipboard operation
arcgis-rest-js copied to clipboard

processParams not converting extent to string

Open benstoltz opened this issue 3 years ago • 1 comments

Describe the bug

addItem and updateItem both expect extent to be a flattened string instead of a bbox ( 1, 2, 3, 4 vs [[1, 2], [3, 4]] ). Apparently around two years ago we made a change to processParams ( #779 ) for catalog to allow array of arrays through. At that time it was decided to do that for all the cases so as to not make an explicit reference to a property. Unfortunately that has broken extent. What ends up happening is in the actual request the bbox is split into two different extent params, the create/update call succeeds, however the item ends up with an extent which is simply an empty array.

Reproduction

Will be provided

Logs

No response

System Info

v3.4.3

Additional Information

No response

benstoltz avatar Sep 20 '22 16:09 benstoltz

@benstoltz please implement the conversion from number[][] to string in the createItem and updateItem functions, and leave the processParams function as-is

dbouwman avatar Sep 20 '22 16:09 dbouwman

Closing per @dbouwman.

gavinr avatar Oct 17 '22 16:10 gavinr