Do not force the presence of PDF format for all books

[benoit74]

A few lines are forcing the presence of the PDF format for all books.

This is causing broken links / weird buttons in all books which do not have a PDF format.

Discussion on this issue already started here by mistake (my bad).

[rgaudin]

I don't understand this ticket. If it's independent to #159, does that mean that we have PDF files in ZIM files always? Even when we don't request them?

[benoit74]

It is relatively independent from #159
It means that no matter the content of the RDF, the scraper consider there is always a PDF format available for all books.
This causes the behavior you saw while testing #163 that when you request PDF among other export formats, you get many books with broken PDF buttons because the format is supposed to be there according to the bookformat table.

[rgaudin]

OK it's more clear: it fakes the availability of PDF for all books, even if there's no real PDF. Understood.

I don't see how it could affect #159 though but I believe we're not understanding each other on what #159 is about 😅

[eshellman]

FWIW, pdf generation at PG is planned for sometime this year.

…

On Jan 20, 2023, at 9:13 AM, rgaudin ***@***.***> wrote: OK it's more clear: it fakes the availability of PDF for all books, even if there's no real PDF. Understood. I don't see how it could affect #159 <#159> though but I believe we're not understanding each other on what #159 <#159> is about 😅 — Reply to this email directly, view it on GitHub <#160 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAHCGMNZGII62OOG6ZDPXYTWTKMPHANCNFSM6AAAAAAUAZHIDE>. You are receiving this because you are subscribed to this thread.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be now be reviewed manually. Thank you for your contributions.