-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Findable DOIs appearing with no creator or title #235
Comments
This is a display problem only, looks like data not being loaded into the fabrica model correctly, probably because of the relation to the form. |
Potential related issue might be datacite/lupo#285 - This was referring to missing publisher field. |
A reassessment of this actually suggest this isn't bracco related, the search index doesn't have the correct information from the DOI. I can reproduce by making an API call with the same XML used. With subsequent updates it does correct itself.
|
A further further reassessment does confirm this was due to bracco and sending empty attributes like creators: [] - These are prioritised over the value in the XML. I've done a fix on test to prevent these being sent, this bug only occured for file upload and was "solved" because we do cancel out the titles/creators when modifying. |
This has been deployed and should now be fixed. |
Expected Behaviour
Findable DOIs cannot be created without all of the required fields
Current Behaviour
A user reports numerous DOIs appearing not to have a creator or title in form and summary view although this information is seen in the file upload view
Steps to Reproduce
E.g
https://doi.datacite.org/dois/10.21405%2Flogtrend.2019.5.1.56/edit
plus the following DOIs
10.21405/logtrend.2019.5.1.12
10.21405/logtrend.2019.5.1.25
10.21405/logtrend.2019.5.1.4
10.21405/logtrend.2019.5.1.43
10.21405/logtrend.2019.5.1.20.
10.21405/logtrend.2019.5.1.50
10.21405/logtrend.2019.5.1.29
10.21405/logtrend.2019.5.1.36
Context (Environment)
Fabrica production
Hypothesis
Detailed Description
Possible Implementation
The text was updated successfully, but these errors were encountered: