Task #96551: Use proper QueryBuilder execute() replacements executeQuery() and executeStatement() - TYPO3 Core - TYPO3 Forge

Custom queries

Accessibility
Easy tasks
Forge Triage
JavaScript issues
JavaScript Issues (w/o Epics/Stories)
My open issues
No feedback within 90 days
Open Bugs
Open issues of 10
Recently modified
Regressions
Stabilization Issues
Usability or UX

Actions

Copy link

Task #96551

closed

Epic #96521: Enhance PhpStan coverage

Use proper QueryBuilder execute() replacements executeQuery() and executeStatement()

Added by Stefan Bürk almost 3 years ago. Updated about 2 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Stefan Bürk

Category:

Database API (Doctrine DBAL)

Target version:

Start date:

2022-01-16

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Sprint Focus:

Related issues 1 (0 open — 1 closed)

Related to TYPO3 Core - Task #96972: Deprecate QueryBuilder::execute()

Closed

Stefan Bürk

2022-02-20

Actions

Issue # Delay: days Cancel

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Stefan Bürk almost 3 years ago

Category set to Database API (Doctrine DBAL)
Parent task set to #96521

Replaceing doctrine/dbal execute() with executeQuery() for count/select queries
and executeStatement() for insert/update/delete queries would have following
benefits:

- solves core phpstan errors which are currently swallowed by checkThisOnly config
- prepare/cleanup the core and avoid using the to-be deprecated method execute() early
- start early to get firm with the new methods and be a demonstration for developers

Actions

Copy link

Updated by Gerrit Code Review almost 3 years ago

Status changed from New to Under Review

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

Actions

Copy link

Updated by Gerrit Code Review almost 3 years ago

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

Actions

Copy link

Updated by Gerrit Code Review almost 3 years ago

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

Actions

Copy link

Updated by Gerrit Code Review almost 3 years ago

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

Actions

Copy link

Updated by Gerrit Code Review almost 3 years ago

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

Actions

Copy link