search

DSpace / RestContract

REST Contract for DSpace 7-8
https://wiki.lyrasis.org/display/DSDOC8x/
38 stars 48 forks source link

Return no content for unset properties #243

Closed J4bbi closed 11 months ago

tdonohue commented 11 months ago

Thanks @J4bbi ! This looks correct & is just a documentation mistake. Merging immediately.

tdonohue commented 11 months ago

@J4bbi : I've reverted this PR. So, please feel free to recreate it and link it up to https://github.com/DSpace/DSpace/issues/9014 Apologies for the misunderstanding.

tdonohue commented 11 months ago

@J4bbi : I will also note though that I'm starting to wonder if the better "fix" here is to simply use the newly built /search/exposed endpoint: https://github.com/DSpace/RestContract/pull/240

The behavior of returning a 404 no longer is a problem if properties are changed to be loaded in bulk , as it will be rare for 404s to be encountered in that scenario.