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 commentedComment #3
alberto56 commentedWorks well, thanks. This patch does away with this error:
Comment #4
esod 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