Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Part of meta-issue #2002650: [meta, no patch] improve maintainability by removing unused local variables
File /core/modules/node/lib/Drupal/node/Tests/NodeTranslationUITest.php
Line 120: Unused local variable $index
Line 189: Unused local variable $disabledNode
Comment | File | Size | Author |
---|---|---|---|
#1 | 2067993-remove-unused-local-variable-index-and-disableNode-1.patch | 1.42 KB | rhm5000 |
Comments
Comment #1
rhm5000 CreditAttribution: rhm5000 commentedComment #2
LinL CreditAttribution: LinL commentedThis a duplicate of #2080059: Remove Unused local variable $index from /core/modules/node/lib/Drupal/node/Tests/NodeTranslationUITest.php.
The other issue has a different solution. Not sure which is better, so not closing either as a duplicate just yet.
Comment #3
LinL CreditAttribution: LinL commentedPatch still applies, the deleted variables are not used. RTBC.
And thanks to mcrittenden on the other issue.
Comment #4
webchickCommitted and pushed to 8.x. Thanks!