Problem/Motivation

We no longer need that with PHP 8.5 minimum.

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3574284

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

mondrake created an issue. See original summary.

mondrake opened merge request !14817

mondrake’s picture

mondrake
Location 🇮🇹
commented
Status: Active » Needs review
smustgrave’s picture

smustgrave commented

Does this need to be deprecated?

mondrake’s picture

mondrake
Location 🇮🇹
commented

I don't think so if we limit this to D12+, as it requires PHP 8.5 where this class is not needed. In D11 it cannot be deprecated because PHP 8.3 still requires the class. Should probably have been removed when removing deprecated code for the Database API, but we missed that.

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Thanks for the explanation. Makes sense.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Reviewed & tested by the community » Fixed

Yep just some cleanup that we missed in 12.x.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • longwave committed 7c466763 on main 
    task: #3574284 Remove PDOConnection from SQLite

By: mondrake

longwave closed merge request !14817

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.