Author: echatellier Date: 2014-07-10 16:46:57 +0200 (Thu, 10 Jul 2014) New Revision: 381 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/381 Log: Ajout de la valorisation des getValidRangeRowModels() pour la validation. Et plein de FIXME parce que l'interet est pas evident 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/content/demande/DemandeUIHandler.java 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/actions/LoadFolderEmailsAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/LoadFolderEmailsAction.java 2014-07-10 14:04:18 UTC (rev 380) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/LoadFolderEmailsAction.java 2014-07-10 14:46:57 UTC (rev 381) @@ -26,10 +26,12 @@ import com.franciaflex.faxtomail.persistence.entities.Email; import com.franciaflex.faxtomail.persistence.entities.MailFolder; +import com.franciaflex.faxtomail.persistence.entities.RangeRow; import com.franciaflex.faxtomail.ui.swing.content.demande.DemandeListUI; import com.franciaflex.faxtomail.ui.swing.content.demande.DemandeListUIHandler; import com.franciaflex.faxtomail.ui.swing.content.demande.DemandeListUIModel; import com.franciaflex.faxtomail.ui.swing.content.demande.DemandeUIModel; +import com.franciaflex.faxtomail.ui.swing.content.demande.RangeRowModel; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; @@ -37,6 +39,7 @@ import org.nuiton.util.pagination.PaginationResult; import java.util.ArrayList; +import java.util.Collection; import java.util.List; import static org.nuiton.i18n.I18n.t; @@ -86,6 +89,22 @@ for (Email email : emails) { 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 + Collection<RangeRow> rangeRows = demand.getRangeRow(); + if (rangeRows != null) { + 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); + } + } + // / getValidRangeRowModels() + demand.setValid(handler.isDemandeValid(demand)); demands.add(demand); } 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-10 14:04:18 UTC (rev 380) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIHandler.java 2014-07-10 14:46:57 UTC (rev 381) @@ -181,6 +181,9 @@ 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é 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-10 14:04:18 UTC (rev 380) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java 2014-07-10 14:46:57 UTC (rev 381) @@ -681,10 +681,10 @@ if (getRangeRow() != null) { oldValue = new ArrayList<RangeRow>(getRangeRow()); } - String refOldValue = getReference(); + String refOldValue = getReference(); // FIXME : why ? editObject.removeRangeRow(rangeRow); firePropertyChange(Email.PROPERTY_RANGE_ROW, oldValue, getRangeRow()); - firePropertyChanged(PROPERTY_REFERENCE, refOldValue, getReference()); + firePropertyChanged(PROPERTY_REFERENCE, refOldValue, getReference()); // FIXME : why ? } public void setRangeRow(Collection<RangeRow> rangeRow) { @@ -695,10 +695,10 @@ if (rangeRow == null) { rangeRow = new ArrayList<RangeRow>(); } - String refOldValue = getReference(); + String refOldValue = getReference(); // FIXME : why ? editObject.setRangeRow(rangeRow); firePropertyChange(Email.PROPERTY_RANGE_ROW, oldValue, getRangeRow()); - firePropertyChanged(PROPERTY_REFERENCE, refOldValue, getReference()); + firePropertyChanged(PROPERTY_REFERENCE, refOldValue, getReference()); // FIXME : why ? } public MailFolder getMailFolder() {