I can't depend on Core classes.

Rewrite DefaultMenuLinkTreeManipulators into AnonymousMenuLinkTreeManipulator instead of simply extending it.

CommentFileSizeAuthor
#2 footermap-2853964-fix-core-bc-break-2.patch10.33 KBmradcliffe
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mradcliffe created an issue. See original summary.

mradcliffe’s picture

mradcliffe
he/him
Primary language English
CreditAttribution: mradcliffe at Kosada commented
Status: Active » Needs review
FileSize
footermap-2853964-fix-core-bc-break-2.patch10.33 KB

This adds the code in DefaultMenuLinkTreeManipulator prior to the backwards-incompatible change in #2389335: Deprecate entity.query service and replace with using the entity storage's getQuery() method.

  • mradcliffe committed fe49387 on 8.x-1.x 
    Issue #2853964 by mradcliffe: Add backwards-compatible version of core...
mradcliffe’s picture

mradcliffe
he/him
Primary language English
CreditAttribution: mradcliffe at Kosada commented
Status: Needs review » Fixed

After thinking about it. I'm going to commit my approach because I can and should no longer rely on core to provide a stable API.

Status: Fixed » Closed (fixed)

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