ARUP-CAS / aiscr-dapro

OAI-PMH Data Provider pro AMČR
https://api.aiscr.cz/
GNU General Public License v3.0
2 stars 0 forks source link

Samostatné nálezy - datestamp #45

Closed motyc closed 3 years ago

motyc commented 4 years ago

Patrně souvisí s dolaďováním historie, ale momentálně se vkládá špatná (defaultní) hodnota. Také by se měly v záznamu objevit tagy:

<datum_zapisu>
<odpovedny_pracovnik_zapisu>
<datum_odeslani>
<odpovedny_pracovnik_odeslani>
<datum_potvrzeni>
<odpovedny_pracovnik_potvrzeni>
<datum_archivace>
<odpovedny_pracovnik_archivace>

Z nich se datestamp odvodí. Obsahovat by měly poslední údaj daného typu z historie.

pbartosova commented 4 years ago

Pozn.: prvni polozky musi byt

pbartosova commented 4 years ago

To vlastne neni problem api ale exportu. Co byl zásah do api, je hotovo.

motyc commented 4 years ago

Zde tomu moc nerozumím, nebo je to špatně. Stav se jmenuje "zápis", proto by mělo jít o "odpovedny_pracovnik_zapisu", ale hlavně mi tam chybí výpis těch prostředních dvou stavů (odeslání a potvrzení). Historie je tak neúplná. Mělo by to odpovídat tomu, co je v prvním komentáři, což neplatí.

pbartosova commented 4 years ago

Názvy pro datum i odpovědného pracovníka odpovídají ze zadání a následnému pojmenování polí. Takže z mého pohledu bych za chybný považovala název stavu, ten byl ale také odsouhlasený. Bylo by dobré to sjednotit , pokud to bude možné v rámci celé amčr, aby byly názvy jasné. Juraj se na to může podívat během analýzy DB.

pbartosova commented 4 years ago

Měla by být opravena hodnota pro datestamp.

pbartosova commented 4 years ago

Chybějící pole jsou chybou exportu, ne api.

motyc commented 4 years ago

Datestamp je v pořádku, to potvrzuji. Název stavu tolik podstatný není, jen to mate. Včera jsme téma historie tak jako tak s Jurajem probírali a určitě v tom změny budou. Přinejmenším z tabulek zmizí daná pole a budou existovat jen závislé tabulky (či tabulka) s historií. Zvlášť u samostatných nálezů, kde máme tabulku bokem, jsou pole zcela redundantní a zbytečná (to se však samozřejmě netýká API, kam je dostat chceme).