From ebc1794b56233ef32a0f008aa4b57c06a5e7381d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=EA=B6=8C=ED=98=81=EC=84=B1?= Date: Wed, 21 Jan 2026 10:29:59 +0900 Subject: [PATCH] =?UTF-8?q?fix:=20ApiResponse::handle()=20=ED=98=B8?= =?UTF-8?q?=EC=B6=9C=20=ED=8C=A8=ED=84=B4=20=EC=88=98=EC=A0=95=20(callable?= =?UTF-8?q?=20=EC=82=AC=EC=9A=A9)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../Controllers/Api/V1/CalendarController.php | 19 ++++++++---------- .../Api/V1/EntertainmentController.php | 6 +++--- .../Api/V1/StatusBoardController.php | 9 +++------ .../Api/V1/TodayIssueController.php | 7 ++++--- app/Http/Controllers/Api/V1/VatController.php | 6 +++--- .../Controllers/Api/V1/WelfareController.php | 20 +++++++++---------- 6 files changed, 31 insertions(+), 36 deletions(-) diff --git a/app/Http/Controllers/Api/V1/CalendarController.php b/app/Http/Controllers/Api/V1/CalendarController.php index f60837c..5197c75 100644 --- a/app/Http/Controllers/Api/V1/CalendarController.php +++ b/app/Http/Controllers/Api/V1/CalendarController.php @@ -42,16 +42,13 @@ public function summary(Request $request) $type = $validated['type'] ?? null; $departmentFilter = $validated['department_filter'] ?? 'all'; - $data = $this->calendarService->getSchedules( - $startDate, - $endDate, - $type, - $departmentFilter - ); - - return ApiResponse::handle( - data: $data, - message: __('message.fetched') - ); + return ApiResponse::handle(function () use ($startDate, $endDate, $type, $departmentFilter) { + return $this->calendarService->getSchedules( + $startDate, + $endDate, + $type, + $departmentFilter + ); + }, __('message.fetched')); } } \ No newline at end of file diff --git a/app/Http/Controllers/Api/V1/EntertainmentController.php b/app/Http/Controllers/Api/V1/EntertainmentController.php index 0b236a9..a5b1f12 100644 --- a/app/Http/Controllers/Api/V1/EntertainmentController.php +++ b/app/Http/Controllers/Api/V1/EntertainmentController.php @@ -32,8 +32,8 @@ public function summary(Request $request): JsonResponse $year = $request->query('year') ? (int) $request->query('year') : null; $quarter = $request->query('quarter') ? (int) $request->query('quarter') : null; - $data = $this->entertainmentService->getSummary($limitType, $companyType, $year, $quarter); - - return ApiResponse::handle($data, __('message.fetched')); + return ApiResponse::handle(function () use ($limitType, $companyType, $year, $quarter) { + return $this->entertainmentService->getSummary($limitType, $companyType, $year, $quarter); + }, __('message.fetched')); } } \ No newline at end of file diff --git a/app/Http/Controllers/Api/V1/StatusBoardController.php b/app/Http/Controllers/Api/V1/StatusBoardController.php index f4abd06..7d6311b 100644 --- a/app/Http/Controllers/Api/V1/StatusBoardController.php +++ b/app/Http/Controllers/Api/V1/StatusBoardController.php @@ -20,11 +20,8 @@ public function __construct( */ public function summary() { - $data = $this->statusBoardService->summary(); - - return ApiResponse::handle( - data: $data, - message: __('message.fetched') - ); + return ApiResponse::handle(function () { + return $this->statusBoardService->summary(); + }, __('message.fetched')); } } \ No newline at end of file diff --git a/app/Http/Controllers/Api/V1/TodayIssueController.php b/app/Http/Controllers/Api/V1/TodayIssueController.php index 2766773..a4330d3 100644 --- a/app/Http/Controllers/Api/V1/TodayIssueController.php +++ b/app/Http/Controllers/Api/V1/TodayIssueController.php @@ -19,9 +19,10 @@ public function __construct( */ public function summary(Request $request): JsonResponse { - $limit = $request->input('limit', 30); - $data = $this->todayIssueService->summary((int) $limit); + $limit = (int) $request->input('limit', 30); - return ApiResponse::handle(['data' => $data], __('message.fetched')); + return ApiResponse::handle(function () use ($limit) { + return $this->todayIssueService->summary($limit); + }, __('message.fetched')); } } \ No newline at end of file diff --git a/app/Http/Controllers/Api/V1/VatController.php b/app/Http/Controllers/Api/V1/VatController.php index 89f90a6..36f4d1f 100644 --- a/app/Http/Controllers/Api/V1/VatController.php +++ b/app/Http/Controllers/Api/V1/VatController.php @@ -31,8 +31,8 @@ public function summary(Request $request): JsonResponse $year = $request->query('year') ? (int) $request->query('year') : null; $period = $request->query('period') ? (int) $request->query('period') : null; - $data = $this->vatService->getSummary($periodType, $year, $period); - - return ApiResponse::handle($data, __('message.fetched')); + return ApiResponse::handle(function () use ($periodType, $year, $period) { + return $this->vatService->getSummary($periodType, $year, $period); + }, __('message.fetched')); } } \ No newline at end of file diff --git a/app/Http/Controllers/Api/V1/WelfareController.php b/app/Http/Controllers/Api/V1/WelfareController.php index 8b7b244..4392ae5 100644 --- a/app/Http/Controllers/Api/V1/WelfareController.php +++ b/app/Http/Controllers/Api/V1/WelfareController.php @@ -38,15 +38,15 @@ public function summary(Request $request): JsonResponse $year = $request->query('year') ? (int) $request->query('year') : null; $quarter = $request->query('quarter') ? (int) $request->query('quarter') : null; - $data = $this->welfareService->getSummary( - $limitType, - $calculationType, - $fixedAmountPerMonth, - $ratio, - $year, - $quarter - ); - - return ApiResponse::handle($data, __('message.fetched')); + return ApiResponse::handle(function () use ($limitType, $calculationType, $fixedAmountPerMonth, $ratio, $year, $quarter) { + return $this->welfareService->getSummary( + $limitType, + $calculationType, + $fixedAmountPerMonth, + $ratio, + $year, + $quarter + ); + }, __('message.fetched')); } } \ No newline at end of file