From 4feb0676c29cac558fc7d7ecebf9e198247564f7 Mon Sep 17 00:00:00 2001 From: Matthew Pollock Date: Thu, 15 Jan 2026 11:12:21 -0500 Subject: [PATCH 1/2] fix error handling in the R client --- .../src/main/resources/r/api.mustache | 13 ++- samples/client/echo_api/r/R/auth_api.R | 20 +++- samples/client/echo_api/r/R/body_api.R | 100 +++++++++++----- samples/client/echo_api/r/R/form_api.R | 30 +++-- samples/client/echo_api/r/R/header_api.R | 10 +- samples/client/echo_api/r/R/path_api.R | 10 +- samples/client/echo_api/r/R/query_api.R | 110 ++++++++++++------ .../petstore/R-httr2-wrapper/R/pet_api.R | 2 - samples/client/petstore/R-httr2/R/pet_api.R | 2 - samples/client/petstore/R/R/pet_api.R | 2 - 10 files changed, 204 insertions(+), 95 deletions(-) diff --git a/modules/openapi-generator/src/main/resources/r/api.mustache b/modules/openapi-generator/src/main/resources/r/api.mustache index 1297561f3973..8887a837e073 100644 --- a/modules/openapi-generator/src/main/resources/r/api.mustache +++ b/modules/openapi-generator/src/main/resources/r/api.mustache @@ -150,7 +150,6 @@ if (typeof(stream_callback) == "closure") { # return void if streaming is enabled return(invisible(NULL)) } - {{/vendorExtensions.x-streaming}} if (local_var_response$status_code >= 200 && local_var_response$status_code <= 299) { return(local_var_response$content) @@ -561,7 +560,7 @@ self$api_client$WriteFile(local_var_resp, data_file) } - ApiResponse$new(content,resp) + ApiResponse$new(content = content, response = resp, status_code = local_var_resp$status_code) {{/isPrimitiveType}} {{^isPrimitiveType}} # save response in a file @@ -609,7 +608,9 @@ {{/useRlangExceptionHandling}} {{/returnExceptionOnFailure}} {{^returnExceptionOnFailure}} - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) {{/returnExceptionOnFailure}} } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { {{#returnExceptionOnFailure}} @@ -626,7 +627,9 @@ {{/useRlangExceptionHandling}} {{/returnExceptionOnFailure}} {{^returnExceptionOnFailure}} - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) {{/returnExceptionOnFailure}} } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { {{#returnExceptionOnFailure}} @@ -643,7 +646,7 @@ {{/useRlangExceptionHandling}} {{/returnExceptionOnFailure}} {{^returnExceptionOnFailure}} - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/auth_api.R b/samples/client/echo_api/r/R/auth_api.R index 8b2999b3758e..7247dc14a564 100644 --- a/samples/client/echo_api/r/R/auth_api.R +++ b/samples/client/echo_api/r/R/auth_api.R @@ -148,11 +148,15 @@ AuthApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -239,11 +243,15 @@ AuthApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/body_api.R b/samples/client/echo_api/r/R/body_api.R index e9213e0dc94f..0d0f2b602c1c 100644 --- a/samples/client/echo_api/r/R/body_api.R +++ b/samples/client/echo_api/r/R/body_api.R @@ -250,11 +250,15 @@ BodyApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -349,11 +353,15 @@ BodyApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -447,11 +455,15 @@ BodyApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -541,11 +553,15 @@ BodyApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -640,11 +656,15 @@ BodyApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -739,11 +759,15 @@ BodyApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -838,11 +862,15 @@ BodyApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -937,11 +965,15 @@ BodyApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -1036,11 +1068,15 @@ BodyApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -1135,11 +1171,15 @@ BodyApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/form_api.R b/samples/client/echo_api/r/R/form_api.R index 1278ba0b2ade..94d5c03c8c6e 100644 --- a/samples/client/echo_api/r/R/form_api.R +++ b/samples/client/echo_api/r/R/form_api.R @@ -181,11 +181,15 @@ FormApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -279,11 +283,15 @@ FormApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -408,11 +416,15 @@ FormApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/header_api.R b/samples/client/echo_api/r/R/header_api.R index 21280ce6b645..f4de20d97ae1 100644 --- a/samples/client/echo_api/r/R/header_api.R +++ b/samples/client/echo_api/r/R/header_api.R @@ -169,11 +169,15 @@ HeaderApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/path_api.R b/samples/client/echo_api/r/R/path_api.R index afe0a36107d0..2105e915396a 100644 --- a/samples/client/echo_api/r/R/path_api.R +++ b/samples/client/echo_api/r/R/path_api.R @@ -184,11 +184,15 @@ PathApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/query_api.R b/samples/client/echo_api/r/R/query_api.R index 7afbf45a2908..c7c882fce06f 100644 --- a/samples/client/echo_api/r/R/query_api.R +++ b/samples/client/echo_api/r/R/query_api.R @@ -290,11 +290,15 @@ QueryApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -401,11 +405,15 @@ QueryApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -512,11 +520,15 @@ QueryApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -607,11 +619,15 @@ QueryApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -702,11 +718,15 @@ QueryApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -798,11 +818,15 @@ QueryApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -894,11 +918,15 @@ QueryApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -989,11 +1017,15 @@ QueryApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -1084,11 +1116,15 @@ QueryApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -1179,11 +1215,15 @@ QueryApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -1283,11 +1323,15 @@ QueryApi <- R6::R6Class( local_var_error_msg <- local_var_resp$response_as_text() if (local_var_resp$status_code >= 300 && local_var_resp$status_code <= 399) { - ApiResponse$new(paste("Server returned ", local_var_resp$status_code, " response status code."), local_var_resp) + ApiResponse$new(content = paste("Server returned ", local_var_resp$status_code, " response status code."), + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 400 && local_var_resp$status_code <= 499) { - ApiResponse$new("API client error", local_var_resp) + ApiResponse$new(content = "API client error", + response = local_var_resp, + status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_resp$response == "") { + if (is.null(local_var_resp$response) || local_var_error_msg == "") { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/petstore/R-httr2-wrapper/R/pet_api.R b/samples/client/petstore/R-httr2-wrapper/R/pet_api.R index 83b4a7141c28..82b9de3e6f64 100644 --- a/samples/client/petstore/R-httr2-wrapper/R/pet_api.R +++ b/samples/client/petstore/R-httr2-wrapper/R/pet_api.R @@ -1028,7 +1028,6 @@ PetApi <- R6::R6Class( if (typeof(stream_callback) == "closure") { # return void if streaming is enabled return(invisible(NULL)) } - if (local_var_response$status_code >= 200 && local_var_response$status_code <= 299) { return(local_var_response$content) } else if (local_var_response$status_code >= 300 && local_var_response$status_code <= 399) { @@ -1165,7 +1164,6 @@ PetApi <- R6::R6Class( if (typeof(stream_callback) == "closure") { # return void if streaming is enabled return(invisible(NULL)) } - if (local_var_response$status_code >= 200 && local_var_response$status_code <= 299) { return(local_var_response$content) } else if (local_var_response$status_code >= 300 && local_var_response$status_code <= 399) { diff --git a/samples/client/petstore/R-httr2/R/pet_api.R b/samples/client/petstore/R-httr2/R/pet_api.R index b60f15378597..79fd10d14940 100644 --- a/samples/client/petstore/R-httr2/R/pet_api.R +++ b/samples/client/petstore/R-httr2/R/pet_api.R @@ -1028,7 +1028,6 @@ PetApi <- R6::R6Class( if (typeof(stream_callback) == "closure") { # return void if streaming is enabled return(invisible(NULL)) } - if (local_var_response$status_code >= 200 && local_var_response$status_code <= 299) { return(local_var_response$content) } else if (local_var_response$status_code >= 300 && local_var_response$status_code <= 399) { @@ -1165,7 +1164,6 @@ PetApi <- R6::R6Class( if (typeof(stream_callback) == "closure") { # return void if streaming is enabled return(invisible(NULL)) } - if (local_var_response$status_code >= 200 && local_var_response$status_code <= 299) { return(local_var_response$content) } else if (local_var_response$status_code >= 300 && local_var_response$status_code <= 399) { diff --git a/samples/client/petstore/R/R/pet_api.R b/samples/client/petstore/R/R/pet_api.R index 40cfdabc2271..34360e6eaf94 100644 --- a/samples/client/petstore/R/R/pet_api.R +++ b/samples/client/petstore/R/R/pet_api.R @@ -1028,7 +1028,6 @@ PetApi <- R6::R6Class( if (typeof(stream_callback) == "closure") { # return void if streaming is enabled return(invisible(NULL)) } - if (local_var_response$status_code >= 200 && local_var_response$status_code <= 299) { return(local_var_response$content) } else if (local_var_response$status_code >= 300 && local_var_response$status_code <= 399) { @@ -1165,7 +1164,6 @@ PetApi <- R6::R6Class( if (typeof(stream_callback) == "closure") { # return void if streaming is enabled return(invisible(NULL)) } - if (local_var_response$status_code >= 200 && local_var_response$status_code <= 299) { return(local_var_response$content) } else if (local_var_response$status_code >= 300 && local_var_response$status_code <= 399) { From 0de41cfc95aba15b940a1bcae70dee24f9aae4af Mon Sep 17 00:00:00 2001 From: Matthew Pollock Date: Thu, 15 Jan 2026 12:26:00 -0500 Subject: [PATCH 2/2] adjust logic for avoiding logical vectors in a conditional statement --- .../src/main/resources/r/api.mustache | 2 +- samples/client/echo_api/r/R/auth_api.R | 4 ++-- samples/client/echo_api/r/R/body_api.R | 20 ++++++++--------- samples/client/echo_api/r/R/form_api.R | 6 ++--- samples/client/echo_api/r/R/header_api.R | 2 +- samples/client/echo_api/r/R/path_api.R | 2 +- samples/client/echo_api/r/R/query_api.R | 22 +++++++++---------- 7 files changed, 29 insertions(+), 29 deletions(-) diff --git a/modules/openapi-generator/src/main/resources/r/api.mustache b/modules/openapi-generator/src/main/resources/r/api.mustache index 8887a837e073..f3004d8f6d4a 100644 --- a/modules/openapi-generator/src/main/resources/r/api.mustache +++ b/modules/openapi-generator/src/main/resources/r/api.mustache @@ -646,7 +646,7 @@ {{/useRlangExceptionHandling}} {{/returnExceptionOnFailure}} {{^returnExceptionOnFailure}} - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/auth_api.R b/samples/client/echo_api/r/R/auth_api.R index 7247dc14a564..c90b3c6eb175 100644 --- a/samples/client/echo_api/r/R/auth_api.R +++ b/samples/client/echo_api/r/R/auth_api.R @@ -156,7 +156,7 @@ AuthApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -251,7 +251,7 @@ AuthApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/body_api.R b/samples/client/echo_api/r/R/body_api.R index 0d0f2b602c1c..128ddf44bb5b 100644 --- a/samples/client/echo_api/r/R/body_api.R +++ b/samples/client/echo_api/r/R/body_api.R @@ -258,7 +258,7 @@ BodyApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -361,7 +361,7 @@ BodyApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -463,7 +463,7 @@ BodyApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -561,7 +561,7 @@ BodyApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -664,7 +664,7 @@ BodyApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -767,7 +767,7 @@ BodyApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -870,7 +870,7 @@ BodyApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -973,7 +973,7 @@ BodyApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -1076,7 +1076,7 @@ BodyApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -1179,7 +1179,7 @@ BodyApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/form_api.R b/samples/client/echo_api/r/R/form_api.R index 94d5c03c8c6e..524941953044 100644 --- a/samples/client/echo_api/r/R/form_api.R +++ b/samples/client/echo_api/r/R/form_api.R @@ -189,7 +189,7 @@ FormApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -291,7 +291,7 @@ FormApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -424,7 +424,7 @@ FormApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/header_api.R b/samples/client/echo_api/r/R/header_api.R index f4de20d97ae1..2726d74ec82b 100644 --- a/samples/client/echo_api/r/R/header_api.R +++ b/samples/client/echo_api/r/R/header_api.R @@ -177,7 +177,7 @@ HeaderApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/path_api.R b/samples/client/echo_api/r/R/path_api.R index 2105e915396a..588dda764939 100644 --- a/samples/client/echo_api/r/R/path_api.R +++ b/samples/client/echo_api/r/R/path_api.R @@ -192,7 +192,7 @@ PathApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) diff --git a/samples/client/echo_api/r/R/query_api.R b/samples/client/echo_api/r/R/query_api.R index c7c882fce06f..ddc26a9c2e3e 100644 --- a/samples/client/echo_api/r/R/query_api.R +++ b/samples/client/echo_api/r/R/query_api.R @@ -298,7 +298,7 @@ QueryApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -413,7 +413,7 @@ QueryApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -528,7 +528,7 @@ QueryApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -627,7 +627,7 @@ QueryApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -726,7 +726,7 @@ QueryApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -826,7 +826,7 @@ QueryApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -926,7 +926,7 @@ QueryApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -1025,7 +1025,7 @@ QueryApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -1124,7 +1124,7 @@ QueryApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -1223,7 +1223,7 @@ QueryApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp) @@ -1331,7 +1331,7 @@ QueryApi <- R6::R6Class( response = local_var_resp, status_code = local_var_resp$status_code) } else if (local_var_resp$status_code >= 500 && local_var_resp$status_code <= 599) { - if (is.null(local_var_resp$response) || local_var_error_msg == "") { + if (is.null(local_var_resp$response) || all(local_var_resp$response == "")) { local_var_resp$response <- "API server error" } return(local_var_resp)