sul-dlss / sul_pub

SUL system for harvest and managing publications for Stanford CAP, with controlled API access.
http://cap.stanford.edu
Other
8 stars 3 forks source link

strip MEDLINE prefix when adding publications via PMID #1654

Closed peetucket closed 9 months ago

peetucket commented 9 months ago

Why was this change made?

Fixes #1653 - see the analysis in the issue description. Basically the new WoS API is returning PMIDs in the record now, and the Profiles team is preferentially picking the PMID in the result and sending that when adding publications via the manual title search (I verified both of these). The WOS records adds a MEDINE: prefix to PMIDs, but when we get a request to add a publication for a specific PMID, we go straight to the Pubmed API to look for that PMID, and to do this, we do NOT want the MEDLINE: prefix. So we should strip it in this case for any incoming PMID we are sent.

How was this change tested?

cap-dev-a and new specs

peetucket commented 9 months ago

closed in favor of #1655