feat(webstatement): tambahkan fitur konfirmasi email dan optimasi proses download statement

- **Penambahan Fitur Konfirmasi Email:**
  - Menambahkan event listener untuk form submit:
    - Menampilkan SweetAlert jika field email telah diisi.
    - Mengonfirmasi pengiriman statement ke alamat email yang diisi pengguna.
    - Submit form hanya setelah user mengonfirmasi.

- **Optimalisasi Proses Download Statement:**
  - Menangani logic download statement dalam rentang periode (period range):
    - Mencatat log keberadaan file untuk setiap periode.
    - Membuat file ZIP yang berisi semua file statement yang tersedia dalam rentang tersebut.
    - Mengelola file sementara untuk proses kompresi dengan pembersihan otomatis.
    - Menambahkan log error dan warning untuk file yang hilang dalam rentang periode.
    - Mendukung mekanisme download file tunggal untuk periode tertentu.
  - Menyesuaikan log dengan detail proses, seperti:
    - Informasi periode yang tersedia dan tidak.
    - Notifikasi penyelesaian atau kegagalan proses download ZIP.
  - Menambahkan logging trace pada exception untuk debugging lebih rinci.

- **Perubahan Validasi Logic:**
  - Validasi baru pada `PrintStatementRequest`:
    - Menentukan `is_period_range` hanya jika `period_to` berbeda dengan `period_from`.

- **Perbaikan dan Penyesuaian Pengiriman Email:**
  - Menambahkan pengecekan field email sebelum menjalankan fungsi kirim email di `PrintStatementController`.
  - Mengintegrasikan fungsi `sendEmail` jika terdapat email pada statement.

- **Penambahan Dokumentasi Kode:**
  - Menambahkan komentar inline di beberapa bagian:
    - Logika konfirmasi email.
    - Proses pembuatan ZIP dan penanganan download.
  - Menjelaskan tiap langkah operasional untuk mempermudah pemahaman dan debugging.

Perubahan ini mengintegrasikan fitur konfirmasi email yang lebih interaktif, meningkatkan proses download statement berjenjang, serta memperbaiki validasi dan logging pada tiap langkah proses.

Signed-off-by: Daeng Deni Mardaeni <ddeni05@gmail.com>

app/Http/Controllers/PrintStatementController.php

@@ -30,7 +30,10 @@
                               ->orderBy('name')
                               ->get();
 
