QueuerService.php:84-90

// Override the mapping with information stored in CMI, then filter out
// everything that isn't enabled and finally flip the array with just ids.
foreach ($this->configFactory->get('purge.plugins')->get('queuers') as $inst) {
  if (isset($this->plugins_enabled[$inst['plugin_id']])) {
    $this->plugins_enabled[$inst['plugin_id']] = $inst['status'];
  }
}

This produces a warning on site install: "Invalid argument supplied for foreach() QueuersService.php:86"

Should be updated to something like this:

// Override the mapping with information stored in CMI, then filter out
// everything that isn't enabled and finally flip the array with just ids.
$queuers = $this->configFactory->get('purge.plugins')->get('queuers');
if ($queuers) {
  foreach ($queuers as $inst) {
    if (isset($this->plugins_enabled[$inst['plugin_id']])) {
      $this->plugins_enabled[$inst['plugin_id']] = $inst['status'];
    }
  }
}
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lahoosascoots created an issue. See original summary.

lahoosascoots’s picture

Status: Active » Needs review
arknoll’s picture

Status: Needs review » Reviewed & tested by the community

Tested and works. Thank you!

nielsvm’s picture

reviewing+testing

nielsvm’s picture

Title: QueuerService assumes there are queuers in CMI » QueuerService and ProcessorsService assumes there are queuers in CMI

  • nielsvm committed d48a58d on 8.x-3.x authored by lahoosascoots
    Issue #2706567 by lahoosascoots, nielsvm, arknoll: QueuerService and...
nielsvm’s picture

Status: Reviewed & tested by the community » Fixed

Thanks guys, brace for 8.x-3.0-beta5!

Status: Fixed » Closed (fixed)

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