Port postgresql module to python3 (#4579)

Iteritems is no longer a dict method in Python3, replace it with
the six wrapper.
reviewable/pr18780/r1
Michael Scherer 8 years ago committed by Toshio Kuratomi
parent c091a3e9ac
commit 2e7cd6e02a

@ -114,6 +114,7 @@ except ImportError:
postgresqldb_found = False postgresqldb_found = False
else: else:
postgresqldb_found = True postgresqldb_found = True
from ansible.module_utils.six import iteritems
class NotSupportedError(Exception): class NotSupportedError(Exception):
pass pass
@ -261,7 +262,7 @@ def main():
"login_password":"password", "login_password":"password",
"port":"port" "port":"port"
} }
kw = dict( (params_map[k], v) for (k, v) in module.params.iteritems() kw = dict( (params_map[k], v) for (k, v) in iteritems(module.params)
if k in params_map and v != '' ) if k in params_map and v != '' )
# If a login_unix_socket is specified, incorporate it here. # If a login_unix_socket is specified, incorporate it here.

@ -170,6 +170,7 @@ except ImportError:
postgresqldb_found = False postgresqldb_found = False
else: else:
postgresqldb_found = True postgresqldb_found = True
from ansible.module_utils.six import iteritems
_flags = ('SUPERUSER', 'CREATEROLE', 'CREATEUSER', 'CREATEDB', 'INHERIT', 'LOGIN', 'REPLICATION') _flags = ('SUPERUSER', 'CREATEROLE', 'CREATEUSER', 'CREATEDB', 'INHERIT', 'LOGIN', 'REPLICATION')
VALID_FLAGS = frozenset(itertools.chain(_flags, ('NO%s' % f for f in _flags))) VALID_FLAGS = frozenset(itertools.chain(_flags, ('NO%s' % f for f in _flags)))
@ -433,7 +434,7 @@ def revoke_privileges(cursor, user, privs):
changed = False changed = False
for type_ in privs: for type_ in privs:
for name, privileges in privs[type_].iteritems(): for name, privileges in iteritems(privs[type_]):
# Check that any of the privileges requested to be removed are # Check that any of the privileges requested to be removed are
# currently granted to the user # currently granted to the user
differences = check_funcs[type_](cursor, user, name, privileges) differences = check_funcs[type_](cursor, user, name, privileges)
@ -451,7 +452,7 @@ def grant_privileges(cursor, user, privs):
changed = False changed = False
for type_ in privs: for type_ in privs:
for name, privileges in privs[type_].iteritems(): for name, privileges in iteritems(privs[type_]):
# Check that any of the privileges requested for the user are # Check that any of the privileges requested for the user are
# currently missing # currently missing
differences = check_funcs[type_](cursor, user, name, privileges) differences = check_funcs[type_](cursor, user, name, privileges)
@ -595,7 +596,7 @@ def main():
"port":"port", "port":"port",
"db":"database" "db":"database"
} }
kw = dict( (params_map[k], v) for (k, v) in module.params.iteritems() kw = dict( (params_map[k], v) for (k, v) in iteritems(module.params)
if k in params_map and v != "" ) if k in params_map and v != "" )
# If a login_unix_socket is specified, incorporate it here. # If a login_unix_socket is specified, incorporate it here.

Loading…
Cancel
Save