fix: invio mail con allegato a utenti senza i permessi per il modulo Coda di invio

Author: Pek5892

modules/emails/ajax/complete.php

@@ -87,7 +87,7 @@
         $addResults($fetchEmails('an_sedi', 'email', 'nomesede'));
 
         // Tutti i referenti
-        $addResults($fetchEmails('an_referenti', 'email', 'nome'));
+        $addResults($fetchEmails('an_referenti', 'email', 'an_referenti.nome'));
 
         echo json_encode($results);
 

modules/emails/src/Mail.php

@@ -22,10 +22,12 @@
 
 use Common\SimpleModelTrait;
 use Illuminate\Database\Eloquent\Model;
+use Models\Module;
 use Models\PrintTemplate;
 use Models\Upload;
 use Models\User;
 use Modules\Newsletter\Newsletter;
+use Monolog\Logger;
 
 class Mail extends Model
 {
@@ -78,7 +80,7 @@ public function addUpload($file_id, $name = null)
 
         // duplica il file
         $upload = $file->copia([
-            'id_module' => \Models\Module::where('name', 'Stato email')->first()->id,
+            'id_module' => Module::where('name', 'Stato email')->first()->id,
             'id_record' => $this->id,
         ]);
 
@@ -109,8 +111,20 @@ public function addPrint($print_id, $name = null)
         // Genera la stampa come PDF
         $print = \Prints::render($print_id, $this->id_record, null, true);
 
+        // Verifica che la stampa sia stata generata correttamente
+        if (!$print || !isset($print['pdf'])) {
+            $logger = \logger_osm();
+            $logger->addRecord(Logger::ERROR, 'Impossibile generare il PDF per la stampa: '.$print_id, [
+                'id_email' => $this->id,
+                'id_record' => $this->id_record,
+                'print_id' => $print_id,
+            ]);
+
+            return;
+        }
+
         // Ottieni il modulo "Coda d'invio"
-        $id_module = \Models\Module::where('name', 'Stato email')->first()->id;
+        $id_module = Module::where('name', 'Stato email')->first()->id;
 
         $name = $name ?: $print['name'];
 
@@ -123,6 +137,19 @@ public function addPrint($print_id, $name = null)
             'id_record' => $this->id,
         ]);
 
+        // Verifica che l'upload sia stato eseguito correttamente
+        if (!$upload) {
+            $logger = \logger_osm();
+            $logger->addRecord(Logger::ERROR, 'Impossibile salvare l\'allegato PDF per la email', [
+                'id_email' => $this->id,
+                'id_record' => $this->id_record,
+                'print_id' => $print_id,
+                'filename' => $name,
+            ]);
+
+            return;
+        }
+
         // Collega il file alla tabella em_email_attachment
         $this->attachments()->attach($upload->id, ['id_email' => $this->id, 'name' => $name, 'type' => 'print']);
     }

src/Models/Upload.php

@@ -491,6 +491,12 @@ protected static function checkUploadPermissions($data)
             return true; // Se non riusciamo a determinare il modulo, permetti per compatibilità
         }
 
+        // Permetti sempre l'upload di un allegato a un email in invio, tramite il modulo Coda di invio
+        $email_module = Module::where('name', 'Stato email')->first();
+        if ($email_module && $id_module == $email_module->id) {
+            return true;
+        }
+
         // Verifica permessi in base al tipo di accesso
         if (\Permissions::isTokenAccess()) {
             // Per accesso tramite token, verifica i permessi del token