Uncaught TypeError: Cannot read property 'length' of undefined about openneuro HOT 10 CLOSED

Happened after user tried to delete one of the .nii files.

from openneuro.

Looks like they deleted the only .nii file and that's important because it leads to a special case where the validator returns 'Invalid' instead of a list of errors.

I'm writing tests and a fix for this on the frontend but maybe it would be better to remove this special case from the validator? It would be more consistent to return a regular validator error quickTest fails.

from openneuro.

Last step to finish this is to upgrade OpenNeuro to [email protected]

from openneuro.

@chrisfilo The bids-validator fixes prevent this from crashing the page but it is still impossible to recover from it. You can't add a subject because the directory is removed automatically and you can't create directories. You can delete the dataset and start over with these fixes and it's much clearer what went wrong with the error code.

Should we support adding subjects after upload so you can recover from this situation without recreating the dataset?

from openneuro.

Yes. Adding new subjects after upload is very useful when you run apps while still acquiring data.

from openneuro.

Even after update this error is still affecting this dataset. Maybe revalidation needs to be forced?

In addition, there is a new one as well now:

Uncaught TypeError: Cannot read property 'body' of undefined
    at scitran.js:169
    at o (request.js:131)
    at request.js:26
    at u.callback (client.js:615)
    at u.<anonymous> (client.js:415)
    at u.t.emit (index.js:133)
    at XMLHttpRequest.e.onreadystatechange (client.js:705)

from openneuro.

The second error is probably #6

from openneuro.

I confirmed the original error on a connection not affected by #6

from openneuro.

@chrisfilo I did miss revalidating this in my deployment steps. I just tried it now and ran into a problem with the dataset hashes not matching up. I'll put together a workaround tomorrow.

from openneuro.

@chrisfilo I cleared the bad validation data but it looks like this dataset was deleted. Revalidation didn't work since the files are missing.

from openneuro.