Project

General

Profile

Actions

Task #79122

closed

Deprecated BackendUtility::getRecordsByField()

Added by Morton Jonuschat almost 8 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Should have
Category:
Database API (Doctrine DBAL)
Target version:
Start date:
2016-12-30
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.1
Tags:
Complexity:
Sprint Focus:

Description

BackendUtility::getRecordsByField() has a flawed design due to passing SQL fragments. This contradicts the goal of using named
parameters for all queries in the core and requires passing the original QueryBuilder object in addition to the stringified constraint.

Replace all calls to the method with direct usage of the QueryBuilder and deprecate the method.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #86778: The backend user authentication checkFullLanguagesAccess does not check the full authentication Closed2018-10-28

Actions
Actions #1

Updated by Gerrit Code Review almost 8 years ago

  • 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 https://review.typo3.org/51078

Actions #2

Updated by Gerrit Code Review almost 8 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/51078

Actions #3

Updated by Gerrit Code Review almost 8 years ago

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

Actions #4

Updated by Gerrit Code Review almost 8 years ago

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

Actions #5

Updated by Benni Mack almost 8 years ago

  • Target version changed from 8.6 to 8 LTS
Actions #6

Updated by Gerrit Code Review almost 8 years ago

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

Actions #7

Updated by Gerrit Code Review almost 8 years ago

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

Actions #8

Updated by Gerrit Code Review almost 8 years ago

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

Actions #9

Updated by Gerrit Code Review almost 8 years ago

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

Actions #10

Updated by Morton Jonuschat over 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #11

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Resolved to Closed
Actions #12

Updated by Ursula Klinger about 6 years ago

  • Related to Bug #86778: The backend user authentication checkFullLanguagesAccess does not check the full authentication added
Actions

Also available in: Atom PDF