[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