Bug #68672
closed
Epic #86307: Extbase allows to fetch deleted/hidden records (respects ignoreEnableFields)
setIgnoreEnableFields() should be inherited to language overlays
0%
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.
Updated by Gerrit Code Review almost 7 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
Updated by Peter Niederlag almost 7 years ago
- Related to Bug #73007: Insufficient sysLanguageStatement does not respect enableFields for translations added
Updated by Gerrit Code Review almost 7 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
Updated by Benni Mack over 6 years ago
- Category changed from Extbase to Extbase + l10n
Updated by Tymoteusz Motylewski over 5 years ago
- Related to Bug #84955: Extbase persistence layer ignores query settings added
Updated by Tymoteusz Motylewski over 5 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.
Updated by Tymoteusz Motylewski over 5 years ago
- Status changed from Under Review to New
Updated by Benni Mack almost 2 years ago
- Status changed from New to Needs Feedback
Hey,
since this issue is really old and we have improved all of these places (mergeIfNotBlank, also optiimized hidden language handling incl. site handling in v9 etc). Can you let me know if your problem still exists?
Updated by Christian Kuhn about 1 year ago
- Related to Bug #100638: setIgnoreEnableFields() does not work on language overlays added
Updated by Christian Kuhn 4 months ago
- Status changed from Needs Feedback to Closed
Hey. I hope it's ok to close here for now: Various things in this area changed over the years and it's very possible this has been adapted. Let's restart with a fresh issue in case we missed something.