Skip to content

Comments

Finish Doctrine DBAL 3.x migration#41454

Open
DeepDiver1975 wants to merge 1 commit intodeps/dbal-3from
finish-dbal-3-migration-6256941112454037654
Open

Finish Doctrine DBAL 3.x migration#41454
DeepDiver1975 wants to merge 1 commit intodeps/dbal-3from
finish-dbal-3-migration-6256941112454037654

Conversation

@DeepDiver1975
Copy link
Member

This PR completes the migration from Doctrine DBAL 2.x to 3.x.
Key changes include updating core database connection interfaces and classes, enhancing the QueryBuilder with new execution methods, and globally updating deprecated method calls throughout the codebase. Special care was taken to maintain compatibility with legacy components and ensure that non-database fetch() calls (like those in Symfony Console) remain untouched.


PR created automatically by Jules for task 6256941112454037654 started by @DeepDiver1975

@google-labs-jules
Copy link

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@update-docs
Copy link

update-docs bot commented Feb 18, 2026

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@CLAassistant
Copy link

CLAassistant commented Feb 18, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@DeepDiver1975
Copy link
Member Author

rebase

@google-labs-jules
Copy link

rebase

Acknowledged. I'm resubmitting the changes to ensure everything is up to date.

@DeepDiver1975 DeepDiver1975 force-pushed the finish-dbal-3-migration-6256941112454037654 branch 2 times, most recently from bbbcc7b to 4db9e66 Compare February 19, 2026 10:53
@DeepDiver1975 DeepDiver1975 force-pushed the deps/dbal-3 branch 2 times, most recently from f88593d to faaa0c8 Compare February 20, 2026 11:51
- Updated IDBConnection, Connection, and OracleConnection with DBAL 3 type hints.
- Replaced deprecated getSchemaManager() with createSchemaManager().
- Added executeQuery() and executeStatement() to IQueryBuilder and QueryBuilder.
- Updated OC_DB_StatementWrapper for DBAL 3 Result compatibility.
- Globally replaced fetchAssoc, fetchArray, fetchColumn, and executeUpdate with DBAL 3 equivalents.
- Preserved Result::free() calls.
- Ensured Symfony Console BufferedOutput::fetch() calls are not incorrectly replaced.
- Updated lib/base.php to allow PHP versions up to 8.4.0.

Co-authored-by: DeepDiver1975 <1005065+DeepDiver1975@users.noreply.github.com>
@DeepDiver1975 DeepDiver1975 force-pushed the finish-dbal-3-migration-6256941112454037654 branch from 4db9e66 to 4455487 Compare February 20, 2026 12:11
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.

2 participants