Merge pull request #4428 from robparrott/devel
Properly handle DEFAULT section with ConfigParser in ini_file modules
This commit is contained in:
commit
94fac29bf7
1 changed files with 3 additions and 3 deletions
|
@ -116,9 +116,9 @@ def do_ini(module, filename, section=None, option=None, value=None, state='prese
|
|||
pass
|
||||
|
||||
if state == 'present':
|
||||
if cp.has_section(section) == False:
|
||||
if section.upper() == 'DEFAULT':
|
||||
module.fail_json(msg="[DEFAULT] is an illegal section name")
|
||||
|
||||
# DEFAULT section is always there by DEFAULT, so never try to add it.
|
||||
if cp.has_section(section) == False and section.upper() != 'DEFAULT':
|
||||
|
||||
cp.add_section(section)
|
||||
changed = True
|
||||
|
|
Loading…
Reference in a new issue