Skip to content

Support doctrine/dbal 4 alongside 3 - #84

Merged
dmolineus merged 1 commit into
contao-bootstrap:masterfrom
ericges:chore/doctrine-dbal-4-support
Aug 19, 2026
Merged

dmolineus merged 1 commit into
contao-bootstrap:masterfrom
ericges:chore/doctrine-dbal-4-support

Conversation

@ericges

@ericges ericges commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Introduce support of DBAL 4:

  • Widen the doctrine/dbal constraint to ^3.4 || ^4.0
  • In DBAL 4 AbstractSchemaManager::tablesExist() only accepts an array, while DBAL 3 still accepted a string (with deprecation)
    • Dynamic grid size column check passed string, raising TypeError on every tl_bs_grid DCA load under DBAL 4
  • The remaining DBAL touchpoints were checked against the 4.3.0 sources and need no change:
    • listTableColumns() is only ever called with the table name
    • int|string return type of insert()/update()/ executeStatement() is either cast or unused

The DBAL ^3.4 constraint is still required in support of older/LTS contao versions.

Widen the doctrine/dbal constraint to `^3.4 || ^4.0`.

DBAL 4 types `AbstractSchemaManager::tablesExist()` as `array`, while DBAL 3
still accepted a string with a deprecation. The dynamic grid size column
check passed a bare string, which would have raised a TypeError on every
tl_bs_grid DCA load under DBAL 4.

The remaining DBAL touchpoints were checked against the 4.3.0 sources and
need no change: `listTableColumns()` is only ever called with the table
name, and the `int|string` return type of `insert()`/`update()`/
`executeStatement()` is either cast or unused.

Dependency constraints already permit DBAL-4-capable releases, so no other
requirement changes: contao/core-bundle from 5.6, contao-bootstrap/core
from 3.1, netzmacht/contao-toolkit from 4.0, mvo/contao-group-widget from
1.6 and meta-palettes from 2.1. On Contao 5.3-5.5 the resolver keeps
selecting DBAL 3, which is why the `^3.4` half of the constraint stays.
@dmolineus dmolineus self-assigned this Aug 19, 2026
@dmolineus
dmolineus merged commit 4815970 into contao-bootstrap:master Aug 19, 2026
4 checks passed
@dmolineus

Copy link
Copy Markdown
Member

Thanks, released as 3.1.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants