Problem/Motivation

Parent issue converted mysqli.install to oo hooks.

Most of the code there is copypasted from earlier mysql.install, now converted to oo hook too.

Proposed resolution

Introduce a common trait to use across the two drivers and reduce duplication.

Remaining tasks

User interface changes

N/A

Introduced terminology

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

Issue fork drupal-3534183

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 !12635

mondrake’s picture

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

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

Seems like a straight forward update. Searched for $requirements['mysql_transaction_level'] to see if any other duplicates and just these 2 instances.

  • catch committed 91157925 on 11.x 
    Issue #3534183 by mondrake: Introduce a trait to manage hook...
catch’s picture

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

Committed/pushed to 11.x, thanks!

catch closed merge request !12635

Status: Fixed » Closed (fixed)

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