Fix for #31477: return 'dest' from copy when file wasn't changed

(cherry picked from commit 89ad40b129)
This commit is contained in:
Tomasz Kontusz 2018-02-04 12:58:35 +01:00 committed by Toshio Kuratomi
parent e55e5cf23e
commit c1c5433733

View file

@ -538,13 +538,13 @@ class ActionModule(ActionBase):
changed = changed or module_return.get('changed', False) 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: if module_executed and len(source_files['files']) == 1:
result.update(module_return) 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: else:
result.update(dict(dest=dest, src=source, changed=changed)) result.update(dict(dest=dest, src=source, changed=changed))