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

File /core/modules/system/lib/Drupal/system/Tests/Cache/GenericCacheBackendUnitTestBase.php

Line 183: Unused local variable $cached

CommentFileSizeAuthor
#4 core-remove-unused-local-variables-genericcachebackendunittestbase-2080115-3.patch793 bytesaaronott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chertzog’s picture

chertzog
he/him
Primary language English
Location Harrisburg, PA, USA
CreditAttribution: chertzog commented
Status: Active » Closed (duplicate)

Closing as duplicate of #2080311: Clean up Drupal\system\Tests\Database\SelectTest.php

angel.h’s picture

angel.h
Primary language English
Location Plovdiv
CreditAttribution: angel.h commented
Status: Closed (duplicate) » Needs work
angel.h’s picture

angel.h
Primary language English
Location Plovdiv
CreditAttribution: angel.h commented
Status: Needs work » Active

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

aaronott’s picture

aaronott CreditAttribution: aaronott commented
FileSize
core-remove-unused-local-variables-genericcachebackendunittestbase-2080115-3.patch793 bytes
aaronott’s picture

aaronott CreditAttribution: aaronott commented
Status: Active » Needs review
johnmcc’s picture

johnmcc CreditAttribution: johnmcc commented

Looks good, applies cleanly.

parthipanramesh’s picture

parthipanramesh CreditAttribution: parthipanramesh commented
Issue summary: View changes

Works fine! Thank you very much!

parthipanramesh’s picture

parthipanramesh CreditAttribution: parthipanramesh commented
Status: Needs review » Reviewed & tested by the community
webchick’s picture

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

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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