diff options
| author | Ted Trask <ttrask01@yahoo.com> | 2010-09-28 07:15:36 +0000 |
|---|---|---|
| committer | Ted Trask <ttrask01@yahoo.com> | 2010-09-28 07:15:36 +0000 |
| commit | b202af98292d6d7b5053e5a934b916153ce89af9 (patch) | |
| tree | bd8a6e2a973d01317fbf462e13d98932fbed2822 /app/acf-util/password-status-html.lsp | |
| parent | 09344bc273f7147a9f81168096d2f8bb3d0fdedc (diff) | |
| download | acf-core-b202af98292d6d7b5053e5a934b916153ce89af9.tar.bz2 acf-core-b202af98292d6d7b5053e5a934b916153ce89af9.tar.xz | |
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.
Diffstat (limited to 'app/acf-util/password-status-html.lsp')
| -rw-r--r-- | app/acf-util/password-status-html.lsp | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/app/acf-util/password-status-html.lsp b/app/acf-util/password-status-html.lsp index 90672ad..01eafe7 100644 --- a/app/acf-util/password-status-html.lsp +++ b/app/acf-util/password-status-html.lsp @@ -11,14 +11,9 @@ </form> <H2>Existing account</H2> <DL> -<% local users = {} -for name,user in pairs(form.value) do - users[#users+1] = name -end -table.sort(users) -for i,name in ipairs(users) do - user = form.value[name] %> - <DT><IMG SRC='<%= html.html_escape(page_info.wwwprefix..page_info.staticdir) %>/tango/16x16/apps/system-users.png' HEIGHT='16' WIDTH='16'> <%= html.html_escape(name) %></DT> +<% for i,user in ipairs(form.value) do + local name = html.html_escape(user.value.userid.value) %> + <DT><IMG SRC='<%= html.html_escape(page_info.wwwprefix..page_info.staticdir) %>/tango/16x16/apps/system-users.png' HEIGHT='16' WIDTH='16'> <%= name %></DT> <DD><TABLE> <TR> <TD STYLE='border:none;'><B><%= html.html_escape(user.value.userid.label) %></B></TD> @@ -32,9 +27,9 @@ for i,name in ipairs(users) do </TR><TR> <TD STYLE='border:none;'><B>Option</B></TD> <TD STYLE='border:none;'> - [<A HREF='edituser?userid=<%= html.html_escape(name) %>&redir=<%= html.html_escape(page_info.orig_action) %>'>Edit this account</A>] - [<A HREF='deleteuser?userid=<%= html.html_escape(name) %>'>Delete this account</A>] - [<A HREF='<%= html.html_escape(page_info.script) %>/acf-util/roles/viewuserroles?userid=<%= html.html_escape(name) %>'>View roles for this account</A>] + [<A HREF='edituser?userid=<%= name %>&redir=<%= html.html_escape(page_info.orig_action) %>'>Edit this account</A>] + [<A HREF='deleteuser?userid=<%= name %>'>Delete this account</A>] + [<A HREF='<%= html.html_escape(page_info.script) %>/acf-util/roles/viewuserroles?userid=<%= name %>'>View roles for this account</A>] </TD> </TR> </TABLE></DD> |