From a97248555eb71bd0cab381003df8877cd705064e Mon Sep 17 00:00:00 2001
From: Hendrik Hagendorn <hendrik@hagendorn.it>
Date: Sat, 27 May 2023 11:07:46 +0200
Subject: [PATCH] allow attaching uploaded files when sending mails

---
 app/Http/Requests/Common/CustomMail.php       |  7 +-
 .../Document/SendDocumentAsCustomMail.php     |  7 +-
 app/Notifications/Sale/Invoice.php            | 21 +++-
 resources/lang/en-GB/general.php              |  1 +
 .../views/modals/invoices/email.blade.php     | 98 ++++++++++++++++---
 5 files changed, 118 insertions(+), 16 deletions(-)

diff --git a/app/Http/Requests/Common/CustomMail.php b/app/Http/Requests/Common/CustomMail.php
index ce449794b..24d3133d6 100644
--- a/app/Http/Requests/Common/CustomMail.php
+++ b/app/Http/Requests/Common/CustomMail.php
@@ -14,9 +14,10 @@ class CustomMail extends FormRequest
     public function rules()
     {
         return [
-            'to'        => 'required|email',
-            'subject'   => 'required|string',
-            'body'      => 'required|string',
+            'to'            => 'required|email',
+            'subject'       => 'required|string',
+            'body'          => 'required|string',
+            'attachments.*' => 'nullable|boolean',
         ];
     }
 }
diff --git a/app/Jobs/Document/SendDocumentAsCustomMail.php b/app/Jobs/Document/SendDocumentAsCustomMail.php
index 986c69570..c5ca10fe7 100644
--- a/app/Jobs/Document/SendDocumentAsCustomMail.php
+++ b/app/Jobs/Document/SendDocumentAsCustomMail.php
@@ -30,10 +30,15 @@ class SendDocumentAsCustomMail extends Job
             $custom_mail['cc'] = user()->email;
         }
 
+        $attachments = collect($this->request->get('attachments', []))
+            ->filter(fn($value) => $value == true)
+            ->keys()
+            ->all();
+
         $notification = config('type.document.' . $document->type . '.notification.class');
 
         // Notify the contact
-        $document->contact->notify(new $notification($document, $this->template_alias, true, $custom_mail));
+        $document->contact->notify(new $notification($document, $this->template_alias, true, $custom_mail, $attachments));
 
         event(new DocumentSent($document));
     }
diff --git a/app/Notifications/Sale/Invoice.php b/app/Notifications/Sale/Invoice.php
index 4c0033915..8c5ebc97b 100644
--- a/app/Notifications/Sale/Invoice.php
+++ b/app/Notifications/Sale/Invoice.php
@@ -35,10 +35,17 @@ class Invoice extends Notification
      */
     public $attach_pdf;
 
+    /**
+     * List of document attachments to attach when sending the email.
+     *
+     * @var array
+     */
+    public $attachments;
+
     /**
      * Create a notification instance.
      */
