Author: fdesbois Date: 2010-01-29 17:21:13 +0000 (Fri, 29 Jan 2010) New Revision: 294 Added: trunk/suiviobsmer-business/src/main/sql/ trunk/suiviobsmer-business/src/main/sql/indexes.sql Modified: trunk/changelog.txt trunk/pom.xml trunk/suiviobsmer-business/pom.xml trunk/suiviobsmer-business/src/main/filters/SuiviObsmer.properties trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/SuiviObsmerGlobal.java trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ActivityCalendarImport.java trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceBoatImpl.java trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceContactImpl.java trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSamplingImpl.java trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceUserImpl.java trunk/suiviobsmer-business/src/main/xmi/suiviobsmer.zargo trunk/suiviobsmer-ui/pom.xml trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/base/AbstractFilteredPage.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/BooleanImage.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Chart.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Layout.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/BoatActivityCalendar.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Boats.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Contacts.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/SampleRowForm.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Synthesis.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/services/ContactModelFactory.java Log: - Evo #2045 : Change email message when user password is updated - Ano #2058 : ActivityCalendar Lazy error - Ano #2064 : Contact export problem - Correction some code for Sonar rules compliance - use last topia snapshot and change version to 0.4.0 -> need migration service from ToPIA Modified: trunk/changelog.txt =================================================================== --- trunk/changelog.txt 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/changelog.txt 2010-01-29 17:21:13 UTC (rev 294) @@ -1,10 +1,14 @@ Release note ============ -0.3.1 +0.4.0 ----- -- [fdesbois] Amélioration import pour prise en charge du code ligne du plan (sur 4 digits dans l'application) +- [fdesbois] Ano #2064 : Export contact : ajouter l'extension .csv + n'exporte que les 20premières lignes +- [fdesbois] Ano #2058 : Erreur affichage calendrier d'activité +- [fdesbois] Evo #2045 : Enlever le mot de passe du mail lors de la modification du compte utilisateur +- [fdesbois] Evo #1987 : Synthèse : Graphe sollicitation des navires +- [fdesbois] Evo #2065 : Amélioration import pour prise en charge du code ligne du plan (sur 4 digits dans l'application) - [fdesbois] Evo #2061 : Synthèse : ajouter le filtre "Programme ou règlement rattachement" - [fdesbois] Ano #2056 : Le filtre sur la date de début pour les contacts ne reste pas Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/pom.xml 2010-01-29 17:21:13 UTC (rev 294) @@ -15,7 +15,7 @@ <groupId>fr.ifremer</groupId> <artifactId>suiviobsmer</artifactId> - <version>0.3.1-SNAPSHOT</version> + <version>0.4.0-SNAPSHOT</version> <modules> <module>suiviobsmer-ui</module> @@ -212,7 +212,7 @@ <!-- libraries version --> <nuitonutils.version>1.1.3</nuitonutils.version> - <topia.version>2.3.0-beta-6</topia.version> + <topia.version>2.3.0-beta-7-SNAPSHOT</topia.version> <eugene.version>2.0.0-beta-3</eugene.version> <tapestry.version>5.1.0.5</tapestry.version> Modified: trunk/suiviobsmer-business/pom.xml =================================================================== --- trunk/suiviobsmer-business/pom.xml 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-business/pom.xml 2010-01-29 17:21:13 UTC (rev 294) @@ -10,7 +10,7 @@ <parent> <groupId>fr.ifremer</groupId> <artifactId>suiviobsmer</artifactId> - <version>0.3.1-SNAPSHOT</version> + <version>0.4.0-SNAPSHOT</version> </parent> <groupId>fr.ifremer.suiviobsmer</groupId> Modified: trunk/suiviobsmer-business/src/main/filters/SuiviObsmer.properties =================================================================== --- trunk/suiviobsmer-business/src/main/filters/SuiviObsmer.properties 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-business/src/main/filters/SuiviObsmer.properties 2010-01-29 17:21:13 UTC (rev 294) @@ -1,5 +1,5 @@ ## Default properties for H2 database -hibernate.hbm2ddl.auto=update +hibernate.hbm2ddl.auto=create hibernate.show_sql=false hibernate.dialect=org.hibernate.dialect.H2Dialect @@ -16,4 +16,4 @@ suiviobsmer.email.from=admin at suiviobsmer.org ## Version -suiviobsmer.version=0.3.0 \ No newline at end of file +suiviobsmer.version=0.4.0 \ No newline at end of file Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/SuiviObsmerGlobal.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/SuiviObsmerGlobal.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/SuiviObsmerGlobal.java 2010-01-29 17:21:13 UTC (rev 294) @@ -44,11 +44,6 @@ private static final Logger log = LoggerFactory.getLogger(SuiviObsmerGlobal.class); - // CREATE UNIQUE INDEX boat_immatriculation_idx ON BOAT(IMMATRICULATION) - // CREATE UNIQUE INDEX activitycalendar_idx ON ACTIVITYCALENDAR (BOAT, YEAR) - // CREATE UNIQUE INDEX shipowner_idx ON SHIPOWNER (CODE) - // CREATE UNIQUE INDEX samplerow_idx ON SAMPLEROW (CODE) - /** * Create a default admin user if no user already exist. * Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ActivityCalendarImport.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ActivityCalendarImport.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ActivityCalendarImport.java 2010-01-29 17:21:13 UTC (rev 294) @@ -200,10 +200,10 @@ calendar.setActivityMonth(new ArrayList<ActivityMonth>()); - if (log.isDebugEnabled()) { - info("Création d'un calendrier " + year + - " pour le navire immatriculé " + boatImmatriculation, currRow); - } +// if (log.isDebugEnabled()) { +// info("Création d'un calendrier " + year + +// " pour le navire immatriculé " + boatImmatriculation, currRow); +// } } availableCalendars.put(key, calendar); } @@ -234,23 +234,6 @@ month.setActive(active); if (active) { -// String nbSeaDaysString = reader.get("SYNA_NOMJDM").trim(); -// Integer nbSeaDays = !nbSeaDaysString.equals("NA") ? -// Integer.valueOf(nbSeaDaysString) : null; -// -// String nbFishingDaysString = reader.get("SYNA_NOMJDP").trim(); -// Integer nbFishingDays = !nbFishingDaysString.equals("NA") ? -// Integer.valueOf(nbFishingDaysString) : null; -// -// String nbBoardingPersonsString = reader.get("SYNA_NOMHE").trim(); -// Integer nbBoardingPersons = !nbBoardingPersonsString.equals("NA") ? -// Integer.valueOf(nbBoardingPersonsString) : null; -// -// String harbourIdString = reader.get("SYNA_TPOR_COD").trim(); -// int harbourId = !harbourIdString.equals("NA") ? Integer.parseInt(harbourIdString) : -1; -// String harbourLibelle = reader.get("SYNA_POR_LIB").trim(); -// harbourLibelle = !harbourLibelle.equals("NA") ? harbourLibelle : null; -// harbourCode = !harbourCode.equals("NA") ? harbourCode : null; Integer nbSeaDays = ImportHelper.readInteger(reader, ACTIVITY_CALENDAR.SYNA_NOMJDM); Integer nbFishingDays = ImportHelper.readInteger(reader, ACTIVITY_CALENDAR.SYNA_NOMJDP); @@ -267,11 +250,6 @@ month.setNbFishingDays(nbFishingDays); month.setNbSeaDays(nbSeaDays); -// int professionOrder = Integer.parseInt(reader.get("META_ORDRE").trim()); // NA -// String professionCode = reader.get("MET_COD").trim(); -// int professionId = Integer.parseInt(reader.get("MET_ID").trim()); -// String professionLibelle = reader.get("MET_LIB").trim(); - // Can be equals to -1 if set to NA in Csv File (not a problem for ordering Profession) int professionOrder = ImportHelper.readInt(reader, ACTIVITY_CALENDAR.META_ORDRE); int professionId = ImportHelper.readInt(reader, ACTIVITY_CALENDAR.MET_ID); @@ -321,7 +299,7 @@ zone.setGradiantCode(gradiantCode); zone.setGradiantLibelle(gradiantLibelle); } else { - info("Ligne inactif", currRow); + //info("Ligne inactif", currRow); } result++; } else { @@ -395,8 +373,8 @@ String msg = "[" + dateString + "] " + ligne + message; writer.write(msg + "\n"); writer.flush(); - if (log.isDebugEnabled()) { - log.debug("write log activityCalendar : " + msg); + if (log.isTraceEnabled()) { + log.trace("write log activityCalendar : " + msg); } } Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceBoatImpl.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceBoatImpl.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceBoatImpl.java 2010-01-29 17:21:13 UTC (rev 294) @@ -500,13 +500,13 @@ // load other complex data from collections List<ActivityMonth> months = result.getActivityMonth(); for (ActivityMonth month : months) { - if (month.getActive()) { +// if (month.getActive()) { List<ActivityProfession> professions = month.getActivityProfession(); for (ActivityProfession profession : professions) { List<ActivityZone> zones = profession.getActivityZone(); zones.size(); } - } +// } } transaction.closeContext(); Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceContactImpl.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceContactImpl.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceContactImpl.java 2010-01-29 17:21:13 UTC (rev 294) @@ -183,24 +183,25 @@ } @Override - public InputStream exportContactCsv(WaoUser user, Collection<Contact> contacts) + public InputStream exportContactCsv(ContactFilter filter) throws SuiviObsmerException { TopiaContext transaction = null; InputStream result = null; + CsvWriter writer = null; try { transaction = rootContext.beginTransaction(); // No contacts from arguments, select all existing from database - if (contacts == null) { - ContactDAO dao = SuiviObsmerModelDAOHelper.getContactDAO(transaction); - if (user.getAdmin()) { - contacts = dao.findAll(); - } else { - // Filtered by company for a simple user - contacts = dao.createQuery().add(Contact.OBSERVER + "." + WaoUser.COMPANY, user.getCompany()). - executeToEntityList(); - } - } +// if (contacts == null) { +// ContactDAO dao = SuiviObsmerModelDAOHelper.getContactDAO(transaction); +// if (user.getAdmin()) { +// contacts = dao.findAll(); +// } else { +// // Filtered by company for a simple user +// contacts = dao.createQuery().add(Contact.OBSERVER + "." + WaoUser.COMPANY, user.getCompany()). +// executeToEntityList(); +// } +// } //String alea = SuiviObsmerContext.createRandomString(16); //String filename = "/tmp/wao-contacts-" + alea + ".csv"; @@ -212,7 +213,7 @@ } FileOutputStream output = new FileOutputStream(file); - CsvWriter writer = new CsvWriter(output, ',', Charset.forName("UTF-8")); + writer = new CsvWriter(output, ',', Charset.forName("UTF-8")); // Headers for (int i = 0; i < ImportHelper.CONTACT_NB_HEADERS; i++) { @@ -224,6 +225,19 @@ DateFormat dateFormat = ImportHelper.getContactDateFormat(); + // Get contacts from database + ContactDAO dao = SuiviObsmerModelDAOHelper.getContactDAO(transaction); + TopiaQuery<Contact> query = dao.createQuery("C"); + + // No page limit in this case + query = filter.prepareQueryForContact(query).resetLimit(); + + if (log.isDebugEnabled()) { + log.debug("Exec query : " + query); + } + + List<Contact> contacts = query.executeToEntityList(); + int curr = 0; for (Contact contact : contacts) { curr++; @@ -296,15 +310,18 @@ record[BOAT.QUARTIER_IMMA.forContactCsv()] = boat.getDistrictCode(); writer.writeRecord(record); + writer.flush(); } - writer.close(); - result = new FileInputStream(file); transaction.closeContext(); } catch (Exception eee) { SuiviObsmerContext.serviceException(transaction, "Impossible d'exporter les contacts", eee); + } finally { + if (writer != null) { + writer.close(); + } } return result; } Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSamplingImpl.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSamplingImpl.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSamplingImpl.java 2010-01-29 17:21:13 UTC (rev 294) @@ -304,6 +304,9 @@ log.debug("Query : " + query); } + // FIXME-FD20100129 Problem with order, need a + // Map<facadeName, Map<sectorName, List<sampleRow>>> + results = query.executeToEntityList(); transaction.closeContext(); Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceUserImpl.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceUserImpl.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceUserImpl.java 2010-01-29 17:21:13 UTC (rev 294) @@ -187,11 +187,9 @@ "\t\t * identifiant = " + user.getLogin() + "\n" + "\t\t * mot de passe = " + password + "\n\n" + "Vous pouvez modifier votre mot de passe en accédant à la page de gestion de votre profil utilisateur.\n\n"; - } else { + } else if (generatePassword) { subject += "Modification de votre compte"; - msg += "\tVos identifiants de connexion à l'application WAO ont été modifiés :\n" + - "\t\t * identifiant = " + user.getLogin() + "\n" + - "\t\t * mot de passe = " + password + "\n\n"; + msg += "\tVotre mot de passe de connexion à l'application WAO a été modifié : " + password + "\n\n"; } msg += "Cordialement,\n\n" + "L'Equipe WAO"; Added: trunk/suiviobsmer-business/src/main/sql/indexes.sql =================================================================== --- trunk/suiviobsmer-business/src/main/sql/indexes.sql (rev 0) +++ trunk/suiviobsmer-business/src/main/sql/indexes.sql 2010-01-29 17:21:13 UTC (rev 294) @@ -0,0 +1,2 @@ +CREATE UNIQUE INDEX activitycalendar_idx ON ACTIVITYCALENDAR (BOAT, YEAR); +CREATE INDEX program_idx ON SAMPLEROW (PROGRAMNAME); \ No newline at end of file Modified: trunk/suiviobsmer-business/src/main/xmi/suiviobsmer.zargo =================================================================== (Binary files differ) Modified: trunk/suiviobsmer-ui/pom.xml =================================================================== --- trunk/suiviobsmer-ui/pom.xml 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-ui/pom.xml 2010-01-29 17:21:13 UTC (rev 294) @@ -10,7 +10,7 @@ <parent> <groupId>fr.ifremer</groupId> <artifactId>suiviobsmer</artifactId> - <version>0.3.1-SNAPSHOT</version> + <version>0.4.0-SNAPSHOT</version> </parent> <groupId>fr.ifremer.suiviobsmer</groupId> Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/base/AbstractFilteredPage.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/base/AbstractFilteredPage.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/base/AbstractFilteredPage.java 2010-01-29 17:21:13 UTC (rev 294) @@ -282,7 +282,7 @@ if (log.isDebugEnabled()) { log.debug("BUSINESS REQUEST [getCompanies]"); } - List<Company> companies = serviceUser.getCompanies(getAvailableDataForFiltersOnly()); + List<Company> companies = serviceUser.getCompanies(isAvailableDataForFiltersOnly()); companySelectModel = new GenericSelectModel<Company>(companies,Company.class, "name","topiaId",propertyAccess); } @@ -306,7 +306,7 @@ if (log.isInfoEnabled()) { log.info("BUSINESS REQUEST [getObservers]"); } - observers = serviceUser.getObservers(getAvailableDataForFiltersOnly()); + observers = serviceUser.getObservers(isAvailableDataForFiltersOnly()); } observerSelectModel = new GenericSelectModel<WaoUser>(observers,WaoUser.class, "fullName","topiaId",propertyAccess); @@ -342,7 +342,7 @@ * * @return true if only active users will be showed, false otherwise */ - protected abstract boolean getAvailableDataForFiltersOnly(); + protected abstract boolean isAvailableDataForFiltersOnly(); public String[] onProvideCompletionsFromBoatName(String input) throws SuiviObsmerException { if (log.isInfoEnabled()) { Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/BooleanImage.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/BooleanImage.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/BooleanImage.java 2010-01-29 17:21:13 UTC (rev 294) @@ -22,10 +22,7 @@ package fr.ifremer.suiviobsmer.ui.components; import org.apache.tapestry5.BindingConstants; -import org.apache.tapestry5.annotations.Log; import org.apache.tapestry5.annotations.Parameter; -import org.apache.tapestry5.ioc.annotations.Inject; -import org.slf4j.Logger; /** * BooleanImage @@ -40,9 +37,6 @@ */ public class BooleanImage { - @Inject - private Logger log; - @Parameter(required = true) private Boolean value; Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Chart.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Chart.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Chart.java 2010-01-29 17:21:13 UTC (rev 294) @@ -38,73 +38,43 @@ import org.jfree.chart.JFreeChart; /** - * Composant qui affiche un graphe de type PieChart. - * - * <pre> - * Utilisation : - * <t:chart width="210" height="190" values="choice" title="title" type="1"/> - * - choice est une liste de chaînes représentant les labels et leur valeurs respectives. ({"val1", "20", "val2", "10"}...) - * - type est le type de représentation du diagramme (1:PIE, 2:PIE3D...) - * Pour généraliser à d'autres types de graphes il faudrait passer un JFreeChart : - * Persist Parameter(required=true) - * private JFreeChart _chart; - * </pre> - * + * * @author rannou * @version $Id$ */ public class Chart { -// /** diagramme de type PIE */ -// public static final int PIE = 1; -// /** diagramme de type PIE3D */ -// public static final int PIE3D = 2; -// /** diagramme de type RING */ -// public static final int RING = 3; - /** titre du diagramme */ @Parameter - private String _title = ""; + private String title = ""; @Parameter(required = true) - private JFreeChart _chart; + private JFreeChart chart; - /** type de représentation du diagramme */ -// @Parameter(required = true) -// private int _type; - @Parameter(required = true) - private int _width; + private int width; @Parameter(required = true) - private int _height; + private int height; @Persist private JFreeChart currentChart; - /** - * list(array) of paired values(label,value): - * [String,Number,String,Number,...] - */ -// @Parameter(required = true) -// private List<Object> _values; - @Inject - private ComponentResources _resources; + private ComponentResources resources; void beginRender(MarkupWriter writer) { - currentChart = _chart; + currentChart = chart; // event link params - Object[] params = new Object[] { _title, _width, _height }; -// params = ArrayUtil.concat(params, _values.toArray()); + Object[] params = new Object[] { title, width, height }; // generate event link - Link link = _resources.createEventLink("chart", params); - Element img = writer.element("img", "src", link); + Link link = resources.createEventLink("chart", params); + writer.element("img", "src", link); - _resources.renderInformalParameters(writer); + resources.renderInformalParameters(writer); } void afterRender(MarkupWriter writer) { @@ -112,7 +82,6 @@ } public StreamResponse onChart(final String title, final int width, final int height) { - // return the image return new StreamResponse() { public String getContentType() { @@ -120,11 +89,6 @@ } public InputStream getStream() throws IOException { -// String alea = SuiviObsmerContext.createRandomString(8); -// String path = SuiviObsmerContext.getProperty(SuiviObsmerContext.PROP_PATH_CHARTS); -// File file = new File(path, "chart_" + alea + ".png"); -// file.getParentFile().mkdirs(); - ByteArrayOutputStream output = new ByteArrayOutputStream(); ChartUtilities.writeChartAsPNG(output, currentChart, width, height); return new ByteArrayInputStream(output.toByteArray()); Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Layout.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Layout.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/components/Layout.java 2010-01-29 17:21:13 UTC (rev 294) @@ -40,7 +40,6 @@ import org.apache.tapestry5.annotations.SessionState; import org.apache.tapestry5.ioc.annotations.Inject; import org.apache.tapestry5.services.Request; -import org.slf4j.Logger; /** * Layout @@ -55,9 +54,6 @@ */ public class Layout { - @Inject - private Logger log; - @Parameter(required = true, defaultPrefix = BindingConstants.LITERAL) private String pageTitle; Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/BoatActivityCalendar.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/BoatActivityCalendar.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/BoatActivityCalendar.java 2010-01-29 17:21:13 UTC (rev 294) @@ -118,8 +118,8 @@ public int getProfessionIndexMax() { int result = 0; - for (ActivityMonth month : activityCalendar.getActivityMonth()) { - int nbProfessions = month.sizeActivityProfession(); + for (ActivityMonth curr : activityCalendar.getActivityMonth()) { + int nbProfessions = curr.sizeActivityProfession(); if (nbProfessions > result) { result = nbProfessions; } Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Boats.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Boats.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Boats.java 2010-01-29 17:21:13 UTC (rev 294) @@ -188,7 +188,7 @@ private InputStream activityCalendarLogFile; - protected static String GZIP_MIMETYPE = "application/x-gzip"; + protected static final String GZIP_MIMETYPE = "application/x-gzip"; InputStream getActivityCalendarLogFile() throws FileNotFoundException { if (activityCalendarLogFile == null) { @@ -292,7 +292,7 @@ } @Override - protected boolean getAvailableDataForFiltersOnly() { + protected boolean isAvailableDataForFiltersOnly() { return true; } @@ -639,8 +639,8 @@ public boolean canCreateNewContactFromElligibleBoat() throws SuiviObsmerException { if (!user.getAdmin() && !isElligibleBoatCompanyActiveFalse()) { - Boat boat = getBoatInfos().getBoat(); - return boat.canCreateContact(user.getCompany()); + Boat currentBoat = getBoatInfos().getBoat(); + return currentBoat.canCreateContact(user.getCompany()); } return false; } Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Contacts.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Contacts.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Contacts.java 2010-01-29 17:21:13 UTC (rev 294) @@ -47,9 +47,7 @@ import java.io.InputStream; import java.text.DateFormat; import java.text.SimpleDateFormat; -import java.util.Calendar; import java.util.Date; -import java.util.GregorianCalendar; import java.util.List; import org.apache.commons.lang.BooleanUtils; import org.apache.commons.lang.StringUtils; @@ -157,7 +155,7 @@ return contactFilter; } - public boolean getFiltersVisible() { + public boolean isFiltersVisible() { boolean companyFiltered = contactFilter.getCompany() != null && user.getAdmin(); return (contactFilter.isFiltered() || companyFiltered) && StringUtils.isEmpty(contactFilter.getBoatName()); /*&& !getDefaultFromDate().equals(contactFilter.getFromDate());*/ } @@ -168,7 +166,7 @@ } @Override - protected boolean getAvailableDataForFiltersOnly() { + protected boolean isAvailableDataForFiltersOnly() { return true; } @@ -234,7 +232,7 @@ public InputStream getStream() throws IOException { InputStream result = null; try { - result = serviceContact.exportContactCsv(user, getContacts().values()); + result = serviceContact.exportContactCsv(getContactFilter()); } catch (SuiviObsmerException eee) { throw new IOException(eee); } @@ -243,6 +241,9 @@ @Override public void prepareResponse(Response response) { + DateFormat dateFormat = new SimpleDateFormat("dd-MM-yyyy"); + Date current = SuiviObsmerContext.getCurrentDate(); + response.setHeader("Content-Disposition", "attachment; filename=\"wao-contacts-" + dateFormat.format(current) + ".csv\""); } }; } Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/SampleRowForm.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/SampleRowForm.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/SampleRowForm.java 2010-01-29 17:21:13 UTC (rev 294) @@ -569,7 +569,7 @@ int size = getSampleMonths().size(); SampleMonth firstMonth = getSampleMonths().get(0); - SampleMonth lastMonth = getSampleMonths().get(size-1); + //SampleMonth lastMonth = getSampleMonths().get(size-1); if (firstMonth != null) { Date firstDate = firstMonth.getPeriodDate(); Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Synthesis.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Synthesis.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Synthesis.java 2010-01-29 17:21:13 UTC (rev 294) @@ -121,7 +121,7 @@ } @Override - protected boolean getAvailableDataForFiltersOnly() { + protected boolean isAvailableDataForFiltersOnly() { return false; } Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/services/ContactModelFactory.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/services/ContactModelFactory.java 2010-01-28 21:53:14 UTC (rev 293) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/services/ContactModelFactory.java 2010-01-29 17:21:13 UTC (rev 294) @@ -137,7 +137,7 @@ Contact contact = (Contact)arg0; return contact.getObserver().getFullName(); } - + @Override public void set(Object arg0, Object arg1) { }