Problem/Motivation

In #3211780: Deprecate Connection::queryTemporary() we deprecated Connection::queryTemporary but forgot to deprecate Connection::$temporaryNameIndex too.

Proposed resolution

Add a deprecation notice so we can remove in D10.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3224199

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

mondrake’s picture

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

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

Look good to me.
Lets deprecate the protected variable.

  • catch committed 3a4c171 on 9.3.x 
    Issue #3224199 by mondrake: Connection::$temporaryNameIndex should be...
catch’s picture

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

Committed 3a4c171 and pushed to 9.3.x. Thanks!

mondrake closed merge request !952

Status: Fixed » Closed (fixed)

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