- Faxtomail-commits - mailman.cloud.codelutin.com

r424 - in trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing: . actions content content/demande
by kmorin＠users.forge.codelutin.com 18 Jul '14

18 Jul '14

Author: kmorin Date: 2014-07-18 12:22:20 +0200 (Fri, 18 Jul 2014) New Revision: 424 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/424 Log: affichage du message de l'utilisateur qui n'a aucun droit seulement au d?\195?\169marrage Removed: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/StartAction.java Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/RunFaxToMail.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/MainUIHandler.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/RunFaxToMail.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/RunFaxToMail.java 2014-07-18 10:17:20 UTC (rev 423) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/RunFaxToMail.java 2014-07-18 10:22:20 UTC (rev 424) @@ -25,7 +25,6 @@ import com.franciaflex.faxtomail.FaxToMailConfiguration; import com.franciaflex.faxtomail.ui.swing.content.MainUI; import com.franciaflex.faxtomail.ui.swing.content.MainUIHandler; -import com.franciaflex.faxtomail.ui.swing.actions.StartAction; import com.franciaflex.faxtomail.ui.swing.util.FaxToMailExceptionHandler; import jaxx.runtime.SwingUtil; import org.apache.commons.logging.Log; @@ -120,11 +119,6 @@ mainUI.setVisible(true); } }); - - // launch start action (use the tutti-start-action file) - StartAction uiAction = context.getActionFactory().createLogicAction( - mainUI.getHandler(), StartAction.class); - context.getActionEngine().runAction(uiAction); } public static void closeFaxToMail(MainUIHandler handler, Integer exitCode) { Deleted: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/StartAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/StartAction.java 2014-07-18 10:17:20 UTC (rev 423) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/StartAction.java 2014-07-18 10:22:20 UTC (rev 424) @@ -1,112 +0,0 @@ -package com.franciaflex.faxtomail.ui.swing.actions; - -/* - * #%L - * FaxToMail :: UI - * %% - * Copyright (C) 2014 Franciaflex - * %% - * This program is free software: you can redistribute it and/or modify - * it under the terms of the GNU General Public License as - * published by the Free Software Foundation, either version 3 of the - * License, or (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public - * License along with this program. If not, see - * <http://www.gnu.org/licenses/gpl-3.0.html>. - * #L% - */ - -import com.franciaflex.faxtomail.ui.swing.content.MainUIHandler; -import org.apache.commons.logging.Log; -import org.apache.commons.logging.LogFactory; - -import static org.nuiton.i18n.I18n.t; - -/** - * Start action (when the start action file is present when - * starting application). - * - * @author kmorin - morin(a)codelutin.com - */ -public class StartAction extends AbstractMainUIFaxToMailAction { - - /** Logger. */ - private static final Log log = LogFactory.getLog(StartAction.class); - - protected AbstractMainUIFaxToMailAction delegateAction; - - public StartAction(MainUIHandler handler) { - super(handler, true); - setActionDescription(t("faxtomail.action.start.tip")); - } - - @Override - public boolean prepareAction() throws Exception { - -// File actionfile = getConfig().getStartActionFile(); - boolean doAction = false;//actionfile.exists(); - - if (doAction) { - - try { - // get action to execute -// String content = FaxToMailIOUtil.readContent(actionfile, _("faxtomail.error.read.startActionFile", actionfile)); -// content = content.trim(); - -// if (InstallDbAction.class.getName().equals(content)) { -// -// // install db -// delegateAction = TuttiActionHelper.createLogicAction(getHandler(), InstallDbAction.class); -// -// if (log.isInfoEnabled()) { -// log.info("Found install db action"); -// } -// } else if (content.startsWith(ImportDbAction.class.getName())) { -// -// // import db -// ImportDbAction action = TuttiActionHelper.createLogicAction(getHandler(), ImportDbAction.class); -// File importFile = new File(content.substring(ImportDbAction.class.getName().length() + 1)); -// action.setImportFile(importFile); -// delegateAction = action; -// if (log.isInfoEnabled()) { -// log.info("Found import db action (with file " + importFile + ")"); -// } -// } else { - doAction = false; -// } - - } finally { - - // delete start action file -// FaxToMailIOUtil.deleteFile( -// actionfile, -// _("faxtomail.error.delete.startActionFile", actionfile)); - } - } - - - if (doAction) { - setActionDescription(delegateAction.getActionDescription()); - doAction = delegateAction.prepareAction(); - } - return doAction; - } - - @Override - public void doAction() throws Exception { - - getActionEngine().runInternalAction(delegateAction); - } - - @Override - protected void releaseAction() { - delegateAction = null; - super.releaseAction(); - } -} Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/MainUIHandler.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/MainUIHandler.java 2014-07-18 10:17:20 UTC (rev 423) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/MainUIHandler.java 2014-07-18 10:22:20 UTC (rev 424) @@ -38,6 +38,7 @@ import jaxx.runtime.swing.JAXXWidgetUtil; import jaxx.runtime.swing.session.SwingSession; import jaxx.runtime.validator.swing.SwingValidator; +import org.apache.commons.collections.CollectionUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.nuiton.i18n.I18n; @@ -161,6 +162,13 @@ ui.getStatus().addWidget(bar, 0); getContext().setScreen(FaxToMailScreen.LIST); + + // affichage d'un message si l'utilisateur n'a aucun droit + DemandeListUI demandeListUI = (DemandeListUI) getCurrentBody(); + if (CollectionUtils.isEmpty(demandeListUI.getModel().getFolders())) { + JOptionPane.showMessageDialog(demandeListUI, t("faxtomail.demandeList.noRigthsFound"), + t("faxtomail.demandeList.title"), JOptionPane.ERROR_MESSAGE); + } } @Override Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java 2014-07-18 10:17:20 UTC (rev 423) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java 2014-07-18 10:22:20 UTC (rev 424) @@ -325,12 +325,6 @@ ui.getNavigationTree().setSelectionPath(new TreePath(node.getPath())); } - - // affichage d'un message si l'utilisateur n'a aucun droit - if (CollectionUtils.isEmpty(model.getFolders())) { - JOptionPane.showMessageDialog(ui, t("faxtomail.demandeList.noRigthsFound"), - t("faxtomail.demandeList.title"), JOptionPane.ERROR_MESSAGE); - } } public void goToNextPage() {

1 0

r423 - trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service
by echatellier＠users.forge.codelutin.com 18 Jul '14

18 Jul '14

Author: echatellier Date: 2014-07-18 12:17:20 +0200 (Fri, 18 Jul 2014) New Revision: 423 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/423 Log: Deverrouillage automatique des demandes qui ne conernent pas la demande qui vient d'?\195?\170tre verrouill?\195?\169e (ancien lock) Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailService.java Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailService.java =================================================================== --- trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailService.java 2014-07-18 09:56:25 UTC (rev 422) +++ trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailService.java 2014-07-18 10:17:20 UTC (rev 423) @@ -568,18 +568,26 @@ */ public Email lockEmail(String emailId, FaxToMailUser currentUser) { EmailTopiaDao emailDao = getPersistenceContext().getEmailDao(); + + // verrouillage du mail courant Email email = emailDao.findByTopiaId(emailId); email.setLockedBy(currentUser); email = emailDao.update(email); getPersistenceContext().commit(); if (log.isDebugEnabled()) { - log.debug("[LOCK] " + emailId + " locked by " + currentUser.getFirstName() + " " + currentUser.getLastName()); + log.debug("[LOCK] " + emailId + " locked by " + currentUser.getLogin()); } - - // TODO echatellier : les utilisateurs peuvent avoir plusieurs verrous actif dans le - // cas des interfaces à onglet et des demandes groupé - // on pourrait peut-etre supprimer automatiquement les locks sur les autres demande - // si elle ne sont pas groupées + + // dévérouillage automatique des mails qui ne font pas partit du même group que celui du mail + // qui vient d'être locké + List<Email> mailsToUnlock = emailDao.forLockedByEquals(currentUser).addNotEquals(Email.PROPERTY_EMAIL_GROUP, email.getEmailGroup()).findAll(); + for (Email mailToUnlock : mailsToUnlock) { + mailToUnlock.setLockedBy(null); + mailToUnlock = emailDao.update(mailToUnlock); + if (log.isDebugEnabled()) { + log.debug("[UNLOCK] " + mailToUnlock.getTopiaId() + " unlocked (automatic)"); + } + } return email; } @@ -596,7 +604,7 @@ email = emailDao.update(email); getPersistenceContext().commit(); if (log.isDebugEnabled()) { - log.debug("[UNLOCK] " + emailId + " unlocked "); + log.debug("[UNLOCK] " + emailId + " unlocked"); } return email; }

1 0

r422 - trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions
by echatellier＠users.forge.codelutin.com 18 Jul '14

18 Jul '14

