proarc / proarc-client

3 stars 0 forks source link

Kontrola hodnot v dateissued v číslech a přílohách #399

Open luckajirku opened 1 year ago

luckajirku commented 1 year ago

Odložím si tu issue s nejasným zadáním zatím k přemýšlení/diskusi. Bylo by fajn, kdyby šlo nějak víc hlídat u periodik datum. Kromě formátu jako takového (je v https://github.com/proarc/proarc-client/issues/378), kdyby PA upozornil na možnou chybu, pokud by číslo (příloha) pod ročníkem neodpovídalo tomu roku, který je v popisu ročníku. Stává se, že v čísle člověk zadá jiný rok (typicky ten současný, nebo na začátku dalšího ročníku dá automaticky rok z ročníku předchozího apod.)... Budou tam i zádrhele jako rozmezí let atd., ale hlídat by se to myslím dalo.

ZdenkaSera commented 1 year ago

👍 - Můžeme k tomu dát "návrh na rozšíření"?

luckajirku commented 1 year ago

jj, tak to bylo myšleno, aby se s tím počítalo. i proto to sem dávám teď, i když je to takové hodně neurčité zatím...

luckajirku commented 1 year ago

Pak je otázka, jestli by to mělo kontrolovat už přímo při ukládání formuláře toho čísla/přílohy (bude fungovat jen když bude objekt ve struktuře, u samostatně založených by neměl proti čemu kontrolovat - ale těch bude minimum). Kde by to dávalo smysl vždycky, to by byla validace na úrovni ročníku, že by to směrem dolů pak zkontrolovalo, jestli roky sedí. Stejně tak u validace při exportu.

ZdenkaSera commented 1 year ago

Týká se ručně zakládaných ročníků/čísel/příloh period a e-periodik, tj. ne objektů stahovaných z katalogu

luckajirku commented 5 months ago

jako úplný základ by tady měla být kontrola povolených hodnot na té které pozici (v závislosti na použitém formátu - https://github.com/proarc/proarc-client/issues/378) - př. MM může mít jen hodnotu 01-12 atp (teď můžu zadat klidně třináctý měsíc).

luckajirku commented 5 months ago

nebo jestli to půjde napojit na kalendář - a mít u dat obdobné kontroly jako při vytváření čísel periodika přes více...

SykoraLukas commented 5 months ago

Při uložení čísla kontrolovat, že rok dateIssue odpovídá roku vydání ročníku.

Kontrola pro NDK Číslo, NDK přílohu čísla, NDK přílohu ročníku.

@kerschfilip ověřit u NDK eDokumentů.

SykoraLukas commented 2 months ago

Týká se ručně zakládaných ročníků/čísel/příloh period a e-periodik, tj. ne objektů stahovaných z katalogu

Toto nejde odlišit. Informaci o tom, že data jsou stažena z katalogu víme jen v době stažení. Poté již ne, takže se bude kontrolovat při každém stažení.

Kontrola tedy bude zda dateIssued odpovídá ročníku, ke kterému je přidán. Vzhledem k tomu, že u určitých okrajových čísel může dojít k tomu, že vyšlo ještě v předchozím nebo následujícím roce, bych udělal toto jako překonatelnou chybu při exportu.

Kontrolovat se kromě toho bude, že je daná hodnota pro den, měsíc a rok v odpovídajícím číselném rozmezí.

SykoraLukas commented 2 months ago

Kontrolováno, zda hodnota odpovídá ročníku. Ve validaci se to hlásí jako překonatelná chyba, aby při exportu toto šlo překonat.

SykoraLukas commented 2 months ago

Kontroluje se na straně API při uložení.