vngrup/dress
vngrup/dress
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: introduce LogHelper for centralized exception handling and implement consistent error logging across admin controllers

Browse Source
This commit is contained in:
Yoga Pangestu 2026-04-22 21:19:33 +07:00
parent 635dcf712b
commit 53a7b23cbe
8 changed files with 446 additions and 256 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/Console/Commands/GeneratePayrollCommand.php
View File

@ -4,10 +4,10 @@
use App\Models\Payroll;
use App\Models\User;
use App\Support\LogHelper;
use Carbon\Carbon;
use Illuminate\Console\Command;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Log;
use Throwable;
class GeneratePayrollCommand extends Command
@ -60,8 +60,8 @@ public function handle()
}
});
} catch (Throwable $e) {
Log::error('Payroll generate failed', [
'error' => $e->getMessage(),
LogHelper::logException($e, 'Payroll generate failed', [
'period' => $periodMonth,
]);
$this->info('Gagal generate data penggajian, silakan hubungi pengembang.');

5
app/Http/Controllers/Admin/Finance/PayrollController.php
View File

@ -5,12 +5,12 @@
use App\Http\Controllers\Controller;
use App\Http\Requests\Admin\Finance\SalaryAdjustmentRequest;
use App\Models\Payroll;
use App\Support\LogHelper;
use Carbon\Carbon;
use Illuminate\Http\RedirectResponse;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Artisan;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Log;
use Inertia\Inertia;
use Inertia\Response;
use Throwable;
@ -75,9 +75,8 @@ public function update(SalaryAdjustmentRequest $request, Payroll $payroll): Redi
]);
});
} catch (Throwable $e) {
Log::error('Payroll update failed', [
LogHelper::logException($e, 'Payroll update failed', [
'payroll_id' => $payroll->id,
'error' => $e->getMessage(),
]);
return back()->withInput()->with('error', 'Gagal memperbarui data. Silakan coba lagi.');

41
app/Http/Controllers/Admin/Manage/OrderController.php
View File

@ -11,6 +11,7 @@
use App\Models\Order;
use App\Models\OrderItem;
use App\Models\Product;
use App\Support\LogHelper;
use Illuminate\Http\RedirectResponse;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\DB;
@ -46,6 +47,7 @@ public function store(OrderRequest $request): RedirectResponse
{
$validated = $request->validated();
try {
DB::transaction(function () use ($validated) {
$totalItemsPrice = collect($validated['items'])->sum('total');
$hpp = collect($validated['items'])->sum(function ($item) {
@ -88,6 +90,17 @@ public function store(OrderRequest $request): RedirectResponse
});
return redirect()->route('order.index')->with('success', 'Pesanan berhasil disimpan');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to store order', [
'customer_name' => $validated['customer_name'],
'items_count' => count($validated['items'] ?? []),
]);
return redirect()
->back()
->withInput()
->with('error', 'Terjadi kesalahan saat menyimpan pesanan');
}
}
public function edit(Order $order): Response
@ -108,6 +121,7 @@ public function update(OrderRequest $request, Order $order): RedirectResponse
{
$validated = $request->validated();
try {
DB::transaction(function () use ($validated, $order) {
$totalItemsPrice = collect($validated['items'])->sum(fn ($item) => $item['qty'] * $item['price']);
$hpp = collect($validated['items'])->sum(function ($item) {
@ -150,10 +164,22 @@ public function update(OrderRequest $request, Order $order): RedirectResponse
});
return redirect()->route('order.index')->with('success', 'Pesanan berhasil diperbarui');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to update order', [
'order_id' => $order->id,
'customer_name' => $validated['customer_name'],
]);
return redirect()
->back()
->withInput()
->with('error', 'Terjadi kesalahan saat memperbarui pesanan');
}
}
public function destroy(Order $order): RedirectResponse
{
try {
DB::transaction(function () use ($order) {
foreach ($order->items as $item) {
$item->product->increment('stock', $item->qty);
@ -162,12 +188,20 @@ public function destroy(Order $order): RedirectResponse
});
return redirect()->back()->with('success', 'Pesanan berhasil dihapus');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to delete order', [
'order_id' => $order->id,
]);
return redirect()->back()->with('error', 'Terjadi kesalahan saat menghapus pesanan');
}
}
public function bulkDestroy(Request $request): RedirectResponse
{
$ids = $request->input('ids');
try {
DB::transaction(function () use ($ids) {
$orders = Order::with('items')->whereIn('id', $ids)->get();
foreach ($orders as $order) {
@ -179,5 +213,12 @@ public function bulkDestroy(Request $request): RedirectResponse
});
return redirect()->back()->with('success', 'Pesanan terpilih berhasil dihapus');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to bulk delete orders', [
'ids' => $ids,
]);
return redirect()->back()->with('error', 'Terjadi kesalahan saat menghapus beberapa pesanan');
}
}
}

