Issue #3326371 by Spokje: Fix PHPStan L1 error "Anonymous function has an unused use $foo."

merge-requests/2928/merge
Alex Pott 2022-12-30 16:39:39 +00:00
parent 23edd01fb4
commit 1a63aa6318
No known key found for this signature in database
GPG Key ID: BDA67E7EE836E5CE
8 changed files with 8 additions and 54 deletions

View File

@ -208,10 +208,9 @@ class BasicAuthTest extends BrowserTestBase {
* @see \Drupal\basic_auth\Authentication\Provider\BasicAuth::challengeException() * @see \Drupal\basic_auth\Authentication\Provider\BasicAuth::challengeException()
*/ */
public function testCacheabilityOf401Response() { public function testCacheabilityOf401Response() {
$session = $this->getSession();
$url = Url::fromRoute('router_test.11'); $url = Url::fromRoute('router_test.11');
$assert_response_cacheability = function ($expected_page_cache_header_value, $expected_dynamic_page_cache_header_value) use ($session, $url) { $assert_response_cacheability = function ($expected_page_cache_header_value, $expected_dynamic_page_cache_header_value) use ($url) {
$this->drupalGet($url); $this->drupalGet($url);
$this->assertSession()->statusCodeEquals(401); $this->assertSession()->statusCodeEquals(401);
$this->assertSession()->responseHeaderEquals('X-Drupal-Cache', $expected_page_cache_header_value); $this->assertSession()->responseHeaderEquals('X-Drupal-Cache', $expected_page_cache_header_value);

View File

@ -103,7 +103,7 @@ class ConfigImportAllTest extends ModuleTestBase {
$validation_modules = array_keys($validation_reasons); $validation_modules = array_keys($validation_reasons);
$this->assertEqualsCanonicalizing($expected_modules, $validation_modules); $this->assertEqualsCanonicalizing($expected_modules, $validation_modules);
$modules_to_uninstall = array_filter($all_modules, function ($module) use ($validation_reasons) { $modules_to_uninstall = array_filter($all_modules, function ($module) {
// Filter required and not enabled modules. // Filter required and not enabled modules.
if (!empty($module->info['required']) || $module->status == FALSE) { if (!empty($module->info['required']) || $module->status == FALSE) {
return FALSE; return FALSE;

View File

@ -120,7 +120,7 @@ class EditModeTest extends WebDriverTestBase {
$web_assert = $this->assertSession(); $web_assert = $this->assertSession();
$page = $this->getSession()->getPage(); $page = $this->getSession()->getPage();
// Wait till all the contextual links are hidden. // Wait till all the contextual links are hidden.
$page->waitFor(1, function () use ($page, $web_assert) { $page->waitFor(1, function () use ($page) {
return empty($page->find('css', '.contextual .trigger.visually-hidden')); return empty($page->find('css', '.contextual .trigger.visually-hidden'));
}); });
$web_assert->elementContains('css', static::ANNOUNCE_SELECTOR, 'Tabbing is no longer constrained by the Contextual module.'); $web_assert->elementContains('css', static::ANNOUNCE_SELECTOR, 'Tabbing is no longer constrained by the Contextual module.');

View File

@ -580,7 +580,7 @@ abstract class EntityResourceTestBase extends ResourceTestBase {
? $link_relation_type->getRegisteredName() ? $link_relation_type->getRegisteredName()
: $link_relation_type->getExtensionUri(); : $link_relation_type->getExtensionUri();
}, array_keys($this->entity->getEntityType()->getLinkTemplates())); }, array_keys($this->entity->getEntityType()->getLinkTemplates()));
$parse_rel_from_link_header = function ($value) use ($link_relation_type_manager) { $parse_rel_from_link_header = function ($value) {
$matches = []; $matches = [];
if (preg_match('/rel="([^"]+)"/', $value, $matches) === 1) { if (preg_match('/rel="([^"]+)"/', $value, $matches) === 1) {
return $matches[1]; return $matches[1];

View File

@ -137,7 +137,7 @@ class UrlTest extends KernelTestBase {
$l = Link::fromTextAndUrl('foo', Url::fromUri('https://www.drupal.org'))->toString(); $l = Link::fromTextAndUrl('foo', Url::fromUri('https://www.drupal.org'))->toString();
// Test a renderable array passed to the link generator. // Test a renderable array passed to the link generator.
$renderer->executeInRenderContext(new RenderContext(), function () use ($renderer, $l) { $renderer->executeInRenderContext(new RenderContext(), function () use ($l) {
$renderable_text = ['#markup' => 'foo']; $renderable_text = ['#markup' => 'foo'];
$l_renderable_text = \Drupal::service('link_generator')->generate($renderable_text, Url::fromUri('https://www.drupal.org')); $l_renderable_text = \Drupal::service('link_generator')->generate($renderable_text, Url::fromUri('https://www.drupal.org'));
$this->assertEquals($l, $l_renderable_text); $this->assertEquals($l, $l_renderable_text);

View File

@ -785,11 +785,6 @@ parameters:
count: 1 count: 1
path: modules/ban/src/Plugin/migrate/destination/BlockedIp.php path: modules/ban/src/Plugin/migrate/destination/BlockedIp.php
-
message: "#^Anonymous function has an unused use \\$session\\.$#"
count: 1
path: modules/basic_auth/tests/src/Functional/BasicAuthTest.php
- -
message: "#^\\#lazy_builder callback 'hello_or_yarhar' at key '0' is not callable\\.$#" message: "#^\\#lazy_builder callback 'hello_or_yarhar' at key '0' is not callable\\.$#"
count: 1 count: 1
@ -990,11 +985,6 @@ parameters:
count: 2 count: 2
path: modules/config/src/Form/ConfigSingleImportForm.php path: modules/config/src/Form/ConfigSingleImportForm.php
-
message: "#^Anonymous function has an unused use \\$validation_reasons\\.$#"
count: 1
path: modules/config/tests/src/Functional/ConfigImportAllTest.php
- -
message: "#^Call to deprecated constant REQUEST_TIME\\: Deprecated in drupal\\:8\\.3\\.0 and is removed from drupal\\:10\\.0\\.0\\. Use \\\\Drupal\\:\\:time\\(\\)\\-\\>getRequestTime\\(\\); $#" message: "#^Call to deprecated constant REQUEST_TIME\\: Deprecated in drupal\\:8\\.3\\.0 and is removed from drupal\\:10\\.0\\.0\\. Use \\\\Drupal\\:\\:time\\(\\)\\-\\>getRequestTime\\(\\); $#"
count: 1 count: 1
@ -1075,11 +1065,6 @@ parameters:
count: 1 count: 1
path: modules/content_translation/tests/src/Functional/ContentTranslationUITestBase.php path: modules/content_translation/tests/src/Functional/ContentTranslationUITestBase.php
-
message: "#^Anonymous function has an unused use \\$web_assert\\.$#"
count: 1
path: modules/contextual/tests/src/FunctionalJavascript/EditModeTest.php
- -
message: "#^Variable \\$unrestricted_tab_count might not be defined\\.$#" message: "#^Variable \\$unrestricted_tab_count might not be defined\\.$#"
count: 1 count: 1
@ -1930,11 +1915,6 @@ parameters:
count: 1 count: 1
path: modules/rest/src/Routing/ResourceRoutes.php path: modules/rest/src/Routing/ResourceRoutes.php
-
message: "#^Anonymous function has an unused use \\$link_relation_type_manager\\.$#"
count: 1
path: modules/rest/tests/src/Functional/EntityResource/EntityResourceTestBase.php
- -
message: "#^Variable \\$created_entity might not be defined\\.$#" message: "#^Variable \\$created_entity might not be defined\\.$#"
count: 4 count: 4
@ -2155,11 +2135,6 @@ parameters:
count: 2 count: 2
path: modules/system/tests/src/Functional/Theme/ThemeUiTest.php path: modules/system/tests/src/Functional/Theme/ThemeUiTest.php
-
message: "#^Anonymous function has an unused use \\$renderer\\.$#"
count: 1
path: modules/system/tests/src/Kernel/Common/UrlTest.php
- -
message: "#^Call to deprecated constant REQUEST_TIME\\: Deprecated in drupal\\:8\\.3\\.0 and is removed from drupal\\:10\\.0\\.0\\. Use \\\\Drupal\\:\\:time\\(\\)\\-\\>getRequestTime\\(\\); $#" message: "#^Call to deprecated constant REQUEST_TIME\\: Deprecated in drupal\\:8\\.3\\.0 and is removed from drupal\\:10\\.0\\.0\\. Use \\\\Drupal\\:\\:time\\(\\)\\-\\>getRequestTime\\(\\); $#"
count: 1 count: 1
@ -3150,11 +3125,6 @@ parameters:
count: 1 count: 1
path: tests/Drupal/Tests/Core/Plugin/TestPluginManager.php path: tests/Drupal/Tests/Core/Plugin/TestPluginManager.php
-
message: "#^Anonymous function has an unused use \\$self\\.$#"
count: 1
path: tests/Drupal/Tests/Core/Render/MetadataBubblingUrlGeneratorTest.php
- -
message: "#^\\#access_callback callback class '\\$this\\(Drupal\\\\Tests\\\\Core\\\\Render\\\\RendererCallbackTest\\)' at key '0' does not implement Drupal\\\\Core\\\\Security\\\\TrustedCallbackInterface\\.$#" message: "#^\\#access_callback callback class '\\$this\\(Drupal\\\\Tests\\\\Core\\\\Render\\\\RendererCallbackTest\\)' at key '0' does not implement Drupal\\\\Core\\\\Security\\\\TrustedCallbackInterface\\.$#"
count: 1 count: 1
@ -3175,21 +3145,6 @@ parameters:
count: 1 count: 1
path: tests/Drupal/Tests/Core/Render/RendererCallbackTest.php path: tests/Drupal/Tests/Core/Render/RendererCallbackTest.php
-
message: "#^Anonymous function has an unused use \\$complex_child_template\\.$#"
count: 1
path: tests/Drupal/Tests/Core/Render/RendererRecursionTest.php
-
message: "#^Anonymous function has an unused use \\$parent_markup\\.$#"
count: 1
path: tests/Drupal/Tests/Core/Render/RendererRecursionTest.php
-
message: "#^Anonymous function has an unused use \\$renderer\\.$#"
count: 1
path: tests/Drupal/Tests/Core/Render/RendererRecursionTest.php
- -
message: "#^Variable \\$value in isset\\(\\) always exists and is not nullable\\.$#" message: "#^Variable \\$value in isset\\(\\) always exists and is not nullable\\.$#"
count: 1 count: 1

View File

@ -55,7 +55,7 @@ class MetadataBubblingUrlGeneratorTest extends UrlGeneratorTest {
$this->renderer->expects($this->exactly($invocations)) $this->renderer->expects($this->exactly($invocations))
->method('render') ->method('render')
->willReturnCallback(function ($build) use ($self) { ->willReturnCallback(function ($build) {
$this->assertArrayHasKey('#cache', $build); $this->assertArrayHasKey('#cache', $build);
}); });

View File

@ -69,7 +69,7 @@ class RendererRecursionTest extends RendererTestBase {
$renderer = $this->renderer; $renderer = $this->renderer;
$this->setUpRequest(); $this->setUpRequest();
$callable = function ($markup) use ($renderer, $complex_child_template) { $callable = function ($markup) {
$this->assertStringStartsWith('<drupal-render-placeholder', $markup, 'Rendered complex child output as expected, without the placeholder replaced, i.e. with just the placeholder.'); $this->assertStringStartsWith('<drupal-render-placeholder', $markup, 'Rendered complex child output as expected, without the placeholder replaced, i.e. with just the placeholder.');
return $markup; return $markup;
}; };
@ -105,7 +105,7 @@ class RendererRecursionTest extends RendererTestBase {
$complex_child = $complex_child_template; $complex_child = $complex_child_template;
$callable = function ($elements) use ($renderer, $complex_child, $parent_markup) { $callable = function ($elements) use ($renderer, $complex_child) {
$elements['#markup'] = $renderer->renderPlain($complex_child); $elements['#markup'] = $renderer->renderPlain($complex_child);
$this->assertEquals('<p>This is a rendered placeholder!</p>', $elements['#markup'], 'Rendered complex child output as expected, with the placeholder replaced.'); $this->assertEquals('<p>This is a rendered placeholder!</p>', $elements['#markup'], 'Rendered complex child output as expected, with the placeholder replaced.');
return $elements; return $elements;