- Issue created by @andrei.vesterli
- 🇲🇩Moldova andrei.vesterli Chisinau
Here is the attached patch with the fix
- last update
about 1 year ago Composer require failure - Open in Jenkins → Open on Drupal.org →Core: 10.1.4 + Environment: PHP 8.1 & MySQL 5.7 w/Bullseye Upgradeslast update
about 1 year ago 1 pass - last update
about 1 year ago Composer require failure - @andreivesterli opened merge request.
- Issue was unassigned.
- Status changed to Needs review
about 1 year ago 9:30pm 3 November 2023 - last update
about 1 year ago Composer require failure - 🇦🇹Austria agoradesign
With the given patch from #2, there's the risk of running into an error:
TypeError: array_merge(): Argument #2 must be of type array, null given in array_merge() (line 191
This seems to happen, when the user does not have the 'use menu link attributes' permission set
I've updated the patch to ensure an empty array, if the value is not set
- 🇨🇿Czech Republic milos.kroulik
Based on the patch in #6 it looks like this is actually a duplicate of https://www.drupal.org/project/menu_link_attributes/issues/3331630 🐛 Compatibility with other modules, options being overwritten. Fixed , which is already RTBC. Can we close this one?
- Status changed to Closed: duplicate
6 months ago 9:21am 14 May 2024 - 🇩🇪Germany Anybody Porta Westfalica
🐛 Compatibility with other modules, options being overwritten. Fixed has already been fixed. Closing this as duplicate.
- 🇧🇬Bulgaria alexrayu
Sadly the other mentioned issue did not resolve this specific error - variables passed to NestedArray::MergeDeep() can still be NULL, which causes a warning. Reworked the previous patch to also check the second variable passed. If someone still has this error in ^1.5.0, you can check this patch.