Problem/Motivation

MenuLinkAdd class constructor use RedirectDestination instead of RedirectDestinationInterface, adding faulty dependency to RedirectDestination class.

Proposed resolution

MenuLinkAdd class constructor should be changed to use the RedirectDestinationInterface

Remaining tasks

User interface changes

none

API changes

none

Data model changes

none

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Bug, because other implementations of RedirectDestination won't be possible
Issue priority Normal, because the impact is not that high
Disruption No disruption
CommentFileSizeAuthor
#1 faulty_dependency_to-2516866-1.patch1.85 KBwillzyx

Comments

willzyx’s picture

willzyx commented
StatusFileSize
new faulty_dependency_to-2516866-1.patch1.85 KB
dawehner’s picture

dawehner
Primary language German
commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
Issue tags: +Quickfix

That is perfect, good catch!

alexpott’s picture

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

Committed 48aa09b and pushed to 8.0.x. Thanks!

  • alexpott committed 48aa09b on 8.0.x 
    Issue #2516866 by willzyx: Faulty dependency to RedirectDestination in...

Status: Fixed » Closed (fixed)

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