[TYPO3-core] RFC: #9901: Enable stdWrap for expAll
Oliver Hader
oliver at typo3.org
Mon Dec 29 15:38:57 CET 2008
Hi Steffen,
Steffen Kamper schrieb:
> Oliver Hader schrieb:
>> Please put the stdWrap call into a "if (isset($this->mconf['expAll.']))
>> { }" block.
>
> imho it's not needed as stdWrap passes through if there is no
> configuration for, this is done at many places.
The fact that is has been done like this before is not an argument... ;)
Checking if a key of an array is set before accessing it and if an
additional call to a function is necessary at all is just "good style".
FYI: The attached changes have been committed to SVN Trunk (rev. 4627).
olly
--
Oliver Hader
TYPO3 4.3 Release Manager
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0009901_followup.patch
Type: text/x-diff
Size: 1293 bytes
Desc: not available
Url : http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20081229/42d55fc1/attachment.patch
More information about the TYPO3-team-core
mailing list