X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=org.argeo.app.ui%2Fsrc%2Forg%2Fargeo%2Fapp%2Fui%2Fpeople%2FPeopleEntryArea.java;h=789698bcf21f4904a5217244478be393cf432a86;hb=f1b87af2e0de38f1f49ab8fe68f9988018914811;hp=4a92c20630456bb42094ece6d36a9e40611d5033;hpb=b32a7ed960be8dbaa50e605a409c14402db5b489;p=gpl%2Fargeo-suite.git

diff --git a/org.argeo.app.ui/src/org/argeo/app/ui/people/PeopleEntryArea.java b/org.argeo.app.ui/src/org/argeo/app/ui/people/PeopleEntryArea.java
index 4a92c20..789698b 100644
--- a/org.argeo.app.ui/src/org/argeo/app/ui/people/PeopleEntryArea.java
+++ b/org.argeo.app.ui/src/org/argeo/app/ui/people/PeopleEntryArea.java
@@ -84,11 +84,14 @@ public class PeopleEntryArea implements SwtUiProvider, CmsUiProvider {
 				List<HierarchyUnit> visible = new ArrayList<>();
 				if (parent != null) {
 					for (HierarchyUnit hu : parent.getDirectHierarchyUnits(true)) {
-						if (CurrentUser.implies(CmsRole.userAdmin, hu.getBase()) //
-						) // IPA
-						{
-							visible.add(hu);
-						}
+						// if parent was visible, it is visible
+						// TODO restrict more?
+
+//						if (CurrentUser.implies(CmsRole.userAdmin, hu.getBase()) //
+//						) // IPA
+//						{
+						visible.add(hu);
+//						}
 					}
 				} else {
 					for (UserDirectory directory : cmsUserManager.getUserDirectories()) {
@@ -96,8 +99,10 @@ public class PeopleEntryArea implements SwtUiProvider, CmsUiProvider {
 								|| CurrentUser.implies(CmsRole.userAdmin,
 										IpaUtils.IPA_ACCOUNTS_RDN + "," + directory.getBase())) // IPA
 						{
-							// we do not show the base level
-							for (HierarchyUnit hu : directory.getDirectHierarchyUnits(true)) {
+							// TODO show base level
+						}
+						for (HierarchyUnit hu : directory.getDirectHierarchyUnits(true)) {
+							if (CurrentUser.implies(CmsRole.userAdmin, hu.getBase())) {
 								visible.add(hu);
 							}
 						}
@@ -109,7 +114,7 @@ public class PeopleEntryArea implements SwtUiProvider, CmsUiProvider {
 
 			@Override
 			public String getText(HierarchyUnit model) {
-				return model.getHierarchyUnitName();
+				return model.getHierarchyUnitLabel(CurrentUser.locale());
 			}
 
 			@Override
@@ -118,9 +123,9 @@ public class PeopleEntryArea implements SwtUiProvider, CmsUiProvider {
 				if (content.hasContentClass(LdapObjs.organization.qName()))
 					return SuiteIcon.organisation;
 				else if (content.hasContentClass(LdapObjs.posixGroup.qName()))
-					return SuiteIcon.group;
+					return SuiteIcon.users;
 				else
-					return SuiteIcon.people;
+					return SuiteIcon.addressBook;
 			}
 
 		};
@@ -177,7 +182,7 @@ public class PeopleEntryArea implements SwtUiProvider, CmsUiProvider {
 				else if (role.isContentClass(LdapObjs.inetOrgPerson.qName()))
 					return SuiteIcon.person;
 				else if (role.isContentClass(LdapObjs.organization.qName()))
-					return SuiteIcon.organisation;
+					return SuiteIcon.organisationContact;
 				else if (role.isContentClass(LdapObjs.groupOfNames.qName()))
 					return SuiteIcon.group;
 				else