Fix module according to feedback

Bystroushaak commented 9 years ago

It will be necessary to add more of the postprocessing. Email feedback from Jaroslav Kvasnica:

1) Celé location je špatně. Element <physicalLocation> by se neměl používat pro elektronické dokumenty. Element <url> pak musí vést na místo, kde je dokument zpřístupněn (K4??). V elementu <relatedItem> by další <location> již nemělo být.

  <mods:location>
    <mods:physicalLocation authority="siglaADR">ABA001</mods:physicalLocation>
    <mods:url displayLabel="electronic resource" usage="primary display">http://edeposit-test.nkp.cz/producents/beta-nakladatelstvi/epublications/controlling-v-msp-nejen-o-cislech/controlling.pdf</mods:url>
    <mods:url displayLabel="electronic resource">http://edeposit.nkp.cz/</mods:url>
  </mods:location>

Takto by to mělo správně vypadat. Pokud budmeme chtít přidat odkaz do K4, tak do elementu <url> a dát ho hned od <location>

    <mods:location>
        <mods:holdingSimple>
            <mods:copyInformation>
            <!-- <mods:electronicLocator>http://edeposit.nkp.cz</mods:electronicLocator> pole 856 $u -->
                <mods:electronicLocator>http://edeposit-test.nkp.cz/producents/zizala/epublications/draci-doupe-plus/drdplus-bestiar.pdf</mods:electronicLocator>
            </mods:copyInformation>
        </mods:holdingSimple>
    </mods:location>

2) u elementu <genre> je špatný obsah. Má tam být <mods:genre>electronic volume</mods:genre> místo "elektronic title". Genre vždy odpovídá tomu, co je uvedeno jako ID u elementu <mods>

3) element <issuance> sice umožňuje použít hodnotu "monografic", ale je lepší používat hodnoty "single unit" pro běžnou monografii a "multipart monograph" pro vícesvazky

4) jméno adresáře ve specifikace je "data" místo "original"

5) u info.xml chybí deklarace schéma <info xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://www.ndk.cz/standardy-digitalizace/info11.xsd"> s tím souvisí i chybějící povinný element <mainmets>, který ty nemůžeš naplnit, ale možná by se měl dát prázdný, kvůli validaci

Bystroushaak commented 9 years ago

All of the issues are now fixed. I will send the new test data to check them.

Bystroushaak commented 9 years ago

I've sent the new test data to mr. Kvasnica.

Bystroushaak commented 8 years ago

Time to update the export procedure and tests, then we can send next data batch.

Bystroushaak commented 8 years ago

Email from mr. Kvasnica:

Ahoj všem,

AIP SAFE nahrála novou verzi epubflow, který upravila podle našich požadavků. Takže bychom potřebovali nové testovací balíčky.

Jen je třeba vrátit některé změny, protože jsme nepočítali, že to změní, tak rychle.

epublow nyní vyžaduje název info.xml ve formátu s ID, jak to bylo původně. (info_XXX.xml)

epublow take už umí pracovat s .md5 souborem a dokonce jej teď vyžaduje

Tak vraťte prosím zpět tyto dvě úpravy a pošlete nám prosím balíčky. Nyní jen s valídními soubory (PDF, EPUB). V této fázi již nemá cenu testovat validátory souborů.

S pozdravem J.

Bystroushaak commented 8 years ago

Disabled features allowed again. See 38b2ca126629445a088d9473115cd0f758c7b35c for details.

edeposit / edeposit.amqp.ltp

Fix module according to feedback #22