[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