The current version of MatcherDumper::getFit() does not work though it seems to be that it's never called.

MatcherDumper::dump() uses the implentation of RouteCompiler, so i'm wondering what to do.

CommentFileSizeAuthor
#3 drupal-1801356-3.patch811 bytesdawehner
#1 drupal-1906322-1.patch834 bytesdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
FileSize
drupal-1906322-1.patch834 bytes

Just posting a patch to make it working, but I'm not sure whether we should actually care about it.

Crell’s picture

Crell CreditAttribution: Crell commented

I'm pretty sure that's vestigial and we can just remove that method entirely.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Active » Needs review
FileSize
drupal-1801356-3.patch811 bytes

Oh right, for a moment I thought that this is part of the interface.

Crell’s picture

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

Thanks, dawehner!

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Kill kill kill.

Status: Fixed » Closed (fixed)

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