-    public function __construct(Document $invoice = null, string $template_alias = null, bool $attach_pdf = false, array $custom_mail = [])
+    public function __construct(Document $invoice = null, string $template_alias = null, bool $attach_pdf = false, array $custom_mail = [], $attachments = [])
     {
         parent::__construct();
 
@@ -46,6 +53,7 @@ class Invoice extends Notification
         $this->template = EmailTemplate::alias($template_alias)->first();
         $this->attach_pdf = $attach_pdf;
         $this->custom_mail = $custom_mail;
+        $this->attachments = $attachments;
     }
 
     /**
@@ -68,6 +76,17 @@ class Invoice extends Notification
             ]);
         }
 
+        // Attach selected attachments
+        if (! empty($this->invoice->attachment)) {
+            foreach ($this->invoice->attachment as $attachment) {
+                if (in_array($attachment->id, $this->attachments)) {
+                    $message->attach($attachment->getAbsolutePath(), [
+                        'mime' => $attachment->mime_type,
+                    ]);
+                }
+            }
+        }
+
         return $message;
     }
 
diff --git a/resources/lang/en-GB/general.php b/resources/lang/en-GB/general.php
index 373f92d87..21be355d0 100644
--- a/resources/lang/en-GB/general.php
+++ b/resources/lang/en-GB/general.php
@@ -230,6 +230,7 @@ return [
     'go_back'               => 'Go back to :type',
     'validation_error'      => 'Validation error',
     'dismiss'               => 'Dismiss',
+    'size'                  => 'Size',
 
     'card' => [
         'cards'                 => 'Card|Cards',
diff --git a/resources/views/modals/invoices/email.blade.php b/resources/views/modals/invoices/email.blade.php
index 3691c9d8e..db03c2da4 100644
--- a/resources/views/modals/invoices/email.blade.php
+++ b/resources/views/modals/invoices/email.blade.php
@@ -1,15 +1,91 @@
 <x-form id="form-email" :route="[$store_route, $invoice->id]">
-    <x-form.section>
-        <x-slot name="body">
-            <x-form.group.text name="to" label="{{ trans('general.to') }}" value="{{ $invoice->contact->email }}" form-group-class="sm:col-span-6" />
+    <x-tabs active="general" class="grid grid-cols-{{ $invoice->attachment ? '2' : '1' }}" override="class" ignore-hash>
+        <x-slot name="navs">
+            <x-tabs.nav id="general">
+                {{ trans('general.general') }}
+            </x-tabs.nav>
 
-            <x-form.group.text name="subject" label="{{ trans('settings.email.templates.subject') }}" value="{{ $notification->getSubject() }}" form-group-class="sm:col-span-6" />
-
-            <x-form.group.editor name="body" label="{{ trans('settings.email.templates.body') }}" :value="$notification->getBody()" rows="5" data-toggle="quill" form-group-class="sm:col-span-6 mb-0" />
-
-            <x-form.group.checkbox name="user_email" :options="['1' => trans('general.email_send_me', ['email' => user()->email])]" checkbox-class="col-span-6" />
-
-            <x-form.input.hidden name="document_id" :value="$invoice->id" />
+            @if ($invoice->attachment)
+                <x-tabs.nav id="attachments">
+                    {{ trans_choice('general.attachments', 2) }}
+                </x-tabs.nav>
+            @endif
         </x-slot>
-    </x-form.section>
+
+        <x-slot name="content">
+            <x-tabs.tab id="general">
+                <x-form.section>
+                    <x-slot name="body">
+                        <x-form.group.text name="to" label="{{ trans('general.to') }}" value="{{ $invoice->contact->email }}" form-group-class="sm:col-span-6" />
+
+                        <x-form.group.text name="subject" label="{{ trans('settings.email.templates.subject') }}" value="{{ $notification->getSubject() }}" form-group-class="sm:col-span-6" />
+
+                        <x-form.group.editor name="body" label="{{ trans('settings.email.templates.body') }}" :value="$notification->getBody()" rows="5" data-toggle="quill" form-group-class="sm:col-span-6 mb-0" />
+
+                        <x-form.group.checkbox name="user_email" :options="['1' => trans('general.email_send_me', ['email' => user()->email])]" checkbox-class="col-span-6" />
+
+                        <x-form.input.hidden name="document_id" :value="$invoice->id" />
+                    </x-slot>
+                </x-form.section>
+            </x-tabs.tab>
+
+            @if ($invoice->attachment)
+                <x-tabs.tab id="attachments">
+                    <x-table>
+                        <x-table.thead>
+                            <x-table.tr class="flex items-center px-1">
+                                <x-table.th class="w-1/12">
+                                </x-table.th>
+
+                                <x-table.th class="w-1/6">
+                                </x-table.th>
+
+                                <x-table.th class="w-4/6">
+                                    {{ trans('general.name') }}
+                                </x-table.th>
+
+                                <x-table.th class="w-1/6">
+                                    {{ trans('general.size') }}
+                                </x-table.th>
+                            </x-table.tr>
+                        </x-table.thead>
+
+                        <x-table.tbody>
+                            @foreach($invoice->attachment as $attachment)
+                                <x-table.tr id="method-{{ $attachment->id }}">
+                                    <x-table.td class="w-1/12">
+                                        <input type="checkbox"
+                                               id="attachment-{{ $attachment->id }}"
+                                               name="{{ $attachment->id }}"
+                                               class="rounded-sm text-purple border-gray-300 cursor-pointer disabled:bg-gray-200 focus:outline-none focus:ring-transparent"
+                                               data-field="attachments"
+                                               @input="e => form.attachments[e.target.name] = e.target.checked | 0">
+                                    </x-table.td>
+                                    <x-table.td class="w-1/6">
+                                        @if ($attachment->aggregate_type == 'image')
+                                            <div class="avatar-attachment">
+                                                <img src="{{ route('uploads.get', $attachment->id) }}" alt="{{ $attachment->basename }}" class="avatar-img h-full rounded object-cover">
+                                            </div>
+                                        @else
+                                            <div class="avatar-attachment">
+                                                <span class="material-icons text-base">attach_file</span>
+                                            </div>
+                                        @endif
+                                    </x-table.td>
+
+                                    <x-table.td class="w-4/6">
+                                        {{ $attachment->basename }}
+                                    </x-table.td>
+
+                                    <x-table.td class="w-1/6">
+                                        {{ $attachment->readableSize() }}
+                                    </x-table.td>
+                                </x-table.tr>
+                            @endforeach
+                        </x-table.tbody>
+                    </x-table>
+                </x-tabs.tab>
+            @endif
+        </x-slot>
+    </x-tabs>
 </x-form>