Existing menu links show validation issues on migration (and ALL menu links pointing to node translations are invalid)

Created on 28 April 2019, over 5 years ago
Updated 16 March 2023, almost 2 years ago

Problem/Motivation

The d7_menu_links migration complains about failed path validation for many links.

The reason: it points to content entities that have not yet been migrated. Its migration dependencies are:

migration_dependencies:
  required:
    - d7_menu

While this guarantees the existence of the menu in which a migrated menu link lives, it does not guarantee that the location the menu link points to exists. This is the problem.

Proposed resolution

  1. Add dependencies.
  2. Add deriver, to allow migrating subsets of menu links (to be able to migrate only menu links targeting only content entities that have already been migrated).

Remaining tasks

  1. Add tests for the deriver.

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

TBD

πŸ› Bug report
Status

Needs work

Version

10.1 ✨

Component
MigrationΒ  β†’

Last updated 3 days ago

Created by

Live updates comments and jobs are added and updated live.
  • Needs subsystem maintainer review

    It is used to alert the maintainer(s) of a particular core subsystem that an issue significantly impacts their subsystem, and their signoff is needed (see the governance policy draft for more information). Also, if you use this tag, make sure the issue component is set to the correct subsystem. If an issue significantly impacts more than one subsystem, use needs framework manager review instead.

  • Needs tests

    The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

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