41
app/Http/Controllers/Admin/Manage/PurchaseController.php
View File

@ -7,6 +7,7 @@
use App\Models\Product;
use App\Models\Purchase;
use App\Models\PurchaseItem;
use App\Support\LogHelper;
use Illuminate\Http\RedirectResponse;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\DB;
@ -38,6 +39,7 @@ public function store(PurchaseRequest $request): RedirectResponse
{
$validated = $request->validated();
try {
DB::transaction(function () use ($validated) {
$total = collect($validated['items'])->sum(fn ($item) => $item['quantity'] * $item['unit_price']);
@ -66,6 +68,17 @@ public function store(PurchaseRequest $request): RedirectResponse
});
return redirect()->route('purchase.index')->with('success', 'Data berhasil disimpan');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to store purchase', [
'purchase_date' => $validated['purchase_date'] ?? null,
'items_count' => count($validated['items'] ?? []),
]);
return redirect()
->back()
->withInput()
->with('error', 'Terjadi kesalahan saat menyimpan pembelian');
}
}
public function edit(Purchase $purchase): Response
@ -82,6 +95,7 @@ public function update(PurchaseRequest $request, Purchase $purchase): RedirectRe
{
$validated = $request->validated();
try {
DB::transaction(function () use ($validated, $purchase) {
$total = collect($validated['items'])->sum(fn ($item) => $item['quantity'] * $item['unit_price']);
@ -111,10 +125,22 @@ public function update(PurchaseRequest $request, Purchase $purchase): RedirectRe
});
return redirect()->route('purchase.index')->with('success', 'Data berhasil diperbarui');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to update purchase', [
'purchase_id' => $purchase->id,
'items_count' => count($validated['items'] ?? []),
]);
return redirect()
->back()
->withInput()
->with('error', 'Terjadi kesalahan saat memperbarui pembelian');
}
}
public function destroy(Purchase $purchase): RedirectResponse
{
try {
DB::transaction(function () use ($purchase) {
foreach ($purchase->items as $item) {
$item->product->decrement('stock', $item->quantity);
@ -123,12 +149,20 @@ public function destroy(Purchase $purchase): RedirectResponse
});
return redirect()->back()->with('success', 'Data berhasil dihapus');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to delete purchase', [
'purchase_id' => $purchase->id,
]);
return redirect()->back()->with('error', 'Terjadi kesalahan saat menghapus pembelian');
}
}
public function bulkDestroy(Request $request): RedirectResponse
{
$ids = $request->input('ids');
try {
DB::transaction(function () use ($ids) {
$purchases = Purchase::with('items')->whereIn('id', $ids)->get();
foreach ($purchases as $purchase) {
@ -140,5 +174,12 @@ public function bulkDestroy(Request $request): RedirectResponse
});
return redirect()->back()->with('success', 'Data terpilih berhasil dihapus');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to bulk delete purchases', [
'ids' => $ids,
]);
return redirect()->back()->with('error', 'Terjadi kesalahan saat menghapus beberapa pembelian');
}
}
}

37
app/Http/Controllers/Admin/Master/ProductController.php
View File

@ -7,6 +7,7 @@
use App\Http\Requests\Admin\Master\ProductRequest;
use App\Models\Category;
use App\Models\Product;
use App\Support\LogHelper;
use Illuminate\Http\RedirectResponse;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\DB;
@ -35,6 +36,7 @@ public function store(ProductRequest $request): RedirectResponse
{
$validated = $request->validated();
try {
DB::transaction(function () use ($request, $validated) {
$product = Product::create([
'name' => $validated['name'],
@ -67,6 +69,24 @@ public function store(ProductRequest $request): RedirectResponse
});
return redirect()->route('product.index')->with('success', 'Data berhasil disimpan');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to store product', [
'input_summary' => [
'name' => $request->input('name'),
'categories_count' => count($request->input('category_ids', [])),
'prices_keys' => array_keys($request->input('prices', [])),
'has_thumbnail' => $request->hasFile('thumbnail'),
'images_count' => $request->hasFile('images')
? count($request->file('images'))
: 0,
],
]);
return redirect()
->back()
->withInput()
->with('error', 'Terjadi kesalahan saat menyimpan data');
}
}
public function edit(Product $product): Response
@ -83,6 +103,7 @@ public function update(ProductRequest $request, Product $product): RedirectRespo
{
$validated = $request->validated();
try {
DB::transaction(function () use ($request, $product, $validated) {
$product->update([
'name' => $validated['name'],
@ -123,6 +144,22 @@ public function update(ProductRequest $request, Product $product): RedirectRespo
});
return redirect()->route('product.index')->with('success', 'Data berhasil diperbarui');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to update product', [
'product_id' => $product->id,
'input_summary' => [
'name' => $request->input('name'),
'categories_count' => count($request->input('category_ids', [])),
'prices_keys' => array_keys($request->input('prices', [])),
'has_thumbnail' => $request->hasFile('thumbnail'),
],
]);
return redirect()
->back()
->withInput()
->with('error', 'Terjadi kesalahan saat memperbarui data');
}
}
public function destroy(Product $product): RedirectResponse

25
app/Http/Controllers/Admin/Master/UserController.php
View File

@ -5,6 +5,7 @@
use App\Http\Controllers\Controller;
use App\Http\Requests\Admin\Master\UserRequest;
use App\Models\User;
use App\Support\LogHelper;
use Illuminate\Http\RedirectResponse;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\DB;
@ -31,6 +32,7 @@ public function store(UserRequest $request): RedirectResponse
{
$validated = $request->validated();
try {
DB::transaction(function () use ($validated) {
$user = User::create([
'username' => $validated['username'],
@ -50,6 +52,17 @@ public function store(UserRequest $request): RedirectResponse
});
return redirect()->route('user.index')->with('success', 'Data berhasil disimpan');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to store user', [
'username' => $validated['username'],
'email' => $validated['email'],
]);
return redirect()
->back()
->withInput()
->with('error', 'Terjadi kesalahan saat menyimpan data');
}
}
public function edit(User $user): Response
@ -65,6 +78,7 @@ public function update(UserRequest $request, User $user): RedirectResponse
{
$validated = $request->validated();
try {
DB::transaction(function () use ($user, $validated) {
$user->update([
'username' => $validated['username'],
@ -83,6 +97,17 @@ public function update(UserRequest $request, User $user): RedirectResponse
});
return redirect()->route('user.index')->with('success', 'Data berhasil diperbarui');
} catch (\Throwable $e) {
LogHelper::logException($e, 'Failed to update user', [
'user_id' => $user->id,
'username' => $validated['username'],
]);
return redirect()
->back()
->withInput()
->with('error', 'Terjadi kesalahan saat memperbarui data');
}
}
public function resetPassword(User $user): RedirectResponse

4
app/Http/Controllers/Admin/System/LogController.php
View File

@ -39,8 +39,8 @@ private function getLogFiles(): array
$files = File::files($path);
return collect($files)
->map(fn($file) => $file->getFilename())
->filter(fn($filename) => str_ends_with($filename, '.log'))
->map(fn ($file) => $file->getFilename())
->filter(fn ($filename) => str_ends_with($filename, '.log'))
->sortDesc()
->take(10)
->values()

47
app/Support/LogHelper.php Normal file
View File

@ -0,0 +1,47 @@
<?php
namespace App\Support;
use Illuminate\Support\Arr;
use Illuminate\Support\Facades\Log;
use Throwable;
class LogHelper
{
/**
* Log an exception with full request and context details.
*
* @param Throwable $e The exception to log
* @param string $message Custom log message
* @param array $extraContext Additional context data
*/
public static function logException(Throwable $e, string $message, array $extraContext = []): void
{
$request = request();
Log::error($message, [
'message' => $e->getMessage(),
'exception' => get_class($e),
'file' => $e->getFile(),
'line' => $e->getLine(),
// stack trace (take top 10)
'trace' => collect($e->getTrace())->take(10)->toArray(),
// request context
'request' => [
'url' => $request->fullUrl(),
'method' => $request->method(),
'ip' => $request->ip(),
'user_agent' => $request->userAgent(),
'payload' => Arr::except($request->all(), ['password', 'token', 'password_confirmation']),
],
// auth context
'user' => optional(auth()->user())->only(['id', 'email']),
// custom context
'extra' => $extraContext,
]);
}
}