Description
A few Solr field suffix issues came up in PR #29 beyond the depositor_tesim fix (tracked in #39). The most obvious one: all_text_timv in core_file.rb:72 uses a timv suffix (datetime multi-valued) for what appears to be a full-text field — that doesn't make sense and the intent is unclear. More broadly, it's worth doing a pass over all the dynamic field suffixes in the to_solr methods and checking them against the schema to make sure we're not storing data in the wrong field types.
Steps
- List every field and suffix used across
to_solr in collection.rb, project.rb, and core_file.rb
- Cross-reference against
solrconfig/tapas/conf/schema.xml dynamic field definitions
- Note any mismatches
- Fix them in the model
to_solr methods and update catalog_controller.rb as needed
- Reindex after changes
Acceptance criteria
- All Solr fields use suffixes that match the data type being stored
all_text_timv resolved — removed, corrected, or at minimum documented with a TODO- No field type mismatches between models and schema
Description
A few Solr field suffix issues came up in PR #29 beyond the
depositor_tesimfix (tracked in #39). The most obvious one:all_text_timvincore_file.rb:72uses atimvsuffix (datetime multi-valued) for what appears to be a full-text field — that doesn't make sense and the intent is unclear. More broadly, it's worth doing a pass over all the dynamic field suffixes in theto_solrmethods and checking them against the schema to make sure we're not storing data in the wrong field types.Steps
to_solrincollection.rb,project.rb, andcore_file.rbsolrconfig/tapas/conf/schema.xmldynamic field definitionsto_solrmethods and updatecatalog_controller.rbas neededAcceptance criteria
all_text_timvresolved — removed, corrected, or at minimum documented with a TODO