X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=swt%2Forg.argeo.app.ui%2Fsrc%2Forg%2Fargeo%2Fapp%2Fui%2Fpeople%2FUsersPart.java;h=c615ebff81aa174d0e1d223e4943ea369e9c0bd7;hb=3757251d0be9b74dda727c51df355be9d7cbb3bf;hp=d82b227d709887e9441e2c348679ca2cbcb22145;hpb=0b492f2cfb8a24ec84e1dd52825624f5e9ab7e81;p=gpl%2Fargeo-suite.git

diff --git a/swt/org.argeo.app.ui/src/org/argeo/app/ui/people/UsersPart.java b/swt/org.argeo.app.ui/src/org/argeo/app/ui/people/UsersPart.java
index d82b227..c615ebf 100644
--- a/swt/org.argeo.app.ui/src/org/argeo/app/ui/people/UsersPart.java
+++ b/swt/org.argeo.app.ui/src/org/argeo/app/ui/people/UsersPart.java
@@ -5,19 +5,13 @@ import java.util.List;
 
 import org.argeo.api.acr.Content;
 import org.argeo.api.acr.ContentSession;
-import org.argeo.api.acr.ldap.LdapAttrs;
-import org.argeo.api.acr.ldap.LdapObjs;
+import org.argeo.api.acr.ldap.LdapObj;
 import org.argeo.api.cms.directory.CmsUserManager;
 import org.argeo.api.cms.directory.HierarchyUnit;
 import org.argeo.api.cms.directory.UserDirectory;
-import org.argeo.api.cms.ux.CmsIcon;
-import org.argeo.app.ui.SuiteIcon;
 import org.argeo.cms.acr.ContentUtils;
-import org.argeo.cms.auth.UserAdminUtils;
-import org.argeo.cms.ux.widgets.Column;
 import org.argeo.cms.ux.widgets.DefaultTabularPart;
 import org.osgi.service.useradmin.Role;
-import org.osgi.service.useradmin.User;
 
 public class UsersPart extends DefaultTabularPart<HierarchyUnit, Content> {
 	private ContentSession contentSession;
@@ -26,33 +20,7 @@ public class UsersPart extends DefaultTabularPart<HierarchyUnit, Content> {
 	public UsersPart(ContentSession contentSession, CmsUserManager cmsUserManager) {
 		this.contentSession = contentSession;
 		this.cmsUserManager = cmsUserManager;
-		addColumn(new Column<Content>() {
-
-			@Override
-			public String getText(Content role) {
-				if (role.hasContentClass(LdapObjs.inetOrgPerson))
-					return UserAdminUtils.getUserDisplayName(role.adapt(User.class));
-				else if (role.hasContentClass(LdapObjs.organization))
-					return role.attr(LdapAttrs.o);
-				else if (role.hasContentClass(LdapObjs.groupOfNames))
-					return role.attr(LdapAttrs.cn);
-				else
-					return null;
-			}
-
-			@Override
-			public CmsIcon getIcon(Content role) {
-				if (role.hasContentClass(LdapObjs.posixAccount))
-					return SuiteIcon.user;
-				else if (role.hasContentClass(LdapObjs.inetOrgPerson))
-					return SuiteIcon.person;
-				else if (role.hasContentClass(LdapObjs.organization))
-					return SuiteIcon.organisationContact;
-				else if (role.hasContentClass(LdapObjs.groupOfNames))
-					return SuiteIcon.group;
-				else
-					return null;
-			}
+		addColumn(new UserColumn() {
 
 			@Override
 			public int getWidth() {
@@ -69,7 +37,7 @@ public class UsersPart extends DefaultTabularPart<HierarchyUnit, Content> {
 		if (ud.getRealm().isPresent()) {
 			for (Role r : ud.getHierarchyUnitRoles(ud, null, true)) {
 				Content content = ContentUtils.roleToContent(cmsUserManager, contentSession, r);
-				if (content.hasContentClass(LdapObjs.inetOrgPerson, LdapObjs.organization))
+				if (content.hasContentClass(LdapObj.inetOrgPerson, LdapObj.organization))
 					roles.add(content);
 			}
 
@@ -79,8 +47,7 @@ public class UsersPart extends DefaultTabularPart<HierarchyUnit, Content> {
 						|| directChild.isType(HierarchyUnit.Type.ROLES))) {
 					for (Role r : ud.getHierarchyUnitRoles(directChild, null, false)) {
 						Content content = ContentUtils.roleToContent(cmsUserManager, contentSession, r);
-						if (content.hasContentClass(LdapObjs.inetOrgPerson, LdapObjs.organization,
-								LdapObjs.groupOfNames))
+						if (content.hasContentClass(LdapObj.inetOrgPerson, LdapObj.organization, LdapObj.groupOfNames))
 							roles.add(content);
 					}
 				}