[TYPO3-core] RFC: small enhancement to t3lib_div::arrayToLogString()

Michael Stucki michael at typo3.org
Wed Jun 7 11:18:19 CEST 2006


+1 (reading the code)

Michael

Martin Kutschker wrote:

> Hi!
> 
> Problem: the inline doc says that argument $valueList may be either an
> array or a string (comma delimited list). Also the docs says that an
> empty list means "all array keys". But only an empty array, not an empty
> string, is accepted. This is a bit annoying/confusing.
> 
> Solution: use t3lib_div::trimExplode() instead of explode(). The former
> may ignore empty keys, but the latter will always return an array with
> at least one element (an empty string).
> 
> The patch also moves count() outside of the loop and uses || instead of
> AND/OR which is more in sync with the usual TYPO3 code.
> 
> Masi

-- 
Use a newsreader! Check out
http://typo3.org/community/mailing-lists/use-a-news-reader/



More information about the TYPO3-team-core mailing list