[fix] Construct a proper emails array (fix #39)

This commit is contained in:
Jérôme Lebleu 2015-06-30 21:02:58 +02:00
parent cd85f6b740
commit a46be28b53

View file

@ -613,14 +613,18 @@ function edit_user ()
filter = "(uid="..user..")", filter = "(uid="..user..")",
attrs = {"mail"} attrs = {"mail"}
} do } do
-- Construct proper emails array
local mail_list = {}
local mail_attr = attribs["mail"]
if type(mail_attr) == "string" then
mail_list = { mail_attr }
elseif type(mail_attr) == "table" then
mail_list = mail_attr
end
-- Filter configuration's domain list to keep only -- Filter configuration's domain list to keep only
-- "allowed" domains -- "allowed" domains
for _, domain in ipairs(conf["domains"]) do for _, domain in ipairs(conf["domains"]) do
if type(attribs["mail"] == "string") then
mail_list = { attribs["mail"] }
else
mail_list = attribs["mail"]
end
for k, mail in ipairs(mail_list) do for k, mail in ipairs(mail_list) do
if string.ends(mail, "@"..domain) then if string.ends(mail, "@"..domain) then
if not is_in_table(domains, domain) then if not is_in_table(domains, domain) then
@ -705,10 +709,16 @@ function edit_user ()
} do } do
-- Another user with one of these emails has been found. -- Another user with one of these emails has been found.
if attribs[conf["ldap_identifier"]] and attribs[conf["ldap_identifier"]] ~= user then if attribs[conf["ldap_identifier"]] and attribs[conf["ldap_identifier"]] ~= user then
if type(attribs["mail"]) == "string" then -- Construct proper emails array
attribs["mail"] = {attribs["mail"]} local mail_list = {}
local mail_attr = attribs["mail"]
if type(mail_attr) == "string" then
mail_list = { mail_attr }
elseif type(mail_attr) == "table" then
mail_list = mail_attr
end end
for _, mail in ipairs(attribs["mail"]) do
for _, mail in ipairs(mail_list) do
if is_in_table(mails, mail) then if is_in_table(mails, mail) then
flash("fail", t("mail_already_used").." "..mail) flash("fail", t("mail_already_used").." "..mail)
end end