Commit 64fa95ca by Marcus McCurdy

Improves lineinfile's error checking

Line in file would raise a TypeError exception if
the 'line' key was in params, but its value was None.
parent 266d2008
...@@ -305,7 +305,7 @@ def main(): ...@@ -305,7 +305,7 @@ def main():
module.fail_json(rc=256, msg='Destination %s is a directory !' % dest) module.fail_json(rc=256, msg='Destination %s is a directory !' % dest)
if params['state'] == 'present': if params['state'] == 'present':
if 'line' not in params: if params.get('line') is None:
module.fail_json(msg='line= is required with state=present') module.fail_json(msg='line= is required with state=present')
# Deal with the insertafter default value manually, to avoid errors # Deal with the insertafter default value manually, to avoid errors
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment