Skip to content

Commit f0c0ecb

Browse files
committed
fix: invio email con allegato per utenti senza permessi per il modulo Coda di invio, e duplicazione generazione allegato
1 parent e0870d3 commit f0c0ecb

4 files changed

Lines changed: 8 additions & 8 deletions

File tree

actions.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -352,7 +352,7 @@
352352
elseif (post('op') == 'send-email') {
353353
$template = Template::find(post('template'));
354354

355-
$mail = Modules\Emails\Mail::build($user, $template, $id_record);
355+
$mail = Modules\Emails\Mail::build(user: $user, template: $template, id_record: $id_record, reset_from_template: false);
356356

357357
// Rimozione allegati predefiniti
358358
$mail->resetPrints();

modules/emails/src/Mail.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ class Mail extends Model
3737

3838
protected $options;
3939

40-
public static function build(?User $user = null, $template = null, $id_record = null, $account = null)
40+
public static function build(?User $user = null, $template = null, $id_record = null, $account = null, $reset_from_template = true)
4141
{
4242
$model = new static();
4343

@@ -52,7 +52,7 @@ public static function build(?User $user = null, $template = null, $id_record =
5252

5353
$model->save();
5454

55-
if (!empty($template)) {
55+
if (!empty($template) && $reset_from_template) {
5656
$model->resetFromTemplate();
5757
}
5858

@@ -135,7 +135,7 @@ public function addPrint($print_id, $name = null)
135135
'id_category' => null,
136136
'id_module' => $id_module,
137137
'id_record' => $this->id,
138-
]);
138+
], true);
139139

140140
// Verifica che l'upload sia stato eseguito correttamente
141141
if (!$upload) {

src/Models/Upload.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,10 +82,10 @@ public function setCategoryAttribute($value)
8282
*
8383
* @return self
8484
*/
85-
public static function build($source = null, $data = null, $name = null, $category = null)
85+
public static function build($source = null, $data = null, $name = null, $category = null, $force = false)
8686
{
8787
// Verifica permessi prima del caricamento
88-
if (!self::checkUploadPermissions($data)) {
88+
if (!self::checkUploadPermissions($data) && !$force) {
8989
return false;
9090
}
9191

src/Uploads.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,15 +80,15 @@ public static function getDirectory($id_module, $id_plugin = null)
8080
*
8181
* @return Upload
8282
*/
83-
public static function upload($source, $data, $options = [])
83+
public static function upload($source, $data, $force = false)
8484
{
8585
// Se è attivo il task sul controllo dello spazio disponibile allora aggiorno la cache
8686
$task = Tasks\Task::where('enabled', 1)->where('name', 'Hook Spazio disponibile')->first();
8787
if (!empty($task)) {
8888
$task->execute();
8989
}
9090

91-
return Upload::build($source, $data);
91+
return Upload::build(source: $source, data: $data, force: $force);
9292
}
9393

9494
/**

0 commit comments

Comments
 (0)