Validar formData en AjaxForms antes de aplicarlo

json_decode devuelve null si el POST llega truncado (post_max_size,
max_input_vars) y provocaba un TypeError en los apply(). Ahora las
acciones de guardado rechazan con invalid-request y las de solo
lectura usan un array vacío.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: NeoRazorX

Core/Lib/AjaxForms/PurchasesController.php

@@ -294,7 +294,7 @@ protected function findProductAction(): bool
     {
         $this->setTemplate(false);
         $model = $this->getModel();
-        $formData = json_decode($this->request->input('data'), true);
+        $formData = json_decode($this->request->input('data'), true) ?? [];
         PurchasesHeaderHTML::apply($model, $formData);
         PurchasesFooterHTML::apply($model, $formData);
         PurchasesModalHTML::apply($model, $formData);
@@ -364,7 +364,7 @@ protected function recalculateAction(bool $renderLines): bool
         $this->setTemplate(false);
         $model = $this->getModel();
         $lines = $model->getLines();
-        $formData = json_decode($this->request->input('data'), true);
+        $formData = json_decode($this->request->input('data'), true) ?? [];
         PurchasesHeaderHTML::apply($model, $formData);
         PurchasesFooterHTML::apply($model, $formData);
         PurchasesLineHTML::apply($model, $lines, $formData);
@@ -393,7 +393,15 @@ protected function saveDocAction(bool $sendOk = true): bool
         }
 
         $model = $this->getModel();
+
+        // si los datos del formulario no llegan o no son JSON válido (petición truncada,
+        // límites post_max_size / max_input_vars), rechazamos en lugar de guardar en blanco
         $formData = json_decode($this->request->input('data'), true);
+        if (false === is_array($formData)) {
+            Tools::log()->warning('invalid-request');
+            $this->sendJsonWithLogs(['ok' => false]);
+            return false;
+        }
 
         // bloqueo optimista: si el estado del documento ha cambiado desde que se cargó el formulario,
         // rechazamos para no borrar líneas a partir de un formulario obsoleto (tarea 4673)

Core/Lib/AjaxForms/SalesController.php

@@ -310,7 +310,7 @@ protected function findProductAction(): bool
     {
         $this->setTemplate(false);
         $model = $this->getModel();
-        $formData = json_decode($this->request->input('data'), true);
+        $formData = json_decode($this->request->input('data'), true) ?? [];
         SalesHeaderHTML::apply($model, $formData);
         SalesFooterHTML::apply($model, $formData);
         SalesModalHTML::apply($model, $formData);
@@ -380,7 +380,7 @@ protected function recalculateAction(bool $renderLines): bool
         $this->setTemplate(false);
         $model = $this->getModel();
         $lines = $model->getLines();
-        $formData = json_decode($this->request->input('data'), true);
+        $formData = json_decode($this->request->input('data'), true) ?? [];
         SalesHeaderHTML::apply($model, $formData);
         SalesFooterHTML::apply($model, $formData);
         SalesLineHTML::apply($model, $lines, $formData);
@@ -409,7 +409,15 @@ protected function saveDocAction(bool $sendOk = true): bool
         }
 
         $model = $this->getModel();
+
+        // si los datos del formulario no llegan o no son JSON válido (petición truncada,
+        // límites post_max_size / max_input_vars), rechazamos en lugar de guardar en blanco
         $formData = json_decode($this->request->input('data'), true);
+        if (false === is_array($formData)) {
+            Tools::log()->warning('invalid-request');
+            $this->sendJsonWithLogs(['ok' => false]);
+            return false;
+        }
 
         // bloqueo optimista: si el estado del documento ha cambiado desde que se cargó el formulario,
         // rechazamos para no borrar líneas a partir de un formulario obsoleto (tarea 4673)
@@ -486,6 +494,11 @@ protected function savePaidAction(): bool
         // cargamos el modelo actualizado y los datos del form
         $model = $this->getModel();
         $formData = json_decode($this->request->input('data'), true);
+        if (false === is_array($formData)) {
+            Tools::log()->warning('invalid-request');
+            $this->sendJsonWithLogs(['ok' => false]);
+            return false;
+        }
 
         // si la factura es de 0 €, la marcamos como pagada
         if (empty($model->total) && $model->hasColumn('pagada')) {