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=09907cb78346553521ee1d2af247ffb6ea607420;hpb=f2d6ceac3fc54f45e825f6f40a741b16c75fa7d3;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 09907cb..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 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 @@ -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