Part of #2068331: Convert comment SQL queries to the Entity Query API. Make the query EFQ and hence database agnostic.

CommentFileSizeAuthor
#2 issue-2290157.patch673 bytesmarcingy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

marcingy’s picture

marcingy CreditAttribution: marcingy commented
Parent issue: » #2068331: Convert comment SQL queries to the Entity Query API
marcingy’s picture

marcingy CreditAttribution: marcingy commented
Status: Active » Needs review
FileSize
issue-2290157.patch673 bytes
marcingy’s picture

marcingy CreditAttribution: marcingy commented
Status: Needs review » Needs work
marcingy’s picture

marcingy CreditAttribution: marcingy commented
Status: Needs work » Needs review
slashrsm’s picture

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

Looks good.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

  • Commit 5baeab7 on 8.x by webchick: 
    Issue #2290157 by marcingy: Convert comment_user_predelete query to...

Status: Fixed » Closed (fixed)

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