[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