loafle/openapi-generator
3
0
Fork 0
forked from loafle/openapi-generator-original
Code Pull Requests Activity

python: use isinstance instead of type() (#16516)

Browse Source 
Use of `isinstance` is preferred (and validated by common linters).
This commit is contained in:
jessemyers-lettuce 2023-09-05 08:45:18 -07:00 committed by GitHub
parent 4b7a808a9f
commit 1b32088c2d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
modules/openapi-generator/src/main/resources/python/api_client.mustache vendored
View File

@ -234,7 +234,7 @@ class ApiClient:
# data needs deserialization or returns HTTP data (deserialized) only # data needs deserialization or returns HTTP data (deserialized) only
if _preload_content or _return_http_data_only: if _preload_content or _return_http_data_only:
response_type = response_types_map.get(str(response_data.status), None) response_type = response_types_map.get(str(response_data.status), None)
if not response_type and type(response_data.status) == int and 100 <= response_data.status <= 599: if not response_type and isinstance(response_data.status, int) and 100 <= response_data.status <= 599:
# if not found, look for '1XX', '2XX', etc. # if not found, look for '1XX', '2XX', etc.
response_type = response_types_map.get(str(response_data.status)[0] + "XX", None) response_type = response_types_map.get(str(response_data.status)[0] + "XX", None)

2
samples/client/echo_api/python/openapi_client/api_client.py
View File

@ -227,7 +227,7 @@ class ApiClient:
# data needs deserialization or returns HTTP data (deserialized) only # data needs deserialization or returns HTTP data (deserialized) only
if _preload_content or _return_http_data_only: if _preload_content or _return_http_data_only:
response_type = response_types_map.get(str(response_data.status), None) response_type = response_types_map.get(str(response_data.status), None)
if not response_type and type(response_data.status) == int and 100 <= response_data.status <= 599: if not response_type and isinstance(response_data.status, int) and 100 <= response_data.status <= 599:
# if not found, look for '1XX', '2XX', etc. # if not found, look for '1XX', '2XX', etc.
response_type = response_types_map.get(str(response_data.status)[0] + "XX", None) response_type = response_types_map.get(str(response_data.status)[0] + "XX", None)

2
samples/openapi3/client/petstore/python-aiohttp/petstore_api/api_client.py
View File

@ -227,7 +227,7 @@ class ApiClient:
# data needs deserialization or returns HTTP data (deserialized) only # data needs deserialization or returns HTTP data (deserialized) only
if _preload_content or _return_http_data_only: if _preload_content or _return_http_data_only:
response_type = response_types_map.get(str(response_data.status), None) response_type = response_types_map.get(str(response_data.status), None)
if not response_type and type(response_data.status) == int and 100 <= response_data.status <= 599: if not response_type and isinstance(response_data.status, int) and 100 <= response_data.status <= 599:
# if not found, look for '1XX', '2XX', etc. # if not found, look for '1XX', '2XX', etc.
response_type = response_types_map.get(str(response_data.status)[0] + "XX", None) response_type = response_types_map.get(str(response_data.status)[0] + "XX", None)

2
samples/openapi3/client/petstore/python/petstore_api/api_client.py
View File

@ -226,7 +226,7 @@ class ApiClient:
# data needs deserialization or returns HTTP data (deserialized) only # data needs deserialization or returns HTTP data (deserialized) only
if _preload_content or _return_http_data_only: if _preload_content or _return_http_data_only:
response_type = response_types_map.get(str(response_data.status), None) response_type = response_types_map.get(str(response_data.status), None)
if not response_type and type(response_data.status) == int and 100 <= response_data.status <= 599: if not response_type and isinstance(response_data.status, int) and 100 <= response_data.status <= 599:
# if not found, look for '1XX', '2XX', etc. # if not found, look for '1XX', '2XX', etc.
response_type = response_types_map.get(str(response_data.status)[0] + "XX", None) response_type = response_types_map.get(str(response_data.status)[0] + "XX", None)