From 89ad40b1299bb99962533b6a046aa153e711ab5e Mon Sep 17 00:00:00 2001 From: Tomasz Kontusz Date: Sun, 4 Feb 2018 12:58:35 +0100 Subject: [PATCH] Fix for #31477: return 'dest' from copy when file wasn't changed --- lib/ansible/plugins/action/copy.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/lib/ansible/plugins/action/copy.py b/lib/ansible/plugins/action/copy.py index a5cbecf35b..28c606fb6c 100644 --- a/lib/ansible/plugins/action/copy.py +++ b/lib/ansible/plugins/action/copy.py @@ -538,13 +538,13 @@ class ActionModule(ActionBase): changed = changed or module_return.get('changed', False) - # the file module returns the file path as 'path', but - # the copy module uses 'dest', so add it if it's not there - if 'path' in module_return and 'dest' not in module_return: - module_return['dest'] = module_return['path'] - if module_executed and len(source_files['files']) == 1: result.update(module_return) + + # the file module returns the file path as 'path', but + # the copy module uses 'dest', so add it if it's not there + if 'path' in result and 'dest' not in result: + result['dest'] = result['path'] else: result.update(dict(dest=dest, src=source, changed=changed))