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

File /core/modules/file/lib/Drupal/file/Plugin/field/widget/FileWidget.php

Line 94: Unused local variable $wrapper_id

CommentFileSizeAuthor
#1 core-remove-unused-vars-filewidget.php-2072553-1.patch757 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
Assigned: legolasbo » Unassigned
Status: Active » Needs review
FileSize
core-remove-unused-vars-filewidget.php-2072553-1.patch757 bytes

Removed unused local variable $wrapper_id. Removing it caused $id_prefix to be unused. Also removed $id_prefix.

phiit’s picture

phiit CreditAttribution: phiit commented
Status: Needs review » Reviewed & tested by the community

Reviewed, looks good. Thanks!

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!

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