nytimes, missing images in EPUB

[ees]

The nytimes recipe works great, but images don't show up when using the EPUB format (on the PRS-505 device itself).

I think the reason is because mobile.nytimes.com names their JPEG images with .cr extensions, and the Reader doesn't approve. As an experiment, if I:

1) save the intermediate HTML (after calling feeds2epub),
2) rename all images to have .jpg extensions,
3) edit the corresponding HTML to refer to the new image filenames,
4) convert the HTML to EPUB

...then the PRS-505 displays images just fine.

I haven't studied Calibre's codebase too deeply yet, so I'll ask: what is the best approach to fixing this? Would it be to edit the nytimes recipe to rename all images appropriately?

Thanks,
-e

[kovidgoyal]

It's a good idea to fix this for all recipe downloads, since it's probably hapening for others as well. I would suggest making this change in the process_images method in the web.fetch.simple module. The best way to do it would be to read in the image using QImage and write it out as jpeg after checking that the image is valid. If invalid, then dont write the image. Go ahead and submit a patch or open a ticket and I'll fix it when I get some time.