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

r714 - in trunk: faxtomail-persistence/src/main/java/com/franciaflex/faxtomail faxtomail-service/src/main/java/com/franciaflex/faxtomail/services faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ldap faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions faxtomail-ui-swing/src/main/java/com/franciafle
by kmorin＠users.forge.codelutin.com 03 Dec '14

03 Dec '14

Author: kmorin Date: 2014-12-03 08:32:52 +0000 (Wed, 03 Dec 2014) New Revision: 714 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/714 Log: refs #6234 Ajouter une sorte de carnet des adresses email du ldap dans l'?\195?\169cran de r?\195?\169ponse fixes #6235 Ajouter un cache sur les emails des dossiers Added: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ldap/Contact.java 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/DecoratorServiceImpl.java trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/LdapService.java trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/LdapServiceImpl.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/FaxToMailUIContext.java 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/ArchiveFromListAction.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/actions/ReplyAction.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/TransmitAction.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUI.css trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUI.jaxx trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIHandler.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIModel.java trunk/faxtomail-ui-swing/src/main/resources/i18n/faxtomail-ui-swing_fr_FR.properties Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfiguration.java =================================================================== --- trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfiguration.java 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfiguration.java 2014-12-03 08:32:52 UTC (rev 714) @@ -191,6 +191,11 @@ return isDevMode; } + public boolean isUseFolderCache() { + boolean isUseFolderCache = applicationConfig.getOptionAsBoolean(FaxToMailConfigurationOption.USE_FOLDER_CACHE.getKey()); + return isUseFolderCache; + } + /** * Get sub properties containing all options starting with "hibernate" and "topia". * Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfigurationOption.java =================================================================== --- trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfigurationOption.java 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/FaxToMailConfigurationOption.java 2014-12-03 08:32:52 UTC (rev 714) @@ -89,6 +89,11 @@ "Mode développement, court-circuite l'envoi de mail", "false", Boolean.class), + USE_FOLDER_CACHE( + "faxtomail.useFolderCache", + "Utilisation du cache des emails des dossiers, pour éviter de recharger les éléments quand on revient sur un dossier", + "false", Boolean.class), + LOG_CONFIGURATION_FILE( "logConfigurationFile", "Chemin vers le fichier de configuration des journaux", Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/DecoratorServiceImpl.java =================================================================== --- trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/DecoratorServiceImpl.java 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/DecoratorServiceImpl.java 2014-12-03 08:32:52 UTC (rev 714) @@ -27,6 +27,7 @@ import java.text.DateFormat; import java.util.Date; +import com.franciaflex.faxtomail.services.service.ldap.Contact; import org.nuiton.decorator.Decorator; import org.nuiton.decorator.DecoratorProvider; @@ -112,6 +113,7 @@ return DateFormat.getDateInstance(DateFormat.MEDIUM).format(bean); } }); + registerMultiJXPathDecorator(Contact.class, "${name}$s <${email}$s>", SEPARATOR, " - "); registerMultiJXPathDecorator(HasLabel.class, "${label}$s", SEPARATOR, " - "); registerDecorator(new Decorator<Attachment>(Attachment.class) { private static final long serialVersionUID = 1L; Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/LdapService.java =================================================================== --- trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/LdapService.java 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/LdapService.java 2014-12-03 08:32:52 UTC (rev 714) @@ -27,7 +27,10 @@ import com.franciaflex.faxtomail.persistence.entities.FaxToMailUser; import com.franciaflex.faxtomail.services.FaxToMailService; import com.franciaflex.faxtomail.services.service.ldap.AuthenticationException; +import com.franciaflex.faxtomail.services.service.ldap.Contact; +import java.util.List; + /** * @author Kevin Morin (Code Lutin) */ @@ -39,4 +42,6 @@ FaxToMailUser authenticateUser(String login, String password) throws AuthenticationException; FaxToMailUser getUserFromPrincipal(String login) throws AuthenticationException; + + List<Contact> getUserAndEmails(); } Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/LdapServiceImpl.java =================================================================== --- trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/LdapServiceImpl.java 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/LdapServiceImpl.java 2014-12-03 08:32:52 UTC (rev 714) @@ -24,11 +24,13 @@ * #L% */ +import java.util.ArrayList; import java.util.Collection; import java.util.HashMap; import java.util.List; import java.util.Map; +import com.franciaflex.faxtomail.services.service.ldap.Contact; import org.apache.commons.lang3.ArrayUtils; import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; @@ -392,4 +394,54 @@ return user; } + + public List<Contact> getUserAndEmails() { + List<Contact> result = new ArrayList<>(); + LDAPConnection connection = null; + try { + connection = new LDAPConnection(getApplicationConfig().getLdapHost(), + getApplicationConfig().getLdapPort(), + getApplicationConfig().getLdapUser(), + getApplicationConfig().getLdapPassword()); + if (connection.isConnected()) { + + FaxToMailUserTopiaDao faxtomailUserDao = getPersistenceContext().getFaxToMailUserDao(); + FaxToMailUserGroupTopiaDao faxtomailUserGroupDao = getPersistenceContext().getFaxToMailUserGroupDao(); + + // get all users ( + String usersBaseDN = "OU=Mac-Groupe,DC=mac-groupe,DC=net"; + String usersFilter = "(objectCategory=CN=Person,CN=Schema,CN=Configuration,DC=mac-groupe,DC=net)"; + + // on recupere d'abord les organisationUnit pour fractionner les requettes + // sinon au dela de 1000 resultats ca veux plus + SearchResult usersResult = connection.search(usersBaseDN, SearchScope.SUB, "(objectClass=organizationalUnit)"); + List<SearchResultEntry> unitEntries = usersResult.getSearchEntries(); + for (SearchResultEntry unitEntry : unitEntries) { + if (log.isDebugEnabled()) { + log.debug("Search for unit " + unitEntry.getDN()); + } + SearchResult userResult = connection.search(unitEntry.getDN(), SearchScope.ONE, usersFilter); + + List<SearchResultEntry> userEntries = userResult.getSearchEntries(); + for (SearchResultEntry userEntry : userEntries) { + + String name = userEntry.getAttributeValue("displayname"); + String email = userEntry.getAttributeValue("mail"); + if (StringUtils.isNotBlank(email)) { + result.add(new Contact(name, email)); + } + } + } + } + + } catch (LDAPException ex) { + throw new RuntimeException("Can't connect to ldap", ex); + } finally { + if (connection != null) { + connection.close(); + } + } + + return result; + } } Added: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ldap/Contact.java =================================================================== --- trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ldap/Contact.java (rev 0) +++ trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/ldap/Contact.java 2014-12-03 08:32:52 UTC (rev 714) @@ -0,0 +1,41 @@ +package com.franciaflex.faxtomail.services.service.ldap; + +import org.jdesktop.beans.AbstractBean; + +/** + * @author Kevin Morin (Code Lutin) + * @since x.x + */ +public class Contact extends AbstractBean { + + public static final String PROPERTY_NAME = "name"; + public static final String PROPERTY_EMAIL = "email"; + + protected String name; + protected String email; + + public Contact(String name, String email) { + this.name = name; + this.email = email; + } + + public String getName() { + return name; + } + + public void setName(String name) { + Object oldValue = getName(); + this.name = name; + firePropertyChange(PROPERTY_NAME, oldValue, name); + } + + public String getEmail() { + return email; + } + + public void setEmail(String email) { + Object oldValue = getEmail(); + this.email = email; + firePropertyChange(PROPERTY_NAME, oldValue, email); + } +} Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/FaxToMailUIContext.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/FaxToMailUIContext.java 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/FaxToMailUIContext.java 2014-12-03 08:32:52 UTC (rev 714) @@ -30,6 +30,8 @@ import java.io.Closeable; import java.io.File; import java.util.ArrayList; +import java.util.Collection; +import java.util.Date; import java.util.List; import java.util.Locale; import java.util.Map; @@ -38,6 +40,8 @@ import javax.swing.JFrame; import javax.swing.JOptionPane; +import com.franciaflex.faxtomail.persistence.entities.Email; +import com.franciaflex.faxtomail.services.service.ldap.Contact; import jaxx.runtime.swing.editor.bean.BeanDoubleList; import jaxx.runtime.swing.editor.bean.BeanFilterableComboBox; import jaxx.runtime.swing.session.BeanDoubleListState; @@ -92,6 +96,7 @@ import com.google.common.collect.Maps; import com.google.common.collect.Sets; import org.nuiton.util.pagination.PaginationParameter; +import org.nuiton.util.pagination.PaginationResult; /** * UI application context. @@ -197,6 +202,11 @@ /** L'email courant (pour quoi faire ?). */ protected DemandeUIModel currentEmail; + //FIXME rustine en attendant de trouver pourquoi ca rame autant + protected boolean useFolderCache; + + protected FolderData currentFolderEmails = new FolderData(); + /** Les dossiers étendus (pour pouvoir les étendres encore au rechargement de l'UI). */ protected List<MailFolder> expandedFolders = new ArrayList<MailFolder>(); @@ -212,6 +222,8 @@ protected List<Priority> priorityCache; /** Range cache. */ protected List<Range> rangeCache; + /** */ + protected List<Contact> contactCache; public static FaxToMailUIContext newContext(FaxToMailConfiguration config) { Preconditions.checkNotNull(config); @@ -245,6 +257,8 @@ topiaApplicationContext = new FaxToMailTopiaApplicationContext(config.getTopiaProperties()); FaxToMailServiceContext serviceContext = newServiceContext(); + useFolderCache = serviceContext.getApplicationConfig().isUseFolderCache(); + InitFaxToMailService initFaxToMailService = serviceContext.getInitFaxToMailService(); initFaxToMailService.init(); @@ -399,6 +413,15 @@ // users UserService userService = serviceContext.getUserService(); setFaxtomailUserCache(userService.getAllActiveUsers()); + + try { + setContactCache(serviceContext.getLdapService().getUserAndEmails()); + + } catch (Exception e) { + if (log.isErrorEnabled()) { + log.error("Error while getting contact from ldap", e); + } + } } /** @@ -726,6 +749,10 @@ this.currentEmail = currentEmail; } + public FolderData getCurrentFolderEmails() { + return currentFolderEmails; + } + public List<MailFolder> getExpandedFolders() { return expandedFolders; } @@ -781,4 +808,55 @@ public void setRangeCache(List<Range> rangeCache) { this.rangeCache = rangeCache; } + + public List<Contact> getContactCache() { + return contactCache; + } + + public void setContactCache(List<Contact> contactCache) { + this.contactCache = contactCache; + } + + public boolean isUseFolderCache() { + return useFolderCache; + } + + public class FolderData { + + protected MailFolder folder; + protected List<DemandeUIModel> demands; + protected PaginationResult<Email> paginationResult; + + public MailFolder getFolder() { + return folder; + } + + public void setFolder(MailFolder folder) { + this.folder = folder; + } + + public List<DemandeUIModel> getDemands() { + return demands; + } + + public void setDemands(List<DemandeUIModel> demands) { + this.demands = new ArrayList<>(demands); + } + + public void removeDemand(DemandeUIModel demand) { + this.demands.remove(demand); + } + + public void removeAllDemands(Collection<DemandeUIModel> demands) { + this.demands.removeAll(demands); + } + + public PaginationResult<Email> getPaginationResult() { + return paginationResult; + } + + public void setPaginationResult(PaginationResult<Email> paginationResult) { + this.paginationResult = paginationResult; + } + } } 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-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveAction.java 2014-12-03 08:32:52 UTC (rev 714) @@ -103,4 +103,13 @@ super.doAction(); } + + @Override + public void postSuccessAction() { + //FIXME enlever la rustine + if (getContext().isUseFolderCache()) { + getContext().getCurrentFolderEmails().removeDemand(getModel()); + } + super.postSuccessAction(); + } } Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveFromListAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveFromListAction.java 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ArchiveFromListAction.java 2014-12-03 08:32:52 UTC (rev 714) @@ -145,7 +145,12 @@ @Override public void postSuccessAction() { super.postSuccessAction(); - getModel().removeEmails(getModel().getSelectedEmails()); + List<DemandeUIModel> selectedEmails = getModel().getSelectedEmails(); + getModel().removeEmails(selectedEmails); + //FIXME enlever la rustine + if (getContext().isUseFolderCache()) { + getContext().getCurrentFolderEmails().removeAllDemands(selectedEmails); + } getModel().setSelectedEmails(null); } 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-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/LoadFolderEmailsAction.java 2014-12-03 08:32:52 UTC (rev 714) @@ -34,6 +34,7 @@ import com.franciaflex.faxtomail.persistence.entities.WaitingState; import com.franciaflex.faxtomail.persistence.entities.MailField; +import com.franciaflex.faxtomail.ui.swing.FaxToMailUIContext; import com.franciaflex.faxtomail.ui.swing.util.DemandeTableModel; import com.google.common.collect.Maps; import com.google.common.collect.Sets; @@ -81,20 +82,31 @@ result = false; } else { - final JXTable dataTable = getUI().getDataTable(); + //FIXME enlever la rustine + FaxToMailUIContext.FolderData currentFolderEmails = getContext().getCurrentFolderEmails(); + if (getContext().isUseFolderCache() && folder.equals(currentFolderEmails.getFolder())) { + model.setEmails(new ArrayList<DemandeUIModel>(currentFolderEmails.getDemands())); + PaginationResult<Email> paginationResult = currentFolderEmails.getPaginationResult(); + model.setPaginationParameter(paginationResult.getCurrentPage()); + model.setPaginationResult(paginationResult); + result = false; - setActionDescription(t("faxtomail.action.loadFolderEmails.tip", folder.getName())); + } else { + final JXTable dataTable = getUI().getDataTable(); - // change name to save the state of the column width for every folder - // (as every folder can have a different header) - getContext().getSwingSession().updateState(); - String columns = handler.populateColumnModel(dataTable, false); - dataTable.setName("dataTable" + columns); - getContext().getSwingSession().add(dataTable, true); + setActionDescription(t("faxtomail.action.loadFolderEmails.tip", folder.getName())); - // fixes #5528 filtre sur les colonnes : il manque l'icone qui indique que la colonne est filtrée - handler.initTableFilter(); - result = true; + // change name to save the state of the column width for every folder + // (as every folder can have a different header) + getContext().getSwingSession().updateState(); + String columns = handler.populateColumnModel(dataTable, false); + dataTable.setName("dataTable" + columns); + getContext().getSwingSession().add(dataTable, true); + + // fixes #5528 filtre sur les colonnes : il manque l'icone qui indique que la colonne est filtrée + handler.initTableFilter(); + result = true; + } } return result; @@ -156,6 +168,14 @@ demands.add(demand); } + //FIXME enlever la rustine + if (getContext().isUseFolderCache()) { + FaxToMailUIContext.FolderData currentFolderEmails = getContext().getCurrentFolderEmails(); + currentFolderEmails.setFolder(folder); + currentFolderEmails.setDemands(demands); + currentFolderEmails.setPaginationResult(paginationResult); + } + model.setEmails(demands); model.setPaginationParameter(paginationResult.getCurrentPage()); model.setPaginationResult(paginationResult); Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ReplyAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ReplyAction.java 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/ReplyAction.java 2014-12-03 08:32:52 UTC (rev 714) @@ -58,8 +58,6 @@ @Override public void doAction() throws Exception { - log.info("do reply"); - ReplyFormUIModel model = getModel(); FaxToMailServiceContext serviceContext = getContext().newServiceContext(); EmailService emailService = serviceContext.getEmailService(); Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/TransmitAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/TransmitAction.java 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/TransmitAction.java 2014-12-03 08:32:52 UTC (rev 714) @@ -118,6 +118,11 @@ @Override public void postSuccessAction() { + //FIXME enlever rustine + if (getContext().isUseFolderCache()) { + getContext().getCurrentFolderEmails().removeAllDemands(getModel().getDemandeUIModels()); + } + super.postSuccessAction(); MailFolderChooserUIHandler handler = getHandler(); Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUI.css =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUI.css 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUI.css 2014-12-03 08:32:52 UTC (rev 714) @@ -39,6 +39,43 @@ selectedItem: { model.getFrom() }; } +.contact-row { + visible: { !model.isReadonly() }; +} + +#contactLabel { + text: "faxtomail.reply.label.contact"; +} + +#contactField { + showReset: true; + showDecorator: false; + bean: {model}; + property: contact; + selectedItem: { model.getContact() }; +} + +#addToButton { + actionIcon: add; + text: "faxtomail.reply.action.addTo"; + toolTipText: "faxtomail.reply.action.addTo.tip"; + enabled: { model.getContact() != null }; +} + +#addCcButton { + actionIcon: add; + text: "faxtomail.reply.action.addCc"; + toolTipText: "faxtomail.reply.action.addCc.tip"; + enabled: { model.getContact() != null }; +} + +#addCciButton { + actionIcon: add; + text: "faxtomail.reply.action.addCci"; + toolTipText: "faxtomail.reply.action.addCci.tip"; + enabled: { model.getContact() != null }; +} + #toLabel { text: "faxtomail.reply.label.to"; } Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUI.jaxx =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUI.jaxx 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUI.jaxx 2014-12-03 08:32:52 UTC (rev 714) @@ -23,6 +23,7 @@ implements='com.franciaflex.faxtomail.ui.swing.util.FaxToMailUI<ReplyFormUIModel, ReplyFormUIHandler>'> <import> + com.franciaflex.faxtomail.services.service.ldap.Contact com.franciaflex.faxtomail.ui.swing.util.FaxToMailUI com.franciaflex.faxtomail.ui.swing.util.FaxToMailUIUtil @@ -31,6 +32,7 @@ java.awt.FlowLayout jaxx.runtime.swing.CardLayout2Ext + jaxx.runtime.swing.editor.bean.BeanFilterableComboBox static org.nuiton.i18n.I18n.t </import> @@ -66,7 +68,7 @@ <cell> <JLabel id="fromLabel"/> </cell> - <cell weightx='1'> + <cell weightx='1' columns="4"> <JPanel id="fromPanel" layout="{ fromPanelLayout }"> <JTextField id="fromField" @@ -79,9 +81,35 @@ </row> <row> <cell> + <JLabel id="contactLabel" styleClass="contact-row"/> + </cell> + <cell weightx='1'> + <BeanFilterableComboBox id="contactField" + constructorParams='this' + genericType="Contact" + styleClass="contact-row"/> + </cell> + <cell> + <JButton id="addToButton" + styleClass="contact-row" + onActionPerformed="handler.addTo()"/> + </cell> + <cell> + <JButton id="addCcButton" + styleClass="contact-row" + onActionPerformed="handler.addCc()"/> + </cell> + <cell> + <JButton id="addCciButton" + styleClass="contact-row" + onActionPerformed="handler.addCci()"/> + </cell> + </row> + <row> + <cell> <JLabel id="toLabel"/> </cell> - <cell weightx='1'> + <cell weightx='1' columns="4"> <JTextField id="toField" onKeyReleased='handler.setText(event, "to")'/> </cell> @@ -90,7 +118,7 @@ <cell> <JLabel id="ccLabel"/> </cell> - <cell weightx='1'> + <cell weightx='1' columns="4"> <JTextField id="ccField" onKeyReleased='handler.setText(event, "cc")'/> </cell> @@ -99,7 +127,7 @@ <cell> <JLabel id="cciLabel"/> </cell> - <cell weightx='1'> + <cell weightx='1' columns="4"> <JTextField id="cciField" onKeyReleased='handler.setText(event, "cci")'/> </cell> @@ -108,7 +136,7 @@ <cell> <JLabel id="subjectLabel"/> </cell> - <cell weightx='1'> + <cell weightx='1' columns="4"> <JTextField id="subjectField" onKeyReleased='handler.setText(event, "subject")'/> </cell> Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIHandler.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIHandler.java 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIHandler.java 2014-12-03 08:32:52 UTC (rev 714) @@ -41,11 +41,13 @@ import javax.swing.*; +import com.franciaflex.faxtomail.services.service.ldap.Contact; import jaxx.runtime.JAXXUtil; import jaxx.runtime.SwingUtil; import jaxx.runtime.validator.swing.SwingValidator; import org.apache.commons.io.FileUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.nuiton.jaxx.application.swing.util.Cancelable; @@ -167,6 +169,8 @@ ui.getFromComboBox().setModel(comboBoxModel); model.setSenderAllowedAddresses(replyToAddresses); + initBeanFilterableComboBox(ui.getContactField(), getContext().getContactCache(), null); + } else { model.setFrom(recipient); } @@ -443,4 +447,49 @@ } } + + public void addTo() { + ReplyFormUIModel model = getModel(); + Contact contact = model.getContact(); + if (contact != null) { + String to = model.getTo(); + String email = contact.getEmail(); + if (StringUtils.isNotBlank(to)) { + to += ";" + email; + } else { + to = email; + } + model.setTo(to); + } + } + + public void addCc() { + ReplyFormUIModel model = getModel(); + Contact contact = model.getContact(); + if (contact != null) { + String cc = model.getCc(); + String email = contact.getEmail(); + if (StringUtils.isNotBlank(cc)) { + cc += ";" + email; + } else { + cc = email; + } + model.setCc(cc); + } + } + + public void addCci() { + ReplyFormUIModel model = getModel(); + Contact contact = model.getContact(); + if (contact != null) { + String cci = model.getCci(); + String email = contact.getEmail(); + if (StringUtils.isNotBlank(cci)) { + cci += ";" + email; + } else { + cci = email; + } + model.setCci(cci); + } + } } Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIModel.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIModel.java 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIModel.java 2014-12-03 08:32:52 UTC (rev 714) @@ -30,6 +30,7 @@ import java.util.List; import java.util.Set; +import com.franciaflex.faxtomail.services.service.ldap.Contact; import org.apache.commons.collections4.CollectionUtils; import org.apache.commons.io.FileUtils; import org.jdesktop.beans.AbstractSerializableBean; @@ -45,6 +46,7 @@ */ public class ReplyFormUIModel extends AbstractSerializableBean { + public static final String PROPERTY_CONTACT = "contact"; public static final String PROPERTY_TO = "to"; public static final String PROPERTY_CC = "cc"; public static final String PROPERTY_CCI = "cci"; @@ -62,6 +64,7 @@ public static final String PROPERTY_EDITABLE = "editable"; public static final String PROPERTY_READ_SENT_DATE = "readSentDate"; + protected Contact contact; protected String to; protected String cc; protected String cci; @@ -96,6 +99,16 @@ copyBinder.copyExcluding(other, this); } + public Contact getContact() { + return contact; + } + + public void setContact(Contact contact) { + Object oldValue = getContact(); + this.contact = contact; + firePropertyChange(PROPERTY_CONTACT, oldValue, contact); + } + public String getTo() { return to; } Modified: trunk/faxtomail-ui-swing/src/main/resources/i18n/faxtomail-ui-swing_fr_FR.properties =================================================================== --- trunk/faxtomail-ui-swing/src/main/resources/i18n/faxtomail-ui-swing_fr_FR.properties 2014-12-02 18:39:34 UTC (rev 713) +++ trunk/faxtomail-ui-swing/src/main/resources/i18n/faxtomail-ui-swing_fr_FR.properties 2014-12-03 08:32:52 UTC (rev 714) @@ -264,6 +264,12 @@ faxtomail.rangeRows.table.header.range.tip=Gamme faxtomail.rangeRows.table.header.savQuantity=Qté SAV faxtomail.rangeRows.table.header.savQuantity.tip=Quantité de SAV +faxtomail.reply.action.addCc=Copie +faxtomail.reply.action.addCc.tip=Ajouter le contact aux destinataires en copie +faxtomail.reply.action.addCci=Copie cachée +faxtomail.reply.action.addCci.tip=Ajouter le contact aux destinataires en copie cachée +faxtomail.reply.action.addTo=À +faxtomail.reply.action.addTo.tip=Ajouter le contact aux destinataires directs faxtomail.reply.action.cancel=Annuler faxtomail.reply.action.cancel.tip=Annuler et fermer la popup faxtomail.reply.action.close=Fermer @@ -279,6 +285,7 @@ faxtomail.reply.forwardsubject=Tr\: %s faxtomail.reply.label.cc=Copie \: faxtomail.reply.label.cci=Copie cachée \: +faxtomail.reply.label.contact=Contact \: faxtomail.reply.label.from=De \: faxtomail.reply.label.subject=Objet \: faxtomail.reply.label.to=À \:

