From b72f93faab11bec71d5cfbcf8d6abc1af5827cfa Mon Sep 17 00:00:00 2001 From: Zach Silveira Date: Sun, 26 Oct 2025 11:25:44 -0400 Subject: [PATCH] fix api call booleans not being used --- app/Http/Controllers/Api/ApplicationsController.php | 13 ++++++------- app/Http/Controllers/Api/DatabasesController.php | 13 ++++++------- app/Http/Controllers/Api/ServicesController.php | 8 ++++---- 3 files changed, 16 insertions(+), 18 deletions(-) diff --git a/app/Http/Controllers/Api/ApplicationsController.php b/app/Http/Controllers/Api/ApplicationsController.php index e9c52d2f50..6b4f1efeeb 100644 --- a/app/Http/Controllers/Api/ApplicationsController.php +++ b/app/Http/Controllers/Api/ApplicationsController.php @@ -1893,7 +1893,6 @@ public function logs_by_uuid(Request $request) public function delete_by_uuid(Request $request) { $teamId = getTeamIdFromToken(); - $cleanup = filter_var($request->query->get('cleanup', true), FILTER_VALIDATE_BOOLEAN); if (is_null($teamId)) { return invalidTokenResponse(); } @@ -1912,10 +1911,10 @@ public function delete_by_uuid(Request $request) DeleteResourceJob::dispatch( resource: $application, - deleteVolumes: $request->query->get('delete_volumes', true), - deleteConnectedNetworks: $request->query->get('delete_connected_networks', true), - deleteConfigurations: $request->query->get('delete_configurations', true), - dockerCleanup: $request->query->get('docker_cleanup', true) + deleteVolumes: $request->boolean('delete_volumes', true), + deleteConnectedNetworks: $request->boolean('delete_connected_networks', true), + deleteConfigurations: $request->boolean('delete_configurations', true), + dockerCleanup: $request->boolean('docker_cleanup', true) ); return response()->json([ @@ -3155,8 +3154,8 @@ public function action_deploy(Request $request) if (is_null($teamId)) { return invalidTokenResponse(); } - $force = $request->query->get('force') ?? false; - $instant_deploy = $request->query->get('instant_deploy') ?? false; + $force = $request->boolean('force', false); + $instant_deploy = $request->boolean('instant_deploy', false); $uuid = $request->route('uuid'); if (! $uuid) { return response()->json(['message' => 'UUID is required.'], 400); diff --git a/app/Http/Controllers/Api/DatabasesController.php b/app/Http/Controllers/Api/DatabasesController.php index 46282fddb2..73690a05be 100644 --- a/app/Http/Controllers/Api/DatabasesController.php +++ b/app/Http/Controllers/Api/DatabasesController.php @@ -2133,7 +2133,6 @@ public function create_database(Request $request, NewDatabaseTypes $type) public function delete_by_uuid(Request $request) { $teamId = getTeamIdFromToken(); - $cleanup = filter_var($request->query->get('cleanup', true), FILTER_VALIDATE_BOOLEAN); if (is_null($teamId)) { return invalidTokenResponse(); } @@ -2149,10 +2148,10 @@ public function delete_by_uuid(Request $request) DeleteResourceJob::dispatch( resource: $database, - deleteVolumes: $request->query->get('delete_volumes', true), - deleteConnectedNetworks: $request->query->get('delete_connected_networks', true), - deleteConfigurations: $request->query->get('delete_configurations', true), - dockerCleanup: $request->query->get('docker_cleanup', true) + deleteVolumes: $request->boolean('delete_volumes', true), + deleteConnectedNetworks: $request->boolean('delete_connected_networks', true), + deleteConfigurations: $request->boolean('delete_configurations', true), + dockerCleanup: $request->boolean('docker_cleanup', true) ); return response()->json([ @@ -2243,7 +2242,7 @@ public function delete_backup_by_uuid(Request $request) return response()->json(['message' => 'Backup configuration not found.'], 404); } - $deleteS3 = filter_var($request->query->get('delete_s3', false), FILTER_VALIDATE_BOOLEAN); + $deleteS3 = $request->boolean('delete_s3', false); try { DB::beginTransaction(); @@ -2376,7 +2375,7 @@ public function delete_execution_by_uuid(Request $request) return response()->json(['message' => 'Backup execution not found.'], 404); } - $deleteS3 = filter_var($request->query->get('delete_s3', false), FILTER_VALIDATE_BOOLEAN); + $deleteS3 = $request->boolean('delete_s3', false); try { if ($execution->filename) { diff --git a/app/Http/Controllers/Api/ServicesController.php b/app/Http/Controllers/Api/ServicesController.php index b3565a9335..2c4d0d3610 100644 --- a/app/Http/Controllers/Api/ServicesController.php +++ b/app/Http/Controllers/Api/ServicesController.php @@ -649,10 +649,10 @@ public function delete_by_uuid(Request $request) DeleteResourceJob::dispatch( resource: $service, - deleteVolumes: $request->query->get('delete_volumes', true), - deleteConnectedNetworks: $request->query->get('delete_connected_networks', true), - deleteConfigurations: $request->query->get('delete_configurations', true), - dockerCleanup: $request->query->get('docker_cleanup', true) + deleteVolumes: $request->boolean('delete_volumes', true), + deleteConnectedNetworks: $request->boolean('delete_connected_networks', true), + deleteConfigurations: $request->boolean('delete_configurations', true), + dockerCleanup: $request->boolean('docker_cleanup', true) ); return response()->json([