Use an entityQuery in getPartialItemIds

Open on Drupal.org →

Created on 13 February 2024, 9 months ago

Updated 27 February 2024, 9 months ago

Problem/Motivation

Currently in ContentEntityRevisions::getPartialItemIds, we currently do 1 query for the entities, and then another for each entity returned by the first query to get all the revision ids. We don't need to do that - we can get all revision ids in a single query. This will massively decrease the number of queries required.

📌 Task

Status

Fixed

Version

2.0

Component

Code

Created by

🇦🇺Australia acbramley

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!12Use an entityQuery in getPartialItemIds
Merged
🇦🇺Australia acbramley
updated 9 months ago

Comments & Activities

Issue created by @acbramley
Merge request !12Resolve #3420969 "Use an entityquery" → (Merged) created by acbramley
Status changed to Needs review 9 months ago1:07am 13 February 2024
Comment 9 months ago →
🇦🇺Australia acbramley
Pipeline finished with Skipped
9 months ago
#93489

Comment 9 months ago →

acbramley → committed 5abb0688 on 2.x

Issue #3420969 by acbramley: Use an entityQuery in getPartialItemIds

Status changed to Fixed 9 months ago1:34am 13 February 2024
Comment 9 months ago →
🇦🇺Australia acbramley
Included some phpcs fixes too.
Comment 9 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024