Project

General

Profile

Actions

Bug #14124

closed

Imagemagick takes up to 6 minutes for displaying PDF preview of 100 pages document

Added by Michael Stucki over 20 years ago. Updated over 18 years ago.

Status:
Closed
Priority:
Should have
Category:
Backend API
Target version:
-
Start date:
2004-04-29
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

IM identify, which is called each time the thumbnail is displayed, will go through every frame (page) of the document although only the first page is needed.

IM option frameprepended is neccessary, 4.2.9 supports that, 5.4.4 doesn't?

Original message from Michael Stucki;

Matthias hat versucht, ein sehr grosses PDF auszuwählen, welches ca. 100 Seiten gross ist. ImageMagick braucht für den identify-Befehl (der wird bei jedem Bild aufgerufen) pro Seite ca. 3 Sekunden. Sprich, er würde nach ca. 6 Minuten fertig werden und das Bild anzeigen.Ich habe einen kleinen Patch gemacht, so dass identify nur noch den ersten Frame (=die erste Seite) aufrufen muss. Dann gehts immerhin nur noch 3 Sekunden. Wenn Du den aktuellen CVS verwendest ist selbst diese Zeit nur einmalig, das heisst, nach dem ersten indexieren wird identify nicht mehr benötigt.Der Patch ist eigentlich nur zwei Zeilen gross, Du kannst das ganz einfach von Hand ändern, schau Dir einfach kurz das File an!(Dominic: Dein ImageMagick muss die FramePrepended-Option unterstützen, sonst klappts nicht. Sie darf in localconf.php auch nicht deaktiviert sein!ImageMagick 4.2.9 hat diese Funktion dabei, 5.4.4 aus Debian Woody kann das anscheinend nicht...)
(issue imported from #M32)


Files

0000032-class.t3lib_stdgraphic.php.diff (639 Bytes) 0000032-class.t3lib_stdgraphic.php.diff Administrator Admin, 2004-04-29 11:56
Actions #1

Updated by Robert Lemke over 20 years ago

That's a classic for Kasper. Because he likes Imagemagick so much ...

Actions #2

Updated by Karsten Dambekalns over 20 years ago

This patch hasn't made it into CVS yet, why? Is more testing needed? Then I'd do that...

Actions #3

Updated by Michael Stucki over 20 years ago

Of course, not. I see no problem with this patch, used it already, but I'm still going to ask Kasper before I commit this to the HEAD branch.

Thanks for the reminder.

Actions #4

Updated by Michael Stucki over 20 years ago

Now fixed in HEAD.

Actions

Also available in: Atom PDF