Fix compatibility with menu_link_weight module

Created on 28 May 2023, over 1 year ago

Updated 31 May 2024, 6 months ago

Problem/Motivation

When I installed and enabled 2.0.0-rc1 of this module on a Drupal 9.5.9 site which I was already using the menu_link_weight module on, the node edit pages displayed as broken.

The page showed a white screen with this text:

The website encountered an unexpected error. Please try again later.
TypeError: strstr(): Argument #1 ($haystack) must be of type string, array given in strstr() (line 127 of modules/contrib/menu_link_weight/menu_link_weight.node.inc).

Steps to reproduce

Install Drupal 9.5.9, install and enable menu_link_weight 1.0.0-beta5, then install and enable menu_select 2.0.0-rc1

Proposed resolution

Please could you see if you could make this module compatible with menu_link_weight, thank you.

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report

Status

Closed: won't fix

Version

2.0

Component

Code

Created by

🇬🇧United Kingdom ChandeepKhosa

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @ChandeepKhosa
Comment over 1 year ago →
🇬🇧United Kingdom ChandeepKhosa
Status changed to Closed: won't fix 6 months ago7:54pm 31 May 2024
Comment 6 months ago →
🇨🇦Canada mcgowanm
Doesn't look like there's much we can do from our end. This module must change the form element and Menu Link Weight expects the standard form element.

I have opened a ticket on their end and provided a basic compatiblity patch so if anyone's finding this in the future head on over: https://www.drupal.org/project/menu_link_weight/issues/3451660#comment-1... ✨ Add support for menu_select Needs work

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024