Skip to content

Commit

Permalink
fix for empty data in diff
Browse files Browse the repository at this point in the history
  • Loading branch information
ansibleguy committed Aug 19, 2024
1 parent 5713513 commit e7896f2
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion plugins/module_utils/helper/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@

def diff_remove_empty(diff: dict) -> dict:
d = diff.copy()
for k in diff.keys():
for k in diff:
if len(diff[k]) == 0:
d.pop(k)

Expand Down
3 changes: 2 additions & 1 deletion plugins/module_utils/helper/wrapper.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,12 @@ def module_process(instance: BaseModule):
if hasattr(instance, 's'):
instance.s.close()

instance.r['diff'] = diff_remove_empty(instance.r['diff'])


def module_wrapper(instance: BaseModule):
if instance.m.params['profiling'] or instance.m.params['debug']:
module_name = inspect_getfile(inspect_stack()[1][0]).rsplit('/', 1)[1].rsplit('.', 1)[0]
return profiler(check=module_process, module_name=module_name, kwargs={'instance': instance})

instance.r['diff'] = diff_remove_empty(instance.r['diff'])
return module_process(instance)
2 changes: 0 additions & 2 deletions plugins/modules/ids_policy_rule.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,6 @@ def run_module():
)

module_wrapper(Rule(module=module, result=result))

result['diff'] = diff_remove_empty(result['diff'])
module.exit_json(**result)


Expand Down

0 comments on commit e7896f2

Please sign in to comment.