Open tfmorris opened 1 year ago
@hornc @mekarpeles Because I created this from a comment, it didn't get any of the standard boilerplate tags which are in the new issue template. It's a bug that I've split out from #7349 because I have a fix in hand for it. It's another Day 0 bug from 2010 which has never worked and was never tested because the test cases were built from the output of the existing implementation.
Oops! This is a duplicate of #2103.
As @hornc pointed out on the PR, there are a number of problems with using the fuller_name
field. In addition to not being visible/editable/searchable, it's also associated with a particular name form which was imported from the MARC record, but there's no way to maintain that correspondence .
I plan to drop the fuller_name
field and instead construct a name to be added to alternate_names
. I've looked at several tens of thousands of records with 100$q from various sources and of various vintages and have found a variety of cases. Here are some that can be handled easily (about 90%+ of the occurrences):
And some which we punt on (at least for now), appending the $q to the normalized name:
A secondary question is which name form should be the primary name
and which should go in alternate_names
.
@seabelis @hornc @mekarpeles any opinions?
Support for $q (and $6 linkage) needs to be added for not only 100 fields, but also 600, 700, and 800 fields.
As mentioned in my comment to #7349 MARC 100$q subfields aren't being imported:
Originally posted by @tfmorris in https://github.com/internetarchive/openlibrary/issues/7349#issuecomment-1368295193