Part of meta-issue #2002650: [meta, no patch] improve maintainability by removing unused local variables

File /core/modules/node/lib/Drupal/node/Tests/PageViewTest.php

Line 31: Unused local variable $html

CommentFileSizeAuthor
#1 drupal-2064277-1.patch629 bytessergeypavlenko
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sergeypavlenko’s picture

sergeypavlenko CreditAttribution: sergeypavlenko commented
Assigned: sergeypavlenko » Unassigned
Status: Active » Needs review
FileSize
drupal-2064277-1.patch629 bytes
alphawebgroup’s picture

alphawebgroup
he/him
Primary language English
Location Kiev / Toronto
CreditAttribution: alphawebgroup commented
Status: Needs review » Reviewed & tested by the community

just applied and reviewed that patch.
don't see any reason to have unused $html variable in this test.
so, rtbc

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed a45c88f and pushed to 8.x. Thanks!

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