Re-work menu_ui_form_node_type_form_alter() so we don't have to pre-load all possible options

Created on 26 July 2014, over 10 years ago
Updated 19 April 2023, over 1 year ago

Problem/Motivation

After #2301317: MenuLinkNG part4: Conversion β†’ we have a remaining @todo in menu_ui.module where it alters the node type form.

We are pre-loading all the possible menu parent options into the $form array, but then using ajax in the form to just show some of them. Since the options array can be very large, this is a potential performance/memory problem as it is now.

Proposed resolution

Re-work into better FAPI code so we implement the necessary process steps to avoid validation errors, or rework this form to be more like the book module form where only I menu at a time has links displayed.

Remaining tasks

todo

User interface changes

todo

API changes

todo

πŸ“Œ Task
Status

Active

Version

9.5

Component
Menu systemΒ  β†’

Last updated about 23 hours ago

Created by

πŸ‡ΊπŸ‡ΈUnited States pwolanin

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024