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

r426 - trunk/src/site/rst
by echatellier＠users.forge.codelutin.com July 18, 2014

July 18, 2014

Author: echatellier Date: 2014-07-18 16:08:45 +0200 (Fri, 18 Jul 2014) New Revision: 426 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/426 Log: Add index Added: trunk/src/site/rst/index.rst Added: trunk/src/site/rst/index.rst =================================================================== --- trunk/src/site/rst/index.rst (rev 0) +++ trunk/src/site/rst/index.rst 2014-07-18 14:08:45 UTC (rev 426) @@ -0,0 +1,29 @@ +.. - +.. * #%L +.. * FaxToMail +.. * $Id$ +.. * $HeadURL$ +.. * %% +.. * Copyright (C) 2014 Franciaflex, Code Lutin +.. * %% +.. * 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% +.. - + +Accueil +======= + +Bienvenue sur le site de FaxToMail. Il contient certaines documentation utilisateur et technique +concernant l'installation et l'utilisation de l'application FaxToMail. Property changes on: trunk/src/site/rst/index.rst ___________________________________________________________________ Added: svn:eol-style + native Added: svn:keywords + Author Date Id Revision HeadURL

1 0

r425 - in trunk/src: . site site/resources site/resources/img site/rst
by echatellier＠users.forge.codelutin.com July 18, 2014

July 18, 2014

