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; }