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

File /core/modules/update/update.report.inc

Line 31: Unused local variable $notification_level
Line 156: Unused local variable $key

CommentFileSizeAuthor
#2 2080685-remove-unused-variables-2.patch1.08 KBberdyshev
#1 2080685-remove-unused-variables.patch1.08 KBchertzog

Comments

chertzog’s picture

chertzog
he/him
Primary language English
Location Harrisburg, PA, USA
commented
Status: Active » Needs review
StatusFileSize
new 2080685-remove-unused-variables.patch1.08 KB
berdyshev’s picture

berdyshev commented
StatusFileSize
new 2080685-remove-unused-variables-2.patch1.08 KB

re-rolled this patch against current HEAD

samhassell’s picture

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

Simple patch removing the unused variables. Applies against current HEAD.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

2: 2080685-remove-unused-variables-2.patch queued for re-testing.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

2: 2080685-remove-unused-variables-2.patch queued for re-testing.

catch’s picture

catch
he/him
Primary language English
commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

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