EPUB-Download for more than one article create a corrupt file

[msone]

More than one articles as EPUB create a corrupt EPUB-file. If I rename this to *.zip, my packer says: there are errors in the file.

My configuration
Wallabag 1.7
Shared Hosting
PHP 5.3.28
MySQL-Storage
Error-Message from the server:
PHP Warning: fwrite() expects parameter 1 to be resource, boolean given in /is/htdocs/xxx/wallabag/inc/3rdparty/libraries/PHPePub/Zip.php on line 719

[tcitworld]

Hi,
Which method did you use to create the epub ? (search/category/tag/all ?)
If the bug occurs with a tag (too), could you give us the links from the concerned articles ? Two articles should be enough, if the bug occurs with them.

[tcitworld]

Also, please tell me if concerned articles have special/unusual characters.

[msone]

I´ve tested a bit. Single articles of "Spiegel Online" always produce a broken EPUB (eg. http://www.spiegel.de/politik/deutschland/ueberwachung-de-maziere-verteidigt-ausspaehen-von-facebook-und-twitter-a-973069.html) .

And these 5 articles together always produce a broken EPUB. Individually or in groups, however, the EPUBs are generated correctly:

http://techniktagebuch.tumblr.com/post/87282203687/30-5-2014
http://www.nubert-forum.de/nuforum/viewtopic.php?f=1&t=36943
http://www.lesen.net/forum/ebook-reader/amazon-kindle/15375-akku-entl%C3%A4d-sich-auch-wenn-er-aus-ist/
http://wasmachendieda.de/2014-05-20/markus-trapp-stabstelle-social-media/
http://papierlos-lesen.de/vergleich-kobo-aura-tolino-vision-kindle-paperwhite-pocketbook-touch-lux-2/

[tcitworld]

OK, I think it's an encoding problem with the umlauts. Will try something.

[tcitworld]

And these 5 articles together always produce a broken EPUB

By which way do you make them together ?

[tcitworld]

OK, so about the first problem : articles from the Spiegel don't work because the Spiegel refers to an image which doesn't exists :

<noscript>
<img src="http://spiegel.ivwbox.de/cgi-bin/ivw/CP/1006;/politik/deutschland/c-20/r-97/a-973069/be-PB64-cG9saXRpay9hcnRpa2Vs/szwprofil-1006?d=89231600" width="1" height="1" border="0" align="right" id="spIvwPixel" alt="" />
</noscript>

The library wallabag uses still wants to include it, so when the epub is produced, it misses and creates errors. We'll have to check that case.

About the second kind of error, I can't manage to reproduce it. Can you give me more details ? I've tried assigning all of them a common tag, or put them all into a single category.

[msone]

Sorry for the late feedback.

Can you give me more details ?

Yes, of course

• If these 5 articles are unread, the download ist broken.
• If I move the last two articles to archive, the download from unread is ok and the download from archive is also ok.
• But if I move the articles from archive back to unread, the epub is corrupt again.
• If I move them all to archive, the file is also corrupt.
• If all articles tagged, the file is corrupt.

[tcitworld]

OK, the problematic article is http://wasmachendieda.de/2014-05-20/markus-trapp-stabstelle-social-media/
If you remove it from your list, you shouldn't have errors anymore.

I've pushed an update to fix this bug in dev branch. It will be available in wallabag 1.7.1, which will only correct some bugs.

[tcitworld]

First bug is also fixed now.

[msone]

Thank you.