Skip to content

[2.x] chore(core): add MariaDB to frontend DatabaseDriver enum#4701

Merged
imorland merged 1 commit into
flarum:2.xfrom
glowingblue:di/database-support
Jun 10, 2026
Merged

[2.x] chore(core): add MariaDB to frontend DatabaseDriver enum#4701
imorland merged 1 commit into
flarum:2.xfrom
glowingblue:di/database-support

Conversation

@DavideIadeluca

@DavideIadeluca DavideIadeluca commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Fixes #0000
With MariaDB and MySQL diverging more and more we've noticed and changed in the Framework that both databases need to be treaded separatly. Some more information on this can be viewed athttps://github.com//pull/4682

Changes proposed in this pull request:
Add MariaDB to the DatabaseDriver enum

Reviewers should focus on:

Screenshot

Necessity

  • Has the problem that is being solved here been clearly explained?
  • If applicable, have various options for solving this problem been considered?
  • For core PRs, does this need to be in core, or could it be in an extension?
  • Are we willing to maintain this for years / potentially forever?

Confirmed

  • Frontend changes: tested on a local Flarum installation.
  • Backend changes: tests are green (run composer test).
  • Core developer confirmed locally this works as intended.
  • Tests have been added, or are not appropriate here.

Required changes:

  • Related documentation PR: (Remove if irrelevant)

@DavideIadeluca DavideIadeluca changed the title chore(core): add MariaDB to frontend DatabaseDriver enum [2.x] chore(core): add MariaDB to frontend DatabaseDriver enum Jun 10, 2026
@DavideIadeluca DavideIadeluca marked this pull request as ready for review June 10, 2026 12:20
@DavideIadeluca DavideIadeluca requested a review from a team as a code owner June 10, 2026 12:20
@imorland imorland added this to the 2.0.0-rc.4 milestone Jun 10, 2026
@imorland imorland merged commit c2448e9 into flarum:2.x Jun 10, 2026
19 checks passed
@imorland imorland deleted the di/database-support branch June 10, 2026 12:27
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