-            return view('webstatement::statements.index', compact('branches'));
+            $branch = Branch::find(Auth::user()->branch_id);
+            $multiBranch = session('MULTI_BRANCH') ?? false;
+
+            return view('webstatement::statements.index', compact('branches', 'branch', 'multiBranch'));
         }
 
         /**
@@ -104,6 +107,10 @@
                 // Process statement availability check
                 $this->checkStatementAvailability($statement);
 
+                $statement = PrintStatementLog::find($statement->id);
+                if($statement->email){
+                    $this->sendEmail($statement->id);
+                }
                 DB::commit();
 
                 return redirect()->route('statements.index')
@@ -141,7 +148,14 @@
 
             try {
                 $disk     = Storage::disk('sftpStatement');
-                $filePath = "{$statement->period_from}/Print/{$statement->branch_code}/{$statement->account_number}.pdf";
+                $filePath = "{$statement->period_from}/{$statement->branch_code}/{$statement->account_number}_{$statement->period_from}.pdf";
+
+                // Log untuk debugging
+                Log::info('Checking SFTP file path', [
+                    'file_path' => $filePath,
+                    'sftp_root' => config('filesystems.disks.sftpStatement.root'),
+                    'full_path' => config('filesystems.disks.sftpStatement.root') . '/' . $filePath
+                ]);
 
                 if ($statement->is_period_range && $statement->period_to) {
                     $periodFrom = Carbon::createFromFormat('Ym', $statement->period_from);
@@ -152,7 +166,7 @@
 
                     for ($period = clone $periodFrom; $period->lte($periodTo); $period->addMonth()) {
                         $periodFormatted = $period->format('Ym');
-                        $periodPath      = $periodFormatted . "/Print/{$statement->branch_code}/{$statement->account_number}.pdf";
+                        $periodPath      = $periodFormatted . "/{$statement->branch_code}/{$statement->account_number}_{$periodFormatted}.pdf";
 
                         if ($disk->exists($periodPath)) {
                             $availablePeriods[] = $periodFormatted;
@@ -274,20 +288,148 @@
 
                 // Generate or fetch the statement file
                 $disk     = Storage::disk('sftpStatement');
-                $filePath = "{$statement->period_from}/Print/{$statement->branch_code}/{$statement->account_number}.pdf";
+                $filePath = "{$statement->period_from}/{$statement->branch_code}/{$statement->account_number}_{$statement->period_from}.pdf";
 
                 if ($statement->is_period_range && $statement->period_to) {
-                    // Handle period range download (existing logic)
+                    // Log: Memulai proses download period range
-                    // ... existing zip creation logic ...
+                    Log::info('Starting period range download', [
+                        'statement_id' => $statement->id,
+                        'period_from'  => $statement->period_from,
+                        'period_to'    => $statement->period_to
+                    ]);
+
+                    /**
+                     * Handle period range download dengan membuat zip file
+                     * yang berisi semua statement dalam rentang periode
+                     */
+                    $periodFrom = Carbon::createFromFormat('Ym', $statement->period_from);
+                    $periodTo   = Carbon::createFromFormat('Ym', $statement->period_to);
+
+                    // Loop through each month in the range
+                    $missingPeriods   = [];
+                    $availablePeriods = [];
+
+                    for ($period = clone $periodFrom; $period->lte($periodTo); $period->addMonth()) {
+                        $periodFormatted = $period->format('Ym');
+                        $periodPath      = $periodFormatted . "/{$statement->branch_code}/{$statement->account_number}_{$periodFormatted}.pdf";
+
+                        if ($disk->exists($periodPath)) {
+                            $availablePeriods[] = $periodFormatted;
+                            Log::info('Period available for download', [
+                                'period' => $periodFormatted,
+                                'path'   => $periodPath
+                            ]);
+                        } else {
+                            $missingPeriods[] = $periodFormatted;
+                            Log::warning('Period not available for download', [
+                                'period' => $periodFormatted,
+                                'path'   => $periodPath
+                            ]);
+                        }
+                    }
+
+                    // If any period is available, create a zip and download it
+                    if (count($availablePeriods) > 0) {
+                        /**
+                         * Membuat zip file temporary untuk download
+                         * dengan semua statement yang tersedia dalam periode
+                         */
+                        $zipFileName = "{$statement->account_number}_{$statement->period_from}_to_{$statement->period_to}.zip";
+                        $zipFilePath = storage_path("app/temp/{$zipFileName}");
+
+                        // Ensure the temp directory exists
+                        if (!file_exists(storage_path('app/temp'))) {
+                            mkdir(storage_path('app/temp'), 0755, true);
+                            Log::info('Created temp directory for zip files');
+                        }
+
+                        // Create a new zip archive
+                        $zip = new ZipArchive();
+                        if ($zip->open($zipFilePath, ZipArchive::CREATE | ZipArchive::OVERWRITE) === true) {
+                            Log::info('Zip archive created successfully', ['zip_path' => $zipFilePath]);
+
+                            // Add each available statement to the zip
+                            foreach ($availablePeriods as $period) {
+                                $periodFilePath = "{$period}/{$statement->branch_code}/{$statement->account_number}_{$period}.pdf";
+                                $localFilePath  = storage_path("app/temp/{$statement->account_number}_{$period}.pdf");
+
+                                try {
+                                    // Download the file from SFTP to local storage temporarily
+                                    file_put_contents($localFilePath, $disk->get($periodFilePath));
+
+                                    // Add the file to the zip
+                                    $zip->addFile($localFilePath, "{$statement->account_number}_{$period}.pdf");
+
+                                    Log::info('Added file to zip', [
+                                        'period'     => $period,
+                                        'local_path' => $localFilePath
+                                    ]);
+                                } catch (Exception $e) {
+                                    Log::error('Failed to add file to zip', [
+                                        'period' => $period,
+                                        'error'  => $e->getMessage()
+                                    ]);
+                                }
+                            }
+
+                            $zip->close();
+                            Log::info('Zip archive closed successfully');
+
+                            // Return the zip file for download
+                            $response = response()->download($zipFilePath, $zipFileName)->deleteFileAfterSend(true);
+
+                            // Clean up temporary PDF files
+                            foreach ($availablePeriods as $period) {
+                                $localFilePath = storage_path("app/temp/{$statement->account_number}_{$period}.pdf");
+                                if (file_exists($localFilePath)) {
+                                    unlink($localFilePath);
+                                    Log::info('Cleaned up temporary file', ['file' => $localFilePath]);
+                                }
+                            }
+
+                            Log::info('Period range download completed successfully', [
+                                'statement_id'      => $statement->id,
+                                'available_periods' => count($availablePeriods),
+                                'missing_periods'   => count($missingPeriods)
+                            ]);
+
+                            return $response;
+                        } else {
+                            Log::error('Failed to create zip archive', ['zip_path' => $zipFilePath]);
+                            return back()->with('error', 'Failed to create zip archive for download.');
+                        }
+                    } else {
+                        Log::warning('No statements available for download in period range', [
+                            'statement_id'    => $statement->id,
+                            'missing_periods' => $missingPeriods
+                        ]);
+                        return back()->with('error', 'No statements available for download in the specified period range.');
+                    }
                 } else if ($disk->exists($filePath)) {
+                    /**
+                     * Handle single period download
+                     * Download file PDF tunggal untuk periode tertentu
+                     */
+                    Log::info('Single period download', [
+                        'statement_id' => $statement->id,
+                        'file_path'    => $filePath
+                    ]);
+
                     return $disk->download($filePath, "{$statement->account_number}_{$statement->period_from}.pdf");
+                } else {
+                    Log::warning('Statement file not found', [
+                        'statement_id' => $statement->id,
+                        'file_path'    => $filePath
+                    ]);
+                    return back()->with('error', 'Statement file not found.');
                 }
             } catch (Exception $e) {
                 DB::rollBack();
 
                 Log::error('Failed to download statement', [
                     'statement_id' => $statement->id,
-                    'error'        => $e->getMessage()
+                    'error'        => $e->getMessage(),
+                    'trace'        => $e->getTraceAsString()
                 ]);
 
                 return back()->with('error', 'Failed to download statement: ' . $e->getMessage());
@@ -334,7 +476,10 @@
             $query = PrintStatementLog::query();
 
             if (!auth()->user()->hasRole('administrator')) {
-                $query->where('user_id', Auth::id());
+                $query->where(function($q) {
+                    $q->where('user_id', Auth::id())
+                        ->orWhere('branch_code', Auth::user()->branch->code);
+                });
             }
 
             // Apply search filter if provided
@@ -468,6 +613,7 @@
         public function sendEmail($id)
         {
             $statement = PrintStatementLog::findOrFail($id);
+
             // Check if statement has email
             if (empty($statement->email)) {
                 return redirect()->back()->with('error', 'No email address provided for this statement.');
@@ -480,7 +626,7 @@
 
             try {
                 $disk     = Storage::disk('sftpStatement');
-                $filePath = "{$statement->period_from}/Print/{$statement->branch_code}/{$statement->account_number}.pdf";
+                $filePath = "{$statement->period_from}/{$statement->branch_code}/{$statement->account_number}_{$statement->period_from}.pdf";
 
                 if ($statement->is_period_range && $statement->period_to) {
                     $periodFrom = Carbon::createFromFormat('Ym', $statement->period_from);
@@ -492,7 +638,7 @@
 
                     for ($period = clone $periodFrom; $period->lte($periodTo); $period->addMonth()) {
                         $periodFormatted = $period->format('Ym');
-                        $periodPath      = $periodFormatted . "/Print/{$statement->branch_code}/{$statement->account_number}.pdf";
+                        $periodPath      = $periodFormatted . "/{$statement->branch_code}/{$statement->account_number}_{$periodFormatted}.pdf";
 
                         if ($disk->exists($periodPath)) {
                             $availablePeriods[] = $periodFormatted;
@@ -517,7 +663,7 @@
                         if ($zip->open($zipFilePath, ZipArchive::CREATE | ZipArchive::OVERWRITE) === true) {
                             // Add each available statement to the zip
                             foreach ($availablePeriods as $period) {
-                                $filePath      = "{$period}/Print/{$statement->branch_code}/{$statement->account_number}.pdf";
+                                $filePath      = "{$period}/{$statement->branch_code}/{$statement->account_number}_{$statement->period_from}.pdf";
                                 $localFilePath = storage_path("app/temp/{$statement->account_number}_{$period}.pdf");
 
                                 // Download the file from SFTP to local storage temporarily

app/Http/Requests/PrintStatementRequest.php

@@ -35,6 +35,7 @@ class PrintStatementRequest extends FormRequest
                 function ($attribute, $value, $fail) {
                     $query = Statement::where('account_number', $this->input('account_number'))
                                       ->where('authorization_status', '!=', 'rejected')
+                                      ->where('is_available', true)
                                       ->where('period_from', $value);
 
                     // If this is an update request, exclude the current record
@@ -103,14 +104,14 @@ class PrintStatementRequest extends FormRequest
             $this->merge([
                 'period_to' => substr($this->period_to, 0, 4) . substr($this->period_to, 5, 2),
             ]);
-        }
 
-        // Convert is_period_range to boolean if it exists
+            // Only set is_period_range to true if period_to is different from period_from
-        if ($this->has('period_to')) {
+            if ($this->period_to !== $this->period_from) {
                 $this->merge([
                     'is_period_range' => true,
                 ]);
             }
+        }
 
         // Set default request_type if not provided
         if (!$this->has('request_type')) {

resources/views/statements/index.blade.php

@@ -11,16 +11,47 @@
                 <h3 class="card-title">Request Print Stetement</h3>
             </div>
             <div class="card-body">
-                <form action="{{ isset($statement) ? route('statements.update', $statement->id) : route('statements.store') }}" method="POST">
+                <form
+                    action="{{ isset($statement) ? route('statements.update', $statement->id) : route('statements.store') }}"
+                    method="POST">
                     @csrf
                     @if (isset($statement))
                         @method('PUT')
                     @endif
 
                     <div class="grid grid-cols-1 gap-5">
+
+                        @if ($multiBranch)
+                            <div class="form-group">
+                                <label class="form-label required" for="branch_id">Branch/Cabang</label>
+                                <select class="input form-control tomselect @error('branch_id') is-invalid @enderror"
+                                    id="branch_id" name="branch_id" required>
+                                    <option value="">Pilih Branch/Cabang</option>
+                                    @foreach ($branches as $branchOption)
+                                        <option value="{{ $branchOption->code }}"
+                                            {{ old('branch_id', $statement->branch_id ?? ($branch->code ?? '')) == $branchOption->code ? 'selected' : '' }}>
+                                            {{ $branchOption->code }} - {{ $branchOption->name }}
+                                        </option>
+                                    @endforeach
+                                </select>
+                                @error('branch_id')
+                                    <div class="invalid-feedback">{{ $message }}</div>
+                                @enderror
+                            </div>
+                        @else
+                            <div class="form-group">
+                                <label class="form-label" for="branch_display">Branch/Cabang</label>
+                                <input type="text" class="input form-control" id="branch_display"
+                                    value="{{ $branch->code ?? '' }} - {{ $branch->name ?? '' }}" readonly>
+                                <input type="hidden" name="branch_id" value="{{ $branch->code ?? '' }}">
+                            </div>
+                        @endif
+
                         <div class="form-group">
                             <label class="form-label required" for="account_number">Account Number</label>
-                            <input type="text" class="input form-control @error('account_number') is-invalid @enderror" id="account_number" name="account_number" value="{{ old('account_number', $statement->account_number ?? '') }}" required>
+                            <input type="text" class="input form-control @error('account_number') is-invalid @enderror"
+                                id="account_number" name="account_number"
+                                value="{{ old('account_number', $statement->account_number ?? '') }}" required>
                             @error('account_number')
                                 <div class="invalid-feedback">{{ $message }}</div>
                             @enderror
@@ -28,7 +59,9 @@
 
                         <div class="form-group">
                             <label class="form-label" for="email">Email</label>
-                            <input type="email" class="input form-control @error('email') is-invalid @enderror" id="email" name="email" value="{{ old('email', $statement->email ?? '') }}" placeholder="Optional email for send statement">
+                            <input type="email" class="input form-control @error('email') is-invalid @enderror"
+                                id="email" name="email" value="{{ old('email', $statement->email ?? '') }}"
+                                placeholder="Optional email for send statement">
                             @error('email')
                                 <div class="invalid-feedback">{{ $message }}</div>
                             @enderror
@@ -38,24 +71,21 @@
                             <label class="form-label required" for="start_date">Start Date</label>
 
                             <input class="input @error('period_from') border-danger bg-danger-light @enderror"
-                                   type="month"
+                                type="month" name="period_from"
-                                   name="period_from"
                                 value="{{ $statement->period_from ?? old('period_from') }}"
                                 max="{{ date('Y-m', strtotime('-1 month')) }}">
                             @error('period_from')
-                            <em class="alert text-danger text-sm">{{ $message }}</em>
+                                <em class="text-sm alert text-danger">{{ $message }}</em>
                             @enderror
                         </div>
 
                         <div class="form-group">
                             <label class="form-label required" for="end_date">End Date</label>
-                            <input class="input @error('period_to') border-danger bg-danger-light @enderror"
+                            <input class="input @error('period_to') border-danger bg-danger-light @enderror" type="month"
-                                   type="month"
+                                name="period_to" value="{{ $statement->period_to ?? old('period_to') }}"
-                                   name="period_to"
-                                   value="{{ $statement->period_to ?? old('period_to') }}"
                                 max="{{ date('Y-m', strtotime('-1 month')) }}">
                             @error('period_to')
-                            <em class="alert text-danger text-sm">{{ $message }}</em>
+                                <em class="text-sm alert text-danger">{{ $message }}</em>
                             @enderror
                         </div>
                     </div>
@@ -70,8 +100,9 @@
             </div>
         </div>
         <div class="col-span-6">
-            <div class="card card-grid min-w-full" data-datatable="false" data-datatable-page-size="10" data-datatable-state-save="false" id="statement-table" data-api-url="{{ route('statements.datatables') }}">
+            <div class="min-w-full card card-grid" data-datatable="false" data-datatable-page-size="10"
-                <div class="card-header py-5 flex-wrap">
+                data-datatable-state-save="false" id="statement-table" data-api-url="{{ route('statements.datatables') }}">
+                <div class="flex-wrap py-5 card-header">
                     <h3 class="card-title">
                         Daftar Statement Request
                     </h3>
@@ -85,7 +116,8 @@
                 </div>
                 <div class="card-body">
                     <div class="scrollable-x-auto">
-                        <table class="table table-auto table-border align-middle text-gray-700 font-medium text-sm" data-datatable-table="true">
+                        <table class="table text-sm font-medium text-gray-700 align-middle table-auto table-border"
+                            data-datatable-table="true">
                             <thead>
                                 <tr>
                                     <th class="w-14">
@@ -124,12 +156,14 @@
                             </thead>
                         </table>
                     </div>
-                    <div class="card-footer justify-center md:justify-between flex-col md:flex-row gap-3 text-gray-600 text-2sm font-medium">
+                    <div
-                        <div class="flex items-center gap-2">
+                        class="flex-col gap-3 justify-center font-medium text-gray-600 card-footer md:justify-between md:flex-row text-2sm">
+                        <div class="flex gap-2 items-center">
                             Show
-                            <select class="select select-sm w-16" data-datatable-size="true" name="perpage"> </select> per page
+                            <select class="w-16 select select-sm" data-datatable-size="true" name="perpage"> </select>
+                            per page
                         </div>
-                        <div class="flex items-center gap-4">
+                        <div class="flex gap-4 items-center">
                             <span data-datatable-info="true"> </span>
                             <div class="pagination" data-datatable-pagination="true">
                             </div>
@@ -143,6 +177,10 @@
 
 @push('scripts')
     <script type="text/javascript">
+        /**
+         * Fungsi untuk menghapus data statement
+         * @param {number} data - ID statement yang akan dihapus
+         */
         function deleteData(data) {
             Swal.fire({
                 title: 'Are you sure?',
@@ -173,6 +211,56 @@
                 }
             })
         }
+
+        /**
+         * Konfirmasi email sebelum submit form
+         * Menampilkan SweetAlert jika email diisi untuk konfirmasi pengiriman
+         */
+        document.addEventListener('DOMContentLoaded', function() {
+            const form = document.querySelector('form');
+            const emailInput = document.getElementById('email');
+
+            // Log: Inisialisasi event listener untuk konfirmasi email
+            console.log('Email confirmation listener initialized');
+
+            form.addEventListener('submit', function(e) {
+                const emailValue = emailInput.value.trim();
+
+                // Jika email diisi, tampilkan konfirmasi
+                if (emailValue) {
+                    e.preventDefault(); // Hentikan submit form sementara
+
+                    // Log: Email terdeteksi, menampilkan konfirmasi
+                    console.log('Email detected:', emailValue);
+
+                    Swal.fire({
+                        title: 'Konfirmasi Pengiriman Email',
+                        text: `Apakah Anda yakin ingin mengirimkan statement ke email: ${emailValue}?`,
+                        icon: 'question',
+                        showCancelButton: true,
+                        confirmButtonColor: '#3085d6',
+                        cancelButtonColor: '#d33',
+                        confirmButtonText: 'Ya, Kirim Email',
+                        cancelButtonText: 'Batal',
+                        reverseButtons: true
+                    }).then((result) => {
+                        if (result.isConfirmed) {
+                            // Log: User konfirmasi pengiriman email
+                            console.log('User confirmed email sending');
+
+                            // Submit form setelah konfirmasi
+                            form.submit();
+                        } else {
+                            // Log: User membatalkan pengiriman email
+                            console.log('User cancelled email sending');
+                        }
+                    });
+                } else {
+                    // Log: Tidak ada email, submit form normal
+                    console.log('No email provided, submitting form normally');
+                }
+            });
+        });
     </script>
 
     <script type="module">
@@ -227,7 +315,8 @@
                 is_available: {
                     title: 'Available',
                     render: (item, data) => {
-                        let statusClass = data.is_available ? 'badge badge-light-success' : 'badge badge-light-danger';
+                        let statusClass = data.is_available ? 'badge badge-light-success' :
+                            'badge badge-light-danger';
                         let statusText = data.is_available ? 'Yes' : 'No';
                         return `<span class="${statusClass}">${statusText}</span>`;
                     },