xml module: Add backup to pretty_print too (#28604)

pull/28460/head
Dag Wieers 7 years ago committed by GitHub
parent b34cfd03b3
commit 7a182fc43e

@ -549,6 +549,8 @@ def pretty(module, tree):
if xml_string != xml_content.read(): if xml_string != xml_content.read():
result['changed'] = True result['changed'] = True
if not module.check_mode: if not module.check_mode:
if module.params['backup']:
result['backup_file'] = module.backup_local(module.params['path'])
tree.write(xml_file, xml_declaration=True, encoding='UTF-8', pretty_print=module.params['pretty_print']) tree.write(xml_file, xml_declaration=True, encoding='UTF-8', pretty_print=module.params['pretty_print'])
finally: finally:
xml_content.close() xml_content.close()
@ -647,12 +649,11 @@ def finish(module, tree, xpath, namespaces, changed=False, msg="", hitcount=0, m
after=etree.tostring(tree, xml_declaration=True, encoding='UTF-8', pretty_print=module.params['pretty_print']), after=etree.tostring(tree, xml_declaration=True, encoding='UTF-8', pretty_print=module.params['pretty_print']),
) )
if module.params['path']: if module.params['path'] and not module.check_mode:
if not module.check_mode: if module.params['backup']:
if module.params['backup']: result['backup_file'] = module.backup_local(module.params['path'])
result['backup_file'] = module.backup_local(module.params['path'])
tree.write(module.params['path'], xml_declaration=True, encoding='UTF-8', pretty_print=module.params['pretty_print']) tree.write(module.params['path'], xml_declaration=True, encoding='UTF-8', pretty_print=module.params['pretty_print'])
if module.params['xmlstring']: if module.params['xmlstring']:
result['xmlstring'] = etree.tostring(tree, xml_declaration=True, encoding='UTF-8', pretty_print=module.params['pretty_print']) result['xmlstring'] = etree.tostring(tree, xml_declaration=True, encoding='UTF-8', pretty_print=module.params['pretty_print'])

Loading…
Cancel
Save