AtlasOfLivingAustralia / biocache-service

Occurrence & mapping webservices
https://biocache-ws.ala.org.au/ws/
Other
9 stars 26 forks source link

dynamicProperties not shown as Additional Properties in Biocache-hub #787

Closed sadeghim closed 1 year ago

sadeghim commented 1 year ago

Some recent records have dynamicProperties as one of the fields in their occurrence but the miscProperties in biocache-service is empty. Here is a sample record in Solr https://biocache.ala.org.au/occurrences/a34e2843-6233-4f78-9a8b-b1a408df036f :

image

While the miscProperties is empty in biocache-service API call: https://biocache.ala.org.au/ws/occurrences/a34e2843-6233-4f78-9a8b-b1a408df036f

nickdos commented 1 year ago

Was looking at a random record just now and saw it has miscProperties in the raw section: https://biocache-ws.ala.org.au/ws/occurrence/8c98a570-eaa3-44ff-a9b7-31a89980776f so it seems that the bug must be data-related in some way.

The example in SOLR above seems to be a JSON blob but it appears to be missing a comma. So I'm wondering if biocache-service is trying to parse the JSON and that is failing silently?

image

sat01a commented 1 year ago

Thanks @nickdos . @sadeghim did you check the data?

sadeghim commented 1 year ago

@sat01a Yes, as @nickdos has pointed out the problem is the incorrect syntax of JSON. But there is no indication anywhere saying that the format is wrong. I don't see it as a bug now, but there should be somewhere in the log mentioning that. Happy to close this one.

sat01a commented 1 year ago

Thanks @sadeghim, please close the ticket.