This is a child task of #1775842: [meta] Convert all variables to state and/or config systems. I think this should be changed to state.

CommentFileSizeAuthor
#6 drupal-1975114-6.patch774 bytesdawehner
#4 drupal-1975114-4.patch52.67 KBdawehner
#1 1975114-menu_default_state-cmi-drupal-1.patch816 bytesACF

Comments

ACF’s picture

ACF commented
Status: Active » Needs review
StatusFileSize
new 1975114-menu_default_state-cmi-drupal-1.patch816 bytes
catch’s picture

catch
he/him
Primary language English
commented
Title: Change menu_default_node_menu to state. » Views references menu_default_node_menu variable which doesn't exist anywhere else
Component: configuration system » views.module
Category: task » bug
Issue tags: +VDC

This looks like a reference to a core variable, but the core variable doesn't exist at all/any more. Moving to views - maybe someone working on VDC knows what this was trying to do.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented

git pickaxe shows this was removed from core during the 6>7 upgrade cycle in #716792: Remove 'Default menu for content' setting from structure>menus>settings.. This core variable is now set per content type, so there seems to be no useful replacement?

This bug should also be raised against Views 7.x-3.x.

dawehner’s picture

dawehner
Primary language German
commented
StatusFileSize
new drupal-1975114-4.patch52.67 KB

So we can't provide anything more useful than navigation.

Status: Needs review » Needs work

The last submitted patch, drupal-1975114-4.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs work » Needs review
StatusFileSize
new drupal-1975114-6.patch774 bytes

Ups, this patch was way too big :)

aspilicious’s picture

aspilicious commented
Status: Needs review » Reviewed & tested by the community

:)

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed a389298 and pushed to 8.x. Thanks!

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