Part of meta-issue #2002650: [meta] 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

#1 2067993-remove-unused-local-variable-index-and-disableNode-1.patch1.42 KBrhm5000
PASSED: [[SimpleTest]]: [MySQL] 58,290 pass(es).
[ View ]


rhm5000’s picture

Status:Active» Needs review
new1.42 KB
PASSED: [[SimpleTest]]: [MySQL] 58,290 pass(es).
[ View ]
LinL’s picture

This 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.

LinL’s picture

Status:Needs review» Reviewed & tested by the community

Patch still applies, the deleted variables are not used. RTBC.

And thanks to mcrittenden on the other issue.

webchick’s picture

Status:Reviewed & tested by the community» Fixed

Committed and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.