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

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

Line 56: Unused local variable $breakpoints

#1 drupal-core-remove-unused-local-variable-2080589.patch604 bytesmrsinguyen
PASSED: [[SimpleTest]]: [MySQL] 58,322 pass(es).
[ View ]


mrsinguyen’s picture

Status:Active» Needs review
new604 bytes
PASSED: [[SimpleTest]]: [MySQL] 58,322 pass(es).
[ View ]
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


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.