k-int / gokb-phase1

Original GOKb repo - Moving to https://github.com/openlibraryenvironment/gokb
http://www.gokb.org
Other
11 stars 5 forks source link

Latest Publisher is showing as duplicate #562

Closed jhsolomon closed 7 years ago

jhsolomon commented 7 years ago

https://gokb-test.openlibraryfoundation.org/gokb/resource/show/org.gokb.cred.JournalInstance%3A10701875#identifiers

jhsolomon commented 7 years ago

I'm not sure if this issue has been corrected or not.

Here is one example where the publisher is showing (correctly) once, but shows a last updated date of 31 October: https://gokb-test.openlibraryfoundation.org/gokb/resource/show/org.gokb.cred.JournalInstance%3A24577075

Whereas Academic Psychiatry is still showing the duplicated publisher name and a last updated date of 2 November: https://gokb-test.openlibraryfoundation.org/gokb/resource/show/org.gokb.cred.JournalInstance%3A10701875

sosguthorpe commented 7 years ago

This is very likely a duplicate of #565

sosguthorpe commented 7 years ago

@jhsolomon My local copy is now correctly importing the publisher history and not just concatenating their names. However the fact that this particular Title shows SpringerSpringer brings to light an issue with the original data.

The title: http://gokb.openlibraryfoundation.org/gokb/resource/show/org.gokb.cred.TitleInstance%3A39490#publishers has 2 entries for Springer as the publisher both without end dates but with differing start dates. The import code will only save the first entry it comes across with that end-date and publisher.

jhsolomon commented 7 years ago

Fix confirmed 11/14