Improve error messaging for validation exceptions

Author: DaneEveritt

app/Exceptions/Model/DataValidationException.php

@@ -2,6 +2,7 @@
 
 namespace Pterodactyl\Exceptions\Model;
 
+use Illuminate\Database\Eloquent\Model;
 use Illuminate\Contracts\Validation\Validator;
 use Pterodactyl\Exceptions\PterodactylException;
 use Illuminate\Contracts\Support\MessageProvider;
@@ -11,21 +12,30 @@ class DataValidationException extends PterodactylException implements HttpExcept
 {
     /**
      * The validator instance.
-     *
-     * @var \Illuminate\Contracts\Validation\Validator
      */
-    public $validator;
+    protected Validator $validator;
+
+    /**
+     * The underlying model instance that triggered this exception.
+     */
+    protected Model $model;
 
     /**
      * DataValidationException constructor.
      */
-    public function __construct(Validator $validator)
+    public function __construct(Validator $validator, Model $model)
     {
-        parent::__construct(
-            'Data integrity exception encountered while performing database write operation. ' . $validator->errors()->toJson()
+        $message = sprintf(
+            'Could not save %s[%s]: failed to validate data: %s',
+            get_class($model),
+            $model->getKey(),
+            $validator->errors()->toJson()
         );
 
+        parent::__construct($message);
+
         $this->validator = $validator;
+        $this->model = $model;
     }
 
     /**
@@ -55,4 +65,14 @@ public function getHeaders()
     {
         return [];
     }
+
+    public function getValidator(): Validator
+    {
+        return $this->validator;
+    }
+
+    public function getModel(): Model
+    {
+        return $this->model;
+    }
 }

app/Http/Controllers/Admin/ServersController.php

@@ -271,7 +271,7 @@ public function updateBuild(Request $request, Server $server)
                 'database_limit', 'allocation_limit', 'backup_limit', 'oom_disabled',
             ]));
         } catch (DataValidationException $exception) {
-            throw new ValidationException($exception->validator);
+            throw new ValidationException($exception->getValidator());
         }
 
         $this->alert->success(trans('admin/server.alerts.build_updated'))->flash();
@@ -315,7 +315,7 @@ public function saveStartup(Request $request, Server $server)
                 ->setUserLevel(User::USER_LEVEL_ADMIN)
                 ->handle($server, $data);
         } catch (DataValidationException $exception) {
-            throw new ValidationException($exception->validator);
+            throw new ValidationException($exception->getValidator());
         }
 
         $this->alert->success(trans('admin/server.alerts.startup_changed'))->flash();

app/Models/Model.php

@@ -57,7 +57,7 @@ protected static function boot()
             try {
                 $model->validate();
             } catch (ValidationException $exception) {
-                throw new DataValidationException($exception->validator);
+                throw new DataValidationException($exception->validator, $model);
             }
 
             return true;

app/Repositories/Eloquent/EloquentRepository.php

@@ -98,7 +98,7 @@ public function create(array $fields, bool $validate = true, bool $force = false
             $saved = $instance->skipValidation()->save();
         } else {
             if (!$saved = $instance->save()) {
-                throw new DataValidationException($instance->getValidator());
+                throw new DataValidationException($instance->getValidator(), $instance);
             }
         }
 
@@ -195,7 +195,7 @@ public function update($id, array $fields, bool $validate = true, bool $force =
             $saved = $instance->skipValidation()->save();
         } else {
             if (!$saved = $instance->save()) {
-                throw new DataValidationException($instance->getValidator());
+                throw new DataValidationException($instance->getValidator(), $instance);
             }
         }