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

File /core/modules/user/user.module

Line 52: Unused local variable $user
Line 525: Unused local variable $find

CommentFileSizeAuthor
#1 core-remove-unused-local-variables-user.module-2057137-1.patch666 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-user.module-2057137-1.patch666 bytes

Removed both variables.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Reviewed & tested by the community

Awesome, thanks!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed 251c8d8 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.