Contrib.social

Path alias fallbacks follow-up: tests & profiling

Open on Drupal.org →
Created on 3 December 2019, over 5 years ago
Updated 17 May 2024, 11 months ago

This is a follow up for #2830291: D8 URL alias language fallbacks → . At the moment, the work done there relies on the core patch from 🐛 path.alias_repository service does not use a proper language fallback mechanism Needs work , so tests could not be set up with testbot. We should probably still have tests that ensure the query alter that changes the result ordering is correct, so this issue is for that.

As well as tests, we should also add profiling to investigate the performance effect added by this change to the query. I'm not sure whether the change in core, or the LH one, will have the greater effect on performance.

📌 Task
Status

Active

Version

2.0

Component

Code

Created by

🇬🇧United Kingdom james.williams

Live updates comments and jobs are added and updated live.
  • 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.

  • needs profiling

    It may affect performance, and thus requires in-depth technical reviews and profiling.

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.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024