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

Fix module according to feedback #22

Open
Bystroushaak opened this issue Mar 13, 2015 · 5 comments
Open

Fix module according to feedback #22

Bystroushaak opened this issue Mar 13, 2015 · 5 comments
Labels

Comments

@Bystroushaak
Copy link
Member

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>
  1. 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>

  2. 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

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

  4. 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
Copy link
Member Author

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

@Bystroushaak
Copy link
Member Author

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

@Bystroushaak
Copy link
Member Author

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

@Bystroushaak
Copy link
Member Author

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
Copy link
Member Author

Disabled features allowed again. See 38b2ca1 for details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant