Author: fdesbois Date: 2010-01-18 14:57:57 +0000 (Mon, 18 Jan 2010) New Revision: 234 Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/SuiviObsmerBusinessException.java trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceUserImpl.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Layout.java Log: - Evol #1972 : Send a mail for password modification - Ano #2015 : An inactive user can't connect to the application (use of SuiviObsmerBusinessException type) Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/SuiviObsmerBusinessException.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/SuiviObsmerBusinessException.java 2010-01-18 14:52:19 UTC (rev 233) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/SuiviObsmerBusinessException.java 2010-01-18 14:57:57 UTC (rev 234) @@ -52,7 +52,7 @@ } public static enum Type { - IMPORT_ERROR, NOT_EXISTS, ALREADY_EXISTS, SYNTAX, CONNECTION; + IMPORT_ERROR, NOT_EXISTS, ALREADY_EXISTS, SYNTAX, BAD_CONNECTION, ILLEGAL_CONNECTION; } public Type getType() { Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceUserImpl.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceUserImpl.java 2010-01-18 14:52:19 UTC (rev 233) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceUserImpl.java 2010-01-18 14:57:57 UTC (rev 234) @@ -71,11 +71,14 @@ String passwordEncoded = SuiviObsmerContext.encodeString(password); user = dao.findByProperties(User.LOGIN, login, User.PASSWORD, passwordEncoded); - if (user != null) { + if (user != null && user.getActive()) { // load company entity user.getCompany(); + } else if (user != null && !user.getActive()) { + throw new SuiviObsmerBusinessException(Type.ILLEGAL_CONNECTION, this.getClass(), + "Vous n'avez plus les droits nécessaires pour vous connecter. Veuillez contacter un administrateur."); } else { - throw new SuiviObsmerBusinessException(Type.CONNECTION, this.getClass(), + throw new SuiviObsmerBusinessException(Type.BAD_CONNECTION, this.getClass(), "Identifiant ou mot de passe incorrects, veuillez réessayer."); } @@ -112,7 +115,7 @@ String subject = "[WAO] Mot de passe oublié"; String msg = "Bonjour,\n\n" + "\tUn nouveau mot de passe a été généré pour votre identifiant " + login + " :\n" + - "\t\t - mot de passe = " + password + "\n\n" + + "\t\t * mot de passe = " + password + "\n\n" + "Vous pouvez modifier votre password en accédant à la page de gestion de votre profile utilisateur.\n\n" + "Cordialement,\n\n" + "L'Equipe WAO"; @@ -157,9 +160,9 @@ if (generatePassword) { password = SuiviObsmerContext.createRandomString(8); user.setPasswordChanged(true); - if (log.isDebugEnabled()) { - log.debug("show generated password : " + password); - } +// if (log.isDebugEnabled()) { +// log.debug("show generated password : " + password); +// } } // For a password set manually by user or generated if (user.isPasswordChanged()) { @@ -169,20 +172,28 @@ dao.update(user); - if (newUser && user.getLogin().contains("@")) { - String subject = "[WAO] Création de votre compte"; - String msg = "Bonjour,\n\n" + - "\tVous avez été inscris sur le site WAO : Web Applicatif Obsmer. Vos identifiants de connexion sont :\n" + - "\t\t - identifiant = " + user.getLogin() + "\n" + - "\t\t - mot de passe = " + password + "\n\n" + - "Vous pouvez modifier votre mot de passe en accédant à la page de gestion de votre profile utilisateur.\n\n" + - "Cordialement,\n\n" + - "L'Equipe WAO"; + if (user.getLogin().contains("@") && user.isPasswordChanged()) { + String subject = "[WAO] "; + String msg = "Bonjour,\n\n"; + if (newUser) { + subject += "Création de votre compte"; + msg += "\tVous avez été inscris sur le site WAO : Web Applicatif Obsmer. Vos identifiants de connexion sont :\n" + + "\t\t * identifiant = " + user.getLogin() + "\n" + + "\t\t * mot de passe = " + password + "\n\n" + + "Vous pouvez modifier votre mot de passe en accédant à la page de gestion de votre profil utilisateur.\n\n"; + } else { + subject += "Modification de votre compte"; + msg += "\tVos identifiants de connexion à l'application WAO ont été modifiés :\n" + + "\t\t * identifiant = " + user.getLogin() + "\n" + + "\t\t * mot de passe = " + password + "\n\n"; + } + msg += "Cordialement,\n\n" + + "L'Equipe WAO"; SuiviObsmerContext.sendEmail(user.getLogin(), subject, msg); - if (log.isDebugEnabled()) { - log.debug("send email to : " + user.getLogin()); - } +// if (log.isDebugEnabled()) { +// log.debug("send email to : " + user.getLogin()); +// } } transaction.commitTransaction(); Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Layout.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Layout.java 2010-01-18 14:52:19 UTC (rev 233) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Layout.java 2010-01-18 14:57:57 UTC (rev 234) @@ -22,6 +22,7 @@ package fr.ifremer.suiviobsmer.ui.components; import fr.ifremer.suiviobsmer.SuiviObsmerBusinessException; +import fr.ifremer.suiviobsmer.SuiviObsmerBusinessException.Type; import fr.ifremer.suiviobsmer.SuiviObsmerContext; import fr.ifremer.suiviobsmer.SuiviObsmerException; import fr.ifremer.suiviobsmer.entity.User; @@ -171,7 +172,11 @@ try { currentUser = serviceUser.connect(login, password); } catch (SuiviObsmerBusinessException eee) { - connexionFeedback.addError(eee.getMessage()); + if (eee.getType().equals(Type.BAD_CONNECTION)) { + connexionFeedback.addError(eee.getMessage()); + } else if (eee.getType().equals(Type.ILLEGAL_CONNECTION)) { + connexionFeedback.addInfo(eee.getMessage()); + } } return null; }