- Issue created by @nico.b
- @nicob opened merge request.
- Status changed to Needs review
about 1 year ago 12:46pm 7 November 2023 I noticed that
$menuLinkEntity = \Drupal::entityQuery('menu_link_content') ->accessCheck(FALSE) ->condition('uuid', $menu['uuid']) ->execute();
always returned an empty array. The condition does no longer to be fulfilled and it seems like this is related to the value of "uuid" being placed in the array key "value" instead of directly in "uuid", although I am not sure when and where this changed - but maybe somebody else can shed some light onto this?
I created a merge request that seems to fix the issue while maintaining backwards compatibility. However, I am not sure whether the fix could have any side effects that I'm unaware of - so feedback welcome :)
- 🇮🇱Israel Amir Simantov
Hi issue.
In my case (Drupal 10.1.7) I don't get an error when importing. I simply don't see the links in the menu (caches cleared). Hi @Amir Simantov,
did you also hit the "Import menu links" button at admin/config/development/menu_export/import and selected the menu previously in the list of menus to be imported/exported (/admin/config/development/menu_export)?
Sorry if the question sounds dumb, but when I started using this module, it was also not clear to me that this is necessary (in addition to importing/exporting the config via Drupal's config sync mechanism).
- 🇮🇱Israel Amir Simantov
Hi,
I have just selected the menus in the checkboxes.
Your question isn't dumb at all... The UX might be, though :)
Anyway, I have uninstalled the module, and my my own module with links - much better.
Thank you! - Status changed to RTBC
10 months ago 8:25pm 6 February 2024 - 🇨🇷Costa Rica marcoaa97
MR worked for me, created a patch to apply the same change.