From db292e08b9ec1cfe36f166ed032e1d0115148520 Mon Sep 17 00:00:00 2001 From: Tristan Fisher Date: Fri, 15 May 2015 17:09:54 -0400 Subject: [PATCH 1/2] standardizes bools in argument_spec --- files/file.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/files/file.py b/files/file.py index 8da87b0707e..3b4b5a466e6 100644 --- a/files/file.py +++ b/files/file.py @@ -34,7 +34,7 @@ module: file version_added: "historical" short_description: Sets attributes of files extends_documentation_fragment: files -description: +description: - Sets attributes of files, symlinks, and directories, or removes files/symlinks/directories. Many other modules support the same options as the M(file) module - including M(copy), M(template), and M(assemble). @@ -48,7 +48,7 @@ options: - 'path to the file being managed. Aliases: I(dest), I(name)' required: true default: [] - aliases: ['dest', 'name'] + aliases: ['dest', 'name'] state: description: - If C(directory), all immediate subdirectories will be created if they @@ -82,7 +82,7 @@ options: default: "no" choices: [ "yes", "no" ] description: - - 'force the creation of the symlinks in two cases: the source file does + - 'force the creation of the symlinks in two cases: the source file does not exist (but will appear later); the destination exists and is a file (so, we need to unlink the "path" file and create symlink to the "src" file in place of it).' ''' @@ -150,10 +150,10 @@ def main(): state = dict(choices=['file','directory','link','hard','touch','absent'], default=None), path = dict(aliases=['dest', 'name'], required=True), original_basename = dict(required=False), # Internal use only, for recursive ops - recurse = dict(default='no', type='bool'), - force = dict(required=False,default=False,type='bool'), + recurse = dict(default=False, type='bool'), + force = dict(required=False, default=False, type='bool'), diff_peek = dict(default=None), - validate = dict(required=False, default=None), + validate = dict(required=False, default=None, type='bool'), src = dict(required=False, default=None), ), add_file_common_args=True, From 90b34e2b67f6c32904a20d5edec9ffdae898cc88 Mon Sep 17 00:00:00 2001 From: Tristan Fisher Date: Mon, 18 May 2015 18:44:31 -0400 Subject: [PATCH 2/2] removes bool type from validate. --- files/file.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/files/file.py b/files/file.py index 3b4b5a466e6..fb3241b4241 100644 --- a/files/file.py +++ b/files/file.py @@ -153,7 +153,7 @@ def main(): recurse = dict(default=False, type='bool'), force = dict(required=False, default=False, type='bool'), diff_peek = dict(default=None), - validate = dict(required=False, default=None, type='bool'), + validate = dict(required=False, default=None), src = dict(required=False, default=None), ), add_file_common_args=True,