1 0

r713 - in trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing: actions content/reply
by kmorin＠users.forge.codelutin.com 02 Dec '14

02 Dec '14

Author: kmorin Date: 2014-12-02 18:39:34 +0000 (Tue, 02 Dec 2014) New Revision: 713 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/713 Log: fixes #6233 Pas d'?\195?\169metteur lors d'une r?\195?\169ponse ?\195?\160 un fax Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/SaveAndOpenReplyFormAction.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIHandler.java Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/SaveAndOpenReplyFormAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/SaveAndOpenReplyFormAction.java 2014-12-02 11:01:52 UTC (rev 712) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/SaveAndOpenReplyFormAction.java 2014-12-02 18:39:34 UTC (rev 713) @@ -68,8 +68,8 @@ String sender = JAXXUtil.getStringValue(currentDemand.getSender()); if (currentDemand.isFax()) { - MailFolder selectedFolder = getContext().getCurrentMailFolder(); - sender = FaxToMailServiceUtils.addFaxDomainToFaxNumber(sender, selectedFolder); +// MailFolder selectedFolder = getContext().getCurrentMailFolder(); + sender = FaxToMailServiceUtils.addFaxDomainToFaxNumber(sender, currentDemand.getMailFolder()); } model.setTo(sender); } Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIHandler.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIHandler.java 2014-12-02 11:01:52 UTC (rev 712) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/reply/ReplyFormUIHandler.java 2014-12-02 18:39:34 UTC (rev 713) @@ -121,8 +121,6 @@ } } - model.setFrom(demand.getRecipient()); - model.setSubject(t("faxtomail.reply.subject", demand.getSubject())); String plainContent = JAXXUtil.getStringValue(demand.getPlainContent()); @@ -132,6 +130,7 @@ plainContent.replaceAll("\n", "\n> ")); model.setMessage(quotedReply); + String recipient = demand.getRecipient(); // this have to be disable for read only mode (useless) if (!model.isReadonly()) { @@ -152,8 +151,11 @@ model.setSenderAllowedDomains(folderReplyDomains); List<String> replyToAddresses = Lists.newArrayList(); - if (demand.getRecipient() != null) { - replyToAddresses.add(demand.getRecipient()); + String selectedItem = null; + if (!demand.isFax() && recipient != null) { + replyToAddresses.add(recipient); + model.setFrom(recipient); + selectedItem = recipient; } if (folderReplyAdresses != null) { replyToAddresses.addAll(folderReplyAdresses); @@ -161,9 +163,12 @@ Collections.sort(replyToAddresses); DefaultComboBoxModel comboBoxModel = SwingUtil.newComboModel(replyToAddresses.toArray()); - comboBoxModel.setSelectedItem(demand.getRecipient()); + comboBoxModel.setSelectedItem(selectedItem); ui.getFromComboBox().setModel(comboBoxModel); model.setSenderAllowedAddresses(replyToAddresses); + + } else { + model.setFrom(recipient); } } });

1 0

r712 - in trunk: faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions
by kmorin＠users.forge.codelutin.com 02 Dec '14

02 Dec '14

Author: kmorin Date: 2014-12-02 11:01:52 +0000 (Tue, 02 Dec 2014) New Revision: 712 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/712 Log: fixes #6228 Certaines lignes de gamme envoy?\195?\169es ?\195?\160 l'EDI ne sont plus li?\195?\169es ?\195?\160 un email Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailServiceImpl.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/SaveDemandeAction.java Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailServiceImpl.java =================================================================== --- trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailServiceImpl.java 2014-11-30 17:14:54 UTC (rev 711) +++ trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailServiceImpl.java 2014-12-02 11:01:52 UTC (rev 712) @@ -1251,7 +1251,8 @@ List<Email> emails = dao.forTopiaIdIn(emailIds).findAll(); for (Email email : emails) { - if (email.getDemandStatus() != DemandStatus.TRANSMISSION_TO_EDI) { + if (email.getDemandStatus() != DemandStatus.TRANSMISSION_TO_EDI + || email.getDemandStatus() != DemandStatus.TRANSMITTED_TO_EDI) { email.setDemandStatus(DemandStatus.QUALIFIED); } email.setMailFolder(newFolder); @@ -1376,6 +1377,7 @@ public void execute(Connection connection) throws SQLException { int importedCount = 0; + List<String> rangeRowTopiaIdsInEdiReturnToDelete = new ArrayList<String>(); // attention, le sql à pour cible postgresql, mssql, h2, il doit rester simple String query = String.format("SELECT %s, %s, %s FROM %s", @@ -1409,6 +1411,8 @@ continue; } + rangeRowTopiaIdsInEdiReturnToDelete.add(rangeRowTopiaId); + if (StringUtils.isNotBlank(error)) { //error String oldError = email.getEdiError(); @@ -1443,7 +1447,9 @@ // delete all rows stat = connection.createStatement(); - stat.execute("DELETE FROM " + EdiReturn.class.getSimpleName()); + stat.execute("DELETE FROM " + EdiReturn.class.getSimpleName() + + " WHERE " + EdiReturn.PROPERTY_RANGE_ROW_TOPIA_ID + + " IN ('" + StringUtils.join(rangeRowTopiaIdsInEdiReturnToDelete, "','") + "')"); // usefull log info (do not remove) if (importedCount > 0 && log.isInfoEnabled()) { Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/SaveDemandeAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/SaveDemandeAction.java 2014-11-30 17:14:54 UTC (rev 711) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/SaveDemandeAction.java 2014-12-02 11:01:52 UTC (rev 712) @@ -32,6 +32,7 @@ import java.util.Map; import com.franciaflex.faxtomail.persistence.entities.DemandStatus; +import com.franciaflex.faxtomail.persistence.entities.MailFolder; import org.apache.commons.lang3.StringUtils; import org.nuiton.topia.persistence.TopiaEntities; import org.nuiton.util.beans.BeanMonitor; @@ -130,6 +131,12 @@ email = serviceContext.getEmailService().saveEmail(email, attachments, replies, clientCode, currentUser, modifiedProperties); model.fromEntity(email); + MailFolder folder = model.getMailFolder(); + if (model.getArchiveDate() != null + || !folder.isFolderWritable() + || !model.getDemandStatus().isEditableStatus()) { + model.setEditable(false); + } getModel().setModify(false); monitor.clearModified();

