Problem/Motivation

In case both book and menu_ui modules are enabled, tokens for node's menu aren't available.

Steps to reproduce

Install site, enable both book and menu_ui modules, check in pathauto that node's menu tokens aren't available.

Proposed resolution

Merge arrays recursively, otherwise they'll override each other

Remaining tasks

Confirm issue and merge fix

User interface changes

no

API changes

no

Data model changes

no

Issue fork token-3427474

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mirom created an issue. See original summary.

mirom opened merge request !67

mirom’s picture

mirom CreditAttribution: mirom as a volunteer commented
Status: Active » Needs review
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented

Can we test this? Enable book in TokenMenuTest?

johnny5th’s picture

johnny5th CreditAttribution: johnny5th at RBN Energy, LLC. commented

I'm also seeing this issue. Disabling book fixes the menu-link tokens.

  • Berdir committed 5b3cfb3c on 8.x-1.x authored by mirom 
    Issue #3427474: Book and Menu tokens can't coexist
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Fixed

Merged. I had a quick look at a test, but had some weird unrelated issues locally. possibly we can move the book tokens to contrib book module and drop this.

mirom’s picture

mirom CreditAttribution: mirom for CSGov commented