Skip to content

Conversation

robyngit
Copy link
Member

This pull request updates MetacatUI to version 2.34.1 and introduces a fix to how resource maps are resolved when multiple versions are present. The ResourceMapResolver can now handle cases where more than one RM is found, selecting the most recent, non-obsoleted version. Several helper methods have been added to the VersionTracker to support this logic, and documentation and version references have been updated throughout the project.

See PR #2726 for more details

robyngit and others added 6 commits August 6, 2025 11:27
We are using the minified version and the old file was an older version.
- Add a resolution step to the ResourceMapResolver that handles cases where multiple ResourceMaps are found in Solr
- Step checks all found RMs are versions of eachother and returns the one that is not yet obsoleted, if it exists
- Fixes bug where resolver failed to find RM when multiple were found, and the metadata doc's previous versions had no RMs (i.e. walk Sys meta step failed)

Issue #2725
- Ensure the Editor retrieves the true latest version of a document by re-fetching system metadata instead of relying on a previously cached "end" of the version chain
- Accounts for sysmeta changes made since the cache was saved (e.g. new version created in another browser) and prevents some cases where the Editor can't save because pid has already been obsoleted
- When walking the version chain in the Resource Map resolver, bypass the cached end-of-chain data and fetch fresh sysmeta to ensure accuracy.

Issue #2725, #2724
- VersionTracker: reduce comment width, fix linting issue
- BioontologyResults, EventLog tests: small unit test updates corresponding to v 2.34.0 releases
…-results

Fix resMap resolution when multiple versions found in index + refresh sysmeta when resolving latest version
@robyngit robyngit marked this pull request as ready for review August 14, 2025 16:51
Copy link
Member

@rushirajnenuji rushirajnenuji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for adding the fix @robyngit! 🚀

@robyngit robyngit merged commit 92d45a0 into main Aug 14, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants