Skip to content
Snippets Groups Projects
Commit e4d6ff5e authored by Stefan Bürk's avatar Stefan Bürk
Browse files

[TASK] Replace sql int casts with `ExpressionBuilder::castInt()`

Since #98503 the new TypoScript parser is used
in the frontend, which used a value list trick
to minimize database queries. That required to
do proper value casting for each list element,
which has been implementend manually with dbms
vendor aware handling.

Recently #103309 enriched the `ExpressionBuilder`
with a couple of additional expression methods,
including `castInt()` expression.

This change replaces the manual db aware integer
casting with the new `ExpressionBuilder::castInt()`
method within the TypoScript parser classes along
with the corresponding `@todo` marker.

Resolves: #103343
Related: #103309
Related: #98503
Releases: main
Change-Id: I462a6b0bb8195c6c0db7b70fb72ac1890ee4aadb
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83355


Tested-by: default avatarcore-ci <typo3@b13.com>
Tested-by: default avatarBenni Mack <benni@typo3.org>
Tested-by: default avatarAnja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: default avatarStefan Bürk <stefan@buerk.tech>
Reviewed-by: default avatarAndreas Kienast <a.fernandez@scripting-base.de>
Tested-by: default avatarStefan Bürk <stefan@buerk.tech>
Tested-by: default avatarAndreas Kienast <a.fernandez@scripting-base.de>
Reviewed-by: default avatarBenni Mack <benni@typo3.org>
Reviewed-by: default avatarAnja Leichsenring <aleichsenring@ab-softlab.de>
parent 269a05e8
Branches
Tags
No related merge requests found
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment