Bug #63406
closed
setRespectStoragePid() of QuerySettings does not work with concrete settings in TCA
Added by Patrik Lang almost 10 years ago.
Updated about 6 years ago.
Description
Hello,
in my opinion there is a bug in the class Typo3DbQueryParser in method addPageIdStatement() on lines 649.
There is a if query which checks if the configured rootLevel param in TCA ctrl section ist set to true (not boolean)
In the TCA documentation you can set this param to 0/1/-1 (@see http://docs.typo3.org/typo3cms/TCAReference/singlehtml/#rootlevel)
When you set the param to -1 (allowed in root AND page tree the additionalWhereClause to check the database pid field will be ignored because the value "-1" is true and the first if query applies but the second not and so no additionalWhereClause is set.
Is this a desired behaviour and I have only missed a separate documentation or is it a bug?
In my Opionion the first if query is superfluous.
I tested it on the version 6.2.7
Kind regards
Patrik
Files
Category is extbase, not DBAL, sorry
- Category changed from 999 to Extbase
- 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 http://review.typo3.org/43675
Reproduce / Verify:
- Install attached Extension "Rootlevel"
- Insert the plugin on a page
- create records on different pages including pid 0
- view the plugin output with and without the patch
Note: The storagePid is configured to pid 9. You can of course overwrite this random number with constant
plugin.tx_rootlevel.persistence.storagePid = 9
- Status changed from New to Under Review
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43675
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43675
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
- Status changed from Resolved to Closed
Also available in: Atom
PDF