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, almost 10 years ago
Updated 19 April 2023, about 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 1 day 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.69.0 2024