1 0

r711 - in trunk: faxtomail-service/src/main/java/com/franciaflex/faxtomail/services faxtomail-ui-web/src/main/java/com/franciaflex/faxtomail/web/job
by kmorin＠users.forge.codelutin.com 30 Nov '14

30 Nov '14

Author: kmorin Date: 2014-11-30 17:14:54 +0000 (Sun, 30 Nov 2014) New Revision: 711 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/711 Log: - fix error when recipient contains [] - catch invalid charset error Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/FaxToMailServiceUtils.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/FaxToMailServiceUtils.java =================================================================== --- trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/FaxToMailServiceUtils.java 2014-11-30 15:55:39 UTC (rev 710) +++ trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/FaxToMailServiceUtils.java 2014-11-30 17:14:54 UTC (rev 711) @@ -25,6 +25,7 @@ */ import java.nio.charset.Charset; +import java.nio.charset.UnsupportedCharsetException; import java.util.Collection; import javax.mail.MessagingException; @@ -56,7 +57,13 @@ public static Charset getCharset(Part part) throws MessagingException { ContentType contentType = new ContentType(part.getContentType()); String charsetName = contentType.getParameter("charset"); - Charset charset = Charsets.toCharset(charsetName); + Charset charset; + try { + charset = Charsets.toCharset(charsetName); + + } catch (UnsupportedCharsetException e) { + charset = Charset.defaultCharset(); + } return charset; } 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-11-30 15:55:39 UTC (rev 710) +++ trunk/faxtomail-ui-web/src/main/java/com/franciaflex/faxtomail/web/job/MailFilterJob.java 2014-11-30 17:14:54 UTC (rev 711) @@ -176,6 +176,9 @@ */ public void checkEmails(EmailAccount account) { Properties properties = new Properties(); + // set the mail.mime.address.strict to false to avoid + // javax.mail.internet.AddressException: Domain contains illegal character errors when recipients contains [] + properties.setProperty("mail.mime.address.strict", "false"); switch (account.getProtocol()) { case IMAPS:

1 0

r710 - in trunk: . faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service faxtomail-ui-swing
by kmorin＠users.forge.codelutin.com 30 Nov '14

30 Nov '14

Author: kmorin Date: 2014-11-30 15:55:39 +0000 (Sun, 30 Nov 2014) New Revision: 710 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/710 Log: fix error + dependency Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailServiceImpl.java trunk/faxtomail-ui-swing/pom.xml trunk/pom.xml Modified: trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailServiceImpl.java =================================================================== --- trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailServiceImpl.java 2014-11-30 15:19:17 UTC (rev 709) +++ trunk/faxtomail-service/src/main/java/com/franciaflex/faxtomail/services/service/EmailServiceImpl.java 2014-11-30 15:55:39 UTC (rev 710) @@ -1867,8 +1867,10 @@ String content = IOUtils.toString(bp.getInputStream(), charset); if (bp.isMimeType("text/plain")) { - Attachment attachment = convertTextToPdf(content, t("faxtomail.email.content.attachment.plainfileName")); - attachments.add(attachment); + if (StringUtils.isNotBlank(content)) { + Attachment attachment = convertTextToPdf(content, t("faxtomail.email.content.attachment.plainfileName")); + attachments.add(attachment); + } } else { result = content; @@ -2021,6 +2023,7 @@ @Override public Attachment convertTextToPdf(String content, String name) throws IOException, DocumentException { + Preconditions.checkArgument(StringUtils.isNotBlank(content)); //File target = File.createTempFile("faxtomail-", ".tmp"); //target.deleteOnExit(); Modified: trunk/faxtomail-ui-swing/pom.xml =================================================================== --- trunk/faxtomail-ui-swing/pom.xml 2014-11-30 15:19:17 UTC (rev 709) +++ trunk/faxtomail-ui-swing/pom.xml 2014-11-30 15:55:39 UTC (rev 710) @@ -241,6 +241,11 @@ </dependency> <dependency> + <groupId>org.apache.commons</groupId> + <artifactId>commons-email</artifactId> + </dependency> + + <dependency> <groupId>javax.activation</groupId> <artifactId>activation</artifactId> </dependency> Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2014-11-30 15:19:17 UTC (rev 709) +++ trunk/pom.xml 2014-11-30 15:55:39 UTC (rev 710) @@ -507,6 +507,12 @@ </dependency> <dependency> + <groupId>org.apache.commons</groupId> + <artifactId>commons-email</artifactId> + <version>${commonsEmailVersion}</version> + </dependency> + + <dependency> <groupId>javax.activation</groupId> <artifactId>activation</artifactId> <version>1.1.1</version>

1 0

r709 - in trunk: 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 30 Nov '14

30 Nov '14

Author: kmorin Date: 2014-11-30 15:19:17 +0000 (Sun, 30 Nov 2014) New Revision: 709 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/709 Log: refs #6198 Certains emails sont mal interp?\195?\170t?\195?\169s Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java trunk/faxtomail-ui-web/src/main/java/com/franciaflex/faxtomail/web/job/MailFilterJob.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-11-30 14:04:47 UTC (rev 708) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java 2014-11-30 15:19:17 UTC (rev 709) @@ -51,6 +51,7 @@ import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; +import org.apache.commons.mail.util.MimeMessageUtils; import org.nuiton.jaxx.application.swing.tab.TabContentModel; import org.nuiton.util.beans.Binder; import org.nuiton.util.beans.BinderFactory; @@ -878,7 +879,8 @@ protected void decomposeEmail() { try { // ce code peut provoquer une NPE avec les données de test - Message message = new MimeMessage(null, new ByteArrayInputStream(getOriginalEmail().getBytes(StandardCharsets.UTF_8))); + Message message = MimeMessageUtils.createMimeMessage(null, getOriginalEmail()); + Charset charset = FaxToMailServiceUtils.getCharset(message); subject = MimeUtility.decodeText(message.getSubject()); 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-11-30 14:04:47 UTC (rev 708) +++ trunk/faxtomail-ui-web/src/main/java/com/franciaflex/faxtomail/web/job/MailFilterJob.java 2014-11-30 15:19:17 UTC (rev 709) @@ -209,7 +209,7 @@ checkEmailsOfFolder(account, inbox); } catch (Exception e) { - log.error("Error while getting emails from the mailbox", e); + log.error("Error while getting emails from the mailbox " + account.getHost(), e); } finally { close(inbox); @@ -465,7 +465,7 @@ String headerLine = headerLines.nextElement(); emailSource.append(headerLine).append("\n"); } - String originalContent = IOUtils.toString(message.getInputStream(), charset); + String originalContent = IOUtils.toString(((MimeMessage) message).getRawInputStream(), charset); emailSource.append("\n").append(originalContent); email.setOriginalEmail(emailSource.toString());

1 0

r708 - in trunk: . faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities faxtomail-ui-swing faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/pdfeditor faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/util
by kmorin＠users.forge.codelutin.com 30 Nov '14

30 Nov '14

Author: kmorin Date: 2014-11-30 14:04:47 +0000 (Sun, 30 Nov 2014) New Revision: 708 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/708 Log: fixes #6199 Certains pdf impriment des pages blanches Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.java trunk/faxtomail-ui-swing/pom.xml trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/pdfeditor/PDFEditorUIHandler.java trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/util/FaxToMailUIUtil.java trunk/pom.xml Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.java =================================================================== --- trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.java 2014-11-28 13:37:30 UTC (rev 707) +++ trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.java 2014-11-30 14:04:47 UTC (rev 708) @@ -394,8 +394,8 @@ if (searchFilter.isAddGroupDemands()) { hqlForFetchStep1 += "OR E." + TopiaEntity.PROPERTY_TOPIA_ID + " IN (" + "select E2." + TopiaEntity.PROPERTY_TOPIA_ID + " FROM " + Email.class.getName() + " E " - + "LEFT OUTER JOIN E." + Email.PROPERTY_EMAIL_GROUP + " AS G " - + "LEFT OUTER JOIN G." + EmailGroup.PROPERTY_EMAIL + " AS E2 " + + "INNER JOIN E." + Email.PROPERTY_EMAIL_GROUP + " AS G " + + "INNER JOIN G." + EmailGroup.PROPERTY_EMAIL + " AS E2 " + query.toString() + ")"; } Modified: trunk/faxtomail-ui-swing/pom.xml =================================================================== --- trunk/faxtomail-ui-swing/pom.xml 2014-11-28 13:37:30 UTC (rev 707) +++ trunk/faxtomail-ui-swing/pom.xml 2014-11-30 14:04:47 UTC (rev 708) @@ -225,22 +225,14 @@ <artifactId>itextpdf</artifactId> </dependency> -  <dependency> - <groupId>org.swinglabs</groupId> - <artifactId>pdf-renderer</artifactId> - </dependency> - - <dependency> <groupId>javax.media.jai</groupId> <artifactId>com.springsource.javax.media.jai.core</artifactId> - <version>1.1.3</version> </dependency> <dependency> <groupId>javax.media.jai</groupId> <artifactId>com.springsource.javax.media.jai.codec</artifactId> - <version>1.1.3</version> </dependency> <dependency> Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/pdfeditor/PDFEditorUIHandler.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/pdfeditor/PDFEditorUIHandler.java 2014-11-28 13:37:30 UTC (rev 707) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/pdfeditor/PDFEditorUIHandler.java 2014-11-30 14:04:47 UTC (rev 708) @@ -102,7 +102,7 @@ String value = (String) actionGroup.getSelectedValue(); model.setSelectedComponent(PDFEditorUIModel.EditionComponent.valueOf(value)); - pdfDecoder = new PdfDecoder(true); + pdfDecoder = new PdfDecoder(); FontMappings.setFontReplacements(); ui.getContainer().addContainerListener(new ContainerListener() { @@ -349,7 +349,7 @@ } protected void displayPage(int pageNb, float previousZoom, int previousRotation) { - if (pageNb < 0 || pageNb > pdfDecoder.getPageCount()) { + if (pageNb < 1 || pageNb > pdfDecoder.getPageCount()) { return; } Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/util/FaxToMailUIUtil.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/util/FaxToMailUIUtil.java 2014-11-28 13:37:30 UTC (rev 707) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/util/FaxToMailUIUtil.java 2014-11-30 14:04:47 UTC (rev 708) @@ -24,14 +24,12 @@ import static org.nuiton.i18n.I18n.t; -import java.awt.Color; -import java.awt.Component; -import java.awt.Desktop; -import java.awt.Font; +import java.awt.*; +import java.awt.image.BufferedImage; import java.awt.image.RenderedImage; -import java.awt.print.Book; import java.awt.print.PageFormat; import java.awt.print.Paper; +import java.awt.print.Printable; import java.awt.print.PrinterException; import java.awt.print.PrinterJob; import java.io.BufferedReader; @@ -59,8 +57,6 @@ import javax.swing.*; import javax.swing.event.HyperlinkEvent; import javax.swing.event.HyperlinkListener; -import javax.swing.text.StyleConstants; -import javax.swing.text.html.HTMLDocument; import javax.swing.text.html.HTMLEditorKit; import javax.swing.tree.DefaultMutableTreeNode; import javax.swing.tree.DefaultTreeCellRenderer; @@ -68,7 +64,6 @@ import javax.swing.tree.TreeModel; import javax.swing.tree.TreePath; -import com.sun.pdfview.PDFPrintPage; import jaxx.runtime.JAXXObject; import jaxx.runtime.JAXXUtil; @@ -81,6 +76,9 @@ import org.apache.commons.lang3.SystemUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; +import org.jpedal.PdfDecoder; +import org.jpedal.exception.PdfException; +import org.jpedal.fonts.FontMappings; import org.nuiton.jaxx.application.ApplicationTechnicalException; import org.nuiton.jaxx.application.swing.util.ApplicationUIUtil; import org.nuiton.util.FileUtil; @@ -114,8 +112,6 @@ import com.sun.media.jai.codec.ImageCodec; import com.sun.media.jai.codec.ImageDecoder; import com.sun.media.jai.codec.SeekableStream; -import com.sun.pdfview.PDFFile; -//import com.sun.pdfview.PDFPrintPage; /** * @author kmorin - morin(a)codelutin.com @@ -500,39 +496,54 @@ */ protected static boolean printWithPdfRenderer(String printName, FileInputStream fis, boolean defaultPrinter) { boolean result = false; + final PdfDecoder pdfDecoder = new PdfDecoder(); + FontMappings.setFontReplacements(); try { - // Create a PDFFile from a File reference - FileChannel fc = fis.getChannel(); - ByteBuffer bb = fc.map(FileChannel.MapMode.READ_ONLY, 0, fc.size()); - PDFFile pdfFile = new PDFFile(bb); // Create PDF Print Page - PDFPrintPage pages = new PDFPrintPage(pdfFile); + pdfDecoder.openPdfFileFromInputStream(fis, false); - // fix fit to page - // see http://lynema.org/2010/12/29/printing-a-pdf-in-java-with-pdfrenderer - PrinterJob pjob = PrinterJob.getPrinterJob(); - PageFormat pf = pjob.defaultPage(); + pdfDecoder.setExtractionMode(0, 0, 1f); + PrinterJob printJob = PrinterJob.getPrinterJob(); + printJob.setJobName(printName); + + PageFormat pf = printJob.defaultPage(); Paper paper = new Paper(); - paper.setImageableArea(0, 0, paper.getWidth() * 2, paper.getHeight()); + paper.setImageableArea(0, 0, paper.getWidth(), paper.getHeight()); pf.setPaper(paper); - // Create Print Job - pjob.setJobName(printName); - Book book = new Book(); - book.append(pages, pf, pdfFile.getNumPages()); - pjob.setPageable(book); + printJob.setPrintable(new Printable() { + public int print(Graphics graphics, PageFormat pageFormat, int pageIndex) throws PrinterException { + try { + if (pageIndex >= pdfDecoder.getPageCount()) { + return NO_SUCH_PAGE; + } + BufferedImage image = pdfDecoder.getPageAsImage(pageIndex + 1); + graphics.drawImage(image, 0, 0, image.getWidth(), image.getHeight(), null); + return PAGE_EXISTS; + } catch (Exception e) { + log.error("error while printing", e); + return NO_SUCH_PAGE; + } + } + }); + // Send print job to default printer - result = defaultPrinter || pjob.printDialog(); + result = defaultPrinter || printJob.printDialog(); if (result) { - pjob.print(); + printJob.print(); } - } catch (PrinterException | IOException ex) { + + } catch (PrinterException | PdfException ex) { if (log.isErrorEnabled()) { log.error("can't print", ex); } + + } finally { + log.info("finally close the pdf file"); + pdfDecoder.closePdfFile(); } return result; Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2014-11-28 13:37:30 UTC (rev 707) +++ trunk/pom.xml 2014-11-30 14:04:47 UTC (rev 708) @@ -306,15 +306,15 @@ </dependency> <dependency> - <groupId>org.swinglabs</groupId> - <artifactId>pdf-renderer</artifactId> - <version>1.0.5</version> + <groupId>javax.media.jai</groupId> + <artifactId>com.springsource.javax.media.jai.core</artifactId> + <version>1.1.3</version> </dependency> <dependency> - <groupId>org.apache.pdfbox</groupId> - <artifactId>pdfbox</artifactId> - <version>1.8.7</version> + <groupId>javax.media.jai</groupId> + <artifactId>com.springsource.javax.media.jai.codec</artifactId> + <version>1.1.3</version> </dependency>

1 0

r707 - trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities
by kmorin＠users.forge.codelutin.com 28 Nov '14

28 Nov '14

Author: kmorin Date: 2014-11-28 13:37:30 +0000 (Fri, 28 Nov 2014) New Revision: 707 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/707 Log: revert test am?\195?\169lioration perfs fixes #6197 Les pi?\195?\168ces jointes avec des "/" dans le nom ne s'ouvrent pas Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/AttachmentFileImpl.java trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.java Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/AttachmentFileImpl.java =================================================================== --- trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/AttachmentFileImpl.java 2014-11-28 10:47:15 UTC (rev 706) +++ trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/AttachmentFileImpl.java 2014-11-28 13:37:30 UTC (rev 707) @@ -30,6 +30,7 @@ import java.io.IOException; import java.io.InputStream; import java.io.OutputStream; +import java.net.URLEncoder; import org.apache.commons.io.FileUtils; import org.apache.commons.io.IOUtils; @@ -55,7 +56,7 @@ File tempFolder = new File(FileUtils.getTempDirectory(), "faxtomail"); tempFolder.mkdir(); - String fileName = getFilename(); + String fileName = URLEncoder.encode(getFilename(), "UTF-8"); tmpLocalFile = new File(tempFolder, fileName); // if the file already exists, then add a number to the name Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.java =================================================================== --- trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.java 2014-11-28 10:47:15 UTC (rev 706) +++ trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.java 2014-11-28 13:37:30 UTC (rev 707) @@ -386,7 +386,7 @@ } // add same fecth liste as emailService#getEmailForFolder() - String hqlForFetchStep1 = "FROM " + Email.class.getName() + " E " + String hqlForFetchStep1 = "select E." + TopiaEntity.PROPERTY_TOPIA_ID + " FROM " + Email.class.getName() + " E " + "WHERE E." + TopiaEntity.PROPERTY_TOPIA_ID + " IN (" + "select E." + TopiaEntity.PROPERTY_TOPIA_ID + " FROM " + Email.class.getName() + " E " + query.toString() + ") "; @@ -399,33 +399,33 @@ + query.toString() + ")"; } -// String hqlForFetchStep2 = "select distinct E FROM " + Email.class.getName() + " E " + addAllFecthes( -// Email.PROPERTY_PRIORITY, -// Email.PROPERTY_DEMAND_TYPE, -// Email.PROPERTY_CLIENT, -// //Email.PROPERTY_RANGE_ROW + "." + RangeRow.PROPERTY_RANGE, -// Email.PROPERTY_WAITING_STATE, -// Email.PROPERTY_TAKEN_BY -// //Email.PROPERTY_REPLIES, -// //Email.PROPERTY_ATTACHMENT, -// //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FAX_TO_MAIL_USER, -// //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, -// //Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL -// ) + " WHERE E." + TopiaEntity.PROPERTY_TOPIA_ID + " in ( :topiaIdsForFetch_ )"; + String hqlForFetchStep2 = "select distinct E FROM " + Email.class.getName() + " E " + addAllFecthes( + Email.PROPERTY_PRIORITY, + Email.PROPERTY_DEMAND_TYPE, + Email.PROPERTY_CLIENT, + //Email.PROPERTY_RANGE_ROW + "." + RangeRow.PROPERTY_RANGE, + Email.PROPERTY_WAITING_STATE, + Email.PROPERTY_TAKEN_BY + //Email.PROPERTY_REPLIES, + //Email.PROPERTY_ATTACHMENT, + //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FAX_TO_MAIL_USER, + //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, + //Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL + ) + " WHERE E." + TopiaEntity.PROPERTY_TOPIA_ID + " in ( :topiaIdsForFetch_ )"; // code copied from topia to perform search - PaginationResult<Email> pageResult = findPage(hqlForFetchStep1, args, pagination); - List<Email> step1ResultTopiaIds = pageResult.getElements(); + PaginationResult<String> pageResult = findPage(hqlForFetchStep1, args, pagination); + List<String> step1ResultTopiaIds = pageResult.getElements(); List<Email> sortedEntities; if (CollectionUtils.isEmpty(step1ResultTopiaIds)) { sortedEntities = Lists.newArrayList(); } else { -// Map<String, Object> step2Args = Maps.newHashMap(); -// step2Args.put("topiaIdsForFetch_", step1ResultTopiaIds); -// List<Email> entities = forHql(hqlForFetchStep2, step2Args).findAll(); + Map<String, Object> step2Args = Maps.newHashMap(); + step2Args.put("topiaIdsForFetch_", step1ResultTopiaIds); + List<Email> entities = forHql(hqlForFetchStep2, step2Args).findAll(); - sortedEntities = step1ResultTopiaIds; + sortedEntities = sortAccordingToIds(entities, step1ResultTopiaIds); } PaginationResult<Email> result = PaginationResult.of(sortedEntities, pageResult.getCount(), pageResult.getCurrentPage()); @@ -445,17 +445,17 @@ // Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, // Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL) for (Email email : result.getElements()) { - Hibernate.initialize(email.getPriority()); - Hibernate.initialize(email.getDemandType()); - Hibernate.initialize(email.getClient()); + //Hibernate.initialize(email.getPriority()); + //Hibernate.initialize(email.getDemandType()); + //Hibernate.initialize(email.getClient()); List<RangeRow> rangeRows = email.getRangeRow(); if (rangeRows != null) { for (RangeRow rangeRow : rangeRows) { Hibernate.initialize(rangeRow.getRange()); } } - Hibernate.initialize(email.getWaitingState()); - Hibernate.initialize(email.getTakenBy()); + //Hibernate.initialize(email.getWaitingState()); + //Hibernate.initialize(email.getTakenBy()); Hibernate.initialize(email.getReplies()); Hibernate.initialize(email.getAttachment()); Collection<History> histories = email.getHistory(); @@ -511,34 +511,34 @@ String query = filteredDemandsOfFolderCondition(folder, emailFilter, args); // add same fecth liste as emailService#getEmailForFolder() - String hqlForFetchStep1 = "FROM " + Email.class.getName() + " E " + query; -// String hqlForFetchStep2 = "select distinct E FROM " + Email.class.getName() + " E " + addAllFecthes( -// Email.PROPERTY_PRIORITY, -// Email.PROPERTY_DEMAND_TYPE, -// Email.PROPERTY_CLIENT, -// //Email.PROPERTY_RANGE_ROW + "." + RangeRow.PROPERTY_RANGE, -// Email.PROPERTY_WAITING_STATE, -// Email.PROPERTY_TAKEN_BY -// //Email.PROPERTY_REPLIES, -// //Email.PROPERTY_ATTACHMENT, -// //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FAX_TO_MAIL_USER, -// //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, -// //Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL -// ) + " WHERE E." + TopiaEntity.PROPERTY_TOPIA_ID + " in ( :topiaIdsForFetch_ )"; + String hqlForFetchStep1 = "select E." + TopiaEntity.PROPERTY_TOPIA_ID + " FROM " + Email.class.getName() + " E " + query; + String hqlForFetchStep2 = "select distinct E FROM " + Email.class.getName() + " E " + addAllFecthes( + Email.PROPERTY_PRIORITY, + Email.PROPERTY_DEMAND_TYPE, + Email.PROPERTY_CLIENT, + //Email.PROPERTY_RANGE_ROW + "." + RangeRow.PROPERTY_RANGE, + Email.PROPERTY_WAITING_STATE, + Email.PROPERTY_TAKEN_BY + //Email.PROPERTY_REPLIES, + //Email.PROPERTY_ATTACHMENT, + //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FAX_TO_MAIL_USER, + //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, + //Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL + ) + " WHERE E." + TopiaEntity.PROPERTY_TOPIA_ID + " in ( :topiaIdsForFetch_ )"; // code copied from topia to perform search - PaginationResult<Email> pageResult = findPage(hqlForFetchStep1, args, pagination); - List<Email> step1ResultTopiaIds = pageResult.getElements(); + PaginationResult<String> pageResult = findPage(hqlForFetchStep1, args, pagination); + List<String> step1ResultTopiaIds = pageResult.getElements(); List<Email> sortedEntities; if (CollectionUtils.isEmpty(step1ResultTopiaIds)) { sortedEntities = Lists.newArrayList(); } else { -// Map<String, Object> step2Args = Maps.newHashMap(); -// step2Args.put("topiaIdsForFetch_", step1ResultTopiaIds); -// List<Email> entities = forHql(hqlForFetchStep2, step2Args).findAll(); + Map<String, Object> step2Args = Maps.newHashMap(); + step2Args.put("topiaIdsForFetch_", step1ResultTopiaIds); + List<Email> entities = forHql(hqlForFetchStep2, step2Args).findAll(); - sortedEntities = step1ResultTopiaIds; + sortedEntities = sortAccordingToIds(entities, step1ResultTopiaIds); } PaginationResult<Email> result = PaginationResult.of(sortedEntities, pageResult.getCount(), pageResult.getCurrentPage()); @@ -558,17 +558,17 @@ // Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, // Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL) for (Email email : result.getElements()) { - Hibernate.initialize(email.getPriority()); - Hibernate.initialize(email.getDemandType()); - Hibernate.initialize(email.getClient()); + //Hibernate.initialize(email.getPriority()); + //Hibernate.initialize(email.getDemandType()); + //Hibernate.initialize(email.getClient()); List<RangeRow> rangeRows = email.getRangeRow(); if (rangeRows != null) { for (RangeRow rangeRow : rangeRows) { Hibernate.initialize(rangeRow.getRange()); } } - Hibernate.initialize(email.getWaitingState()); - Hibernate.initialize(email.getTakenBy()); + //Hibernate.initialize(email.getWaitingState()); + //Hibernate.initialize(email.getTakenBy()); Hibernate.initialize(email.getReplies()); Hibernate.initialize(email.getAttachment()); Collection<History> histories = email.getHistory(); @@ -918,7 +918,6 @@ /* * @deprecated because this need to be supported by topia (see http://forge.nuiton.org/issues/3427) - * Do not use with mssql, it is REAAAAAAAAAALLY slow */ @Deprecated protected String addAllFecthes(String... fetchProperties) {

1 0

r706 - in trunk: faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/pdfeditor
by kmorin＠users.forge.codelutin.com 28 Nov '14

28 Nov '14

Author: kmorin Date: 2014-11-28 10:47:15 +0000 (Fri, 28 Nov 2014) New Revision: 706 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/706 Log: test am?\195?\169lioration perfs Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.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/pdfeditor/PDFEditorUIHandler.java Modified: trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.java =================================================================== --- trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.java 2014-11-27 16:05:17 UTC (rev 705) +++ trunk/faxtomail-persistence/src/main/java/com/franciaflex/faxtomail/persistence/entities/EmailTopiaDao.java 2014-11-28 10:47:15 UTC (rev 706) @@ -386,7 +386,7 @@ } // add same fecth liste as emailService#getEmailForFolder() - String hqlForFetchStep1 = "select E." + TopiaEntity.PROPERTY_TOPIA_ID + " FROM " + Email.class.getName() + " E " + String hqlForFetchStep1 = "FROM " + Email.class.getName() + " E " + "WHERE E." + TopiaEntity.PROPERTY_TOPIA_ID + " IN (" + "select E." + TopiaEntity.PROPERTY_TOPIA_ID + " FROM " + Email.class.getName() + " E " + query.toString() + ") "; @@ -399,33 +399,33 @@ + query.toString() + ")"; } - String hqlForFetchStep2 = "select distinct E FROM " + Email.class.getName() + " E " + addAllFecthes( - Email.PROPERTY_PRIORITY, - Email.PROPERTY_DEMAND_TYPE, - Email.PROPERTY_CLIENT, - //Email.PROPERTY_RANGE_ROW + "." + RangeRow.PROPERTY_RANGE, - Email.PROPERTY_WAITING_STATE, - Email.PROPERTY_TAKEN_BY - //Email.PROPERTY_REPLIES, - //Email.PROPERTY_ATTACHMENT, - //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FAX_TO_MAIL_USER, - //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, - //Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL - ) + " WHERE E." + TopiaEntity.PROPERTY_TOPIA_ID + " in ( :topiaIdsForFetch_ )"; +// String hqlForFetchStep2 = "select distinct E FROM " + Email.class.getName() + " E " + addAllFecthes( +// Email.PROPERTY_PRIORITY, +// Email.PROPERTY_DEMAND_TYPE, +// Email.PROPERTY_CLIENT, +// //Email.PROPERTY_RANGE_ROW + "." + RangeRow.PROPERTY_RANGE, +// Email.PROPERTY_WAITING_STATE, +// Email.PROPERTY_TAKEN_BY +// //Email.PROPERTY_REPLIES, +// //Email.PROPERTY_ATTACHMENT, +// //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FAX_TO_MAIL_USER, +// //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, +// //Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL +// ) + " WHERE E." + TopiaEntity.PROPERTY_TOPIA_ID + " in ( :topiaIdsForFetch_ )"; // code copied from topia to perform search - PaginationResult<String> pageResult = findPage(hqlForFetchStep1, args, pagination); - List<String> step1ResultTopiaIds = pageResult.getElements(); + PaginationResult<Email> pageResult = findPage(hqlForFetchStep1, args, pagination); + List<Email> step1ResultTopiaIds = pageResult.getElements(); List<Email> sortedEntities; if (CollectionUtils.isEmpty(step1ResultTopiaIds)) { sortedEntities = Lists.newArrayList(); } else { - Map<String, Object> step2Args = Maps.newHashMap(); - step2Args.put("topiaIdsForFetch_", step1ResultTopiaIds); - List<Email> entities = forHql(hqlForFetchStep2, step2Args).findAll(); +// Map<String, Object> step2Args = Maps.newHashMap(); +// step2Args.put("topiaIdsForFetch_", step1ResultTopiaIds); +// List<Email> entities = forHql(hqlForFetchStep2, step2Args).findAll(); - sortedEntities = sortAccordingToIds(entities, step1ResultTopiaIds); + sortedEntities = step1ResultTopiaIds; } PaginationResult<Email> result = PaginationResult.of(sortedEntities, pageResult.getCount(), pageResult.getCurrentPage()); @@ -445,17 +445,17 @@ // Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, // Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL) for (Email email : result.getElements()) { - //Hibernate.initialize(email.getPriority()); - //Hibernate.initialize(email.getDemandType()); - //Hibernate.initialize(email.getClient()); + Hibernate.initialize(email.getPriority()); + Hibernate.initialize(email.getDemandType()); + Hibernate.initialize(email.getClient()); List<RangeRow> rangeRows = email.getRangeRow(); if (rangeRows != null) { for (RangeRow rangeRow : rangeRows) { Hibernate.initialize(rangeRow.getRange()); } } - //Hibernate.initialize(email.getWaitingState()); - //Hibernate.initialize(email.getTakenBy()); + Hibernate.initialize(email.getWaitingState()); + Hibernate.initialize(email.getTakenBy()); Hibernate.initialize(email.getReplies()); Hibernate.initialize(email.getAttachment()); Collection<History> histories = email.getHistory(); @@ -511,34 +511,34 @@ String query = filteredDemandsOfFolderCondition(folder, emailFilter, args); // add same fecth liste as emailService#getEmailForFolder() - String hqlForFetchStep1 = "select E." + TopiaEntity.PROPERTY_TOPIA_ID + " FROM " + Email.class.getName() + " E " + query; - String hqlForFetchStep2 = "select distinct E FROM " + Email.class.getName() + " E " + addAllFecthes( - Email.PROPERTY_PRIORITY, - Email.PROPERTY_DEMAND_TYPE, - Email.PROPERTY_CLIENT, - //Email.PROPERTY_RANGE_ROW + "." + RangeRow.PROPERTY_RANGE, - Email.PROPERTY_WAITING_STATE, - Email.PROPERTY_TAKEN_BY - //Email.PROPERTY_REPLIES, - //Email.PROPERTY_ATTACHMENT, - //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FAX_TO_MAIL_USER, - //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, - //Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL - ) + " WHERE E." + TopiaEntity.PROPERTY_TOPIA_ID + " in ( :topiaIdsForFetch_ )"; + String hqlForFetchStep1 = "FROM " + Email.class.getName() + " E " + query; +// String hqlForFetchStep2 = "select distinct E FROM " + Email.class.getName() + " E " + addAllFecthes( +// Email.PROPERTY_PRIORITY, +// Email.PROPERTY_DEMAND_TYPE, +// Email.PROPERTY_CLIENT, +// //Email.PROPERTY_RANGE_ROW + "." + RangeRow.PROPERTY_RANGE, +// Email.PROPERTY_WAITING_STATE, +// Email.PROPERTY_TAKEN_BY +// //Email.PROPERTY_REPLIES, +// //Email.PROPERTY_ATTACHMENT, +// //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FAX_TO_MAIL_USER, +// //Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, +// //Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL +// ) + " WHERE E." + TopiaEntity.PROPERTY_TOPIA_ID + " in ( :topiaIdsForFetch_ )"; // code copied from topia to perform search - PaginationResult<String> pageResult = findPage(hqlForFetchStep1, args, pagination); - List<String> step1ResultTopiaIds = pageResult.getElements(); + PaginationResult<Email> pageResult = findPage(hqlForFetchStep1, args, pagination); + List<Email> step1ResultTopiaIds = pageResult.getElements(); List<Email> sortedEntities; if (CollectionUtils.isEmpty(step1ResultTopiaIds)) { sortedEntities = Lists.newArrayList(); } else { - Map<String, Object> step2Args = Maps.newHashMap(); - step2Args.put("topiaIdsForFetch_", step1ResultTopiaIds); - List<Email> entities = forHql(hqlForFetchStep2, step2Args).findAll(); +// Map<String, Object> step2Args = Maps.newHashMap(); +// step2Args.put("topiaIdsForFetch_", step1ResultTopiaIds); +// List<Email> entities = forHql(hqlForFetchStep2, step2Args).findAll(); - sortedEntities = sortAccordingToIds(entities, step1ResultTopiaIds); + sortedEntities = step1ResultTopiaIds; } PaginationResult<Email> result = PaginationResult.of(sortedEntities, pageResult.getCount(), pageResult.getCurrentPage()); @@ -558,17 +558,17 @@ // Email.PROPERTY_HISTORY + "." + History.PROPERTY_FIELDS, // Email.PROPERTY_EMAIL_GROUP + "." + EmailGroup.PROPERTY_EMAIL) for (Email email : result.getElements()) { - //Hibernate.initialize(email.getPriority()); - //Hibernate.initialize(email.getDemandType()); - //Hibernate.initialize(email.getClient()); + Hibernate.initialize(email.getPriority()); + Hibernate.initialize(email.getDemandType()); + Hibernate.initialize(email.getClient()); List<RangeRow> rangeRows = email.getRangeRow(); if (rangeRows != null) { for (RangeRow rangeRow : rangeRows) { Hibernate.initialize(rangeRow.getRange()); } } - //Hibernate.initialize(email.getWaitingState()); - //Hibernate.initialize(email.getTakenBy()); + Hibernate.initialize(email.getWaitingState()); + Hibernate.initialize(email.getTakenBy()); Hibernate.initialize(email.getReplies()); Hibernate.initialize(email.getAttachment()); Collection<History> histories = email.getHistory(); @@ -918,6 +918,7 @@ /* * @deprecated because this need to be supported by topia (see http://forge.nuiton.org/issues/3427) + * Do not use with mssql, it is REAAAAAAAAAALLY slow */ @Deprecated protected String addAllFecthes(String... fetchProperties) { 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-11-27 16:05:17 UTC (rev 705) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/demande/DemandeUIModel.java 2014-11-28 10:47:15 UTC (rev 706) @@ -862,6 +862,11 @@ } @Override + public int hashCode() { + return editObject.hashCode(); + } + + @Override public boolean equals(Object obj) { if (obj == null || !obj.getClass().isAssignableFrom(DemandeUIModel.class)) { return false; @@ -874,6 +879,8 @@ try { // ce code peut provoquer une NPE avec les données de test Message message = new MimeMessage(null, new ByteArrayInputStream(getOriginalEmail().getBytes(StandardCharsets.UTF_8))); + Charset charset = FaxToMailServiceUtils.getCharset(message); + subject = MimeUtility.decodeText(message.getSubject()); toRecipients = new ArrayList<String>(); @@ -895,7 +902,7 @@ decomposeMultipartEmail(message); } else { - String content = IOUtils.toString(message.getInputStream()); + String content = IOUtils.toString(message.getInputStream(), charset); plainContent = content; } @@ -930,6 +937,7 @@ if (bp.isMimeType("text/*") && !Part.ATTACHMENT.equals(disposition)) { Charset charset = FaxToMailServiceUtils.getCharset(bp); String content = IOUtils.toString(bp.getInputStream(), charset); + if (bp.isMimeType("text/plain")) { plainContent = content; } else { Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/pdfeditor/PDFEditorUIHandler.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/pdfeditor/PDFEditorUIHandler.java 2014-11-27 16:05:17 UTC (rev 705) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/content/pdfeditor/PDFEditorUIHandler.java 2014-11-28 10:47:15 UTC (rev 706) @@ -349,6 +349,9 @@ } protected void displayPage(int pageNb, float previousZoom, int previousRotation) { + if (pageNb < 0 || pageNb > pdfDecoder.getPageCount()) { + return; + } try { float zoom = getModel().getZoom();

1 0

r705 - trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions
by kmorin＠users.forge.codelutin.com 27 Nov '14

27 Nov '14

Author: kmorin Date: 2014-11-27 16:05:17 +0000 (Thu, 27 Nov 2014) New Revision: 705 Url: http://forge.codelutin.com/projects/faxtomail/repository/revisions/705 Log: fixes #6192 Le tableau de gammes n'est pas charg?\195?\169 quand on ouvre un ?\195?\169l?\195?\169ment group?\195?\169 Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/OpenGroupedDemandAction.java Modified: trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/OpenGroupedDemandAction.java =================================================================== --- trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/OpenGroupedDemandAction.java 2014-11-27 13:45:24 UTC (rev 704) +++ trunk/faxtomail-ui-swing/src/main/java/com/franciaflex/faxtomail/ui/swing/actions/OpenGroupedDemandAction.java 2014-11-27 16:05:17 UTC (rev 705) @@ -27,6 +27,7 @@ import com.franciaflex.faxtomail.persistence.entities.Email; import com.franciaflex.faxtomail.persistence.entities.FaxToMailUser; import com.franciaflex.faxtomail.persistence.entities.MailFolder; +import com.franciaflex.faxtomail.persistence.entities.RangeRow; import com.franciaflex.faxtomail.services.FaxToMailServiceContext; import com.franciaflex.faxtomail.services.service.EmailService; import com.franciaflex.faxtomail.services.service.exceptions.AlreadyLockedMailException; @@ -34,6 +35,7 @@ import com.franciaflex.faxtomail.ui.swing.content.MainUIHandler; import com.franciaflex.faxtomail.ui.swing.content.demande.DemandeUIModel; import com.franciaflex.faxtomail.ui.swing.content.demande.DemandesUI; +import com.franciaflex.faxtomail.ui.swing.content.demande.RangeRowModel; import com.franciaflex.faxtomail.ui.swing.content.demande.demandgroup.DemandGroupUI; import com.franciaflex.faxtomail.ui.swing.content.demande.demandgroup.DemandGroupUIHandler; import org.apache.commons.lang3.StringUtils; @@ -41,6 +43,9 @@ import javax.swing.*; +import java.util.ArrayList; +import java.util.Collection; + import static org.nuiton.i18n.I18n.t; /** @@ -152,6 +157,21 @@ takeEmail); demandToOpen.fromEntity(email); + ArrayList<RangeRowModel> validRangeRowModels = new ArrayList<RangeRowModel>(); + Collection<RangeRow> rangeRows = demandToOpen.getRangeRow(); + if (rangeRows != null) { + for (RangeRow rangeRow : rangeRows) { + if (rangeRow != null) { + RangeRowModel rangeRowModel = new RangeRowModel(); + rangeRowModel.fromEntity(rangeRow); + if (rangeRowModel.isValid()) { + validRangeRowModels.add(rangeRowModel); + } + } + } + } + demandToOpen.setValidRangeRowModels(validRangeRowModels); + MailFolder folder = demandToOpen.getMailFolder(); if (demandToOpen.getArchiveDate() != null || !folder.isFolderWritable()

1 0