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

File /core/modules/user/lib/Drupal/user/Form/UserPermissionsForm.php

Line 117: Unused local variable $rid

CommentFileSizeAuthor
#1 drupal-core-remove-unused-local-variable-2081205.patch654 bytesmrsinguyen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrsinguyen’s picture

mrsinguyen CreditAttribution: mrsinguyen commented
Status: Active » Needs review
FileSize
drupal-core-remove-unused-local-variable-2081205.patch654 bytes
Nitesh Sethia’s picture

Nitesh Sethia CreditAttribution: Nitesh Sethia commented
Assigned: Unassigned » Nitesh Sethia

#1 patch had been working fine,

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave commented
Status: Needs review » Reviewed & tested by the community
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

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