Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Part of meta-issue #2002650: [meta, no patch] improve maintainability by removing unused local variables
File /core/modules/locale/lib/Drupal/locale/Tests/LocaleContentTest.php
Line 66: Unused local variable $base_url
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 2080527-3-remove-unused-variables.patch | 9.31 KB | chertzog |
| #2 | 2080527-remove-unused-variables.patch | 9.27 KB | chertzog |
Comments
Comment #1
chertzogClosing the following as duplicates:
#2080513: Remove Unused local variable $config from /core/modules/locale/lib/Drupal/locale/PoDatabaseWriter.php
#2080515: Remove Unused local variable $base_url from /core/modules/locale/lib/Drupal/locale/Tests/LocaleTranslationUiTest.php
#2080517: Remove Unused local variable $config from /core/modules/locale/lib/Drupal/locale/Tests/LocaleImportFunctionalTest.php
#2080519: Remove Unused local variable $config from /core/modules/locale/lib/Drupal/locale/Tests/LocaleUpdateTest.php
#2080521: Remove Unused local variable $translations from /core/modules/locale/lib/Drupal/locale/Tests/LocaleStringTest.php
#2080525: Remove Unused local variable $base_url from /core/modules/locale/lib/Drupal/locale/Tests/LocalePathTest.php
Comment #2
chertzogComment #4
chertzogComment #5
areke commentedThe patch applies cleanly and looks good. Thank you.
Comment #6
catchCommitted/pushed to 8.x, thanks!