Skip to content

Metadata editor / add a configuration to define mandatory keywords from a thesaurus#9354

Merged
josegar74 merged 2 commits into
geonetwork:mainfrom
GeoCat:thesaurus-required
Jun 23, 2026
Merged

Metadata editor / add a configuration to define mandatory keywords from a thesaurus#9354
josegar74 merged 2 commits into
geonetwork:mainfrom
GeoCat:thesaurus-required

Conversation

@josegar74

Copy link
Copy Markdown
Member

The directive gnKeywordSelector has the required attribute to display the field as mandatory, but it was not used.

This change request adds an attribute in the thesaurus element of the metadata editor configuration to allow to define a thesaurus as mandatory:

<thesaurus 
  key="local.theme.pg-privacy" fieldset="false" maxtags="1"  transformations="to-iso19139-keyword"
  mandatory="true" />
image

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

@josegar74 josegar74 added this to the 4.4.12 milestone Jun 23, 2026
Add a reference table for the <thesaurus> element attributes (key,
maxtags, orderById, fieldset, browsable, transformations and the new
mandatory attribute) in the custom editor documentation. Note that
mandatory is a visual required indicator only and does not enforce
keyword selection on save.

@juanluisrp juanluisrp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. I've added some updated documentation too.

@josegar74 josegar74 merged commit a7f08b5 into geonetwork:main Jun 23, 2026
8 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.

3 participants