From b202af98292d6d7b5053e5a934b916153ce89af9 Mon Sep 17 00:00:00 2001 From: Ted Trask Date: Tue, 28 Sep 2010 07:15:36 +0000 Subject: Made major changes to authenticator and roles to improve efficiency. Changed authenticator.lua to request individual users, rather than always loading everyone. Modified get_userinfo and removed get_userinfo_roles and get_userinfo_skin to remove unnecessary cfe processing. This was especially bad when retrieving the cfe options for skins and roles. This does break the interface used by other modules like acf-tinydns. Validation and cfe processing were moved into password-model, where they belong. Added global variables to authenticator and roles libraries so they can reuse data that was already generated. Modified logon-controller to only check for zero users if logon fails. --- app/acf-util/roles-model.lua | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'app/acf-util/roles-model.lua') diff --git a/app/acf-util/roles-model.lua b/app/acf-util/roles-model.lua index 6b12dd9..e12d49a 100644 --- a/app/acf-util/roles-model.lua +++ b/app/acf-util/roles-model.lua @@ -36,7 +36,8 @@ end -- Return roles/permissions for specified user get_user_roles = function(self, userid) - rls = cfe({ type="list", value=authenticator.get_userinfo_roles(self, userid).value, label="Roles" }) + local userinfo = authenticator.get_userinfo(self, userid) or {} + rls = cfe({ type="list", value=userinfo.roles or {}, label="Roles" }) permissions = cfe({ type="table", value=roles.get_roles_perm(self, rls.value), label="Permissions" }) return cfe({ type="group", value={roles=rls, permissions=permissions} }) end -- cgit v1.2.3