XMP: Import of PDF with XMP data does not show XMP

[koppor]

Test PDF: https://github.com/JabRef/jabref/blob/feature/xmp-provement/src/test/resources/pdfs/KoppAZ-MADR-ZEUS-2018/KoppAZ-MADR-ZEUS-2018.pdf

Drag'n'drop on Maintable results in following dialog:

In case XMP data is found, it reads as follows:

[johannes-manner]

The problem is, that the xmp metadata section is faulty:

Line 9,10 and line 64,65: You have to delete the lines 9 and 65 and the xmp import works fine :)

How has this happend, that the ?xpacket is twice in the metadata section? The org.apache.xmpbox.xml.DomXmpParser throws an exception "xmp should end after xpacket end processing instruction". Maybe you can support me with more context :)

[koppor]

I sent it through the validator - and it is OK:

OK, I See. Can JabRef at the XMP export remove

<?xpacket begin="" id="W5M0MpCehiHzreSzNTczkc9d"?>

and

<?xpacket end="w"?>

?

The result is now:

@inproceedings{KoppEtAl-MADR2018,
  author = {Oliver Kopp and Anita Armbruster and Olaf Zimmermann},
  booktitle = {Proceedings of the 10th Central European Workshop on Services and their Composition ({ZEUS} 2018)},
  keywords = {ADR, MADR, architecture decision records, architectural decision records, Nygard},
  month = {#jan#},
  pages = {55--62},
  publisher = {CEUR-WS.org},
  series = {{CEUR} Workshop Proceedings},
  title = {Markdown Architectural Decision Records: Format and Tool Support},
  url = {https://adr.github.io/madr/},
  volume = {2072},
  year = {2018}
}

So, it basically looks great, but the month should be displayed as month = jan. I do not know where this comes from.

[johannes-manner]

Is it necessary to remove the duplicated begin-tags?

If I write the xmp data with JabRef, there is no duplicated entries. Otherwise I have to modify the InputStream before the dom xmp parsing happens, because I have no control over the org.apache.xmpbox.xml.DomXmpParser.DomXmpParser.

A workaround would be to capture the exception and return a empty bib entry with a comment, that the xmp is not valid or another error message?

I will check the month problem!

[koppor]

Do we need this XML processing instructions at export? If we don't export them, the latex package xmpincl can be used without modifications Johannes Manner <notifications@github.com> schrieb am Mi., 18. Apr. 2018, 12:13:

…

Is it necessary to remove the duplicated begin-tags? If I write the xmp data with JabRef, there is no duplicated entries. Otherwise I have to modify the InputStream before the dom xmp parsing happens, because I have no control over the org.apache.xmpbox.xml.DomXmpParser.DomXmpParser. A workaround would be to capture the exception and return a empty bib entry with a comment, that the xmp is not valid or another error message? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#338 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABTafiQ0woETydzFZmWiq_b9MZwYvn5Gks5tpxHWgaJpZM4TSCXn> .

[johannes-manner]

Okay, now I understand the problem. The export xmp should not contain the begin-tag, as mentioned in the xmpincl package.

I reproduced your problem now. Normally, we don't need this begin-xml-processing instruction!

[koppor]

Fixed by JabRef#3964