Author: fdesbois Date: 2010-01-14 19:14:05 +0000 (Thu, 14 Jan 2010) New Revision: 200 Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/bean/BoatFilterImpl.java trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleMonthImpl.java trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSamplingImpl.java trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSynthesisImpl.java trunk/suiviobsmer-business/src/main/xmi/suiviobsmer.zargo Log: - Delete old getPeriod method - Add condition to avoid unused elligibleBoat join in Synthesis Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/bean/BoatFilterImpl.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/bean/BoatFilterImpl.java 2010-01-14 18:44:17 UTC (rev 199) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/bean/BoatFilterImpl.java 2010-01-14 19:14:05 UTC (rev 200) @@ -57,4 +57,9 @@ } return query; } + + @Override + public boolean isBoatFiltered() { + return !StringUtils.isEmpty(getBoatName()) || !StringUtils.isEmpty(getBoatDistrictCode()) || getBoatImmatriculation() != null; + } } Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleMonthImpl.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleMonthImpl.java 2010-01-14 18:44:17 UTC (rev 199) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleMonthImpl.java 2010-01-14 19:14:05 UTC (rev 200) @@ -26,9 +26,9 @@ import fr.ifremer.suiviobsmer.SuiviObsmerModelDAOHelper; import java.io.Serializable; import java.util.Calendar; -import java.util.Date; import java.util.GregorianCalendar; import org.nuiton.topia.TopiaContext; +import org.nuiton.util.PeriodDates; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -49,12 +49,13 @@ private static final long serialVersionUID = 1L; - @Override - @Deprecated - public void setPeriod(Date date) { - setPeriodDate(date); - } - +// @Override +// public PeriodDates getPeriod() { +// PeriodDates period = new PeriodDates(getPeriodDate(), getPeriodDate()); +// period.initDayOfMonthExtremities(); +// return period; +// } + protected Calendar getCalendar() { Calendar calendar = new GregorianCalendar(); if (getPeriodDate() != null) { 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-14 18:44:17 UTC (rev 199) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSamplingImpl.java 2010-01-14 19:14:05 UTC (rev 200) @@ -458,7 +458,7 @@ public SampleMonth getNewSampleMonth(Date period, SampleRow row) { SampleMonth month = new SampleMonthImpl(); month.setSampleRow(row); - month.setPeriod(period); + month.setPeriodDate(period); return month; } Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSynthesisImpl.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSynthesisImpl.java 2010-01-14 18:44:17 UTC (rev 199) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSynthesisImpl.java 2010-01-14 19:14:05 UTC (rev 200) @@ -5,6 +5,7 @@ import fr.ifremer.suiviobsmer.SuiviObsmerException; import fr.ifremer.suiviobsmer.SuiviObsmerModelDAOHelper; import fr.ifremer.suiviobsmer.bean.BoatFilter; +import fr.ifremer.suiviobsmer.entity.Contact; import fr.ifremer.suiviobsmer.entity.ElligibleBoat; import fr.ifremer.suiviobsmer.entity.SampleMonth; import fr.ifremer.suiviobsmer.entity.SampleMonthDAO; @@ -77,9 +78,13 @@ query = filter.prepareQueryForUser(query, "M.sampleRow"); query = filter.prepareQueryForSampling(query, "M"); - query.addFrom(ElligibleBoat.class.getName() + " E").add("E IN elements(M.sampleRow)"); - query = filter.prepareQueryForBoat(query, "E"); + if (filter.isBoatFiltered()) { + // ElligibleBoat is sufficient ? no need to check contacts (more difficult query) ? + query.addFrom(ElligibleBoat.class.getName() + " E").add("E IN elements(M.sampleRow)"); + query = filter.prepareQueryForBoat(query, "E"); + } + if (log.isTraceEnabled()) { log.trace("Exec query : " + query); } Modified: trunk/suiviobsmer-business/src/main/xmi/suiviobsmer.zargo =================================================================== (Binary files differ)