[TYPO3-core] RFC: Bug #2540: t3lib_db new method exec_SELECT_mm_rec_query for recursive datastructures

Martin Kutschker Martin.Kutschker at n0spam-blackbox.net
Mon Apr 30 10:28:43 CEST 2007


Michael Stucki schrieb:
> Comments:
> This new method provides the ability to use recursive Data Structures in 
> mm_queries.

Spent some time figuring out what "recursive" means in the contect of the 
function. So maybe instead of $local_table it can be just $table.

As for the code: lots of tests with trigraphs if $local_table is non-empty? 
Why? The SQL will be broken anyway if it isn't. If you do want a check do a 
"if ($local_table==='' || $mm_table==='') { rerurn FALSE; }" at the beginning.

And commenting the name: the "_rec_" part is perhaps misleading as it it is 
currently mostly used as a shorthand for "record". Please use "recursive" 
(unless you rename it, see above why).

Masi


More information about the TYPO3-team-core mailing list