Author: fdesbois Date: 2010-01-21 09:37:59 +0000 (Thu, 21 Jan 2010) New Revision: 253 Modified: trunk/src/site/rst/index.rst trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleRowImpl.java trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleRowLogImpl.java trunk/suiviobsmer-business/src/main/xmi/suiviobsmer.zargo trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/data/AbstractMappedGridDataSource.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/SamplingPlan.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/services/ContactModelFactory.java Log: - Ano #2019 : Add header for multiple zone in samplingPlan (refactor getFirstFishingZone by two methods getFacade and getSectors) - Clean some FIXME Modified: trunk/src/site/rst/index.rst =================================================================== --- trunk/src/site/rst/index.rst 2010-01-20 12:25:57 UTC (rev 252) +++ trunk/src/site/rst/index.rst 2010-01-21 09:37:59 UTC (rev 253) @@ -1,4 +1,4 @@ Accueil ======= -TODO +Application permettant le suivi des embarquements effectués sur des navires de pêche dans le cadre d'observations scientifiques. Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleRowImpl.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleRowImpl.java 2010-01-20 12:25:57 UTC (rev 252) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleRowImpl.java 2010-01-21 09:37:59 UTC (rev 253) @@ -24,6 +24,7 @@ import fr.ifremer.suiviobsmer.SuiviObsmerContext; import fr.ifremer.suiviobsmer.SuiviObsmerException; import java.io.Serializable; +import java.util.ArrayList; import java.util.Calendar; import java.util.Date; import java.util.GregorianCalendar; @@ -77,15 +78,49 @@ * Return the first fishingZone of the list linked with the SampleRow. * @return a FishingZone or null if there is no fishingZone linked with the SampleRow */ +// @Override +// public FishingZone getFirstFishingZone() { +// if (getFishingZone().size() > 0) { +// return getFishingZone().get(0); +// } +// return null; +// } + + /** + * Get the facade of the sampleRow. The row has normally only one facade in the FishingZone list. + * + * @return the facade name of the row + */ @Override - public FishingZone getFirstFishingZone() { + public String getFacade() { if (getFishingZone().size() > 0) { - return getFishingZone().get(0); + return getFishingZone().get(0).getFacadeName(); } return null; } /** + * Get the sectors of the sampleRow. The sectors will be concat as a single String with "-" as separator. + * + * @return a String with all sectors linked to the sampleRow + */ + @Override + public String getSectors() { + List<String> results = new ArrayList<String>(); + for (FishingZone zone : getFishingZone()) { + String sector = zone.getSectorName(); + if (!results.contains(sector)) { + results.add(sector); + } + } + String result = ""; + for (String sector : results) { + result += sector + " - "; + } + return result.substring(0, result.length()-3); + } + + /** * Check if the sampleRow has an ElligibleBoat corresponding to the boat in argument. * The test return true only if it's a global ElligbleBoat set by an admin and not by a user (companyActive). * @param boat Boat to test Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleRowLogImpl.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleRowLogImpl.java 2010-01-20 12:25:57 UTC (rev 252) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/entity/SampleRowLogImpl.java 2010-01-21 09:37:59 UTC (rev 253) @@ -23,7 +23,6 @@ import java.text.DateFormat; import java.text.SimpleDateFormat; -import java.util.List; import org.apache.commons.lang.StringUtils; import org.nuiton.util.PeriodDates; @@ -69,7 +68,6 @@ return; } - // FIXME doesn't work if (!nvl(oldProfession.getCode()).equals(nvl(newProfession.getCode()))) { addChange("Le métier de la ligne est passé de " + oldProfession.getCode() + " à " + newProfession.getCode()); } Modified: trunk/suiviobsmer-business/src/main/xmi/suiviobsmer.zargo =================================================================== (Binary files differ) Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/data/AbstractMappedGridDataSource.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/data/AbstractMappedGridDataSource.java 2010-01-20 12:25:57 UTC (rev 252) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/data/AbstractMappedGridDataSource.java 2010-01-21 09:37:59 UTC (rev 253) @@ -76,8 +76,8 @@ mapResults = execute(startIndex, endIndex, getSortConstraint(sortConstraints)); listResults = new ArrayList<E>(mapResults.values()); - // FIXME manage sort columns in execute ???? obvious - sortResults(sortConstraints); + // FIXME manage sort columns in execute ???? seems obvious --> needed for Contacts, done for Boats + //sortResults(sortConstraints); } catch (SuiviObsmerException eee) { throw new TapestryException("", eee); } 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-20 12:25:57 UTC (rev 252) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/Boats.java 2010-01-21 09:37:59 UTC (rev 253) @@ -143,7 +143,8 @@ boats = null; filterFeedback.addInfo("Vous êtes en cours de sélection d'un navire pour la ligne du plan " + sampleRow.getCode()); professionId = getSampleRow().getProfession().getTopiaId(); - fishingZoneId = getSampleRow().getFirstFishingZone().getTopiaId(); + // FIXME FD-20100121 Missing filter for fishingZone --> Need use of AbstratFilteredPage + //fishingZoneId = getSampleRow().getFirstFishingZone().getTopiaId(); getBoatFilter().setSampleRowCode(getSampleRow().getCode()); } } Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/SamplingPlan.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/SamplingPlan.java 2010-01-20 12:25:57 UTC (rev 252) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/SamplingPlan.java 2010-01-21 09:37:59 UTC (rev 253) @@ -196,18 +196,16 @@ } public boolean isFacadeChanged() { - FishingZone zone = row.getFirstFishingZone(); - if (!zone.getFacadeName().equals(currentFacadeName)) { - currentFacadeName = zone.getFacadeName(); + if (!row.getFacade().equals(currentFacadeName)) { + currentFacadeName = row.getFacade(); return true; } return false; } public boolean isSectorChanged() { - FishingZone zone = row.getFirstFishingZone(); - if (!zone.getSectorName().equals(currentSectorName)) { - currentSectorName = zone.getSectorName(); + if (!row.getSectors().equals(currentSectorName)) { + currentSectorName = row.getSectors(); return true; } 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-20 12:25:57 UTC (rev 252) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/services/ContactModelFactory.java 2010-01-21 09:37:59 UTC (rev 253) @@ -293,7 +293,7 @@ @Override public Object get(Object arg0) { Contact contact = (Contact)arg0; - return contact.getSampleRow().getFirstFishingZone().getFacadeName(); + return contact.getSampleRow().getFacade(); } @Override @@ -317,7 +317,7 @@ @Override public Object get(Object arg0) { Contact contact = (Contact)arg0; - return contact.getSampleRow().getFirstFishingZone().getSectorName(); + return contact.getSampleRow().getSectors(); } @Override