- Issue created by @joachim
- First commit to issue fork.
- Merge request !12839Issue #3537396: Update comments referencing block_themes_installed() → (Open) created by Unnamed author
- 🇦🇺Australia jesseh
Merge request created to update the remaining comments to the new method added in https://www.drupal.org/project/drupal/issues/3533049 📌 Bulk convert the remaining hooks to OOP Active .
- 🇧🇪Belgium borisson_ Mechelen, 🇧🇪
I had a coding standards nitpick, otherwise this looks great.
- 🇦🇺Australia jesseh
I updated the comment to resolve the coding standards issue. I didn't take the approach above as I'm not familiar enough with PHPUnit and whether the "@covers" annotation would impact code coverage reports.
-
quietone →
committed 99c2af65 on 11.2.x
Issue #3537396 by jesseh, borisson_: Removed function...
-
quietone →
committed 99c2af65 on 11.2.x
-
quietone →
committed b74f08a1 on 11.x
Issue #3537396 by jesseh, borisson_: Removed function...
-
quietone →
committed b74f08a1 on 11.x
- 🇳🇿New Zealand quietone
I used 'git log -Sblock_themes_installed' to find where the function was changed and to confirm the new method name in the MR is correct. That agrees with what jesseh reported in #5. Also checked that there are no other instances of "block_themes_installed".
The comment is an improvement and it is wrapped correctly.