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

File /core/modules/views/lib/Drupal/views/Tests/ModuleTest.php

Line 208: Unused local variable $id
Line 209: Unused local variable $display_id
Line 269: Unused local variable $id
Line 270: Unused local variable $display_id

CommentFileSizeAuthor
#1 core-remove-unused-vars-moduletest.php-2072609-1.patch1.33 KBlegolasbo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
CreditAttribution: legolasbo commented
Assigned: legolasbo » Unassigned
Status: Active » Needs review
FileSize
core-remove-unused-vars-moduletest.php-2072609-1.patch1.33 KB

Removed unused variables

enhdless’s picture

enhdless CreditAttribution: enhdless commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Patch works great!

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

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