Bug #84955

Epic #86307: Extbase allows to fetch deleted/hidden records (respects ignoreEnableFields)

Extbase persistence layer ignores query settings

Added by Feeela no-lastname-given over 2 years ago. Updated 11 months ago.

Status:
Needs Feedback
Priority:
Must have
Assignee:
-
Category:
Extbase
Target version:
-
Start date:
2018-05-09
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.0
Tags:
extbase,persistence,enableFields
Complexity:
easy
Is Regression:
Sprint Focus:

Description

Description:

\TYPO3\CMS\Extbase\Persistence\Generic\Backend::getObjectByIdentifier() ignores any settings made to the defaultQuerySettings made for any repository. Since some repository methods internally use getObjectByIdentifier(), this method also must respect the default query settings. Currently there is no way of retrieving a hidden or deleted entity using Extbase. (Besides overwriting the core classes…)

Expected behavior:

When I disable the test for "enableFields", this setting should be used in any methods, that pull data from the persistence layer.

For example in a repository inside a initializeObject() method:

    /** @var $defaultQuerySettings QuerySettingsInterface */
    $defaultQuerySettings = $this->objectManager->get(QuerySettingsInterface::class);
    $defaultQuerySettings->setIgnoreEnableFields(true);

Actual behavior:

Hidden and deleted entities are excluded by default without any possibility to include them e.g. in controller actions (where the entities are being resolved before the action is executed).


Related issues

Related to TYPO3 Core - Bug #68672: setIgnoreEnableFields() should be inherited to language overlaysNew2015-07-31

Actions
Related to TYPO3 Core - Bug #73007: Insufficient sysLanguageStatement does not respect enableFields for translationsClosed2016-01-29

Actions
#1

Updated by Stephan Großberndt over 2 years ago

  • Subject changed from Extbase peristence layer ignores query settings to Extbase persistence layer ignores query settings
#2

Updated by Tymoteusz Motylewski about 2 years ago

  • Related to Bug #68672: setIgnoreEnableFields() should be inherited to language overlays added
#3

Updated by Tymoteusz Motylewski about 2 years ago

  • Related to Bug #73007: Insufficient sysLanguageStatement does not respect enableFields for translations added
#4

Updated by Tymoteusz Motylewski about 2 years ago

  • Parent task set to #86307
#5

Updated by Alexander Schnitzler 11 months ago

  • Status changed from New to Needs Feedback

I am currently looking at the code base of TYPO3 9.5 and the only method that uses getObjectByIdentifier is findByIdentifier, which deliberately does so.
Can you explain what exact method you call on the repository that causes this bug?

Also available in: Atom PDF