[enh] code simplification

This commit is contained in:
Julien Malik 2016-09-13 09:36:41 +02:00
parent 00b41a03b0
commit 1900a28bf3

View file

@ -59,20 +59,20 @@ def alias_create(auth, alias, mailforward):
}) })
# Check that the mail domain exists # Check that the mail domain exists
if alias[alias.find('@')+1:] not in domain_list(auth)['domains']: alias_domain = alias[alias.find('@')+1:]
if alias_domain not in domain_list(auth)['domains']:
raise MoulinetteError(errno.EINVAL, raise MoulinetteError(errno.EINVAL,
m18n.n('mail_domain_unknown', m18n.n('mail_domain_unknown',
alias[alias.find('@')+1:])) alias_domain))
# Adapt values for LDAP # Adapt values for LDAP
rdn = 'mail=%s,ou=aliases' % alias rdn = 'mail=%s,ou=aliases' % alias
attr_dict = { attr_dict = {
'objectClass' : ['mailAccount', 'mailAlias'], 'objectClass' : ['mailAccount', 'mailAlias'],
'mail' : alias 'mail' : alias,
'maildrop' : mailforward.split(",")
} }
attr_dict['maildrop'] = mailforward.split(",")
if auth.add(rdn, attr_dict): if auth.add(rdn, attr_dict):
msignals.display(m18n.n('alias_created'), 'success') msignals.display(m18n.n('alias_created'), 'success')
return { 'alias' : alias, 'mailforward' : attr_dict['maildrop'] } return { 'alias' : alias, 'mailforward' : attr_dict['maildrop'] }