> On Oct. 4, 2015, 10:32 p.m., Albert Astals Cid wrote: > > This patch does not give me an image for the first page of the linked > > document. Any idea what may be wrong?
Do you have warnings like this in terminal? OEBPS//@public@vhost@g@gutenberg@html@files@50080@50080-h@ima...@cover.jpg - No such file It can be a problem with non-canonical path in libepub. I fixed it in my fork of library and totally forgot about that testing this issue. - Boris ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125500/#review86356 ----------------------------------------------------------- On Oct. 3, 2015, 8:27 a.m., Boris Egorov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125500/ > ----------------------------------------------------------- > > (Updated Oct. 3, 2015, 8:27 a.m.) > > > Review request for Okular and Albert Astals Cid. > > > Repository: okular > > > Description > ------- > > Since Qt 5.0, QUrl::toLocalFile() returns empty path for URLs where the > scheme is not "file"[1]. > > You can see how it fails if you open some epub with images. For example, > http://www.gutenberg.org/ebooks/50080, "EPUB (with images)". There will > be lots of warnings like this: > > libepub (II): OEBPS/ - No such file > > And no images will be loaded. This patch fixes a problem by adding a > scheme "file". > > 1: http://doc.qt.io/qt-4.8/qurl.html#toLocalFile > > > Diffs > ----- > > generators/epub/epubdocument.cpp 3b0ef56 > > Diff: https://git.reviewboard.kde.org/r/125500/diff/ > > > Testing > ------- > > Open metioned file with patch applied. There is no warnings and images loaded > successfully. > > > Thanks, > > Boris Egorov > >
_______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
