Bug #68672

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

setIgnoreEnableFields() should be inherited to language overlays

Added by Martin R. Krause over 5 years ago. Updated over 2 years ago.

Status:
New
Priority:
Should have
Assignee:
-
Category:
Extbase + l10n
Target version:
-
Start date:
2015-07-31
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
hard
Is Regression:
No
Sprint Focus:

Description

Using the setIgnoreEnableFields() for query settings in an extbase repository fails, when the record gets an language overlay. It should be inherited to the overlay query.

The usage of the enable fields for the query of the language overlay is pretty much hardcoded in

typo3_src-6.2.12/typo3/sysext/frontend/Classes/Page/PageRepository.php:431

In addition the logic of the language overlay for Extbase record is inconsistent:

E.g.: Providing that starttime has l10n_mode "mergeIfNotBlank" (which is default), a translation of a record can be shown, even if its l10_parent is still hidden – only the enable fields of the translated record matter (which is OK). The other way around, requesting the record from an Extbase repository does not work, as both records (original and translation) need to allow for display of the record in order to show the translation.


Related issues

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

Actions
Related to TYPO3 Core - Bug #84955: Extbase persistence layer ignores query settingsNeeds Feedback2018-05-09

Actions
#1

Updated by Gerrit Code Review over 3 years ago

  • Status changed from New to Under Review

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53445

#2

Updated by Peter Niederlag over 3 years ago

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

Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53445

#4

Updated by Benni Mack about 3 years ago

  • Category changed from Extbase to Extbase + l10n
#5

Updated by Tymoteusz Motylewski over 2 years ago

  • Related to Bug #84955: Extbase persistence layer ignores query settings added
#6

Updated by Tymoteusz Motylewski over 2 years ago

FYI, I have included tests from this review in https://review.typo3.org/#/c/58276/
to make it easier to fix this issue.

#7

Updated by Tymoteusz Motylewski over 2 years ago

  • Parent task set to #86307
#8

Updated by Tymoteusz Motylewski over 2 years ago

  • Status changed from Under Review to New

Also available in: Atom PDF