- 🇩🇪Germany Anybody Porta Westfalica
Nothing seems to happen at ACL anymore. Should this really be postponed on that? PHP8.1 is the recommended PHP version and this is filling up logs.
- 🇳🇴Norway gisle Norway
What do you suggest? It is Postponed until ACL is fixed. Is there anything to do but wait until that happens.
- 🇨🇭Switzerland LpSolit
@gisle: did you notice my patch about ACL (see issue 3306205)? I got no comment about it.
- 🇳🇴Norway gisle Norway
Yes. I see that [#33006205] is suggested as an "untested patch", but with no community reviews. It is notbon my schedule to review it.
- 🇩🇪Germany Anybody Porta Westfalica
- 🇪🇸Spain lapurddrupal
Patch #14 on Drupal 9.5.4 and PHP 8.12 worked for me. Thanks to you!!!
- 🇨🇦Canada brunodbo
This might be related to this core issue: 📌 Unserialize(): Passing null to parameter #1 ($data) of type string is deprecated in Drupal\Core\Entity\Sql\SqlContentEntityStorage Needs work
- 🇺🇦Ukraine Taran2L Lviv
So, patch from #14 works as expected and does fix the issue, but it has an extra safe code that is not needed.
Approach from #24 is a valid try to improve the module by storing config not as serialized string, but like a normal array, but this approach requires a) update to config schema b) hook_update to resave all settings => clearly not the scope of this issue.
Also, postponing this on ACL fixes can be worked around. See the attached patch
- 🇺🇦Ukraine Taran2L Lviv
Okay, enabling deprecations is not part of the scope either, let's try this minimalistic patch
- Status changed to Needs review
over 1 year ago 10:10am 11 April 2023 - last update
over 1 year ago Composer require failure - last update
over 1 year ago 4 pass, 4 fail - last update
over 1 year ago 8 pass thanks @Taran2L. I can test this later. In any case, should we hold off until the new ACL release it published, and use a new version of your latest patch that doesn't depend on the dev branch of ACL? Or do you think this is ready to go as is?
@gisle, thought?
hey @Taran2L and @hswong3i, I am unable to reproduce this error.
On my dev instance, I have the following in
settings.local.php
:# https://www.drupal.org/forum/support/post-installation/2018-07-18/enable-drupal-8-backend-errorlogdebugging-mode $config['system.logging']['error_level'] = 'verbose'; error_reporting(E_ALL); ini_set('display_errors', TRUE); ini_set('display_startup_errors', TRUE);
I am running PHP 8.1, and I have followed the steps in the OP: Navigate to Manage » Structure » Content types. Under operations, Select "Access control" for any content type.
I am not seeing this warning in any of my php/nginx log files. Any advice?
- Status changed to Fixed
over 1 year ago 5:10pm 25 April 2023 - 🇳🇴Norway gisle Norway
The fix in patch in comment #54 has been pushed to the latest development snapshot.
thanks @gisle and @Taran2L!
I am working on the ACL issue(s). Savlis popped in to do some work, but seems to have gone absent again. I'm hopefully he will return to clean up a few more loose ends and publish a new release. 🤞
- Status changed to Fixed
over 1 year ago 6:54pm 9 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.