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.

Sign up for GitHub

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

Jump to bottom

Improves opds validation #488

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Improves opds validation #488

wants to merge 2 commits into from

Commits on Oct 14, 2020

  1. Update OPDS_renderer.php 

    HTML should not be written into the XML stream as they are both tag based languages.  Depending on the complexity of the rich text in Calibre that populates the content field, this can stop validation and break some readers.  Notable PocketReader on Android which will not process an invalid OPDS.

4.1.3.3.  Processing Model
   2.  If the value of "type" is "html", the content of atom:content
       MUST NOT contain child elements and SHOULD be suitable for
       handling as HTML [HTML].  The HTML markup MUST be escaped; for
       example, "<br>" as "&lt;br>".  The HTML markup SHOULD be such
       that it could validly appear directly within an HTML <DIV>
       element.  Atom Processors that display the content MAY use the
       markup to aid in displaying it.
    @cbckly
    cbckly committed Oct 14, 2020
    Configuration menu
    Copy the full SHA
    1bf5e46 View commit details
    Browse the repository at this point in the history

  2. Update OPDS_renderer.php

    @cbckly
    cbckly committed Oct 14, 2020
    Configuration menu
    Copy the full SHA
    5924e58 View commit details
    Browse the repository at this point in the history