Project

General

Profile

Actions

Bug #73007

closed

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

Insufficient sysLanguageStatement does not respect enableFields for translations

Added by Philipp Wrann almost 9 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Extbase + l10n
Start date:
2016-01-29
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
easy
Is Regression:
No
Sprint Focus:

Description

The statement added to sql queries to load translated entities only respects the deleted falg.

If you define the starttime, endtime, hidden and feaccess features for translations no filters are applied on queries.

This results in a QueryResult containing a number of X Rows.

After mapping those elements will be hidden because in languageOverlay the fields ARE respected.

In my case i display the number of results found by the orm, followed by a paginated widget showing those results.

So it says "11 Results found"

But only 6 Results are rendered.

The Problem can be found in:
https://github.com/TYPO3/TYPO3.CMS/blob/master/typo3/sysext/extbase/Classes/Persistence/Generic/Storage/Typo3DbQueryParser.php#L733

If additional enablefields are configured and not set as l10n_mode=exclude they must be appended as well.

This can be solved without touching anything else in the translation setup, a patch would only affect the subquery statement

The bug exists in 6.2 and 7 LTS


Related issues 5 (2 open3 closed)

Related to TYPO3 Core - Bug #76974: Translation handling with starttime/stoptimeClosed2016-07-07

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

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

Actions
Related to TYPO3 Core - Bug #100638: setIgnoreEnableFields() does not work on language overlaysNew2023-04-17

Actions
Has duplicate TYPO3 Core - Bug #81040: Extbase QueryResult::count() incorrect for strict language modeClosed2017-04-27

Actions
Actions #1

Updated by Gerrit Code Review almost 9 years ago

  • Status changed from New to Under Review

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

Actions #2

Updated by Gerrit Code Review almost 9 years ago

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

Actions #3

Updated by Mathias Schreiber over 8 years ago

  • Status changed from Under Review to New
Actions #4

Updated by Markus Klein over 8 years ago

  • Status changed from New to Accepted
Actions #5

Updated by Gerrit Code Review over 7 years ago

  • Status changed from Accepted 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/52072

Actions #6

Updated by Gerrit Code Review over 7 years ago

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

Actions #7

Updated by Cornel Widmer over 7 years ago

  • Related to Bug #76974: Translation handling with starttime/stoptime added
Actions #8

Updated by Axel Helmert over 7 years ago

  • Has duplicate Bug #81040: Extbase QueryResult::count() incorrect for strict language mode added
Actions #9

Updated by Peter Niederlag over 7 years ago

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

Updated by Riccardo De Contardi about 7 years ago

  • Category changed from Extbase to Extbase + l10n
Actions #11

Updated by Gerrit Code Review over 6 years ago

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

Actions #12

Updated by Tymoteusz Motylewski about 6 years ago

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

Updated by Tymoteusz Motylewski about 6 years ago

  • Parent task set to #86307
Actions #14

Updated by Susanne Moog over 5 years ago

  • Status changed from Under Review to Needs Feedback

Review was abandoned and maybe fixed in the mean time. Please check if the issue still persists in current TYPO3 versions.

Actions #15

Updated by Susanne Moog over 4 years ago

  • Status changed from Needs Feedback to Closed

Extbase language handling was reworked and no feedback here for quite some time, closing the issue now. If you still have problems with this, please open a new issue with steps to reproduce and example code. Thanks!

Actions #16

Updated by Christian Kuhn over 1 year ago

  • Related to Bug #100638: setIgnoreEnableFields() does not work on language overlays added
Actions

Also available in: Atom PDF