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

File /core/tests/Drupal/Tests/Component/PhpStorage/MTimeProtectedFileStorageTest.php

Line 87: Unused local variable $storageFactory

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrsinguyen’s picture

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

Status: Needs review » Reviewed & tested by the community

this looks good: removes the var. phpstorm agrees it was unused and is ok without it. tests come back green. no extra changes.

still applies.

but will conflict with #1908440: Relax MTimeProtectedFileStorage permissions for DX, drush integration and world domination
[edit: wont conflict anymore, other issue is no longer removing it]

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed dd5b41b and pushed to 8.x. Thanks!

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