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

File /core/modules/system/lib/Drupal/system/Tests/Plugin/CacheDecoratorLanguageTest.php

Line 68: Unused local variable $plugin_id
Line 81: Unused local variable $plugin_id
Line 89: Unused local variable $plugin_id
Line 111: Unused local variable $plugin_id
Line 116: Unused local variable $plugin_id

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chertzog’s picture

Status: Active » Closed (duplicate)
angel.h’s picture

Status: Closed (duplicate) » Needs work
angel.h’s picture

Status: Needs work » Active

This issue shouldn't have been closed - see https://drupal.org/node/2080311#comment-7909573.

rhm5000’s picture

Status: Active » Needs review
FileSize
2.62 KB
parthipanramesh’s picture

Issue summary: View changes
Status: Needs review » Needs work

sorry, but I can't apply this patch..

deneo’s picture

Status: Needs work » Needs review
FileSize
2.66 KB

patch

Status: Needs review » Needs work

The last submitted patch, 6: 2080113-remove-unused-variable-5.patch, failed testing.

deneo’s picture

Status: Needs work » Needs review
FileSize
2.64 KB
areke’s picture

Status: Needs review » Reviewed & tested by the community

The patch applies cleanly and takes care of the issue. Thank you!

catch’s picture

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.