Author: echatellier Date: 2014-07-18 11:56:25 +0200 (Fri, 18 Jul 2014) New Revision: 422 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/422 Log: Remove todo Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveAction.java Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveAction.java 2014-07-18 09:55:59 UTC (rev 421) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveAction.java 2014-07-18 09:56:25 UTC (rev 422) @@ -81,7 +81,6 @@ MailFolder archiveChild = null; do { if (folder.getChildren() != null) { - // TODO echatellier : ajout du tryFind car ca n'a pas l'air obligatoire, mais pas certains Optional<MailFolder> optArchiveChild = Iterables.tryFind(folder.getChildren(), new Predicate<MailFolder>() { @Override public boolean apply(MailFolder mailFolder) {

1 0

r421 - trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions
by echatellier＠users.forge.codelutin.com 18 Jul '14

18 Jul '14

Author: echatellier Date: 2014-07-18 11:55:59 +0200 (Fri, 18 Jul 2014) New Revision: 421 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/421 Log: Remove fixme Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/LoadFolderEmailsAction.java Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/LoadFolderEmailsAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/LoadFolderEmailsAction.java 2014-07-18 09:52:54 UTC (rev 420) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/LoadFolderEmailsAction.java 2014-07-18 09:55:59 UTC (rev 421) @@ -103,9 +103,6 @@ for (RangeRow rangeRow : rangeRows) { RangeRowModel rangeRowModel = new RangeRowModel(); rangeRowModel.fromEntity(rangeRow); - // FIXME echatellier 20140710 : wtf is that ? incomprehensible sans explication - // on ajout la row dans getValidRangeRowModels() sans condition ? donc elle contient forcement toutes les rangeRowModel - // et ne doit pas vraiment avoir d'utilité demand.getValidRangeRowModels().add(rangeRowModel); } }

1 0

r420 - trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande
by echatellier＠users.forge.codelutin.com 18 Jul '14

18 Jul '14

Author: echatellier Date: 2014-07-18 11:52:54 +0200 (Fri, 18 Jul 2014) New Revision: 420 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/420 Log: Change fixme Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java 2014-07-18 09:50:27 UTC (rev 419) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java 2014-07-18 09:52:54 UTC (rev 420) @@ -193,14 +193,13 @@ } /** - * Surcharge pour dupliquer correctement les pieces jointes. - * - * FIXME echatellier 20140520 c'est pas terrible, mais c'est comme ca :p + * Surcharge pour dupliquer correctement les pieces jointes sans charger le contenu binaire en mémoire. */ public void fromEntity(Email entity, boolean full) { //super.fromEntity(entity); fromBeanBinder.copyExcluding(entity, this, Email.PROPERTY_ATTACHMENT); - // specific attachment copy + + // On ne copie pas entierrement les attachments pour ne pas force le chargement du contenu binaire du fichier en mémoire. Collection<Attachment> attachmentCopy = new ArrayList<Attachment>(); if (entity.getAttachment() != null) { for (Attachment attachment : entity.getAttachment()) {

1 0

r419 - trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande
by echatellier＠users.forge.codelutin.com 18 Jul '14

18 Jul '14

Author: echatellier Date: 2014-07-18 11:50:27 +0200 (Fri, 18 Jul 2014) New Revision: 419 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/419 Log: Remove tested DODO Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandesUIHandler.java Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandesUIHandler.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandesUIHandler.java 2014-07-18 09:48:53 UTC (rev 418) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandesUIHandler.java 2014-07-18 09:50:27 UTC (rev 419) @@ -373,7 +373,6 @@ model.setOriginalDemand(currentDemand); - //TODO kmorin 20140626 tester la réponse à un fax String sender = JAXXUtil.getStringValue(currentDemand.getSender()); if (currentDemand.isFax()) { MailFolder selectedFolder = getContext().getCurrentMailFolder();

1 0

r418 - trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande
by echatellier＠users.forge.codelutin.com 18 Jul '14

18 Jul '14

Author: echatellier Date: 2014-07-18 11:48:53 +0200 (Fri, 18 Jul 2014) New Revision: 418 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/418 Log: Remove tested DODO Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java 2014-07-18 09:29:44 UTC (rev 417) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java 2014-07-18 09:48:53 UTC (rev 418) @@ -549,7 +549,6 @@ model.setOriginalDemand(demand); - //TODO kmorin 20140626 tester la réponse à un fax String sender = JAXXUtil.getStringValue(demand.getSender()); if (demand.isFax()) { MailFolder selectedFolder = getModel().getSelectedFolder();

1 0

r417 - in trunk: faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande faxtomail-ui-web/src/main/java/com/franciaflex/faxtomail/web/job
by kmorin＠users.forge.codelutin.com 18 Jul '14

18 Jul '14

Author: kmorin Date: 2014-07-18 11:29:44 +0200 (Fri, 18 Jul 2014) New Revision: 417 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/417 Log: mise a jour des fixme Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ClientService.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/LoadFolderEmailsAction.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIHandler.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandesUIHandler.java trunk/faxtomail-ui-web/src/main/java/com/franciaflex/faxtomail/web/job/MailFilterJob.java Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ClientService.java =================================================================== --- trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ClientService.java 2014-07-18 08:47:33 UTC (rev 416) +++ trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ClientService.java 2014-07-18 09:29:44 UTC (rev 417) @@ -93,29 +93,7 @@ } if (CollectionUtils.isNotEmpty(clients)) { - MailFolder folder = email.getMailFolder(); - if (clients.size() == 1) { - client = clients.get(0); - - // FIXME echatellier 20140709 : a revoir, folder == null dans tous les cas - // que fait ce code ? - } else if (folder != null) { - Multimap<String, Client> clientsByBrand = Multimaps.index(clients, new Function<Client, String>() { - @Override - public String apply(Client client) { - return client.getBrand(); - } - }); - Set<String> brands = clientsByBrand.keySet(); - while (client == null && folder != null) { - // TODO echatellier 20140709 : il y a une relation particuliere entre les noms des dossiers - // et les marques ? - if (brands.contains(folder.getName())) { - client = new ArrayList<>(clientsByBrand.get(folder.getName())).get(0); - } - folder = folder.getParent(); - } - } + client = clients.get(0); } } Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/LoadFolderEmailsAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/LoadFolderEmailsAction.java 2014-07-18 08:47:33 UTC (rev 416) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/LoadFolderEmailsAction.java 2014-07-18 09:29:44 UTC (rev 417) @@ -96,8 +96,8 @@ DemandeUIModel demand = new DemandeUIModel(); demand.fromEntity(email, false); - // FIXME echatellier 20140710 : ajout du code suivant sinon getValidRangeRowModels() est toujours vide - // et les lignes sont toujours rouge car jamais valides + // on ajoute toutes les lignes de rangeRow, + // qui sont toutes valides puisqu'on les récupère de la base et qu'on ne sauvegarde que les lignes valides Collection<RangeRow> rangeRows = demand.getRangeRow(); if (rangeRows != null) { for (RangeRow rangeRow : rangeRows) { Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java 2014-07-18 08:47:33 UTC (rev 416) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeListUIHandler.java 2014-07-18 09:29:44 UTC (rev 417) @@ -549,8 +549,7 @@ model.setOriginalDemand(demand); - //FIXME echatellier 20140716 : ajouter dans le TODO ce qui doit être testé - //TODO kmorin 20140626 to test + //TODO kmorin 20140626 tester la réponse à un fax String sender = JAXXUtil.getStringValue(demand.getSender()); if (demand.isFax()) { MailFolder selectedFolder = getModel().getSelectedFolder(); Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIHandler.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIHandler.java 2014-07-18 08:47:33 UTC (rev 416) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIHandler.java 2014-07-18 09:29:44 UTC (rev 417) @@ -181,9 +181,8 @@ RangeRowModel rangeRowModel = new RangeRowModel(); rangeRowModel.fromEntity(rangeRow); rangeRowModels.add(rangeRowModel); - // FIXME echatellier 20140710 : wtf is that ? incomprehensible sans explication - // on ajout la row dans getValidRangeRowModels() sans condition ? donc elle contient forcement toutes les rangeRowModel - // et ne doit pas vraiment avoir d'utilité + // on ajoute toutes les lignes de rangeRow, + // qui sont toutes valides puisqu'on les récupère de la base et qu'on ne sauvegarde que les lignes valides model.getValidRangeRowModels().add(rangeRowModel); } } Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java 2014-07-18 08:47:33 UTC (rev 416) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java 2014-07-18 09:29:44 UTC (rev 417) @@ -681,10 +681,11 @@ if (getRangeRow() != null) { oldValue = new ArrayList<RangeRow>(getRangeRow()); } - String refOldValue = getReference(); // FIXME : why ? + // on met à jour la référence, qui est la concaténation des numéros de commande et de "notre" référence + String refOldValue = getReference(); editObject.removeRangeRow(rangeRow); firePropertyChange(Email.PROPERTY_RANGE_ROW, oldValue, getRangeRow()); - firePropertyChanged(PROPERTY_REFERENCE, refOldValue, getReference()); // FIXME : why ? + firePropertyChanged(PROPERTY_REFERENCE, refOldValue, getReference()); } public void setRangeRow(Collection<RangeRow> rangeRow) { @@ -695,10 +696,11 @@ if (rangeRow == null) { rangeRow = new ArrayList<RangeRow>(); } - String refOldValue = getReference(); // FIXME : why ? + // on met à jour la référence, qui est la concaténation des numéros de commande et de "notre" référence + String refOldValue = getReference(); editObject.setRangeRow(rangeRow); firePropertyChange(Email.PROPERTY_RANGE_ROW, oldValue, getRangeRow()); - firePropertyChanged(PROPERTY_REFERENCE, refOldValue, getReference()); // FIXME : why ? + firePropertyChanged(PROPERTY_REFERENCE, refOldValue, getReference()); } public MailFolder getMailFolder() { Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandesUIHandler.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandesUIHandler.java 2014-07-18 08:47:33 UTC (rev 416) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandesUIHandler.java 2014-07-18 09:29:44 UTC (rev 417) @@ -373,7 +373,7 @@ model.setOriginalDemand(currentDemand); - //TODO kmorin 20140626 to test + //TODO kmorin 20140626 tester la réponse à un fax String sender = JAXXUtil.getStringValue(currentDemand.getSender()); if (currentDemand.isFax()) { MailFolder selectedFolder = getContext().getCurrentMailFolder(); 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-18 08:47:33 UTC (rev 416) +++ trunk/faxtomail-ui-web/src/main/java/com/franciaflex/faxtomail/web/job/MailFilterJob.java 2014-07-18 09:29:44 UTC (rev 417) @@ -302,9 +302,7 @@ } } - // 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 + // on garde le mail sur le serveur pour le traiter suite à l'ajout d'un futur filtre continue; }

1 0

r416 - in trunk: faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions faxtomail-ui-web/src/main/webapp/WEB-INF/content/admin
by echatellier＠users.forge.codelutin.com 18 Jul '14

18 Jul '14

Author: echatellier Date: 2014-07-18 10:47:33 +0200 (Fri, 18 Jul 2014) New Revision: 416 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/416 Log: Restauration des dossiers d'archive Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveAction.java trunk/faxtomail-ui-web/src/main/webapp/WEB-INF/content/admin/configuration-input.jsp Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveAction.java 2014-07-18 08:05:58 UTC (rev 415) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveAction.java 2014-07-18 08:47:33 UTC (rev 416) @@ -39,8 +39,11 @@ import com.google.common.collect.Iterables; /** + * Action d'archivage d'un mail. + * + * Passe le mail et statut archivé et déplace le mail dans le dossier d'archive le plus proche. + * * @author Kevin Morin (Code Lutin) - * */ public class ArchiveAction extends SaveDemandeAndExitAction { @@ -69,7 +72,11 @@ model.setDemandStatus(DemandStatus.ARCHIVED); model.setTakenBy(null); - /* FIXME see #5433: Supprimer les dossiers d'archives + // déplacement du mail dans le dossier d'archive le plus proche + // cela permet de déplacer les archives dans d'autre dossier pour leur appliquer + // des droits différents + // les dossiers d'archive ne sont pas visible dans l'arbre des dossiers, mais les archives + // peuvent être consultées dans la recherche MailFolder folder = model.getMailFolder(); MailFolder archiveChild = null; do { @@ -89,9 +96,11 @@ } while (archiveChild == null && folder != null); + // le dossier d'archive peut ne pas exister. Dans ce cas on laisse le mail dans le dossier + // d'origine if (archiveChild != null) { model.setMailFolder(archiveChild); - }*/ + } super.doAction(); } Modified: trunk/faxtomail-ui-web/src/main/webapp/WEB-INF/content/admin/configuration-input.jsp =================================================================== --- trunk/faxtomail-ui-web/src/main/webapp/WEB-INF/content/admin/configuration-input.jsp 2014-07-18 08:05:58 UTC (rev 415) +++ trunk/faxtomail-ui-web/src/main/webapp/WEB-INF/content/admin/configuration-input.jsp 2014-07-18 08:47:33 UTC (rev 416) @@ -240,9 +240,7 @@ <span class="glyphicon" ng-class="{'glyphicon-chevron-right': !collapsed, 'glyphicon-chevron-down': collapsed}"></span> </a> <a ng-click="editMailFolder(mailFolder)">{{mailFolder.name}}</a> - <%-- FIXME see #5433: Supprimer les dossiers d'archives <a class="pull-right btn btn-info btn-xs" data-nodrag ng-click="newArchiveSubFolder(this)" style="margin-left: 8px;" ng-show="!mailFolder.hasArchiveFolder && !mailFolder.archiveFolder" tooltip="Créer un dossier d'archive"><span class="glyphicon glyphicon-plus"></span></a> - --%> <a class="pull-right btn btn-primary btn-xs" data-nodrag ng-click="newSubFolder(this)" style="margin-left: 8px;" ng-show="!mailFolder.archiveFolder" tooltip="Créer un sous-dossier"><span class="glyphicon glyphicon-plus"></span></a> <span class="pull-right" tooltip="{{mailFolder.$cumulativeCount > 0 && 'Ce dossier contient ' + mailFolder.$cumulativeCount + ' demandes et ne peut pas être supprimé !'|| 'Supprimer ce dossier'}}"> <a class="btn btn-danger btn-xs" data-nodrag ng-click="deleteFolder(this)" ng-disabled="mailFolder.$cumulativeCount > 0"> @@ -808,7 +806,6 @@ </div> </div> </div> - <%-- FIXME see #5433: Supprimer les dossiers d'archives <div class="col-md-8" ng-show="selectedMailFolder.archiveFolder"> <h3>Édition du dossier {{selectedMailFolder.name}}</h3> @@ -888,7 +885,7 @@ </div> </div> </div> - </div>--%> + </div> </div> </div>

1 0

r415 - in trunk: faxtomail-persistence/src/main/java/com/franciaflex/faxtomail faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service faxtomail-ui-web/src/main/webapp/WEB-INF/content/admin
by echatellier＠users.forge.codelutin.com 18 Jul '14

18 Jul '14

Author: echatellier Date: 2014-07-18 10:05:58 +0200 (Fri, 18 Jul 2014) New Revision: 415 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/415 Log: Ajout de l'encodage requis pour le fichier Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfiguration.java trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfigurationOption.java trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ReferentielService.java trunk/faxtomail-ui-web/src/main/webapp/WEB-INF/content/admin/import-input.jsp Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfiguration.java =================================================================== --- trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfiguration.java 2014-07-18 07:56:13 UTC (rev 414) +++ trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfiguration.java 2014-07-18 08:05:58 UTC (rev 415) @@ -58,16 +58,6 @@ private static final Log log = LogFactory.getLog(FaxToMailConfiguration.class); - /*private static FaxToMailConfiguration instance; - - public static FaxToMailConfiguration getInstance() { - return instance; - } - - public static void setInstance(FaxToMailConfiguration instance) { - FaxToMailConfiguration.instance = instance; - }*/ - protected final String[] optionKeyToNotSave; protected File configFile; @@ -219,6 +209,10 @@ return jpaParameters; } + public String getImportFileEncoding() { + return applicationConfig.getOption(FaxToMailConfigurationOption.IMPORT_FILE_ENCODING.getKey()); + } + public String getSmtpHost() { return applicationConfig.getOption(FaxToMailConfigurationOption.SMTP_HOST.getKey()); } Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfigurationOption.java =================================================================== --- trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfigurationOption.java 2014-07-18 07:56:13 UTC (rev 414) +++ trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfigurationOption.java 2014-07-18 08:05:58 UTC (rev 415) @@ -24,10 +24,12 @@ * #L% */ +import org.apache.commons.io.Charsets; import org.nuiton.config.ConfigOptionDef; import org.nuiton.util.Version; import javax.swing.*; + import java.awt.*; import java.io.File; import java.net.URL; @@ -92,6 +94,11 @@ "Chemin vers le fichier de configuration des journaux", null, String.class), + IMPORT_FILE_ENCODING( + "faxtomail.import.file.encoding", + "Encodage des fichiers des fichiers d'import", + Charsets.UTF_8.toString(), String.class), + LDAP_MOCK( "faxtomail.ldap.mock", "Utilisation d'un service ldap mock", "false", Boolean.class), Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ReferentielService.java =================================================================== --- trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ReferentielService.java 2014-07-18 07:56:13 UTC (rev 414) +++ trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ReferentielService.java 2014-07-18 08:05:58 UTC (rev 415) @@ -25,6 +25,8 @@ */ import java.io.InputStream; +import java.io.InputStreamReader; +import java.io.UnsupportedEncodingException; import java.util.ArrayList; import java.util.Collection; import java.util.HashMap; @@ -116,7 +118,7 @@ Binder<Client, Client> clientBinder = BinderFactory.newBinder(Client.class); Import<Client> importer = null; try { - importer = Import.newImport(clientImportModel, inputStream); + importer = Import.newImport(clientImportModel, new InputStreamReader(inputStream, getApplicationConfig().getImportFileEncoding())); for (Client client : importer) { Client current = dao.forCodeEquals(client.getCode()).addEquals(Client.PROPERTY_BRAND, client.getBrand()).findAnyOrNull(); @@ -138,7 +140,7 @@ } getPersistenceContext().commit(); - } catch (ImportRuntimeException e) { + } catch (ImportRuntimeException|UnsupportedEncodingException e) { String message; if (e.getCause() != null) { message = e.getCause().getMessage(); @@ -146,7 +148,6 @@ message = e.getMessage(); } throw new ApplicationTechnicalException(message, e); - } finally { IOUtils.closeQuietly(importer); IOUtils.closeQuietly(inputStream); @@ -159,12 +160,12 @@ EmailAccountTopiaDao dao = getPersistenceContext().getEmailAccountDao(); EmailAccountImportModel emailAccountImportModel = new EmailAccountImportModel(';'); - try (Import<EmailAccount> importer = Import.newImport(emailAccountImportModel, inputStream)) { + try (Import<EmailAccount> importer = Import.newImport(emailAccountImportModel, new InputStreamReader(inputStream, getApplicationConfig().getImportFileEncoding()))) { for (EmailAccount emailAccount : importer) { result.add(dao.create(emailAccount)); } getPersistenceContext().commit(); - } catch (ImportRuntimeException e) { + } catch (ImportRuntimeException|UnsupportedEncodingException e) { String message; if (e.getCause() != null) { message = e.getCause().getMessage(); @@ -184,13 +185,13 @@ MailFilterTopiaDao dao = getPersistenceContext().getMailFilterDao(); EmailFilterImportModel emailFilterImportModel = new EmailFilterImportModel(';', foldersByName); - try (Import<MailFilter> importer = Import.newImport(emailFilterImportModel, inputStream)) { + try (Import<MailFilter> importer = Import.newImport(emailFilterImportModel, new InputStreamReader(inputStream, getApplicationConfig().getImportFileEncoding()))) { for (MailFilter emailFilter : importer) { result.add(dao.create(emailFilter)); } getPersistenceContext().commit(); - } catch (ImportRuntimeException e) { + } catch (ImportRuntimeException|UnsupportedEncodingException e) { String message; if (e.getCause() != null) { message = e.getCause().getMessage(); @@ -211,7 +212,7 @@ EtatAttenteTopiaDao etatAttenteTopiaDao = getPersistenceContext().getEtatAttenteDao(); EtatAttenteImportModel importModel = new EtatAttenteImportModel(';'); - try (Import<EtatAttente> importer = Import.newImport(importModel, inputStream)) { + try (Import<EtatAttente> importer = Import.newImport(importModel, new InputStreamReader(inputStream, getApplicationConfig().getImportFileEncoding()))) { for (EtatAttente etatAttente : importer) { EtatAttente currentEtatAttente = etatAttenteTopiaDao.forLabelEquals(etatAttente.getLabel()).findAnyOrNull(); @@ -224,7 +225,7 @@ } getPersistenceContext().commit(); - } catch (ImportRuntimeException e) { + } catch (ImportRuntimeException|UnsupportedEncodingException e) { String message; if (e.getCause() != null) { message = e.getCause().getMessage(); @@ -245,7 +246,7 @@ PriorityTopiaDao priorityTopiaDao = getPersistenceContext().getPriorityDao(); PriorityImportModel importModel = new PriorityImportModel(';'); - try (Import<Priority> importer = Import.newImport(importModel, inputStream)) { + try (Import<Priority> importer = Import.newImport(importModel, new InputStreamReader(inputStream, getApplicationConfig().getImportFileEncoding()))) { for (Priority priority : importer) { Priority currentPriority = priorityTopiaDao.forLabelEquals(priority.getLabel()).findAnyOrNull(); @@ -258,7 +259,7 @@ } getPersistenceContext().commit(); - } catch (ImportRuntimeException e) { + } catch (ImportRuntimeException|UnsupportedEncodingException e) { String message; if (e.getCause() != null) { message = e.getCause().getMessage(); @@ -279,7 +280,7 @@ RangeTopiaDao rangeTopiaDao = getPersistenceContext().getRangeDao(); RangeImportModel importModel = new RangeImportModel(';'); - try (Import<Range> importer = Import.newImport(importModel, inputStream)) { + try (Import<Range> importer = Import.newImport(importModel, new InputStreamReader(inputStream, getApplicationConfig().getImportFileEncoding()))) { for (Range range : importer) { Range currentRange = rangeTopiaDao.forLabelEquals(range.getLabel()).findAnyOrNull(); @@ -292,7 +293,7 @@ } getPersistenceContext().commit(); - } catch (ImportRuntimeException e) { + } catch (ImportRuntimeException|UnsupportedEncodingException e) { String message; if (e.getCause() != null) { message = e.getCause().getMessage(); @@ -312,7 +313,7 @@ DemandTypeTopiaDao demandTypeTopiaDao = getPersistenceContext().getDemandTypeDao(); DemandTypeImportModel importModel = new DemandTypeImportModel(';'); - try (Import<DemandType> importer = Import.newImport(importModel, inputStream)) { + try (Import<DemandType> importer = Import.newImport(importModel, new InputStreamReader(inputStream, getApplicationConfig().getImportFileEncoding()))) { for (DemandType demandType : importer) { DemandType currentDemandType = demandTypeTopiaDao.forLabelEquals(demandType.getLabel()).findAnyOrNull(); @@ -325,7 +326,7 @@ } getPersistenceContext().commit(); - } catch (ImportRuntimeException e) { + } catch (ImportRuntimeException|UnsupportedEncodingException e) { String message; if (e.getCause() != null) { message = e.getCause().getMessage(); Modified: trunk/faxtomail-ui-web/src/main/webapp/WEB-INF/content/admin/import-input.jsp =================================================================== --- trunk/faxtomail-ui-web/src/main/webapp/WEB-INF/content/admin/import-input.jsp 2014-07-18 07:56:13 UTC (rev 414) +++ trunk/faxtomail-ui-web/src/main/webapp/WEB-INF/content/admin/import-input.jsp 2014-07-18 08:05:58 UTC (rev 415) @@ -44,7 +44,8 @@ <div class="form-group"> <label for="clientField" class="control-label">Fichier client :</label> <input type="file" id="clientField" name="clientFile" class="form-control"> - <p class="help-block">Format du fichier csv : <code>Id_Correspondance;Societe;Nom;Caracteristique1;Caracteristique2;Caracteristique3;Numero_Fax;Adresse_Mail;Code_Client</code></p> + <p class="help-block">Format du fichier csv : <code>Id_Correspondance;Societe;Nom;Caracteristique1;Caracteristique2;Caracteristique3;Numero_Fax;Adresse_Mail;Code_Client</code><br /> + Encodage du fichier csv : <code>${applicationConfig.importFileEncoding}</code></p> </div> <button type="submit" class="btn btn-primary navbar-btn">Valider</button> <hr /> @@ -52,7 +53,8 @@ <div class="form-group"> <label for="etatAttenteField" class="control-label">Fichier état d'attente :</label> <input type="file" id="etatAttenteField" name="etatAttenteFile" class="form-control"> - <p class="help-block">Format du fichier csv : <code>etatattente</code></p> + <p class="help-block">Format du fichier csv : <code>etatattente</code><br /> + Encodage du fichier csv : <code>${applicationConfig.importFileEncoding}</code></p> </div> <button type="submit" class="btn btn-primary navbar-btn">Valider</button> <hr /> @@ -60,7 +62,8 @@ <div class="form-group"> <label for="demandTypeField" class="control-label">Fichier types de demande :</label> <input type="file" id="demandTypeField" name="demandTypeFile" class="form-control"> - <p class="help-block">Format du fichier csv : <code>demandetype</code></p> + <p class="help-block">Format du fichier csv : <code>demandetype</code><br /> + Encodage du fichier csv : <code>${applicationConfig.importFileEncoding}</code></p> </div> <button type="submit" class="btn btn-primary navbar-btn">Valider</button> <hr /> @@ -68,7 +71,8 @@ <div class="form-group"> <label for="rangeField" class="control-label">Fichier gamme :</label> <input type="file" id="rangeField" name="rangeFile" class="form-control"> - <p class="help-block">Format du fichier csv : <code>gamme</code></p> + <p class="help-block">Format du fichier csv : <code>gamme</code><br /> + Encodage du fichier csv : <code>${applicationConfig.importFileEncoding}</code></p> </div> <button type="submit" class="btn btn-primary navbar-btn">Valider</button> <hr /> @@ -76,7 +80,8 @@ <div class="form-group"> <label for="priorityField" class="control-label">Fichier priorite :</label> <input type="file" id="priorityField" name="priorityFile" class="form-control"> - <p class="help-block">Format du fichier csv : <code>priorite</code></p> + <p class="help-block">Format du fichier csv : <code>priorite</code><br /> + Encodage du fichier csv : <code>${applicationConfig.importFileEncoding}</code></p> </div> <button type="submit" class="btn btn-primary navbar-btn">Valider</button>

1 0