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

File /core/modules/system/lib/Drupal/system/Tests/Database/AlterTest.php

Line 37: Unused local variable $record

CommentFileSizeAuthor
#3 core-remove-unused-local-variables-altertest-2080309-3.patch827 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) » Active

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

aaronott’s picture

aaronott CreditAttribution: aaronott commented
Status: Active » Needs review
FileSize
core-remove-unused-local-variables-altertest-2080309-3.patch827 bytes
parthipanramesh’s picture

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

Applied without any issues. Thanks!

webchick’s picture

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

Well that's definitely much nicer!

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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