Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I noticed that in some cases, the method \Drupal\siteimprove\SiteimproveUtils::getEntityUrls does not return an array. Where this method is used (\Drupal\siteimprove\SiteimproveUtils::setSessionUrl), however, an array is always expected.
I'll add a small patch to fix the return of the getEntityUrls method.
Comment | File | Size | Author |
---|---|---|---|
#2 | 3128478-2.patch | 699 bytes | SpadXIII |
Comments
Comment #2
SpadXIII CreditAttribution: SpadXIII at SIM commentedComment #3
alberto56 CreditAttribution: alberto56 at Dcycle commentedWorks well, thanks. This patch does away with this error:
Comment #4
esod CreditAttribution: esod at Memorial Sloan Kettering Cancer Center commentedI confirm this patch is working. With the 8x-1.5 update we are seeing this error
Warning: Invalid argument supplied for foreach() in Drupal\siteimprove\SiteimproveUtils->setSessionUrl() (line 154 of /var/www/builds/mskcc_deploy-develop/web/modules/contrib/siteimprove/src/SiteimproveUtils.php)
on every node save, so it's important to apply this patch before going live.
Comment #5
beltofteComment #7
beltofteFixed. Thanks guys!
Comment #8
beltofte