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

File /core/modules/aggregator/lib/Drupal/aggregator/Tests/AggregatorTestBase.php

Line 170: Unused local variable $items

CommentFileSizeAuthor
#1 drupal-core-remove-unused-local-variable-2080699.patch723 bytesmrsinguyen

Comments

mrsinguyen’s picture

mrsinguyen commented
Status: Active » Needs review
StatusFileSize
new drupal-core-remove-unused-local-variable-2080699.patch723 bytes
barrett’s picture

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

The patch applies cleanly and applicable tests continue to pass. I know process is usually two reviews before RTBC, but this is a minor change so that seems like overkill.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

1: drupal-core-remove-unused-local-variable-2080699.patch queued for re-testing.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

1: drupal-core-remove-unused-local-variable-2080699.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.