[TYPO3-core] RFC #16173: Refactor FLUIDTEMPLATE cObject to use the new Fluid standalone view
Bastian Waidelich
bastian at typo3.org
Tue Nov 2 09:14:41 CET 2010
Ernesto Baschny [cron IT] wrote:
Hi Ernesto,
> I think these changes don't make sense:
>
> ...
> -if (isset($conf['layoutRootPath']) || isset($conf['layoutRootPath.']))
> +if (isset($conf['layoutRootPath']))
> $layoutRootPath = $this->cObj->stdWrap($conf['layoutRootPath'],
> $conf['layoutRootPath.']);
> ...
>
> Because that would not catch a setup like for example:
>
> layoutRootPath.value = fileadmin/....
That's true. I changed it, because
layoutRootPath.wrap = foo | bar
would not work for example, as layoutRootPath is resolved implicitly.
But it probably doesn't really make sense as is.
The second version of Joeys stdWrap patch (#16198) fixes this.
Still: only setting stdWrap properties on layout/partialRootPath without
explicitly setting the value won't work - do you think, that's a problem?
Best,
Bastian
More information about the TYPO3-team-core
mailing list