Project

General

Profile

Fehler #607

Design4.0 Reportgenerator erzeugt sehr viele Leerzeilen

Added by Cem Aydin over 1 year ago. Updated about 1 year ago.

Status:
Erledigt
Priority:
Normal
Assignee:
Target version:
Start date:
07/17/2023
Due date:
% Done:

100%

Estimated time:

Description

Zum Beispiel im Buchungsjournal, bei circa 100 Buchungen ist die Page Source ~26000 Zeilen.

Im alten Design sind es ~4500 Zeilen.

Wenn man sehr viele Buchungen hat, und dann auf Seitenquelltext anzeigen geht, dann stürzt im Prinzip der Browser ab :)

In der normalen Ansicht merkt man dass zwar nicht, da der Browser das raus filtert aber ich würde das trotzdem mal korrigieren, sollte ja trivial sein.

History

#1

Updated by Bernd Bleßmann over 1 year ago

Cem Aydin schrieb:

Zum Beispiel im Buchungsjournal, bei circa 100 Buchungen ist die Page Source ~26000 Zeilen.

Im alten Design sind es ~4500 Zeilen.

HP hat bei den Template-Direktiven fast immer die Bindestriche entfernt, die dafür sorgen, dass im geparsten Code kein Umbruch entsteht:

  [% END %]

vs.
  [%- END %]

Oder ähnlich. Ich vermute, dass es daran liegt.

Das betrifft auch andere Teile, z.B. das menuV3. Bzw. eigentlich alle Templates.

#2

Updated by Cem Aydin over 1 year ago

  • Status changed from Neu to In Bearbeitung
  • % Done changed from 0 to 80

HP hat bei den Template-Direktiven fast immer die Bindestriche entfernt, die dafür sorgen, dass im geparsten Code kein Umbruch entsteht:

Ja, genau.

Das betrifft auch andere Teile, z.B. das menuV3. Bzw. eigentlich alle Templates.

Ja, wobei solange die Zeilenumbrüche nicht in einer Schleife sind und sich das dadurch verzehntausendfacht, ist das denke ich weniger schlimm..

Habe jetzt mal einen pull request gemacht, wo das für die report generator template behoben ist. Zwar auch nicht ganz schön da das etwas verwirrend ist mit den vor- und nach-Zeilenumbrüche... habe es so gemacht dass es im HTML Output einigermassen vernünftig aussieht.

https://github.com/kivitendo/kivitendo-erp/pull/166

http://template-toolkit.org/docs/manual/Syntax.html#section_Chomping_Whitespace

LG, Cem

#3

Updated by Cem Aydin about 1 year ago

  • Status changed from In Bearbeitung to Erledigt
  • % Done changed from 80 to 100

Also available in: Atom PDF