Skip to content

[FEATURE] Include facets from all associated metadata sections for fulltext pages#1441

Merged
sebastian-meyer merged 5 commits intokitodo:mainfrom
michaelkubina:bugfix_fulltext_faceting
Feb 1, 2025
Merged

[FEATURE] Include facets from all associated metadata sections for fulltext pages#1441
sebastian-meyer merged 5 commits intokitodo:mainfrom
michaelkubina:bugfix_fulltext_faceting

Conversation

@michaelkubina
Copy link
Collaborator

Problem: Previously, full-text pages only indexed facets from the top-level element (e.g., Type: Monograph, Volume), which meant that additional metadata in other metadata sections were not utilized, even though pages were explicitly associated with these through the StructLink/structMap-Logical. Basically all extensive efforts made in Kitodo.Production to structure the document internally (apart from the bibliographic record) were not even used on the page-level.

Solution: This PR extends the indexing to include facets from all DMD sections associated with the page through the StructLink and StructMap-Logical as well as the corresponding (structure) type. As a result, full-text hits can now be filtered down to more granular structural elements they have been associated with (Table, Illustration, Article, Index, Entry, etc.). Additionally, any deeper metadata beneath the top-level element (bibliographic record) is now included in the indexing process as well, expanding the possibilities to filter fulltext-results further - e.g. filtering for Authors of journal-articles instead of only the editor/publisher of a journal.

@sebastian-meyer sebastian-meyer self-requested a review February 1, 2025 01:45
@sebastian-meyer sebastian-meyer added the ☇ bug A non-security related bug. label Feb 1, 2025
@sebastian-meyer sebastian-meyer added ↷ feature A new feature or enhancement. and removed ☇ bug A non-security related bug. labels Feb 1, 2025
@sebastian-meyer sebastian-meyer changed the title [BUGFIX] Fix fulltext pages to include facets from all associated metadata sections [FEATURE] Include facets from all associated metadata sections for fulltext pages Feb 1, 2025
@sebastian-meyer sebastian-meyer merged commit b227afc into kitodo:main Feb 1, 2025
8 checks passed
@codecov
Copy link

codecov bot commented Feb 1, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (d64f433) to head (681e72e).
Report is 1 commits behind head on main.

Additional details and impacted files
@@     Coverage Diff      @@
##   main   #1441   +/-   ##
============================
============================

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

↷ feature A new feature or enhancement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants