Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TASK] Replace \PDO::PARAM_* constants with Connection::PARAM_* #514

Merged
merged 1 commit into from
Oct 20, 2023

Conversation

sbuerk
Copy link
Collaborator

@sbuerk sbuerk commented Oct 20, 2023

This change replaces the \PDO::PARAM_* constants with the TYPO3
\TYPO3\CMS\Core\Database\Connection::PARAM_* alternatives. This
aligns the testing-framework with the core and the communicated
best-practice.

The direct dependency to ext-pdo is removed right away.

Note: doctrine/dbal 4+ dropps support for the \PDO integer constants.
Using the Connection constants allows us to change the mapping
in that class and consuming code, like extensions or this package
automaticlly uses the correct enum class on upgrade.

Used command(s):

composer remove ext-pdo

Releases: main, 7

This change replaces the `\PDO::PARAM_*` constants with the TYPO3
`\TYPO3\CMS\Core\Database\Connection::PARAM_*` alternatives. This
aligns the testing-framework with the core and the communicated
best-practice.

The direct dependency to `ext-pdo` is removed right away.

Note: `doctrine/dbal 4+` dropps support for the \PDO integer constants.
       Using the `Connection` constants allows us to change the mapping
       in that class and consuming code, like extensions or this package
       automaticlly uses the correct enum class on upgrade.

Used command(s):

```shell
composer remove ext-pdo
```

Releases: main, 7
@sbuerk sbuerk merged commit 5c61903 into 7 Oct 20, 2023
8 checks passed
@sbuerk sbuerk deleted the 7-stefan-1 branch October 20, 2023 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant