Skip to content

[FINNA-4570] Add container_title_str_mv field#110

Merged
EreMaijala merged 16 commits into
NatLibFi:devfrom
LuomaJuha:add-container-title-str-mv
May 26, 2026
Merged

[FINNA-4570] Add container_title_str_mv field#110
EreMaijala merged 16 commits into
NatLibFi:devfrom
LuomaJuha:add-container-title-str-mv

Conversation

@LuomaJuha
Copy link
Copy Markdown
Contributor

@LuomaJuha LuomaJuha commented May 12, 2026

  • Adds container_title_str_mv to records in solrupdater if it does not already exist and hierarchy_parent_title is present.
  • Ead and Ead3 assigns the value from hierarchy_top_title
  • Added tests to solrupdater to test adding the field properly

@LuomaJuha LuomaJuha changed the title Add container title facet field [FINNA-4570] Add container title facet field May 13, 2026
@LuomaJuha LuomaJuha marked this pull request as ready for review May 13, 2026 08:54
@LuomaJuha LuomaJuha changed the title [FINNA-4570] Add container title facet field [FINNA-4570] Add container_title_str_mv field May 13, 2026
@LuomaJuha LuomaJuha requested review from EreMaijala and removed request for EreMaijala May 13, 2026 10:35
@LuomaJuha LuomaJuha requested a review from EreMaijala May 13, 2026 11:11
Comment thread src/RecordManager/Finna/Solr/SolrUpdater.php Outdated
Comment thread src/RecordManager/Finna/Solr/SolrUpdater.php Outdated
Comment thread tests/fixtures/Finna/SolrUpdaterTest/ead.xml Outdated
@LuomaJuha LuomaJuha requested a review from EreMaijala May 19, 2026 07:44
Comment thread src/RecordManager/Finna/Solr/SolrUpdater.php Outdated
@LuomaJuha LuomaJuha requested a review from EreMaijala May 26, 2026 05:13
@EreMaijala EreMaijala merged commit ae8caf4 into NatLibFi:dev May 26, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants