search

Grandt / PHPePub

PHP Classes for dynamically generating EPub files.
http://www.phpclasses.org/package/6115
241 stars 83 forks source link

Generated epub file does not display on iBook. #27

Open tassoskoutlas opened 10 years ago

tassoskoutlas commented 10 years ago

Hi, I generated an epub version of a drupal node through the print module. The epub does not open in iBook and does not validate correctly.

“This page contains the following errors: error on line 21 at column 27: AttValue: " or ' expected Below is a rendering of the page up to the first error.”

A similar issue has been filed with the drupal issue queue https://www.drupal.org/node/2147659 but was closed and the author was redirected here.

Grandt commented 10 years ago

Can I get you to send me a sample ePub with this error. It looks like there's a problem with the framework delivering bad XHTML files. (ePub doesn't support all aspects of XHTML 1.1)

line 21 at column 27: AttValue: " or ' expected

sounds like there's an attribute in the tag, with missing quotes around its value. ie.

is illegal in XHTML, and should be <p class="header"> or

Cheers A.Grandt

On 09-07-2014 12:15, Tassos wrote:

Hi, I generated an epub version of a drupal node through the print module. The epub does not open in iBook and does not validate correctly.

“This page contains the following errors: error on line 21 at column 27: AttValue: " or ' expected Below is a rendering of the page up to the first error.”

A similar issue has been filed with the drupal issue queue https://www.drupal.org/node/2147659 but was closed and the author was redirected here.

— Reply to this email directly or view it on GitHub https://github.com/Grandt/PHPePub/issues/27.

tassoskoutlas commented 10 years ago

You can find it here http://speedy.sh/KVySh/Site-Install-An-article-2014-07-09.epub.txt

Grandt commented 10 years ago

Line 21 is:

    <meta name="robots" content=noindex, nofollow />          <link

rel='shortcut icon' href='http://bl.dev/misc/favicon.ico' type='image/x-icon' />

The problem is that content=noindex, nofollow is invalid in XHTML, actually it should be invalid in all forms of HTML, from 1.0 to 5, as it's two arguments with no quotes.

The problem is that the equp.html is full of elements that shouldn't be in an ebook, this is the danger of dynamically building these. For the most part, most can be excused, but because of all the missing scripts, you are going to get errors from epubcheck, though the above error is probably the showstopper in iBooks. And I'll assume Google books won't parse it either.

Second, RDFa is not a valid epub format.

Changing the first few lines to the supported XHTML 1.1 header:

<?xml version="1.0" encoding="utf-8"?> <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">

And fix line 21, by adding the missing quotes, causes ePub check to check the rest of the file returning the text I've pasted here below. As you can see, the XHTML is full of extra xml tags, that have not been correctly declared by a schema, probably because they are a part of the RDFa schema, but still. All other files haven't been added, most readers can survive that though. Epubcheck Version 3.0.1 Validating against EPUB version 2.0 ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html: Invalid path: OEBPS/../../misc/throbber.gif ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1318,157): attribute "typeof" not allowed here; expected attribute "alt", "dir", "height", "ismap", "lang", "longdesc", "src", "style", "title", "usemap", "width" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1323,151): attribute "about" not allowed here; expected attribute "dir", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1323,151): attribute "typeof" not allowed here; expected attribute "dir", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1325,45): attribute "property" not allowed here; expected attribute "class", "dir", "id", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1325,45): attribute "datatype" not allowed here; expected attribute "class", "dir", "id", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1330,132): attribute "property" not allowed here; expected attribute "class", "dir", "id", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1330,132): attribute "content" not allowed here; expected attribute "class", "dir", "id", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1330,132): attribute "datatype" not allowed here; expected attribute "class", "dir", "id", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1330,132): attribute "rel" not allowed here; expected attribute "class", "dir", "id", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1333,234): attribute "rel" not allowed here; expected attribute "dir", "id", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1333,234): attribute "resource" not allowed here; expected attribute "dir", "id", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1333,357): attribute "typeof" not allowed here; expected attribute "alt", "class", "dir", "height", "id", "ismap", "lang", "longdesc", "src", "style", "title", "usemap", "width" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1333,539): attribute "property" not allowed here; expected attribute "dir", "id", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1338,230): attribute "rel" not allowed here; expected attribute "dir", "id", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1338,320): attribute "typeof" not allowed here; expected attribute "accesskey", "charset", "class", "coords", "dir", "hreflang", "id", "lang", "rel", "rev", "shape", "style", "tabindex", "target", "title", "type" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1338,320): attribute "property" not allowed here; expected attribute "accesskey", "charset", "class", "coords", "dir", "hreflang", "id", "lang", "rel", "rev", "shape", "style", "tabindex", "target", "title", "type" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1338,320): attribute "datatype" not allowed here; expected attribute "accesskey", "charset", "class", "coords", "dir", "hreflang", "id", "lang", "rel", "rev", "shape", "style", "tabindex", "target", "title", "type" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1338,428): attribute "rel" not allowed here; expected attribute "dir", "id", "lang", "style", "title" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1338,518): attribute "typeof" not allowed here; expected attribute "accesskey", "charset", "class", "coords", "dir", "hreflang", "id", "lang", "rel", "rev", "shape", "style", "tabindex", "target", "title", "type" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1338,518): attribute "property" not allowed here; expected attribute "accesskey", "charset", "class", "coords", "dir", "hreflang", "id", "lang", "rel", "rev", "shape", "style", "tabindex", "target", "title", "type" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1338,518): attribute "datatype" not allowed here; expected attribute "accesskey", "charset", "class", "coords", "dir", "hreflang", "id", "lang", "rel", "rev", "shape", "style", "tabindex", "target", "title", "type" or "xml:lang" ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1332,3): 'images/status-active.gif': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1338,3): 'images/status-active.gif': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1401,73): 'OEBPS/images/ui-bg_flat_75_ffffff_40x100.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1403,71): 'OEBPS/images/ui-bg_highlight-soft_75_cccccc_1x100.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1408,147): 'OEBPS/images/ui-bg_glass_75_e6e6e6_1x400.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1410,227): 'OEBPS/images/ui-bg_glass_75_dadada_1x400.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1412,143): 'OEBPS/images/ui-bg_glass_65_ffffff_1x400.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1418,155): 'OEBPS/images/ui-bg_glass_55_fbf9ee_1x400.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1420,138): 'OEBPS/images/ui-bg_glass_95_fef1ec_1x400.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1431,39): 'OEBPS/images/ui-icons_222222_256x240.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1432,30): 'OEBPS/images/ui-icons_222222_256x240.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1433,29): 'OEBPS/images/ui-icons_222222_256x240.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1434,30): 'OEBPS/images/ui-icons_888888_256x240.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1435,53): 'OEBPS/images/ui-icons_454545_256x240.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1436,28): 'OEBPS/images/ui-icons_454545_256x240.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1437,31): 'OEBPS/images/ui-icons_2e83ff_256x240.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1438,58): 'OEBPS/images/ui-icons_cd0a0a_256x240.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1633,22): 'OEBPS/images/ui-bg_flat_0_aaaaaa_40x100.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1634,122): 'OEBPS/images/ui-bg_flat_0_aaaaaa_40x100.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1660,3): 'OEBPS/images/background.png': referenced resource missing in the package. ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1318,157): 'http://bl.dev/themes/bartik/logo.png': remote resource reference not allowed; resource must be placed in the OCF ERROR: ../Site-Install - An article - 2014-07-09-2.epub/OEBPS/epub.html(1333,357): 'http://bl.dev/sites/default/files/field/image/thumb_2.gif': remote resource reference not allowed; resource must be placed in the OCF Check finished with warnings or errors On 09-07-2014 12:15, Tassos wrote: > Hi, > I generated an epub version of a drupal node through the print module. > The epub does not open in iBook and does not validate correctly. > > “This page contains the following errors: > error on line 21 at column 27: AttValue: " or ' expected > Below is a rendering of the page up to the first error.” > > A similar issue has been filed with the drupal issue queue > https://www.drupal.org/node/2147659 but was closed and the author was > redirected here. > > — > Reply to this email directly or view it on GitHub > https://github.com/Grandt/PHPePub/issues/27.
dalwar23 commented 3 years ago

I had the same issue and it was fixed by adding " around an attribute.

<meta name="robots" content=noindex, nofollow />

Try:

<meta name="robots" content="noindex", nofollow />