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

File /core/lib/Drupal/Core/Database/Query/Update.php

Line 225: Unused local variable $field

CommentFileSizeAuthor
#1 core-remove-unused-local-variables-update.php-2060823-1.patch617 byteslegolasbo
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
Status: Active » Needs review
FileSize
core-remove-unused-local-variables-update.php-2060823-1.patch617 bytes

Attached patch removes the unused variable

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Checked the variable ...

webchick’s picture

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

Committed and pushed to 8.x. Thanks!

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
CreditAttribution: legolasbo commented
Assigned: legolasbo » Unassigned

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