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

CommentFileSizeAuthor
#1 2067993-remove-unused-local-variable-index-and-disableNode-1.patch1.42 KBmichaelhiiva

Comments

michaelhiiva’s picture

michaelhiiva commented
Status: Active » Needs review
StatusFileSize
new 2067993-remove-unused-local-variable-index-and-disableNode-1.patch1.42 KB
linl’s picture

linl commented

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

linl commented
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

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

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