From d1ecab6f615ec20c55c09fee6277423d752343fd Mon Sep 17 00:00:00 2001
From: Christian Kuhn <lolli@schwarzbu.ch>
Date: Fri, 27 May 2016 13:36:13 +0200
Subject: [PATCH] [TASK] Obsolete unserialize(serialize())
There is a funny comment for this call within TypoScriptParser.
Reading the code, there seems to be no part that works with
references on this variable in setVal().
The patch now removes this odd construct. In the
unlikely case some issue still pops up later, we would
at least have a clear way to reproduce then ...
Change-Id: I70f5f8915386a7a4b5038d66611b4ec359f5c1fa
Resolves: #76323
Releases: master
Reviewed-on: https://review.typo3.org/48327
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
---
.../sysext/core/Classes/TypoScript/Parser/TypoScriptParser.php | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/typo3/sysext/core/Classes/TypoScript/Parser/TypoScriptParser.php b/typo3/sysext/core/Classes/TypoScript/Parser/TypoScriptParser.php
index 06db119aa5c3..2b4be2070dbb 100644
--- a/typo3/sysext/core/Classes/TypoScript/Parser/TypoScriptParser.php
+++ b/typo3/sysext/core/Classes/TypoScript/Parser/TypoScriptParser.php
@@ -454,8 +454,7 @@ class TypoScriptParser
} else {
$res = $this->getVal($theVal, $this->setup);
}
- $this->setVal($objStrName, $setup, unserialize(serialize($res)), 1);
- // unserialize(serialize(...)) may look stupid but is needed because of some reference issues. See Kaspers reply to "[TYPO3-core] good question" from December 15 2005.
+ $this->setVal($objStrName, $setup, $res, 1);
break;
case '>':
if ($this->syntaxHighLight) {
--
GitLab