Firstchild checkbox not saved when Mix module is enabled

Created on 8 December 2023, 11 months ago
Updated 9 December 2023, 11 months ago

Problem/Motivation

When Mix module is enabled, the firschild checkbox is not saved.

Steps to reproduce

Install and enable firstchild and Mix modules
Edit menu (/admin/structure/menu) and check First child checkbox
Save

Open again the save menu entry and checkbox is not checked

In the frontend, clicking on the menu does not redirect to the first child menu

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

πŸ› Bug report
Status

Postponed: needs info

Version

2.1

Component

Code

Created by

πŸ‡«πŸ‡·France olivierh65

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @olivierh65
  • Assigned to pingevt
  • Status changed to Postponed: needs info 11 months ago
  • πŸ‡ΊπŸ‡ΈUnited States pingevt

    When taking a look at this, I tested with Mix 1.9.0. When creating a brand new menu link without children, seems there is an error with the mix module. I'm seeing this in the logs when trying to enable First Child menu link:

    Error: Attempt to assign property "options" on null in mix_menu_link_content_form_entity_builder() (line 651 of /Users/peteinge/Sites/drupal10/web/modules/contrib/mix/mix.module).

    At a quick glance there is no error checking or anything on the mix module's side. This should be addressed before we work on trying to make this compatible with another module.

    After creating a menu structure, and trying to change a parent link, I'm seeing the issue as described above. Diving deeper, the mix module is wiping out all other options on a menu link item in _mix_build_menu_link_options(). I would consider this an issue for the mix module to fix, and merge other options that might be present. This module can't do anything if another module is wiping out the data.

    I'd consider this a Mix module issue.

Production build 0.71.5 2024