Problem/Motivation

Configuration for localgov_menu_link_group fails due to special characters.

Steps to reproduce

  1. Install localgov_drupal
  2. Enable localgov_france
  3. Export the config drush cex -y
  4. Edit config/sync/localgov_menu_link_group.localgov_menu_link_group.localgov_menu_link_group_news.yml and replace "Actualités" by "Actualites"
  5. Execute
    drush cim</li>
      <li>Execute <code>drush cr
  6. Edit again config/sync/localgov_menu_link_group.localgov_menu_link_group.localgov_menu_link_group_news.yml and replace "Actualites" by "Actualités"
  7. Execute
    drush cim</li>
      <li>Execute <code>drush cr

Proposed resolution

`prepareMenuLinkIndexForGroup` in MenuLinkGrouper is trying to generate a machine name based on the label, however the label may contains special characters like "Actualités". This is then failing while importing the config.

The proposal would be to make use of the transliteration service to prevent this to happen and use it over `$group->label()`

Remaining tasks

  1. Create a MR with the fix
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

  • 1.x Comparechanges, plain diff MR !2

Comments

luismagr created an issue. See original summary.

luismagr’s picture

Project: LocalGov Drupal » LocalGov Menu Link Group
Version: 3.3.1 » 1.1.8
luismagr’s picture

Issue summary: View changes

luismagr’s picture

Status: Active » Needs review
luismagr’s picture

Hi!

Would it be possible to credit @Penyaskito for this? He pointed out us to the right direction here https://drupal.slack.com/archives/C2PB969UY/p1765567387022489

Really appreciate his help on this