Author: glandais Date: 2008-03-04 20:04:04 +0000 (Tue, 04 Mar 2008) New Revision: 1286 Modified: trunk/simexplorer-is/simexplorer-is-security/src/java/fr/cemagref/simexplorer/is/security/dao/DaoActor.java trunk/simexplorer-is/simexplorer-is-security/src/java/fr/cemagref/simexplorer/is/security/dao/DaoActorImpl.java trunk/simexplorer-is/simexplorer-is-service/src/java/fr/cemagref/simexplorer/is/service/AuthenticationServiceImpl.java trunk/simexplorer-is/src/site/fr/rst/todo.rst Log: Delete Actor Modified: trunk/simexplorer-is/simexplorer-is-security/src/java/fr/cemagref/simexplorer/is/security/dao/DaoActor.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-security/src/java/fr/cemagref/simexplorer/is/security/dao/DaoActor.java 2008-03-04 18:25:34 UTC (rev 1285) +++ trunk/simexplorer-is/simexplorer-is-security/src/java/fr/cemagref/simexplorer/is/security/dao/DaoActor.java 2008-03-04 20:04:04 UTC (rev 1286) @@ -186,22 +186,6 @@ */ public List<User> getUsersOfGroup(Group group); - /** - * Gets the groups of user. - * - * @param user the user - * - * @return the groups of user - */ - public List<Group> getGroupsOfUser(User user); + public List<Group> getGroupsOfActor(Actor actor); - /** - * Gets the groups of group. - * - * @param group the group - * - * @return the groups of group - */ - public List<Group> getGroupsOfGroup(Group group); - } Modified: trunk/simexplorer-is/simexplorer-is-security/src/java/fr/cemagref/simexplorer/is/security/dao/DaoActorImpl.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-security/src/java/fr/cemagref/simexplorer/is/security/dao/DaoActorImpl.java 2008-03-04 18:25:34 UTC (rev 1285) +++ trunk/simexplorer-is/simexplorer-is-security/src/java/fr/cemagref/simexplorer/is/security/dao/DaoActorImpl.java 2008-03-04 20:04:04 UTC (rev 1286) @@ -25,14 +25,14 @@ import javax.persistence.NoResultException; import javax.persistence.PersistenceContext; import javax.persistence.Query; -import javax.persistence.TemporalType; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.codelutin.util.CollectionUtil; import fr.cemagref.simexplorer.is.security.entities.Actor; import fr.cemagref.simexplorer.is.security.entities.Group; import fr.cemagref.simexplorer.is.security.entities.LoginAction; -import fr.cemagref.simexplorer.is.security.entities.Permission; import fr.cemagref.simexplorer.is.security.entities.User; /** @@ -41,6 +41,9 @@ @Stateless(name = "DaoActor") public class DaoActorImpl implements DaoActor { + /** The Constant log. */ + private static final Log log = LogFactory.getLog(DaoActorImpl.class); + /** The em. */ @PersistenceContext(unitName = "simexploreris-security") private EntityManager em; @@ -131,10 +134,15 @@ return a; } + private void setActorVisible(Actor actor) { + actor.setVisible(true); + } + /* (non-Javadoc) * @see fr.cemagref.simexplorer.is.security.dao.DaoActor#saveGroup(fr.cemagref.simexplorer.is.security.entities.Group) */ public void saveGroup(Group group) { + setActorVisible(group); em.persist(group); } @@ -142,6 +150,7 @@ * @see fr.cemagref.simexplorer.is.security.dao.DaoActor#saveUser(fr.cemagref.simexplorer.is.security.entities.User) */ public void saveUser(User user) { + setActorVisible(user); em.persist(user); } @@ -149,6 +158,7 @@ * @see fr.cemagref.simexplorer.is.security.dao.DaoActor#updateGroup(fr.cemagref.simexplorer.is.security.entities.Group) */ public Group updateGroup(Group group) { + setActorVisible(group); return em.merge(group); } @@ -156,6 +166,7 @@ * @see fr.cemagref.simexplorer.is.security.dao.DaoActor#updateUser(fr.cemagref.simexplorer.is.security.entities.User) */ public User updateUser(User user) { + setActorVisible(user); return em.merge(user); } @@ -214,7 +225,7 @@ * @see fr.cemagref.simexplorer.is.security.dao.DaoActor#getGroups() */ public List<Group> getGroups() { - List<Group> groups = CollectionUtil.toGenericList(em.createQuery("select g from Group g").getResultList(), + List<Group> groups = CollectionUtil.toGenericList(em.createQuery("select g from Group g where g.visible = true").getResultList(), Group.class); return groups; } @@ -223,7 +234,7 @@ * @see fr.cemagref.simexplorer.is.security.dao.DaoActor#getGroupsOwnedBy(fr.cemagref.simexplorer.is.security.entities.User) */ public List<Group> getGroupsOwnedBy(User user) { - List<Group> groups = CollectionUtil.toGenericList(em.createQuery("select g from Group g where g.owner=:owner") + List<Group> groups = CollectionUtil.toGenericList(em.createQuery("select g from Group g where g.owner=:owner and g.visible = true") .setParameter("owner", user).getResultList(), Group.class); return groups; } @@ -232,7 +243,7 @@ * @see fr.cemagref.simexplorer.is.security.dao.DaoActor#getUsers() */ public List<User> getUsers() { - List<User> users = CollectionUtil.toGenericList(em.createQuery("select u from User u").getResultList(), + List<User> users = CollectionUtil.toGenericList(em.createQuery("select u from User u where u.visible = true").getResultList(), User.class); return users; } @@ -242,29 +253,18 @@ */ public List<User> getUsersOfGroup(Group group) { List<User> users = CollectionUtil.toGenericList(em.createQuery( - "select distinct u from User u where :group member of u.groups").setParameter("group", group) + "select distinct u from User u where :group member of u.groups and u.visible = true").setParameter("group", group) .getResultList(), User.class); return users; } - /* (non-Javadoc) - * @see fr.cemagref.simexplorer.is.security.dao.DaoActor#getGroupsOfUser(fr.cemagref.simexplorer.is.security.entities.User) - */ - public List<Group> getGroupsOfUser(User user) { + public List<Group> getGroupsOfActor(Actor actor) { List<Group> groups = CollectionUtil.toGenericList(em.createQuery( - "select distinct u.groups from User u where u = :user").setParameter("user", user).getResultList(), - Group.class); - return groups; + "select distinct group from Actor actor join actor.groups as group where actor = :actor and group.visible = true") + .setParameter("actor", actor).getResultList(), + Group.class); + + return groups; } - /* (non-Javadoc) - * @see fr.cemagref.simexplorer.is.security.dao.DaoActor#getGroupsOfGroup(fr.cemagref.simexplorer.is.security.entities.Group) - */ - public List<Group> getGroupsOfGroup(Group group) { - List<Group> groups = CollectionUtil.toGenericList(em.createQuery( - "select distinct g.groups from Group g where g = :group").setParameter("group", group).getResultList(), - Group.class); - return groups; - } - } Modified: trunk/simexplorer-is/simexplorer-is-service/src/java/fr/cemagref/simexplorer/is/service/AuthenticationServiceImpl.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-service/src/java/fr/cemagref/simexplorer/is/service/AuthenticationServiceImpl.java 2008-03-04 18:25:34 UTC (rev 1285) +++ trunk/simexplorer-is/simexplorer-is-service/src/java/fr/cemagref/simexplorer/is/service/AuthenticationServiceImpl.java 2008-03-04 20:04:04 UTC (rev 1286) @@ -720,7 +720,7 @@ */ public Group[] getGroupsOfUser(String token, User user) throws SimExplorerException { if (canViewElements(token)) { - List<Group> list = daoActor.getGroupsOfUser(user); + List<Group> list = daoActor.getGroupsOfActor(user); return list.toArray(new Group[list.size()]); } throw new SimExplorerSecurityException(); @@ -731,7 +731,7 @@ */ public Group[] getGroupsOfGroup(String token, Group group) throws SimExplorerException { if (canViewElements(token)) { - List<Group> groupsOfGroup = daoActor.getGroupsOfGroup(group); + List<Group> groupsOfGroup = daoActor.getGroupsOfActor(group); return groupsOfGroup.toArray(new Group[groupsOfGroup.size()]); } throw new SimExplorerSecurityException(); Modified: trunk/simexplorer-is/src/site/fr/rst/todo.rst =================================================================== --- trunk/simexplorer-is/src/site/fr/rst/todo.rst 2008-03-04 18:25:34 UTC (rev 1285) +++ trunk/simexplorer-is/src/site/fr/rst/todo.rst 2008-03-04 20:04:04 UTC (rev 1286) @@ -5,8 +5,7 @@ Serveur central =============== - - Règles de gestion pour la suppression d'un groupe/utilisateur (affichage selon flag) - - dump H2, backup, restore + - retour synchronisation récursif Client léger ============ @@ -23,6 +22,7 @@ - Synchronisation d'un élément avec le serveur (tests) - export, import (tests) - configuration + - API Fonctionnalités seconde réunion =============================== @@ -31,7 +31,6 @@ ------- - (NOK) Mettre à jour les entêtes de fichier avec la bonne licence GPL - - ( OK) Decouper les métadonnées en deux parties : celles propres à SimExplorer et celles propres au SI. - (NOK) Refondre les entités afin de les générer depuis un modèle et gérer la sérialisation par outils comme JAXB/XStream, ceci dans un module propre (et non inclu avec les autres classes storage) @@ -57,9 +56,10 @@ - Documentation utilisateur - Enoncer les règles métiers qui ont été développées dans un document - indépendant du code (pas du Javadoc) + indépendant du code - Documentation développeur - - Documentation intégration (backup, dumps, configuration) + - Documentation intégration (backup, dumps, configuration) + - Configuration log4j (hibernate, etc) Reste à faire (Autre) ===================== @@ -67,7 +67,6 @@ - Gestion des attachments transparente pour SimExplorer - Document format du fichier zip - Types de fichiers joints (uniquement texte pour le moment), visualisation? - - Changer les logs de jboss : hibernate est en debug par exemple... Règles modifiées ================