andreareider / epubcheck

Automatically exported from code.google.com/p/epubcheck
0 stars 0 forks source link

Empty tags in toc.ncx are not recognized. #302

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
PROBLEM:

In toc.ncx empty <text> elements for <docTitle> or <docAuthor> or <navLabel> is 
not flagged as a warning.

METHOD:

What steps will reproduce the problem?
1. See attached EPUB file.
2. Open OEBPS/toc.ncx
3. Take a look at the elements mentioned above:

  <docTitle>
    <text></text>
  </docTitle>

  <docAuthor>
    <text></text>
  </docAuthor>

  <navMap>
    ...
    <navPoint id="chapter-2" playOrder="3">
      <navLabel>
        <text></text>
      </navLabel>
      <content src="chapter-2.html"/>
    </navPoint>

RESULT:

No warning issued.

EXPECTED:

Issue warning that there are empty <text> elements.

PLATFORM:

java version "1.6.0_51"
Java(TM) SE Runtime Environment (build 1.6.0_51-b11-456-10M4508)
Java HotSpot(TM) Client VM (build 20.51-b01-456, mixed mode)

Epubcheck Version 3.0.1

ADDITIONAL INFO:

Example test file attached.

Original issue reported on code.google.com by jens.tro...@gmail.com on 30 Aug 2013 at 11:40

Attachments:

GoogleCodeExporter commented 8 years ago
epubcheck doesn't complain because this is valid NCX / XML. Therefore: no ERROR 
or WARNING for this issue...

However, I have to admit that a HINT (introduced in v3.0.1) would be 
appropriate to report empty text elements as it is totally clear that the 
element shouldn't be empty...

Original comment by tobias.c...@gmail.com on 24 Sep 2013 at 5:42

GoogleCodeExporter commented 8 years ago
Actually, websites like Bookbaby reject ebooks with such empty elements, and 
refuse to publish them.  As such, this should at least warrant a warning, or 
hint, yes.

Original comment by jens.tro...@gmail.com on 25 Sep 2013 at 5:10

GoogleCodeExporter commented 8 years ago
THIS ISSUE MOVED TO GITHUB:
https://github.com/IDPF/epubcheck/issues/

Original comment by rdeltour@gmail.com on 7 Nov 2013 at 10:49