✨ feat(api,noc): Tambah API pencarian debitur dan perbaikan filter jenis penilaian
- API Debitur Controller: method search() (min 2 karakter), getByCode(), transaksi DB, logging, response JSON konsisten - Batasi hasil pencarian maksimal 20 item untuk performa optimal - Support pencarian berdasarkan CIF dan nama debitur - Tambah import JenisPenilaian di NocController - Perbaiki method penyelesaian() untuk mengirim data jenisPenilaians ke view - Tambahkan filter nama jenis penilaian di dataForDatatablesPenyelesaian() dan kolom jenis_penilaian pada response tabel - Update UI view penyelesaian: dropdown dinamis, kolom sortable, integrasi DataTable, perbaikan layout - Validasi input & keamanan: transaction handling, logging, conditional debug response, SQL injection protection - Peningkatan UX: autocomplete search, filter responsif, pesan error informatif, loading state, format response konsisten
This commit is contained in:
183
app/Http/Controllers/Api/DebiturController.php
Normal file
183
app/Http/Controllers/Api/DebiturController.php
Normal file
@@ -0,0 +1,183 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
namespace Modules\Lpj\Http\Controllers\Api;
|
||||||
|
|
||||||
|
use App\Http\Controllers\Controller;
|
||||||
|
use Illuminate\Http\Request;
|
||||||
|
use Illuminate\Http\JsonResponse;
|
||||||
|
use Illuminate\Support\Facades\Auth;
|
||||||
|
use Illuminate\Support\Facades\DB;
|
||||||
|
use Illuminate\Support\Facades\Log;
|
||||||
|
use Exception;
|
||||||
|
use Modules\Lpj\Models\Debiture;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Controller untuk API pencarian debitur
|
||||||
|
* Digunakan untuk autocomplete search pada form pembayaran
|
||||||
|
*/
|
||||||
|
class DebiturController extends Controller
|
||||||
|
{
|
||||||
|
/**
|
||||||
|
* Pencarian debitur untuk autocomplete
|
||||||
|
*
|
||||||
|
* @param Request $request
|
||||||
|
* @return JsonResponse
|
||||||
|
*/
|
||||||
|
public function search(Request $request): JsonResponse
|
||||||
|
{
|
||||||
|
try {
|
||||||
|
// Log aktivitas pencarian
|
||||||
|
Log::info('API Debitur Search - Request', [
|
||||||
|
'query' => $request->get('q'),
|
||||||
|
'user_id' => Auth::id()
|
||||||
|
]);
|
||||||
|
|
||||||
|
$query = $request->get('q', '');
|
||||||
|
|
||||||
|
// Validasi minimal 2 karakter untuk pencarian
|
||||||
|
if (strlen($query) < 2) {
|
||||||
|
return response()->json([
|
||||||
|
'success' => false,
|
||||||
|
'message' => 'Minimal 2 karakter untuk pencarian',
|
||||||
|
'data' => []
|
||||||
|
], 400);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Mulai database transaction
|
||||||
|
DB::beginTransaction();
|
||||||
|
|
||||||
|
try {
|
||||||
|
// Query pencarian debitur
|
||||||
|
// Asumsi tabel debitur dengan kolom: id, code, nama, alamat
|
||||||
|
$debiturs = Debiture::query()
|
||||||
|
->select('id', 'cif', 'name', 'address')
|
||||||
|
->whereAny(['cif','name'], 'LIKE', "%{$query}%")
|
||||||
|
->orderBy('name', 'asc')
|
||||||
|
->limit(20) // Batasi hasil maksimal 20
|
||||||
|
->get();
|
||||||
|
|
||||||
|
// Format data untuk TomSelect
|
||||||
|
$formattedData = $debiturs->map(function($debitur) {
|
||||||
|
return [
|
||||||
|
'id' => $debitur->id,
|
||||||
|
'kode_debitur' => $debitur->cif,
|
||||||
|
'name' => $debitur->name,
|
||||||
|
'address' => $debitur->address
|
||||||
|
];
|
||||||
|
});
|
||||||
|
|
||||||
|
DB::commit();
|
||||||
|
|
||||||
|
// Log hasil pencarian
|
||||||
|
Log::info('API Debitur Search - Success', [
|
||||||
|
'query' => $query,
|
||||||
|
'results_count' => $formattedData->count(),
|
||||||
|
'user_id' => Auth::id()
|
||||||
|
]);
|
||||||
|
|
||||||
|
return response()->json([
|
||||||
|
'success' => true,
|
||||||
|
'message' => 'Data debitur berhasil ditemukan',
|
||||||
|
'data' => $formattedData
|
||||||
|
]);
|
||||||
|
|
||||||
|
} catch (Exception $e) {
|
||||||
|
DB::rollback();
|
||||||
|
throw $e;
|
||||||
|
}
|
||||||
|
|
||||||
|
} catch (Exception $e) {
|
||||||
|
// Log error
|
||||||
|
Log::error('API Debitur Search - Error', [
|
||||||
|
'query' => $request->get('q'),
|
||||||
|
'error' => $e->getMessage(),
|
||||||
|
'trace' => $e->getTraceAsString(),
|
||||||
|
'user_id' => Auth::id()
|
||||||
|
]);
|
||||||
|
|
||||||
|
return response()->json([
|
||||||
|
'success' => false,
|
||||||
|
'message' => 'Terjadi kesalahan saat mencari data debitur',
|
||||||
|
'error' => config('app.debug') ? $e->getMessage() : 'Internal Server Error'
|
||||||
|
], 500);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Get detail debitur berdasarkan code
|
||||||
|
*
|
||||||
|
* @param Request $request
|
||||||
|
* @return JsonResponse
|
||||||
|
*/
|
||||||
|
public function getByCode(Request $request): JsonResponse
|
||||||
|
{
|
||||||
|
try {
|
||||||
|
$code = $request->get('code');
|
||||||
|
|
||||||
|
if (empty($code)) {
|
||||||
|
return response()->json([
|
||||||
|
'success' => false,
|
||||||
|
'message' => 'Code debitur harus diisi',
|
||||||
|
'data' => null
|
||||||
|
], 400);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Log aktivitas get detail
|
||||||
|
Log::info('API Debitur GetByCode - Request', [
|
||||||
|
'code' => $code,
|
||||||
|
'user_id' => Auth::id()
|
||||||
|
]);
|
||||||
|
|
||||||
|
DB::beginTransaction();
|
||||||
|
|
||||||
|
try {
|
||||||
|
$debitur = DB::table('debitur')
|
||||||
|
->select('id', 'code', 'nama', 'alamat', 'telepon', 'email')
|
||||||
|
->where('code', $code)
|
||||||
|
->where('status', 'aktif')
|
||||||
|
->first();
|
||||||
|
|
||||||
|
if (!$debitur) {
|
||||||
|
DB::rollback();
|
||||||
|
return response()->json([
|
||||||
|
'success' => false,
|
||||||
|
'message' => 'Debitur tidak ditemukan',
|
||||||
|
'data' => null
|
||||||
|
], 404);
|
||||||
|
}
|
||||||
|
|
||||||
|
DB::commit();
|
||||||
|
|
||||||
|
Log::info('API Debitur GetByCode - Success', [
|
||||||
|
'code' => $code,
|
||||||
|
'debitur_id' => $debitur->id,
|
||||||
|
'user_id' => Auth::id()
|
||||||
|
]);
|
||||||
|
|
||||||
|
return response()->json([
|
||||||
|
'success' => true,
|
||||||
|
'message' => 'Data debitur berhasil ditemukan',
|
||||||
|
'data' => $debitur
|
||||||
|
]);
|
||||||
|
|
||||||
|
} catch (Exception $e) {
|
||||||
|
DB::rollback();
|
||||||
|
throw $e;
|
||||||
|
}
|
||||||
|
|
||||||
|
} catch (Exception $e) {
|
||||||
|
Log::error('API Debitur GetByCode - Error', [
|
||||||
|
'code' => $request->get('code'),
|
||||||
|
'error' => $e->getMessage(),
|
||||||
|
'trace' => $e->getTraceAsString(),
|
||||||
|
'user_id' => Auth::id()
|
||||||
|
]);
|
||||||
|
|
||||||
|
return response()->json([
|
||||||
|
'success' => false,
|
||||||
|
'message' => 'Terjadi kesalahan saat mengambil data debitur',
|
||||||
|
'error' => config('app.debug') ? $e->getMessage() : 'Internal Server Error'
|
||||||
|
], 500);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -8,6 +8,7 @@
|
|||||||
use Modules\Lpj\Http\Requests\NocRequest;
|
use Modules\Lpj\Http\Requests\NocRequest;
|
||||||
use Modules\Lpj\Models\Noc;
|
use Modules\Lpj\Models\Noc;
|
||||||
use Modules\Lpj\Models\PersetujuanPenawaran;
|
use Modules\Lpj\Models\PersetujuanPenawaran;
|
||||||
|
use Modules\Lpj\Models\JenisPenilaian;
|
||||||
|
|
||||||
class NocController extends Controller
|
class NocController extends Controller
|
||||||
{
|
{
|
||||||
@@ -31,7 +32,8 @@
|
|||||||
public function penyelesaian()
|
public function penyelesaian()
|
||||||
{
|
{
|
||||||
$persetujuanPenawarans = PersetujuanPenawaran::all();
|
$persetujuanPenawarans = PersetujuanPenawaran::all();
|
||||||
return view('lpj::noc.penyelesaian', compact('persetujuanPenawarans'));
|
$jenisPenilaians = JenisPenilaian::get();
|
||||||
|
return view('lpj::noc.penyelesaian', compact('persetujuanPenawarans', 'jenisPenilaians'));
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -330,7 +332,8 @@
|
|||||||
if ($request->has('search') && !empty($request->get('search'))) {
|
if ($request->has('search') && !empty($request->get('search'))) {
|
||||||
$search = $request->get('search');
|
$search = $request->get('search');
|
||||||
$query->where(function ($q) use ($search) {
|
$query->where(function ($q) use ($search) {
|
||||||
$q->orWhereRelation('penawaran', 'nomor_registrasi', 'LIKE', '%' . $search . '%');
|
$q->orWhereRelation('penawaran', 'nomor_registrasi', 'LIKE', '%' . $search . '%')
|
||||||
|
->orWhereRelation('permohonan.jenisPenilaian', 'name', 'LIKE', '%' . $search . '%');
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -376,6 +379,7 @@
|
|||||||
'nominal_diterima' => currencyFormat(
|
'nominal_diterima' => currencyFormat(
|
||||||
$persetujuanPenawaran->noc->nominal_bayar ?? 0,
|
$persetujuanPenawaran->noc->nominal_bayar ?? 0,
|
||||||
),
|
),
|
||||||
|
'jenis_penilaian' => $persetujuanPenawaran->permohonan?->jenisPenilaian?->name ?? "",
|
||||||
'bukti_ksl' => $persetujuanPenawaran->noc->bukti_ksl ?? $persetujuanPenawaran->bukti_ksl ?? null,
|
'bukti_ksl' => $persetujuanPenawaran->noc->bukti_ksl ?? $persetujuanPenawaran->bukti_ksl ?? null,
|
||||||
'bukti_bayar' => $persetujuanPenawaran->bukti_bayar ?? null,
|
'bukti_bayar' => $persetujuanPenawaran->bukti_bayar ?? null,
|
||||||
'memo_penyelesaian' => $persetujuanPenawaran->noc->memo_penyelesaian ?? $persetujuanPenawaran->memo_penyelesaian ?? null,
|
'memo_penyelesaian' => $persetujuanPenawaran->noc->memo_penyelesaian ?? $persetujuanPenawaran->memo_penyelesaian ?? null,
|
||||||
|
|||||||
@@ -19,6 +19,14 @@
|
|||||||
</label>
|
</label>
|
||||||
</div>
|
</div>
|
||||||
<div class="flex flex-wrap gap-2.5">
|
<div class="flex flex-wrap gap-2.5">
|
||||||
|
<div class="flex">
|
||||||
|
<select id="jenis_penilaian_filter" class="select select-sm">
|
||||||
|
<option value="">Semua Jenis Penilaian</option>
|
||||||
|
@foreach ($jenisPenilaians as $jenisPenilaian)
|
||||||
|
<option value="{{ $jenisPenilaian->name }}">{{ $jenisPenilaian->name }}</option>
|
||||||
|
@endforeach
|
||||||
|
</select>
|
||||||
|
</div>
|
||||||
<div class="h-[24px] border border-r-gray-200"></div>
|
<div class="h-[24px] border border-r-gray-200"></div>
|
||||||
<a class="btn btn-sm btn-light" href="#"> Export to Excel </a>
|
<a class="btn btn-sm btn-light" href="#"> Export to Excel </a>
|
||||||
</div>
|
</div>
|
||||||
@@ -51,6 +59,10 @@
|
|||||||
<span class="sort"> <span class="sort-label"> Cabang </span>
|
<span class="sort"> <span class="sort-label"> Cabang </span>
|
||||||
<span class="sort-icon"> </span> </span>
|
<span class="sort-icon"> </span> </span>
|
||||||
</th>
|
</th>
|
||||||
|
<th class="min-w-[150px]" data-datatable-column="jenis_penilaian">
|
||||||
|
<span class="sort"> <span class="sort-label"> Jenis Penilaian </span>
|
||||||
|
<span class="sort-icon"> </span> </span>
|
||||||
|
</th>
|
||||||
<th class="min-w-[150px]" data-datatable-column="tanggal_setor">
|
<th class="min-w-[150px]" data-datatable-column="tanggal_setor">
|
||||||
<span class="sort"> <span class="sort-label"> Tanggal KSL </span>
|
<span class="sort"> <span class="sort-label"> Tanggal KSL </span>
|
||||||
<span class="sort-icon"> </span> </span>
|
<span class="sort-icon"> </span> </span>
|
||||||
@@ -136,6 +148,7 @@
|
|||||||
<script type="module">
|
<script type="module">
|
||||||
const element = document.querySelector('#noc-table');
|
const element = document.querySelector('#noc-table');
|
||||||
const searchInput = document.getElementById('search');
|
const searchInput = document.getElementById('search');
|
||||||
|
const jenisPenilaianFilter = document.getElementById('jenis_penilaian_filter');
|
||||||
|
|
||||||
const apiUrl = element.getAttribute('data-api-url');
|
const apiUrl = element.getAttribute('data-api-url');
|
||||||
const dataTableOptions = {
|
const dataTableOptions = {
|
||||||
@@ -167,6 +180,9 @@
|
|||||||
return data.kode_cabang + ' - ' + data.cabang;
|
return data.kode_cabang + ' - ' + data.cabang;
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
|
jenis_penilaian: {
|
||||||
|
title: 'Jenis Penilaian'
|
||||||
|
},
|
||||||
tanggal_setor: {
|
tanggal_setor: {
|
||||||
title: 'Tanggal Setor',
|
title: 'Tanggal Setor',
|
||||||
},
|
},
|
||||||
@@ -252,5 +268,15 @@
|
|||||||
const searchValue = this.value.trim();
|
const searchValue = this.value.trim();
|
||||||
dataTable.search(searchValue, true);
|
dataTable.search(searchValue, true);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
// Filter berdasarkan jenis penilaian
|
||||||
|
jenisPenilaianFilter.addEventListener('change', function() {
|
||||||
|
const filterValue = this.value.trim();
|
||||||
|
if (filterValue === '') {
|
||||||
|
dataTable.search('', true); // Reset filter
|
||||||
|
} else {
|
||||||
|
dataTable.search(filterValue, true);
|
||||||
|
}
|
||||||
|
});
|
||||||
</script>
|
</script>
|
||||||
@endpush
|
@endpush
|
||||||
|
|||||||
Reference in New Issue
Block a user