Skip to content

Commit

Permalink
Fix per operation pythonic override. Closes #1078.
Browse files Browse the repository at this point in the history
  • Loading branch information
jshcodes committed Nov 21, 2023
1 parent e4aec1f commit acddc5e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,8 @@
- `installation_tokens.py`
+ Fixed: API operations generating leveraging the raw attribute are not properly displaying results when leveraging result object expansion. Closes #1076.
- `_result/_result.py`
+ Fixed: Per-operation pythonic override is not working as expected. Closes #1078.
- `_util/_functions.py`

# Other
+ Changed: Updated field mapping for Uber Class path variables to a cleaner solution.
Expand Down
5 changes: 4 additions & 1 deletion src/falconpy/_util/_functions.py
Original file line number Diff line number Diff line change
Expand Up @@ -667,6 +667,9 @@ def process_service_request(calling_object, # pylint: disable=R0914 # (19/15)
calling_object.pythonic
)
expand_result = passed_keywords.get("expand_result", False) if passed_keywords else kwargs.get("expand_result", False)
do_pythonic = calling_object.pythonic
if passed_keywords.get("pythonic", None) is not None:
do_pythonic = passed_keywords.get("pythonic")
new_keywords = {
"caller": calling_object,
"method": target_endpoint[1],
Expand All @@ -681,7 +684,7 @@ def process_service_request(calling_object, # pylint: disable=R0914 # (19/15)
"body_required": kwargs.get("body_required", None),
"expand_result": expand_result,
"container": container,
"pythonic": calling_object.pythonic,
"pythonic": do_pythonic,
"perform": True
}

Expand Down

0 comments on commit acddc5e

Please sign in to comment.