DomainLoader::loadMultipleSorted ignores optional $ids argument when passed, which effectively loads everything even if it was told to load a subset of domains. attached is a super simple patch to make sure this doesn't happen.

CommentFileSizeAuthor
Use__ids_in_loading_multiple_sorted_domains_.patch541 bytesFooZee
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

FooZee created an issue. See original summary.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented
Status: Needs review » Reviewed & tested by the community

Nice catch.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented
Status: Reviewed & tested by the community » Fixed

Committed.

In future, please roll patches from the project root. /domain, not /domain/MODULE.

  • agentrickard committed e240bc8 on 8.x-1.x authored by FooZee 
    Issue #2815833 by FooZee: DomainLoader::loadMultipleSorted ignores...

Status: Fixed » Closed (fixed)

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