Author: echatellier Date: 2014-07-16 17:35:15 +0200 (Wed, 16 Jul 2014) New Revision: 400 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/400 Log: Suppression du mail m?\195?\170me s'il est rejet?\195?\169 (enfin surtout s'il est rejet?\195?\169) Ajout d'un fixme... encore. Modified: trunk/faxtomail-ui-web/src/main/java/com/franciaflex/faxtomail/web/job/MailFilterJob.java Modified: trunk/faxtomail-ui-web/src/main/java/com/franciaflex/faxtomail/web/job/MailFilterJob.java =================================================================== --- trunk/faxtomail-ui-web/src/main/java/com/franciaflex/faxtomail/web/job/MailFilterJob.java 2014-07-16 15:14:12 UTC (rev 399) +++ trunk/faxtomail-ui-web/src/main/java/com/franciaflex/faxtomail/web/job/MailFilterJob.java 2014-07-16 15:35:15 UTC (rev 400) @@ -301,6 +301,10 @@ } } } + + // FIXME echatellier 20140716 : le mail doit-il rester sur le serveur potentiellement traité suite à l'ajout d'un futur filtre + // doit-il est supprimé ? + // you decide : delete mail or add proper comment continue; } @@ -376,6 +380,10 @@ recipient = FaxToMailServiceUtils.addFaxDomainToFaxNumber(recipient, mailFolder); } emailService.rejectEmail(senderEmail, recipient, t("faxtomail.email.subject.re", message.getSubject()), rejectMessage); + + // important, delete mail + deleteMail(message); + continue; } } @@ -388,7 +396,6 @@ Date now = new Date(); - Decorator<Date> dateDecorator = decoratorService.getDecoratorByType(Date.class, DecoratorService.DATE); String projectRef = t("faxtomail.email.projectReference.default", dateDecorator.toString(now)); email.setProjectReference(projectRef); @@ -437,16 +444,15 @@ if (log.isDebugEnabled()) { log.debug(" ==> Message placé dans le dossier " + email.getMailFolder().getName()); } - // suppression des mails sur le serveur distant (automatique par default) - if (config.isMailDelete()) { - message.setFlag(Flags.Flag.DELETED, true); - } + // important, delete mail + deleteMail(message); + } catch (Exception e) { log.error("Error while reading the email", e); } } - if (log.isDebugEnabled()){ + if (log.isDebugEnabled()) { log.debug("End of emails"); } @@ -456,6 +462,19 @@ } /** + * Supprime le mail sur le serveur (si nécéssaire). + * + * @param message message to delete + * @throws MessagingException + */ + protected void deleteMail(Message message) throws MessagingException { + // suppression des mails sur le serveur distant (non actif par default) + if (config.isMailDelete()) { + message.setFlag(Flags.Flag.DELETED, true); + } + } + + /** * Decompose a multipart part. * - sets the email content if the part contains a text bodypart * - adds attachments to the email