Skip to content

Commit 5711ef0

Browse files
committed
fix(targetbase): fix double quote renderiing in targets
Signed-off-by: btry <tbugier@teclib.com>
1 parent f20acbf commit 5711ef0

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

inc/targetchange.class.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -981,7 +981,6 @@ public function save(PluginFormcreatorForm_Answer $formanswer) {
981981
];
982982
foreach ($changeFields as $changeField) {
983983
$data[$changeField] = $this->fields[$changeField];
984-
$data[$changeField] = addslashes($data[$changeField]);
985984
$data[$changeField] = str_replace("\r\n", '\r\n', $data[$changeField]);
986985
if (strpos($data[$changeField], '##FULLFORM##') !== false) {
987986
$data[$changeField] = str_replace('##FULLFORM##', $formanswer->getFullForm(), $data[$changeField]);
@@ -993,10 +992,11 @@ public function save(PluginFormcreatorForm_Answer $formanswer) {
993992

994993
$data[$changeField] = $this->parseTags($data[$changeField], $formanswer);
995994

996-
// This targer does not supports rich text
995+
// This target does not supports rich text
997996
$data[$changeField] = strip_tags($data[$changeField], '<p>');
998997
$data[$changeField] = str_replace('<p>', '', $data[$changeField]);
999998
$data[$changeField] = str_replace('</p>', '\r\n', $data[$changeField]);
999+
$data[$changeField] = Toolbox::addslashes_deep($data[$changeField]);
10001000
}
10011001

10021002
$data['_users_id_recipient'] = $_SESSION['glpiID'];

inc/targetticket.class.php

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1113,10 +1113,10 @@ public function save(PluginFormcreatorForm_Answer $formanswer) {
11131113
// Parse data
11141114
// TODO: generate instances of all answers of the form and use them for the fullform computation
11151115
// and the computation from a admin-defined target ticket template
1116-
$data['name'] = addslashes($this->fields['name']);
1116+
$data['name'] = $this->fields['name'];
11171117
$data['name'] = $this->parseTags($data['name'], $formanswer);
1118+
$data['name'] = Toolbox::addslashes_deep($data['name']);
11181119

1119-
$data['content'] = addslashes($this->fields['content']);
11201120
$data['content'] = str_replace("\r\n", '\r\n', $data['content']);
11211121
if (strpos($data['content'], '##FULLFORM##') !== false) {
11221122
$data['content'] = str_replace('##FULLFORM##', $formanswer->getFullForm(), $data['content']);
@@ -1128,8 +1128,9 @@ public function save(PluginFormcreatorForm_Answer $formanswer) {
11281128

11291129
$data['content'] = $this->parseTags($data['content'], $formanswer);
11301130
if (version_compare(PluginFormcreatorCommon::getGlpiVersion(), 9.4) >= 0 || $CFG_GLPI['use_rich_text']) {
1131-
$data['content'] = htmlentities($data['content']);
1131+
$data['content'] = htmlentities($data['content'], ENT_NOQUOTES);
11321132
}
1133+
$data['content'] = Toolbox::addslashes_deep($data['content']);
11331134
$data['_users_id_recipient'] = $_SESSION['glpiID'];
11341135
$data['_tickettemplates_id'] = $this->fields['tickettemplates_id'];
11351136

0 commit comments

Comments
 (0)