Remove static closure declarations

Author: stevebauman

src/Filesystem/MigrationStorage.php

@@ -58,7 +58,7 @@ public function findAll(): Collection
     {
         $files = $this->filesystem->glob($this->directory.'/*_*.php');
 
-        return collect($files)->sort()->map(static function (string $filePath) {
+        return collect($files)->sort()->map(function (string $filePath) {
             return new MigrationFile($filePath);
         });
     }

src/Migrator.php

@@ -62,7 +62,7 @@ public function migrateAll(): self
         $files = $this->migrationStorage->findAll();
         $migratedFileNames = $this->migrationRepository->getAll();
 
-        $nonMigratedFiles = $files->filter(static function (MigrationFile $file) use ($migratedFileNames) {
+        $nonMigratedFiles = $files->filter(function (MigrationFile $file) use ($migratedFileNames) {
             return ! $migratedFileNames->contains($file->getName());
         });
 
@@ -125,7 +125,7 @@ public function showStatus(): self
 
         $headers = ['Ran?', 'Last batch?', 'Migration'];
 
-        $rows = $files->map(static function (MigrationFile $file) use ($migratedFileNames, $migratedLastBatchFileNames) {
+        $rows = $files->map(function (MigrationFile $file) use ($migratedFileNames, $migratedLastBatchFileNames) {
             return [
                 $migratedFileNames->contains($file->getName()) ? '<info>Yes</info>' : '<comment>No</comment>',
                 $migratedLastBatchFileNames->contains($file->getName()) ? '<info>Yes</info>' : '<comment>No</comment>',
@@ -185,7 +185,7 @@ protected function rollback(Collection $fileNames): self
         } elseif ($fileNames->count() != $files->count()) {
             $this->output->writeln(
                 '<error>Migration is not found:</error> '.
-                implode(',', $fileNames->diff($files->map(static function (MigrationFile $file) {
+                implode(',', $fileNames->diff($files->map(function (MigrationFile $file) {
                     return $file->getName();
                 }))->toArray())
             );

tests/Integration/Adapters/IndexManagerAdapterTest.php

@@ -54,7 +54,7 @@ public function test_index_can_be_created_with_modifier(string $indexNamePrefix)
 
         $indexName = 'test';
 
-        $modifier = static function (Mapping $mapping, Settings $settings) {
+        $modifier = function (Mapping $mapping, Settings $settings) {
             $mapping->text('title');
             $settings->index(['number_of_replicas' => 2]);
         };
@@ -99,7 +99,7 @@ public function test_mapping_can_be_updated_using_modifier(string $indexNamePref
 
         $indexName = 'test';
 
-        $modifier = static function (Mapping $mapping) {
+        $modifier = function (Mapping $mapping) {
             $mapping->disableSource()->text('title');
         };
 
@@ -121,7 +121,7 @@ public function test_settings_can_be_updated_using_modifier(string $indexNamePre
 
         $indexName = 'test';
 
-        $modifier = static function (Settings $settings) {
+        $modifier = function (Settings $settings) {
             $settings->index(['number_of_replicas' => 2, 'refresh_interval' => -1]);
         };
 
@@ -143,7 +143,7 @@ public function test_settings_can_be_pushed_using_modifier(string $indexNamePref
 
         $indexName = 'test';
 
-        $modifier = static function (Settings $settings) {
+        $modifier = function (Settings $settings) {
             $settings->index(['number_of_replicas' => 2]);
         };
 

tests/Integration/Filesystem/MigrationStorageTest.php

@@ -104,7 +104,7 @@ public function test_all_files_within_migrations_directory_can_be_retrieved(): v
                 '2018_12_01_081000_create_test_index',
                 '2019_08_10_142230_update_test_index_mapping',
             ],
-            $files->map(static function (MigrationFile $file) {
+            $files->map(function (MigrationFile $file) {
                 return $file->getName();
             })->toArray()
         );

tests/Integration/RealOpenSearchMigrationCommandTest.php

@@ -58,7 +58,7 @@ public function test_index_adapter_operations_run_against_opensearch(): void
         $aliasName = $this->indexPrefix.'adapter_real_alias';
 
         try {
-            $index->create('adapter_real_test', static function (Mapping $mapping, Settings $settings): void {
+            $index->create('adapter_real_test', function (Mapping $mapping, Settings $settings): void {
                 $mapping
                     ->text('title')
                     ->keyword('status');
@@ -71,7 +71,7 @@ public function test_index_adapter_operations_run_against_opensearch(): void
 
             $this->assertTrue($this->client()->indices()->exists(['index' => $indexName]));
 
-            $index->putMapping('adapter_real_test', static function (Mapping $mapping): void {
+            $index->putMapping('adapter_real_test', function (Mapping $mapping): void {
                 $mapping->keyword('category');
             });
 

tests/migrations/2019_08_10_142230_update_test_index_mapping.php

@@ -8,15 +8,15 @@ class UpdateTestIndexMapping implements MigrationInterface
 {
     public function up(): void
     {
-        Index::putMapping('test', static function (Mapping $mapping) {
+        Index::putMapping('test', function (Mapping $mapping) {
             $mapping->enableSource();
             $mapping->text('title');
         });
     }
 
     public function down(): void
     {
-        Index::putMapping('test', static function (Mapping $mapping) {
+        Index::putMapping('test', function (Mapping $mapping) {
             $mapping->disableSource();
         });
     }

tests/real_migrations/2026_01_01_000000_create_real_test_index.php

@@ -11,7 +11,7 @@ class CreateRealTestIndex implements MigrationInterface
      */
     public function up(): void
     {
-        Index::create('real_test', static function (Mapping $mapping) {
+        Index::create('real_test', function (Mapping $mapping) {
             $mapping->text('title');
         });
     }