Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validátor verze 2.2.1 - datum/čas podle ISO 8601 #132

Closed
isenkyr opened this issue Dec 16, 2021 · 6 comments
Closed

Validátor verze 2.2.1 - datum/čas podle ISO 8601 #132

isenkyr opened this issue Dec 16, 2021 · 6 comments
Assignees
Labels
bug next release věc je vyřešena v příštím releasu, potřeba ještě otestovat, jakmile ten výjde ready to test Opraveno a připraveno k otestování validatorConfig slovníky a fDMF v adresáři validatorConfig
Milestone

Comments

@isenkyr
Copy link

isenkyr commented Dec 16, 2021

Dobrý den,

včera jsem si všiml, že byla zveřejněna další verze Komplexního validátoru v.2.2.1. Verzi jsem otestoval a dostal chybové hlášení mets_vsg001-0000yf.xml: /mets:mets/mets:metsHdr/@LASTMODDATE: hodnota '2021-09-22T12:07:54' nesplňuje žádnou ze specifikací [regulární výraz '^(?[0-9]{4})(?1[0-2]|0[1-9])(?3[01]|0[1-9]|[12][0-9])T(?2[0-3]|[01][0-9])(?[0-5][0-9])(?[0-5]0-9?)?(?Z|+-([0-5][0-9])?)$',regulární výraz '^(?[0-9]{4})-(?1[0-2]|0[1-9])-(?3[01]|0[1-9]|[12][0-9])T(?2[0-3]|[01][0-9]):(?[0-5][0-9])(:(?[0-5][0-9])(.(?[0-9]{3}))?)?(?Z|+-(:([0-5][0-9])?)?)$']

Z uvedeného mi plyne, že součástí datumu má být také časové pásmo, čemuž rozumím. Problém bych, ale viděl v tom že standard toto přesně nedefinuje. Současně také nedošlo k žádné úpravě ve standardu a mění se validační pravidla. Dále pak co s ostatními datumy, které nemají ve standardu explicitně definovanou časovou zónu, ale mají být podle normy ISO8601, např. mods:recordCreationDate, mets:file[@created], info.xml element created a další?

Děkuji I.Šenkýř

@FilipPavcik
Copy link
Collaborator

FilipPavcik commented Dec 16, 2021

Dobrý deň,

máte pravdu, je to tak ako píšete. Štandard presne nestanovuje v rámci dátumu časovú zónu a dátumy by mali byť uvádzané podľa normy ISO8601. (V METS hlavičke na úroveň vteřin.)

@rzeh4n Vedeli by ste sa na to pozrieť a prípadne to upraviť? Ďakujem.

@rzeh4n rzeh4n self-assigned this Dec 16, 2021
@rzeh4n rzeh4n changed the title Validátor verze 2.2.1 Validátor verze 2.2.1 - datum/čas podle ISO 8601 Dec 16, 2021
@rzeh4n rzeh4n added this to the 2.2.2 milestone Dec 16, 2021
@jbil7
Copy link
Collaborator

jbil7 commented Mar 13, 2023

Domnívám se, že chyba by se dala opravit, a sice v souboru $VALIDATOR_HOME/validatorConfig/fDMF/$DMF_VERSION/metsProfiles/primary_mets_header.xml vždy u regexp (v tomto případě u druhého, pro extended format) přidat otazník mezi dolar a poslední pravou kulatou závorku na konci regexu:

<regexp>^(?&lt;year&gt;[0-9]{4})-(?&lt;month&gt;1[0-2]|0[1-9])-(?&lt;day&gt;3[01]|0[1-9]|[12][0-9])T(?&lt;hour&gt;2[0-3]|[01][0-9]):(?&lt;minute&gt;[0-5][0-9])(:(?&lt;second&gt;[0-5][0-9])(.(?&lt;millis&gt;[0-9]{3}))?)?(?&lt;timezone&gt;Z|[+-](2[0-3]|[01][0-9])(:([0-5][0-9])?)?)?$</regexp>

Tímto zásahem se bude se skupinou timezone pracovat jako s nepovinnou.

@rzeh4n
Copy link
Collaborator

rzeh4n commented Apr 25, 2023

dávno opraveno

@rzeh4n rzeh4n closed this as completed Apr 25, 2023
@jbil7
Copy link
Collaborator

jbil7 commented Oct 2, 2023

Problém v novém release 2.3 pořád trvá, viz např.: fDMF pro mono v.2.0, mono v.1.4, perio v.1.8 atd.

Z toho důvodu issue znovu otevírám.

@jbil7 jbil7 reopened this Oct 2, 2023
@jbil7 jbil7 added validatorConfig slovníky a fDMF v adresáři validatorConfig bug labels Oct 18, 2023
@rzeh4n
Copy link
Collaborator

rzeh4n commented May 9, 2024

Opravil jsem, kde to jeste zustalo v bb40ac9

@rzeh4n rzeh4n added the next release věc je vyřešena v příštím releasu, potřeba ještě otestovat, jakmile ten výjde label May 9, 2024
@jbil7 jbil7 added the ready to test Opraveno a připraveno k otestování label Jun 2, 2024
@jbil7 jbil7 modified the milestones: 2.2.2, 2.4.2 Jul 2, 2024
@jbil7
Copy link
Collaborator

jbil7 commented Jul 15, 2024

Uzavírám jako opravené.

@jbil7 jbil7 closed this as completed Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug next release věc je vyřešena v příštím releasu, potřeba ještě otestovat, jakmile ten výjde ready to test Opraveno a připraveno k otestování validatorConfig slovníky a fDMF v adresáři validatorConfig
Projects
None yet
Development

No branches or pull requests

4 participants