Bug #36154
closedDescription of pidSelect in class.db_list.inc is misleading
100%
Description
The description says "list of IDs", where it should be "query part for a list of IDs or a single ID"
If you don't dive into the code but just check the description while using the variable i.e. within a hooked method, this will lead to an error or unwanted behaviour.
Setting it to a single integer will just remove pid from the query giving you a result from all possible pages.
Setting it to a list of integers will trow an error message.
In our case it was the cause for a bug in grid elements, showing child elements still in their containers even though they where moved to another page.
Found it in 4.5.x, but still there in current master
Updated by Gerrit Code Review over 12 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10549
Updated by Gerrit Code Review over 12 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10549
Updated by Gerrit Code Review about 12 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10549
Updated by Gerrit Code Review about 12 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10549
Updated by Jo Hasenau about 12 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset ea9c821a8236da285d6ce484e7a2a2e750f53619.
Updated by Riccardo De Contardi almost 7 years ago
- Status changed from Resolved to Closed