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.The config_perms_perms() function is expected to return an array of the configured permissions, or a specific permission if a machine name has been provided. When the configuration is loaded from the cache, the actual configuration is stored at the data property for the cache object, but the function uses the $perms variable as if it was an array.
This will create problems when the return value of the config_perms_perms() is used as an array in foreach loops, for example. This could be found in config_perms_menu_alter().
The solution to this would be to normalize the $perms variable in order to maintain a consistent data structure within the config_perms_perms() function. The result of this would be that the rest of the code within the module can safely rely on that the structure of the returned variable will always be the same.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | config_perms-cache_retrieval-2177151-1.patch | 716 bytes | olofbokedal |
Comments
Comment #1
olofbokedal CreditAttribution: olofbokedal commentedComment #2
mlncn CreditAttribution: mlncn commentedThis is essentially the same fix as in #2200925: On cache clear: Invalid argument supplied for foreach() - config_perms.module but this one tests for cache existence or rebuild explicitly, and has a better explanatory comment.
Tested and it works, which is also important ;-) Nice work, this issue and patch were also first so marking the other duplicate.
Comment #3
mlncn CreditAttribution: mlncn commentedFixed in commit 32f1f27 with credit to olofjohansson.