Merge pull request #207 from TheDragonCode/patch/2025-04-02/13-25

Method `shouldBeAsync` was renamed with `needAsync`

Author: andrey-helldar

src/Operation.php

@@ -51,7 +51,7 @@ public function needBefore(): bool
     /**
      * Defines whether the operation will run synchronously or asynchronously.
      */
-    public function shouldBeAsync(): bool
+    public function needAsync(): bool
     {
         return app(ConfigData::class)->async;
     }

src/Services/MigratorService.php

@@ -60,7 +60,7 @@ public function runUp(string $filename, int $batch, OptionsData $options): void
             return;
         }
 
-        if ($this->hasAsync($operation, $options)) {
+        if ($this->needAsync($operation, $options)) {
             OperationJob::dispatch($name);
 
             $this->notification->twoColumn($name, StatusEnum::Pending->toColor());
@@ -69,7 +69,7 @@ public function runUp(string $filename, int $batch, OptionsData $options): void
         }
 
         $this->notification->task($name, function () use ($operation, $name, $batch) {
-            $this->hasOperation($operation, '__invoke')
+            $this->hasMethod($operation, '__invoke')
                 ? $this->runOperation($operation, '__invoke')
                 : $this->runOperation($operation, 'up');
 
@@ -93,7 +93,7 @@ public function runDown(string $filename, OptionsData $options): void
 
     protected function runOperation(Operation $operation, string $method): void
     {
-        if ($this->hasOperation($operation, $method)) {
+        if ($this->hasMethod($operation, $method)) {
             try {
                 $this->runMethod($operation, $method, $operation->withinTransactions());
 
@@ -107,14 +107,14 @@ protected function runOperation(Operation $operation, string $method): void
         }
     }
 
-    protected function hasOperation(Operation $operation, string $method): bool
+    protected function hasMethod(Operation $operation, string $method): bool
     {
         return method_exists($operation, $method);
     }
 
-    protected function hasAsync(Operation $operation, OptionsData $options): bool
+    protected function needAsync(Operation $operation, OptionsData $options): bool
     {
-        return ! $options->sync && $operation->shouldBeAsync();
+        return ! $options->sync && $operation->needAsync();
     }
 
     protected function runMethod(Operation $operation, string $method, bool $transactions): void

tests/fixtures/app/async/2021_04_06_212742_every_time.php

@@ -25,7 +25,7 @@ public function shouldOnce(): bool
         return false;
     }
 
-    public function shouldBeAsync(): bool
+    public function needAsync(): bool
     {
         return true;
     }

tests/fixtures/app/async/2023_04_06_212637_foo_bar.php

@@ -20,7 +20,7 @@ protected function table(): Builder
         return DB::table('test');
     }
 
-    public function shouldBeAsync(): bool
+    public function needAsync(): bool
     {
         return true;
     }