TAMULib / SAGE

Search Aggregation Engine
MIT License
6 stars 1 forks source link

The sage-1_0 should use "*_string" rather than "*_str", "string" rather than "whole", and other core related changes. #551

Closed kaladay closed 11 months ago

kaladay commented 11 months ago

Description

Using the incorrect copyfield name results in a failure when populating the index.

The filter query does not work as expected for "whole" strings. Change these "whole" strings used in core types to "string" (this will need further runtime testing).

The PR #549 introduced the "multivalue" field across the board. Play it safe and do not add the explicit "multivalue" attribute to the core types.

The date type cannot be used for date_published and date_created because the provided dates may not be in a supported format.

Both collection and resource are multivalued in DSpace. Change the solr core to support multiple values for both collection and resource.

Type of change

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist:

coveralls commented 11 months ago

Coverage Status

coverage: 45.792% (-0.02%) from 45.815% when pulling c42600d1ab83f93de90bc848c03fc65614083809 on solrconfig_should_use_string into 074fc8269e3a83b970b2da1110c084ce8ada915b on sprint-staging.