sul-dlss-deprecated / dlme-traject

Traject configs for the DLME project (DEPRECATED -- MOVED UNDER DLME-TRANSFORM)
0 stars 0 forks source link

Replace language-specific metadata fields with hash #139

Closed jacobthill closed 5 years ago

jacobthill commented 5 years ago

Some metadata fields ('agg_data_provider_ar', 'agg_data_provider_country_ar', 'agg_provider_ar', 'agg_provider_country_ar') have the language value recorded in the name of the field. These should be replaced with a hash value as used in https://github.com/sul-dlss/dlme-transform/pull/240 and https://github.com/sul-dlss/dlme-traject/pull/138).

Requirements:

This should not effect metadata_mappings.json or the methods used to retrieve those values, only the IR validation and the configs.

mjgiarlo commented 5 years ago

@jacobthill Can you unpack

for each config file that has 'agg_data_provider_ar', etc. replace with the new method

:question:

jacobthill commented 5 years ago

e.g. replace to_field 'agg_data_provider_ar', data_provider_ar with to_field 'agg_data_provider', data_provider_ar, lang('ar-Arab') and pass 'agg_data_provider' as an argument to each_record convert_to_language_hash. Repeat with all four fields that end in '_ar' (they should be at the bottom of each config under the comment # Arabic Agg. If I understand your implementation correctly I think that is what is required. Does that make sense?

mjgiarlo commented 5 years ago

@jacobthill :100: clear; I was being obtuse. :smiley:

I can make that change unless you would prefer to do it. What do you think?

jacobthill commented 5 years ago

If you have time, that would be helpful. Thank you

mjgiarlo commented 5 years ago

If you have time, that would be helpful. Thank you

will do!

mjgiarlo commented 5 years ago

@jacobthill ok, all the changes to test this out are now in the master branches of dlme-traject and dlme-transform. Ready for testing tomorrow. cc: @aaron-collier