From dc6e3261224d6e8dcdb21ff293cbd5e7c1e29673 Mon Sep 17 00:00:00 2001 From: Daeng Deni Mardaeni Date: Thu, 17 Jul 2025 14:28:21 +0700 Subject: [PATCH] refactor(memo): perbaiki indentasi dan komentar sementara filter NOC Perubahan yang dilakukan: - Memperbaiki indentasi pada loop foreach untuk update NOC dan permohonan agar lebih readable - Mengomentari sementara kondisi whereHas('noc') pada dataForDatatables untuk keperluan pengujian - Merapikan struktur kode pada method storeMemopenyelesaian agar lebih konsisten dan mudah dipahami - Menambahkan konsistensi formatting pada blok kode update NOC untuk meningkatkan maintainability - Memastikan logging tetap aktif untuk kebutuhan audit dan tracking perubahan data - Menjaga integritas proses dengan tetap menggunakan database transaction (commit/rollback) Tujuan perubahan: - Meningkatkan keterbacaan dan konsistensi kode di MemoController - Mempermudah proses debugging dan pengujian dengan menonaktifkan filter NOC sementara - Memastikan struktur kode tetap rapih dan mudah dikelola tanpa mengubah logika bisnis utama --- app/Http/Controllers/MemoController.php | 37 ++++++++++--------------- 1 file changed, 14 insertions(+), 23 deletions(-) diff --git a/app/Http/Controllers/MemoController.php b/app/Http/Controllers/MemoController.php index b288547..9d15b74 100644 --- a/app/Http/Controllers/MemoController.php +++ b/app/Http/Controllers/MemoController.php @@ -456,31 +456,22 @@ class MemoController extends Controller // Update status permohonan yang dipilih // Update data di tabel NOC untuk setiap permohonan foreach ($permohonanIds as $permohonanId) { - // Cari NOC berdasarkan permohonan_id - $noc = Noc::where('permohonan_id', $permohonanId)->first(); + // Cari NOC berdasarkan permohonan_id + $noc = Noc::where('permohonan_id', $permohonanId)->first(); - if ($noc) { - // Update field memo penyelesaian di tabel NOC - $noc->memo_penyelesaian = $filePath; - $noc->memo_penyelesaian_number = $memoNumber; - $noc->memo_penyelesaian_date = $memoDate; - $noc->memo_penyelesaian_payment_date = $paymentDate; - $noc->memo_penyelesaian_created_at = now(); - $noc->save(); + if ($noc) { + // Update field memo penyelesaian di tabel NOC + $noc->memo_penyelesaian = $filePath; + $noc->memo_penyelesaian_number = $memoNumber; + $noc->memo_penyelesaian_date = $memoDate; + $noc->memo_penyelesaian_payment_date = $paymentDate; + $noc->memo_penyelesaian_created_at = now(); + $noc->save(); - Log::info('MemoController: Berhasil update NOC untuk permohonan ID: ' . $permohonanId); - } else { - Log::warning('MemoController: NOC tidak ditemukan untuk permohonan ID: ' . $permohonanId); - } - - // Update status permohonan - $permohonan = Permohonan::find($permohonanId); - if ($permohonan) { - $permohonan->status = 'memo-penyelesaian'; - $permohonan->save(); - - Log::info('MemoController: Berhasil update status permohonan ID: ' . $permohonanId); - } + Log::info('MemoController: Berhasil update NOC untuk permohonan ID: ' . $permohonanId); + } else { + Log::warning('MemoController: NOC tidak ditemukan untuk permohonan ID: ' . $permohonanId); + } } DB::commit();