Author: echatellier Date: 2014-07-18 16:04:42 +0200 (Fri, 18 Jul 2014) New Revision: 425 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/425 Log: Update web site. Add doc about csv encoding Added: trunk/src/site/ trunk/src/site/resources/ trunk/src/site/resources/img/ trunk/src/site/resources/img/notepad.png trunk/src/site/resources/img/txt-to-csv.png trunk/src/site/rst/ trunk/src/site/rst/import.rst trunk/src/site/site_fr.xml Removed: trunk/src/rst/ Modified: trunk/src/site/rst/install.rst trunk/src/site/rst/mssql.rst Added: trunk/src/site/resources/img/notepad.png =================================================================== (Binary files differ) Property changes on: trunk/src/site/resources/img/notepad.png ___________________________________________________________________ Added: svn:mime-type + image/png Added: trunk/src/site/resources/img/txt-to-csv.png =================================================================== (Binary files differ) Property changes on: trunk/src/site/resources/img/txt-to-csv.png ___________________________________________________________________ Added: svn:mime-type + image/png Added: trunk/src/site/rst/import.rst =================================================================== --- trunk/src/site/rst/import.rst (rev 0) +++ trunk/src/site/rst/import.rst 2014-07-18 14:04:42 UTC (rev 425) @@ -0,0 +1,98 @@ +.. - +.. * #%L +.. * FaxToMail +.. * $Id$ +.. * $HeadURL$ +.. * %% +.. * Copyright (C) 2014 Franciaflex, Code Lutin +.. * %% +.. * 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 par fichier CSV +====================== + +Format +------ + +Les fichiers d'import doivent être formater suivant le standart CSV (Comma-separated values) et +utiliser le séparateur ``;``. + +Exemple avec le fichier d'import des États d'attente:: + + etatattente + etat 1 + etat 2 + etat 3 + +Ce fichier ne comporte qu'une seule colonne ``etatattente``, les séparateurs ``;`` ne sont donc pas requis. + +Voici un exemple plus complet avec le fichier client:: + + Id_Correspondance;Societe;Nom;Caracteristique1;Caracteristique2;Caracteristique3;Numero_Fax;Adresse_Mail;Code_Client + 253042;FX;John Doe;A;71 Allo;NULL;03 12 34 56 78;john(a)doe.fr;99153 + 253142;FX;Gerard Menvussa;A+;40 Nabi;NULL;05 12 23 34 45;gerard.menvussa(a)code.com;99154 + 253242;FX;Ella Elli;NULL;38 LA;NULL;04 98 87 76 65;NULL;99184 + +Ce fichier comporte 9 colonnes séparée par le séparateur ``;``. Il comporte une ligne d'entete et +3 lignes de données. + + +Encodage +-------- + +Il est très difficile de déterminé le format d'encodage du fichier texte (CSV). L'application FaxToMail +impose donc l'utilisation du format d'encodage standardisé unicode (UTF-8). + +Cependant, certains système d'exploitation, comme Windows, n'utilisent pas ce standard. + +Il faut donc spécifier lors de l'enregistrement du fichier: + * l'encodage des caractères à utiliser + * le format du fichier (séparateur ``;``, délimiteur...) + +notepad++ +~~~~~~~~~ + +Pour éditer un fichier au format texte, le logiciel `notepad++`_ est plus évolué que les outils +basique mis à la disposition par défaut par Windows. Il dispose de plus d'une option pour spécifier +le format d'encodage lors de la sauvegarde. + +.. _notepad++: http://notepad-plus-plus.org/ + +.. image:: img/notepad.png + +Openoffice/Libreoffice +~~~~~~~~~~~~~~~~~~~~~~ + +Openoffice et Libreoffice sont deux equivalents libre de la suite bureautique Microsoft Office. Ce +logiciel propose plus facilement les options d'encodage lors de l'enregistrement du fichier CSV. + +Il s'agit d'une simple option à sélection dans la fenêtre de sauvegarde. + +Excel +~~~~~ + +Pour spécifier l'encodage sous Microsoft Excel, c'est un peu compliqué car l'option n'est pas +facilement accessible. + +Pour pouvoir changer cette option, il faut: + * ouvrir la boite de dialogue "Enregistrer sous..." + * Choisir de sauver le document en type "Tous les fichiers (*.*) + * modifier le nom du fichier en xxx.csv + +comme montré dans cette image: + +.. image:: http://www.ablebits.com/_img-blog/excel-csv/txt-to-csv.png Property changes on: trunk/src/site/rst/import.rst ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native Modified: trunk/src/site/rst/install.rst =================================================================== --- trunk/src/rst/install.rst 2014-07-18 09:29:44 UTC (rev 417) +++ trunk/src/site/rst/install.rst 2014-07-18 14:04:42 UTC (rev 425) @@ -28,33 +28,44 @@ Téléchargement -------------- - * Installer Java JRE 7 (minimum) : http://www.oracle.com/technetwork/java/javase/downloads/jre7-downloads-1880… - * Télécharger Tomcat 7 : http://apache.crihan.fr/dist/tomcat/tomcat-7/v7.0.53/bin/apache-tomcat-7.0.… - * Télécharger Apache 2.4 : http://www.apachelounge.com/download/VC11/binaries/httpd-2.4.9-win64-VC11.z… +* Installer Java JRE 7 (minimum) : http://www.oracle.com/technetwork/java/javase/downloads/jre7-downloads-1880… +* Télécharger Tomcat 7 : http://apache.crihan.fr/dist/tomcat/tomcat-7/v7.0.53/bin/apache-tomcat-7.0.… +* Télécharger Apache 2.4 : http://www.apachelounge.com/download/VC11/binaries/httpd-2.4.9-win64-VC11.z… - * Installer Java - * Installer tomcat. Tomcat s'installe automatiquement en tant que service windows. - * Décompresser httpd-2.4.9-win64-VC11.zip dans ''C:\Apache24'' - * Installer apache en tant que service windows : ''httpd.exe -k install'' - * Si un message à propos de msvcr110.dll s'affiche, c'est que "Microsoft Visual C++ Redistributable" n'est pas installé - * http://answers.microsoft.com/en-us/windows/forum/windows_8-winapps/the-prog… - * http://www.microsoft.com/en-us/download/details.aspx?id=30679 - * Démarrer les services "Apache tomcat 7.0 Tomcat7" et "Apache2.4" +* Installer Java +* Installer tomcat. Tomcat s'installe automatiquement en tant que service windows. +* Décompresser httpd-2.4.9-win64-VC11.zip dans ''C:\Apache24'' +* Installer apache en tant que service windows : ''httpd.exe -k install'': + * Si un message à propos de msvcr110.dll s'affiche, c'est que "Microsoft Visual C++ Redistributable" n'est pas installé + * http://answers.microsoft.com/en-us/windows/forum/windows_8-winapps/the-prog… + * http://www.microsoft.com/en-us/download/details.aspx?id=30679 +* Démarrer les services "Apache tomcat 7.0 Tomcat7" et "Apache2.4" + À ce stade, les adresses http://localhost/ et http://localhost:8080 doivent répondre. Apache ------ - * Dans le fichier 'C:\Apache24\conf\httpd.conf', décommenter les lignes:: +Étapes: +* Dans le fichier 'C:\Apache24\conf\httpd.conf', décommenter les lignes + +:: + LoadModule proxy_module modules/mod_proxy.so LoadModule proxy_ajp_module modules/mod_proxy_ajp.so - * Ajouter à la fin du fichier:: +* Ajouter à la fin du fichier + +:: + Include conf/extra/httpd-faxtomail.conf - * Créer le fichier ''conf/extra/httpd-faxtomail.conf'' avec en contenu:: +* Créer le fichier ''conf/extra/httpd-faxtomail.conf'' avec en contenu + +:: + <VirtualHost *:80> ServerAdmin webmaster(a)franciaflex.com ServerName 192.168.100.247 @@ -74,10 +85,14 @@ Tomcat ------ - * Télécharger le war depuis l'adresse : http://forge.codelutin.com/projects/faxtomail/files - * Enregistrez le dans le dossier ''C:\Program Files\Apache Software Foundation\Tomcat 7.0\webapps\'' avec le nom ''ROOT.war'' (supprimer le dossier ROOT) - * Créer le fichier ''C:\Windows\System32\faxToMail.properties'' avec en contenu:: +* Télécharger le war depuis l'adresse : http://forge.codelutin.com/projects/faxtomail/files +* Enregistrez le dans le dossier ''C:\Program Files\Apache Software Foundation\Tomcat 7.0\webapps\'' avec le nom ''ROOT.war'' (supprimer le dossier ROOT) +* Créer le fichier ''C:\Windows\System32\faxToMail.properties'' avec en contenu + +:: + faxtomail.storage=C:\\faxtomail\\data - * Relancer le service ''Apache Tomcat 7.0 Tomcat7'' +* Relancer le service ''Apache Tomcat 7.0 Tomcat7'' + L'application doit maintenant être installée et répondre sur l'adresse : http://192.168.100.247/ Modified: trunk/src/site/rst/mssql.rst =================================================================== --- trunk/src/rst/mssql.rst 2014-07-18 09:29:44 UTC (rev 417) +++ trunk/src/site/rst/mssql.rst 2014-07-18 14:04:42 UTC (rev 425) @@ -37,5 +37,5 @@ hibernate.dialect=org.hibernate.dialect.SQLServer2008Dialect hibernate.connection.driver_class=net.sourceforge.jtds.jdbc.Driver hibernate.connection.url=jdbc:jtds:sqlserver://192.168.100.247:1433/faxtomail - hibernate.connection.username=fx - hibernate.connection.password=FX2013! + hibernate.connection.username=user + hibernate.connection.password=pass Added: trunk/src/site/site_fr.xml =================================================================== --- trunk/src/site/site_fr.xml (rev 0) +++ trunk/src/site/site_fr.xml 2014-07-18 14:04:42 UTC (rev 425) @@ -0,0 +1,41 @@ +<?xml version="1.0" encoding="UTF-8"?> + +<project name="${project.name}"> + + <skin> + <groupId>org.apache.maven.skins</groupId> + <artifactId>maven-fluido-skin</artifactId> + <version>1.3.1</version> + </skin> + + <publishDate format="dd/MM/yyyy" position="right" /> + <version position="right" /> + + <poweredBy> + <logo href="http://maven.apache.org" name="Maven" img="images/logos/maven-feather.png" /> + </poweredBy> + + <body> + + <breadcrumbs> + <item name="${project.name}" href="${project.url}" /> + </breadcrumbs> + + <menu name="Documentation"> + <item name="Accueil" href="index.html" /> + + <item name="Utilisateur" href="import.html"> + <item name="Import" href="import.html" /> + </item> + + <item name="Développeur" href="install.html"> + <item name="Installation" href="install.html" /> + <item name="SQL Serveur" href="mssql.html" /> + </item> + </menu> + + <menu ref="reports" /> + + </body> + +</project> Property changes on: trunk/src/site/site_fr.xml ___________________________________________________________________ Added: svn:mime-type + text/xml Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native

1 0

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

July 18, 2014

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 July 18, 2014

July 18, 2014

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 July 18, 2014

July 18, 2014

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 July 18, 2014

July 18, 2014

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 July 18, 2014

July 18, 2014

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 July 18, 2014

July 18, 2014

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 July 18, 2014

July 18, 2014

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 July 18, 2014

July 18, 2014

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