Project

General

Profile

Bug #6305

wkhtml: commandline should not be separated by '&'

Added by Carsten Rose over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
High
Assignee:
Target version:
Start date:
26.06.2018
Due date:
% Done:

0%

Estimated time:
Spent time:
Discuss:

Description

  • Aktuell werden die Commandline Parameter fuer wkhtml, die in einem QFQ Record gebaut werden, mit '&' voneinander getrennt.
  • Siehe: https://docs.typo3.org/typo3cms/drafts/github/T3DocumentationStarter/Public-Info-053/Manual.html#rendering-pdf-letters
  • Wenn z.B. die wkhtml Header Angabe (--header=...) eine URL enthaelt die wiederum GET Parameter enthaelt, kollidieren die '&' fuer den 'p:' Tag mit den '&' in der '--header={{URL}}?indexp.php?id=head&L=1' Angabe.
  • Entweder die '&' innerhalb der header URL werden escaped (sinnvoll) oder es muss eine neue Notation definiert werden.
  • keyValueStringParser.php: explodeEscape()

Associated revisions

Revision 88779e39 (diff)
Added by Carsten Rose over 1 year ago

Manual.rst: note how to user double backslash to escape arguments in wkhtml call.

History

#2 Updated by Carsten Rose over 1 year ago

  • Description updated (diff)

#4 Updated by Carsten Rose over 1 year ago

  • Description updated (diff)
  • Assignee changed from Carsten Rose to Elias Villiger

#5 Updated by Elias Villiger over 1 year ago

  • Status changed from New to Feedback

Der Code muss nicht geändert werden. Die Übergabe eines intakten &-Zeichens benötigt einfach ein doppeltes Escapen:

SELECT 'd:download.pdf|t:Download|U:id=40&--header-center=Sons\\&Co' AS _pdf

Der Grund dafür ist, dass mySQL auch escaped, und daher der obige string von SQL wie folgt zurückgegeben wird:

d:download.pdf|t:Download|U:id=40&--header-center=Sons\&Co

Alles weitere wird dann von QFQ korrekt gehandhabt, d.h. der Header wird auf dem generierten PDF mit & Zeichen angezeigt.

@Carsten: Das macht Sinn und lässt sich nicht vereinfachen, oder?

#6 Updated by Elias Villiger over 1 year ago

  • Assignee changed from Elias Villiger to Carsten Rose

#7 Updated by Carsten Rose over 1 year ago

  • Status changed from Feedback to Closed

Hallo Elias

ja, das macht Sinn, und ist ok. Ich nehme es in die Doku auf.

CU
Carsten

#8 Updated by Carsten Rose about 1 year ago

  • Target version changed from 18.10.3 to 18.9.2

Also available in: Atom PDF