Skip to content

Commit 64efa12

Browse files
authored
Merge pull request #704 from Patta/sanitize-frontend-output-export
[TASK] Sanitize CSV and XLS frontend output export
2 parents 9303803 + 5d92e13 commit 64efa12

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

Resources/Private/Templates/Output/ExportCsv.html

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,15 +22,17 @@
2222

2323
<f:if condition="{vh:condition.isArray(val: '{answer.value}')}">
2424
<f:else>
25-
"<vh:string.removeQuote>{answer.value}</vh:string.removeQuote>";
25+
"<vh:string.removeQuote><vh:string.sanitizeCsvCell>{answer.value}</vh:string.sanitizeCsvCell></vh:string.removeQuote>";
2626
</f:else>
2727
<f:then>
2828
"<vh:string.removeQuote>
29-
<f:for each="{answer.value}" as="singleValue">
30-
<f:if condition="{singleValue}">
31-
{singleValue},
32-
</f:if>
33-
</f:for>
29+
<vh:string.sanitizeCsvCell>
30+
<f:for each="{answer.value}" as="singleValue">
31+
<f:if condition="{singleValue}">
32+
{singleValue},
33+
</f:if>
34+
</f:for>
35+
</vh:string.sanitizeCsvCell>
3436
</vh:string.removeQuote>";
3537
</f:then>
3638
</f:if>

Resources/Private/Templates/Output/ExportXls.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,12 @@
3232
<f:then>
3333
<f:for each="{answer.value}" as="singleValue">
3434
<f:if condition="{singleValue}">
35-
{singleValue},
35+
<vh:string.sanitizeCsvCell>{singleValue}</vh:string.sanitizeCsvCell>,
3636
</f:if>
3737
</f:for>
3838
</f:then>
3939
<f:else>
40-
{answer.value}
40+
<vh:string.sanitizeCsvCell>{answer.value}</vh:string.sanitizeCsvCell>
4141
</f:else>
4242
</f:if>
4343
</f:if>

0 commit comments

Comments
 (0)