Skip to content

glossary property in the manifest is not allowed on content documents #1170

Closed

Description

@acabal

Tested on epubcheck 4.2.4.

At Standard Ebooks we're trying to add a glossary according to the EPUB Dictionaries and Glossaries 1.0 spec.

In the manifest file, we add two items:

<item href="glossary-search-key-map.xml" id="glossary-search-key-map.xml" media-type="application/vnd.epub.search-key-map+xml" properties="search-key-map glossary"/>
<item href="text/glossary.xhtml" id="glossary.xhtml" media-type="application/xhtml+xml" properties="glossary"/>

According to the spec, we should be able to add the glossary property both to the search key map file to to indicate that the file applies to a glossary, and to the XHTML content document file to indicate that it contains the actual glossary.

However epubcheck outputs:

ERROR(OPF-015): /path/to/glossary.xhtml(-1,-1): The property 'glossary' should not be declared in the OPF file.

It seems to allow glossary for the search key map, but not for the XHTML file.

Activity

  1. changed the title [-]glossary property in the manifest is not allowed[/-] [+]glossary property in the manifest is not allowed on content documents[/+] on Jul 30, 2020
  2. rdeltour commented on Aug 4, 2020

    @rdeltour
    Member

    Have you checked that your glossary.xhtml XHTML Content Document includes at least an element with the epub:type glossary? This is mandatory for the glossary property to be allowed on the manifest item.

    If that's not the issue, then it's possibly a bug in EPUBCheck 😊

  3. self-assigned this
    on Aug 4, 2020
  4. added this to the v4.2.5 milestone on Aug 4, 2020
  5. acabal commented on Aug 4, 2020

    @acabal
    ContributorAuthor

    Yes, glossary.xhtml has an element with epub:type="glossary", though it's on a <section> element in glossary.xhtml, not a <dl> element.

  6. rdeltour commented on Aug 5, 2020

    @rdeltour
    Member

    OK, thanks for your answer 😊
    Are you running EPUBCheck with the dict profile?
    Would you be able to share a test EPUB?

  7. rdeltour commented on Aug 5, 2020

    @rdeltour
    Member

    I was able to reproduce, so no need to send a test EPUB after all 😉
    I'll dig deeper, but it does seem like a bug. Thanks for the report!

  8. added
    type: false-positiveThis issue is about valid content being incorrectly rejected
    and removed
    status: waiting for feedbackThe development team needs feedback from the issue’s creator
    type: unverifiedThe issue couldn't be reproduced
    on Aug 5, 2020
  9. added a commit that references this issue on Feb 26, 2021
    58d773c
  10. added
    status: has PRThe issue is being processed in a pull request
    and removed on Feb 26, 2021
  11. added a commit that references this issue on Feb 26, 2021
    d1727d8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

spec: EPUB Dictionaries and GlossariesImpacting the support of the EPUB Dictionaries and Glossaries specificationstatus: has PRThe issue is being processed in a pull requesttype: false-positiveThis issue is about valid content being incorrectly rejected

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    ⌂ 换网址