Empty-ied metadata fields should be removed entirely after meditor

[bendichter]

https://dandiarchive.org/dandiset/001170/draft is failing to validate but I am not sure why: contributor.3: {'url': '', 'name': 'Pirhayatifard, Delaram', 'email': '[email protected]', 'schemaKey': 'Person', 'includeInCitation': True} is not valid under any of the given schemas

[satra]

url is empty

[satra]

@mvandenburgh - any ideas how to fix this? i thought we had run into this in a previous issue and addressed it. one could clear everything out and fill it in again without touching the url field, but i recall we had done something about this.

[yarikoptic]

I have replicated

• in meditor, add space as URL, save record
• go edit again, remove space and save

progression of records

❯ curl --silent -X 'GET' 'https://api-staging.dandiarchive.org/api/dandisets/214362/versions/draft/' -H 'accept: application/json' | jq '.contributor'
[
  {
    "name": "Halchenko, Yaroslav",
    "email": "[email protected]",
    "roleName": [
      "dcite:ContactPerson"
    ],
    "schemaKey": "Person",
    "affiliation": [],
    "includeInCitation": true
  }
]
❯ curl --silent -X 'GET' 'https://api-staging.dandiarchive.org/api/dandisets/214362/versions/draft/' -H 'accept: application/json' | jq '.contributor'
[
  {
    "url": " ",
    "name": "Halchenko, Yaroslav",
    "email": "[email protected]",
    "roleName": [
      "dcite:ContactPerson"
    ],
    "schemaKey": "Person",
    "affiliation": [],
    "includeInCitation": true
  }
]
❯ curl --silent -X 'GET' 'https://api-staging.dandiarchive.org/api/dandisets/214362/versions/draft/' -H 'accept: application/json' | jq '.contributor'
[
  {
    "url": "",
    "name": "Halchenko, Yaroslav",
    "email": "[email protected]",
    "roleName": [
      "dcite:ContactPerson"
    ],
    "schemaKey": "Person",
    "affiliation": [],
    "includeInCitation": true
  }
]

so I think meditor needs to clean up (remove) empty fields. To resolve this one in particular, I will go and fixup metadata directly in DB for now.

[yarikoptic]

I have tried to fix directly in DB via admin interface, but only caused 500. in logs:

heroku[router]: at=info method=POST path="/admin/api/version/1809/change/?_changelist_filters=q%3D1170" host=api.dandiarchive.org request_id=6f5d04a2-SENSORED fwd="SENSORED" dyno=web.2 connect=0ms service=676ms status=500 bytes=512 protocol=https

attn @dandi/archive-admin happen you see this -- it was me ;-/ I guess someone should fix via GET/fix/PUT

[bendichter]

I was able to resolve this as @satra suggested, but removing this contributor and adding them again, though I would really prefer a solution where empty strings are converted to null

[kabilar]

Thanks all. Will add this to the backlog.