Bug #55707
closed
Missing stdWrap for recursive in select query
100%
Description
typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer.php
Line 7397 ++
public function getQuery($table, $conf, $returnQueryArray = FALSE) {
// Resolve stdWrap in these properties first
$properties = array(
'pidInList',
'uidInList',
'languageField',
'selectFields',
'max',
'begin',
'groupBy',
'orderBy',
'join',
'leftjoin',
'rightjoin',
'recursive' <-----THIS IS MISSING, ADDED AS EXAMPLE
);
recursive is not on the list...
which means
A: Change the manual here:
http://docs.typo3.org/typo3cms/TyposcriptReference/Functions/Select/Index.html
Property recursive Since TYPO3 4.6: integer /stdWrap
or
B: Please add recursive to the list...
Many thanks and best regards,
Matthias
Files
Updated by Markus Klein about 10 years ago
Hi Matthias!
Would you mind pushing your patch to our review system?
http://wiki.typo3.org/CWT
Updated by Matthias Kappenberg about 10 years ago
- File Core_Issue_ID_55707.diff Core_Issue_ID_55707.diff added
Hi Markus,
thank you for the answer :-)
Is the attached file correct?
Or should I commit it via git?
Best regards,
Matthias
Updated by Oliver Hader almost 10 years ago
It seems that "recursive" just has been forgotten in the original change of issue #18822 for TYPO3 CMS 4.6
Thus, it's at least not a regression since it never worked with stdWrap on "recursive". Thus, either add it to 6.1/6.2 or remove it from the documentations.
Updated by Gerrit Code Review almost 10 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30043
Updated by Gerrit Code Review almost 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30043
Updated by Oliver Hader almost 10 years ago
- Is Regression changed from Yes to No
Updated by Gerrit Code Review over 9 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/30043
Updated by Gerrit Code Review over 9 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/30043
Updated by Gerrit Code Review over 9 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/30043
Updated by Gerrit Code Review over 9 years ago
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32209
Updated by Gerrit Code Review over 9 years ago
Patch set 1 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32210
Updated by Matthias Kappenberg over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset baed348e8a2d8ed3298501dc7d17d31cb95204c5.
Updated by Chris topher over 9 years ago
stdWrap for the recursive property had already been documented in TSref (see Documentation/Functions/Select/Index.rst). I have just fixed the wrong "Since TYPO3 4.5/4.6" comment in https://github.com/TYPO3-Documentation/TYPO3CMS-Reference-Typoscript/commit/291efe4296baba42b58d925057ebd2236ffa33bf and in the backports back to 4.7.
Updated by