- Issue created by @KenKodz
- Assigned to KenKodz
- Status changed to Needs review
12 months ago 2:06pm 4 December 2023 - πΊπΈUnited States KenKodz
My merge request may not be needed. It may just need that line removed. It appears that the next line in the code is effectively the same thing, but I haven't tested it.
- π¬π§United Kingdom pstewart
Hah, I should've checked the issue list before releasing rc1 shouldn't I lol. As this issue just relates to the test suite this doesn't affect anything there though, so rc1 is still fine. You're right that
\Drupal::moduleHandler()
and\Drupal::service('module_handler')
are doing exactly the same thing - they both get themodule_handler
from the container, so we only need one of them.