Feature #17283
closed
Multidimensional piVars couldn't be set by _DEFAULT_PI_VARS
Added by Marc Bastian Heinrichs almost 17 years ago. Updated about 4 years ago.
100%
Description
piVars['var1']['var2'] couldn't be set, because TS
._DEFAULT_PI_VARS.var1.var2 = test
would be rendered as
piVars['var1.']['var2']
(issue imported from #M5572)
Files
Updated by Mangan over 12 years ago
- Target version deleted (
0)
Hello,
in 4.5(.2) LTE this is not fixed, yet. Anyone out there to edit that one or two lines?
Thanks!
Updated by Steffen Gebert over 12 years ago
- Category deleted (
Communication) - TYPO3 Version changed from 4.1 to 4.5
Mangan, could you try to create a patch for that?
Updated by Mangan over 12 years ago
Hi Steffen,
I guess it has something do do with
function pi_setPiVarDefaults() {
if (is_array($this->conf['_DEFAULT_PI_VARS.'])) {
$this->piVars = t3lib_div::array_merge_recursive_overrule($this->conf['_DEFAULT_PI_VARS.'],is_array($this->piVars)?$this->piVars:array());
}
}
in class.tslib_pibase.php but I don't see through it all.
See file attached for debug output
From TS: _DEFAULT_PI_VARS.conf.ff_data.recordsForPage=3
From GET-PARAM: &extension[conf][ff_data][recordsForPage]=5
I tried to override some flexform settings using _DEFAULT_PI_VARS in Typoscript.
In the extension I use a $this->pi_setPiVarDefaults();
This works for values like _DEFAULT_PI_VARS.recordsForPage=3 that can be accessed with $this->pibase->piVars['recordsForPage'] in the extension.
Updated by Alexander Opitz over 9 years ago
- Status changed from New to Needs Feedback
Hi,
as this issue is very old. Does the problem still exists within newer versions of TYPO3 CMS (6.2.4)?
Updated by Marc Bastian Heinrichs over 9 years ago
- Status changed from Needs Feedback to New
Since the code in pi_setPiVarDefaults is still the same, I guess it's still not possible.
Updated by 
Updated by Gerrit Code Review over 5 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58846
Updated by Gerrit Code Review over 5 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58846
Updated by Gerrit Code Review over 5 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58846
Updated by Georg Ringer over 5 years ago
To test this, use the following TS
plugin.tx_felogin_pi1 {
_DEFAULT_PI_VARS {
fo = bar
bla {
blu = 1
}
}
}
and debug $this-piVars in the frontendlogin controller
Updated by Gerrit Code Review over 5 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58846
Updated by Gerrit Code Review over 5 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58846
Updated by Gerrit Code Review over 5 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58846
Updated by Gerrit Code Review over 5 years ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58846
Updated by Gerrit Code Review over 5 years ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58846
Updated by Gerrit Code Review over 5 years ago
Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58846
Updated by Gerrit Code Review over 4 years ago
Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/58846
Updated by Gerrit Code Review over 4 years ago
Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/58846
Updated by Georg Ringer over 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 2978ff3b9a739d28cf5bfc435c893a70601231ce.
Updated by Benni Mack about 4 years ago
- Sprint Focus changed from PRC to Needs Decision