Hi Mitesh,

I am getting following notice:
Notice: Undefined index: class in menu_link_modal_set_attributes() (line 113 of modules/contrib/menu_link_modal/menu_link_modal.module).

CommentFileSizeAuthor
#4 undefined-index-2912358-4.patch896 bytesmiteshmap
#3 2912358_class_index_undefined.patch733 bytessolide-echt
#2 notice.png13.58 KBnarendrar

Comments

NarendraR created an issue. See original summary.

narendrar’s picture

narendrar
He/Him
Primary language English
Location Jaipur, India
commented
StatusFileSize
new notice.png13.58 KB
solide-echt’s picture

solide-echt commented
StatusFileSize
new 2912358_class_index_undefined.patch733 bytes

Attached is a patch that, at least in my case, fixed the problem. The ternary operator produces an empty array if the menu item has no classes yet (which I believe is true unless some other module adds classes too); I've added a check to manually add the ['class'] array.

miteshmap’s picture

miteshmap
Primary language Gujarati
commented
StatusFileSize
new undefined-index-2912358-4.patch896 bytes

Define class attributes as array if it's not set.

  • miteshmap committed ee1374f on 8.x-1.x 
    Issue #2912358 by solide-echt, miteshmap, NarendraR: Undefined index...
miteshmap’s picture

miteshmap
Primary language Gujarati
commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

dj12djdjs’s picture

dj12djdjs commented
Version: 8.x-1.2 » 8.x-1.3

It appears that I get this notice as well:
Notice: Undefined index: class in menu_link_modal_set_attributes() (line 104 in /modules/menu_link_modal/menu_link_modal.module)

dj12djdjs’s picture

dj12djdjs commented

This issue was fixed with the latest update.