fix naming of variable in discardedAttributeViolationReporter

getsentry · Jan 7, 2024 · 492f114 · 492f114
1 parent 38718d2
commit 492f114
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/Sentry/Laravel/Integration.php b/src/Sentry/Laravel/Integration.php
@@ -292,11 +292,11 @@ public function __construct(?callable $callback)
 
             public function __invoke(Model $model, array $attributes): void
             {
-                $attributes = implode(', ', $attributes);
-                SentrySdk::getCurrentHub()->withScope(function (Scope $scope) use ($model, $attributes) {
+                $attributes_imploded = implode(', ', $attributes);
+                SentrySdk::getCurrentHub()->withScope(function (Scope $scope) use ($model, $attributes_imploded) {
                     $scope->setContext('violation', [
                         'model'     => get_class($model),
-                        'attributes' => $attributes,
+                        'attributes' => $attributes_imploded,
                         'origin'    => $this->resolveEventOrigin(),
                         'kind'      => 'discarded_attributes',
                     ]);
@@ -308,7 +308,7 @@ public function __invoke(Model $model, array $attributes): void
                         EventHint::fromArray([
                             'exception' => new MassAssignmentException(sprintf(
                                 'Add fillable property [%s] to allow mass assignment on [%s].',
-                                $attributes,
+                                $attributes_imploded,
                                 get_class($model)
                             )),
                             'mechanism' => new ExceptionMechanism(ExceptionMechanism::TYPE_GENERIC, true),
@@ -318,7 +318,7 @@ public function __invoke(Model $model, array $attributes): void
 
                 // Forward the violation to the next handler if there is one
                 if ($this->callback !== null) {
-                    call_user_func($this->callback, $model, $attribute);
+                    call_user_func($this->callback, $model, $attributes);
                 }
             }
         };