search

contao / core

Contao 3 → see contao/contao for Contao 4
GNU Lesser General Public License v3.0
490 stars 214 forks source link

ce_downloads verursacht TCPDF-Feher #6268

Closed do-while closed 11 years ago

do-while commented 11 years ago

Wenn die Seite ein Downloads-Element enthält, läuft die PDF-Ausgabe (Syndication) auf Fehler.

Warning: Division by zero in ..\system\modules\core\vendor\tcpdf\tcpdf.php on line 11736

#0 ..\system\modules\core\vendor\tcpdf\tcpdf.php(11736): __error(2, 'Division by zer...', 'D:\Softleister\...', 11736, Array)
#1 ..\system\modules\core\vendor\tcpdf\tcpdf.php(11692): TCPDF->_outellipticalarc(32.78, 171.184166667, 0, 0, 0, 0, 360, false, 8, true, true, false)
#2 ..\system\modules\core\vendor\tcpdf\tcpdf.php(11845): TCPDF->Ellipse(32.78, 171.184166667, 0, 0, 0, 0, 360, 'F', Array, Array, 8)
#3 ..\system\modules\core\vendor\tcpdf\tcpdf.php(20141): TCPDF->Circle(32.78, 171.184166667, 0, 0, 360, 'F', Array, Array, 8)
#4 ..\system\modules\core\vendor\tcpdf\tcpdf.php(18072): TCPDF->putHtmlListBullet(1, '!', 0)
#5 ..\system\modules\core\modules\ModuleArticle.php(342): TCPDF->writeHTML('

Meine Nachforschungen zeigen, dass das Problem im TCPDF liegt. Das Problem tritt auf, wenn in einem <li> ein Bild und ein Text enthalten ist. Ausreichend für den Fehler ist bereits dieser kurze Codeschnippsel:

<ul>
  <li><img src="assets/contao/images/iconGIF.gif" alt=""> ABC</li>
</ul>
leofeyer commented 11 years ago

Welche Contao- bzw. TCPDF-Version? Tritt das Problem in der TCPDF-Demo ebenfalls auf?

do-while commented 11 years ago

Wahrscheinlich ja, ich bekomme nur nicht in den Einstellungen die Fehlermeldungen aktiviert. Mit abgeschalteten Meldungen fällt einem erstmal nichts auf. Ich habe hier mit einer 3.1.2 getestet (Music Academy). Auch die neuste 6.0.037 von TCPDF brachte keine Verbesserung. Ich habe deshalb dort auch ein Ticket erstellt. https://sourceforge.net/p/tcpdf/bugs/841/

do-while commented 11 years ago

Nachdem ich bei TCPDF erfahren musste, dass es dort kein derartiges Problem gibt, habe ich den betreffenden Bereich in Contao mal debuggt. Im ModuleArticle.php in Zeile 274 beginnt die Anpassung des Artikelinhalts an TCPDF. Dort wird aus der Zeile 

<li><img src="assets/contao/images/iconGIF.gif" alt=""> ABC</li>

die folgende gemacht

<li><br><img src="assets/contao/images/iconGIF.gif" alt=""> ABC</li>

Dieses zusätzliche <br>-Tag verkraftet TCPDF nicht. Ich sehe aber gerade, dass die Umsetzung so schon sehr lange unverändert ist. Ich frage im TCPDF noch mal nach...

do-while commented 11 years ago

Nachricht von Nicola Asuni (Entwickler der TCPDF): "OK. Fixed in TCPDF 6.0.038. Thank you for reporting this issue."

@leofeyer Bitte TCPDF-Update in Contao übernehmen.

Gruß, Hagen

leofeyer commented 11 years ago

Aktualisiert in 76762293381d17148c359f42ab111305fcc4b006.