The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.https://cgit.drupalcode.org/node_access_grants/tree/src/NodeAccessGrants...
The $implementations should be initialised with an empty array.
If this module is installed and there are no modules that use it (no implementations), then the 'foreach' will be throwing warnings for each page load.
And if your site has watchdog enabled - the DB fills in quite quickly (5G in 1 hr)
Warning: Invalid argument supplied for foreach() in Drupal\node_access_grants\NodeAccessGrantsCollection->grants() (line 43 of modules/contrib/node_access_grants/src/NodeAccessGrantsCollection.php).
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | node_access_grants-3040218-4.patch | 1.34 KB | janehollander |
| #3 | node_access_grants-3040218-3.patch | 783 bytes | Snehal Brahmbhatt |
Comments
Comment #2
alex.skrypnykComment #3
Snehal Brahmbhatt commented@alex.designworks, It gives an error because getting $this->implementations null, When no permission set for content. Review the attached patch & let me know if you have any query regarding it.
Comment #4
janehollander commentedThe same if statement should be applied to the grants method as well. Here is patch off #3 which will do just that.
Comment #6
upchuk commented