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

File /core/modules/picture/lib/Drupal/picture/Tests/PictureAdminUITest.php

Line 56: Unused local variable $breakpoints

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrsinguyen’s picture

Status: Active » Needs review
FileSize
604 bytes
YesCT’s picture

@mrsinguyen I added hints to the meta for reviewers. Would you like to review a few?

lokapujya’s picture

Status: Needs review » Reviewed & tested by the community

Applies, and is correct.

YesCT’s picture

@lokapujya

how did you check? Did you apply it, and then use an IDE and ask it to find unused vars in the file?

lokapujya’s picture

Good question. I observed that breakpoints is an unused locally declared array and also ran an inspector in my IDE.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

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