Project

General

Profile

Actions
Copy link

Bug #6305

closed

wkhtml: commandline should not be separated by '&'

Added by Carsten Rose almost 6 years ago. Updated over 5 years ago.

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

0%

Estimated time:
Discuss:
Prio Planung:
Vote:

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()
Actions
Copy link
 #2

Updated by Carsten Rose almost 6 years ago

  • Description updated (diff)
Actions
Copy link
 #4

Updated by Carsten Rose almost 6 years ago

  • Description updated (diff)
  • Assignee changed from Carsten Rose to Elias Villiger
Actions
Copy link
 #5

Updated by Elias Villiger almost 6 years 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?

Actions
Copy link
 #6

Updated by Elias Villiger almost 6 years ago

  • Assignee changed from Elias Villiger to Carsten Rose
Actions
Copy link
 #7

Updated by Carsten Rose almost 6 years 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

Actions
Copy link
 #8

Updated by Carsten Rose over 5 years ago

  • Target version changed from 18.10.3 to 18.9.2
Actions
Copy link

Also available in: Atom PDF