Merge pull request #156 from Askedio/revert-155-fix/replace-try-catch-with-method_exists

Revert "fix: replace `try...catch(BadMethodCallException)` blocks with a `method_exists()` condition"

Author: gcphost

src/Listeners/CascadeQueryListener.php

@@ -4,6 +4,7 @@
 
 use Askedio\SoftCascade\QueryBuilderSoftCascade;
 use Askedio\SoftCascade\Traits\ChecksCascading;
+use BadMethodCallException;
 use Illuminate\Database\Connection;
 use Illuminate\Database\Eloquent\SoftDeletingScope;
 use Illuminate\Database\Events\QueryExecuted;
@@ -76,10 +77,11 @@ public function handle(): void
         if (!is_null($event)) {
             $builder = $event['builder'];
 
-            // add `withTrashed()`, if the model has SoftDeletes
-            // otherwise, we can just skip it
-            if (method_exists($builder, 'withTrashed')) {
+            try {
                 $builder->withTrashed();
+            } catch (BadMethodCallException $e) {
+                // add `withTrashed()`, if the model has SoftDeletes
+                // otherwise, we can just skip it
             }
 
             $keyName = $builder->getModel()->getKeyName();

src/SoftCascade.php

@@ -7,6 +7,7 @@
 use Askedio\SoftCascade\Exceptions\SoftCascadeNonExistentRelationActionException;
 use Askedio\SoftCascade\Exceptions\SoftCascadeRestrictedException;
 use Askedio\SoftCascade\Traits\ChecksCascading;
+use BadMethodCallException;
 use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Relations\BelongsToMany;
 use Illuminate\Database\Eloquent\Relations\MorphOneOrMany;
@@ -285,10 +286,10 @@ protected function withTrashed(Builder $builder): Builder
         }
 
         // if the Model does not use SoftDeletes, withTrashed() will be unavailable.
-        if (method_exists($builder, 'withTrashed')) {
+        try {
             return $builder->withTrashed();
+        } catch (BadMethodCallException) {
+            return $builder;
         }
-
-        return $builder;
     }
 }