Skip to content

Improve persons & terms list checks#358

Merged
joewiz merged 3 commits intomasterfrom
fix-persons-terms-checks
May 15, 2025
Merged

Improve persons & terms list checks#358
joewiz merged 3 commits intomasterfrom
fix-persons-terms-checks

Conversation

@joewiz
Copy link
Member

@joewiz joewiz commented May 15, 2025

  • move ID-related checks into frus-id-checks.sch
  • leave only structural checks in frus.sch
  • add schematron quick fixes to add IDs to persName & terms and ensure uniqueness, using legacy rules for assigning IDs within a volume
    • Person: "p_" + initials + "_1" (where 1 is changed to 2 if another person has already been assigned this ID in the volume, 2 to 3, and so on)
    • Term: "t_" + capitals + "_1" (where 1 is changed to 2 if another term...)

joewiz added 2 commits May 15, 2025 10:38
- move ID-related checks into frus-id-checks.sch
- leave only structural checks in frus.sch
- add schematron quick fixes to add IDs to persName & terms and ensure uniqueness, using legacy rules for assigning IDs within a volume
  - Person:  "p_" + initials + "_1"  (where 1 is changed to 2 if another person has already been assigned this ID in the volume, 2 to 3, and so on)
  - Term: "t_" + capitals + "_1" (where 1 is changed to 2 if another term...)
@joewiz joewiz force-pushed the fix-persons-terms-checks branch from 35d23a2 to b5f99bd Compare May 15, 2025 14:39
@joewiz joewiz merged commit 160559b into master May 15, 2025
@joewiz joewiz deleted the fix-persons-terms-checks branch May 15, 2025 15:36
@github-actions
Copy link

🎉 This PR is included in version 0.7.21 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant