Skip to content

Comments

Fix time search filter in OAI harvester configuration#8902

Open
koplas wants to merge 2 commits intogeonetwork:mainfrom
koplas:time-oai
Open

Fix time search filter in OAI harvester configuration#8902
koplas wants to merge 2 commits intogeonetwork:mainfrom
koplas:time-oai

Conversation

@koplas
Copy link

@koplas koplas commented Jul 1, 2025

Specifying the time range results in an error as the backend expects an ISO formatted date.

Checklist

  • I have read the contribution guidelines
  • Pull request provided for main branch, backports managed with label
  • Good housekeeping of code, cleaning up comments, tests, and documentation
  • Clean commit history broken into understandable chucks, avoiding big commits with hundreds of files, cautious of reformatting and whitespace changes
  • Clean commit messages, longer verbose messages are encouraged
  • API Changes are identified in commit messages
  • Testing provided for features or enhancements using automatic tests
  • User documentation provided for new features or enhancements in manual
  • Build documentation provided for development instructions in README.md files
  • Library management using pom.xml dependency management. Update build documentation with intended library use and library tutorials or documentation

Specifying the time range results in an error as the backend
expects an ISO formatted date.
@josegar74 josegar74 added this to the 4.4.9 milestone Jul 24, 2025
Copy link
Member

@josegar74 josegar74 left a comment

Choose a reason for hiding this comment

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

Tested and works, the following error does not longer happens:

2025-09-16T15:59:33,345 ERROR [geonetwork.domain] - Error parsing ISO DateTimes 'Tue Sep 16 2025 00:00:00 GMT+0200 (Central European Summer Time)'. Error is: Text 'ue Sep 16 2025 00:00:00 GMT+0200 (Central European Summer Time)' could not be parsed at index 0
java.time.format.DateTimeParseException: Text 'ue Sep 16 2025 00:00:00 GMT+0200 (Central European Summer Time)' could not be parsed at index 0

But after saving the form, refreshing the form doesn't display the values selected in the UI, it displays the date selectors without a selected date.

@koplas
Copy link
Author

koplas commented Sep 21, 2025

But after saving the form, refreshing the form doesn't display the values selected in the UI, it displays the date selectors without a selected date.

I currently don't have a quick fix for this. In a week I should have time to investigate this.

@jahow jahow modified the milestones: 4.4.9, 4.4.10 Oct 7, 2025
@koplas koplas requested a review from josegar74 October 8, 2025 08:49
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.

3 participants