May 2013 - Wao-commits - mailman.cloud.codelutin.com

r1658 - trunk/wao-business/src/main/java/fr/ifremer/wao/entity
by bleny＠users.forge.codelutin.com 15 May '13

15 May '13

Author: bleny Date: 2013-05-15 11:14:32 +0200 (Wed, 15 May 2013) New Revision: 1658 Url: http://forge.codelutin.com/projects/wao/repository/revisions/1658 Log: refs #1967 fixes bug when computing differences when updating terrestrial locations on a sample row Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/entity/SampleRowLogImpl.java Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/entity/SampleRowLogImpl.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/entity/SampleRowLogImpl.java 2013-05-13 13:58:38 UTC (rev 1657) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/entity/SampleRowLogImpl.java 2013-05-15 09:14:32 UTC (rev 1658) @@ -233,6 +233,49 @@ // using the three list to add info if ( ! removedTerrestrialLocations.isEmpty()) { + addChange("Les lieux terrestres suivants ont été dissociés : " + + StringUtils.join(removedTerrestrialLocations, ", ")); + } +// if ( ! keptTerrestrialLocations.isEmpty()) { +// addChange("Les lieux terrestres suivants sont toujours associées : " + +// StringUtils.join(keptTerrestrialLocations, ", ")); +// } + if ( ! addedTerrestrialLocations.isEmpty()) { + addChange("Les lieux terrestres suivants ont été ajoutés : " + + StringUtils.join(addedTerrestrialLocations, ", ")); + } + } + + protected void compareTerrestrialDistricts() { + Collection<TerrestrialLocation> oldTerrestrialLocations; + if (oldRow == null) { + oldTerrestrialLocations = Collections.emptyList(); + } else { + oldTerrestrialLocations = oldRow.getTerrestrialLocations(); + } + Collection<TerrestrialLocation> newTerrestrialLocations = newRow.getTerrestrialLocations(); + + // Lists for boats by status + List<String> removedTerrestrialLocations = new ArrayList<String>(); + List<String> keptTerrestrialLocations = new ArrayList<String>(); + List<String> addedTerrestrialLocations = new ArrayList<String>(); + + // filling the three lists + for (TerrestrialLocation oldTerrestrialLocation : oldTerrestrialLocations) { + if ( ! newTerrestrialLocations.contains(oldTerrestrialLocation)) { + removedTerrestrialLocations.add(oldTerrestrialLocation.getDistrictCode()); + } + } + for (TerrestrialLocation newTerrestrialLocation : newTerrestrialLocations) { + if (oldTerrestrialLocations.contains(newTerrestrialLocation)) { + keptTerrestrialLocations.add(newTerrestrialLocation.getDistrictCode()); + } else { + addedTerrestrialLocations.add(newTerrestrialLocation.getDistrictCode()); + } + } + + // using the three list to add info + if ( ! removedTerrestrialLocations.isEmpty()) { addChange("Les quartiers maritimes suivants ont été dissociées : " + StringUtils.join(removedTerrestrialLocations, ", ")); } @@ -567,7 +610,7 @@ compareFishingZones(); } if (ObsProgram.OBSVENTE.equals(obsProgram)) { - compareTerrestrialLocations(); + compareTerrestrialDistricts(); comparePrograms(); comparePeriods(); compareProfession();

1 0

r1657 - in trunk: src/site/rst/obsMer src/site/rst/obsVente wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations wao-business/src/test/resources/import
by bleny＠users.forge.codelutin.com 13 May '13

13 May '13

Author: bleny Date: 2013-05-13 15:58:38 +0200 (Mon, 13 May 2013) New Revision: 1657 Url: http://forge.codelutin.com/projects/wao/repository/revisions/1657 Log: fixes #2398 always use ',' as separator for multiple values in a single column fix tests Modified: trunk/src/site/rst/obsMer/administrateur.rst trunk/src/site/rst/obsVente/administrateur.rst trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DCF5CodesParserFormatter.java trunk/wao-business/src/test/resources/import/echantillonnage.csv Modified: trunk/src/site/rst/obsMer/administrateur.rst =================================================================== --- trunk/src/site/rst/obsMer/administrateur.rst 2013-05-13 13:45:20 UTC (rev 1656) +++ trunk/src/site/rst/obsMer/administrateur.rst 2013-05-13 13:58:38 UTC (rev 1657) @@ -76,9 +76,9 @@ - PLAN_CODE (string) : Code de la ligne (unique, obligatoire) au format AAAA_MIIII (année, n° de ligne), exemple 2012_M0003 - SOCIETE_NOM (string) : Nom de la société -- PECHE_DIVISION (string) : Zones de pêche liés à la ligne, chaque zone est séparé par un " / " -> ex : VIIa / VIIb (obligatoire) +- PECHE_DIVISION (string) : Zones de pêche liés à la ligne, chaque zone est séparé par une virgule -> ex : VIIa,VIIb (obligatoire) - PECHE_AUTRE (string) : Autres information sur les zones de pêche pour la ligne -- METIER_CODE_DCF5 (string) : Code DCF niveau 5 du métier (obligatoire) +- METIER_CODE_DCF5 (string) : Code DCF niveau 5 du métier (obligatoire) séparés par une virgule - METIER_MAILLAGE (string) : Maillage du métier - METIER_TAILLE (string) : Taille du métier - METIER_AUTRE (string) : Autre information du métier Modified: trunk/src/site/rst/obsVente/administrateur.rst =================================================================== --- trunk/src/site/rst/obsVente/administrateur.rst 2013-05-13 13:45:20 UTC (rev 1656) +++ trunk/src/site/rst/obsVente/administrateur.rst 2013-05-13 13:58:38 UTC (rev 1657) @@ -59,9 +59,9 @@ - PLAN_CODE (string) : Code de la ligne (unique, obligatoire) au format AAAA_VIIII (année, n° de ligne), exemple 2012_V0003 - SOCIETE_NOM (string) : Nom de la société -- PECHE_DIVISION (string) : Zones de pêche liés à la ligne, chaque zone est séparé par un " / " -> ex : VIIa / VIIb (obligatoire) +- PECHE_DIVISION (string) : Zones de pêche liés à la ligne, chaque zone est séparé par une virgule -> ex : VIIa,VIIb (obligatoire) - PECHE_AUTRE (string) : Autres information sur les zones de pêche pour la ligne -- METIER_CODE_DCF5 (string) : Code DCF niveau 5 du métier (obligatoire) +- METIER_CODE_DCF5 (string) : Code DCF niveau 5 du métier (obligatoire) séparés par une virgule - METIER_MAILLAGE (string) : Maillage du métier - METIER_TAILLE (string) : Taille du métier - METIER_AUTRE (string) : Autre information du métier Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DCF5CodesParserFormatter.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DCF5CodesParserFormatter.java 2013-05-13 13:45:20 UTC (rev 1656) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DCF5CodesParserFormatter.java 2013-05-13 13:58:38 UTC (rev 1657) @@ -37,7 +37,7 @@ public class DCF5CodesParserFormatter implements ValueParserFormatter<List<DCF5Code>> { - protected static final String DCF5_CODES_SEPARATOR = ", "; + protected static final String DCF5_CODES_SEPARATOR = ","; @Override public String format(List<DCF5Code> dcf5Codes) { Modified: trunk/wao-business/src/test/resources/import/echantillonnage.csv =================================================================== --- trunk/wao-business/src/test/resources/import/echantillonnage.csv 2013-05-13 13:45:20 UTC (rev 1656) +++ trunk/wao-business/src/test/resources/import/echantillonnage.csv 2013-05-13 13:58:38 UTC (rev 1657) @@ -1,14 +1,14 @@ "PLAN_CODE";"SOCIETE_NOM";"PECHE_DIVISION";"PECHE_AUTRE";"METIER_CODE_DCF5";"METIER_MAILLAGE";"METIER_TAILLE";"METIER_AUTRE";"METIER_LIBELLE";"METIER_ESPECES";"PROGRAMME_CODE";"PROGRAMME_DEBUT";"PROGRAMME_FIN";"12/2009";"01/2010";"02/2010";"03/2010";"04/2010";"05/2010";"06/2010";"07/2010";"08/2010";"09/2010";"10/2010";"11/2010";"12/2010";"01/2011";"02/2011";"03/2011";"PLAN_DUREE_MOY";"PLAN_NB_OBSERV";"PLAN_COMMENT";"TAUX_COUVERTURE_APPLIQUE" -"2010_M0001";"TARTANPION";"IId / I";;"DRB_ALG";;;;;;"MA-2009";"03/2010";"01/2011";;;;0;0;0;0;0;0;0;2;0;0;4;;;"1.5";1;"Un petit commentaire";5 -"2010_M0002";"TARTANPION";"IId";;"DRH_CAT et DRH_CRU";;;;"Fileyage";"poissons tubes";"MANY";"01/2008";"12/2008";;20;0;0;21;0;0;21;0;0;21;0;0;;;;1;1;;5 -"2010_M0003";;"IId / IV";;"DRH";;;;"balottage en plein air";"divers choses visqueuses";"MA-2009";"03/2010";"01/2011";;;;0;22;0;0;25;0;0;21;0;0;25;;;2;1;;5 -"2009_M0001";;"IId / IV";;"GTR_DEF et GNS_DEF";"110-119";;;"Fileyage à poissons tubes";"poissons plats";"PPP2010";"12/2009";"10/2010";0;1;1;1;2;1;1;1;1;1;2;;;;;;1;1;;5 -"2009_M0002";;"IId";;"GTR_DEF et GNS_DEF";"9888-99";;;"Fileyage à sirènes ";"nymphes";"PPP2010";"12/2009";"10/2010";1;1;1;2;3;4;4;4;4;3;3;;;;;;1;1;;10 -"2009_M0003";"TARTANPION";"IV";;"GTR_MOL et GNS";;;;"Fileyages à baleines à bosse";"chameaux";"PPP2010";"12/2009";"10/2010";2;2;2;2;2;2;2;2;2;2;4;;;;;;1;1;;10 +"2010_M0001";"TARTANPION";"IId,I";;"DRB_ALG";;;;;;"MA-2009";"03/2010";"01/2011";;;;0;0;0;0;0;0;0;2;0;0;4;;;"1.5";1;"Un petit commentaire";5 +"2010_M0002";"TARTANPION";"IId";;"DRH_CAT,DRH_CRU";;;;"Fileyage";"poissons tubes";"MANY";"01/2008";"12/2008";;20;0;0;21;0;0;21;0;0;21;0;0;;;;1;1;;5 +"2010_M0003";;"IId,IV";;"DRH";;;;"balottage en plein air";"divers choses visqueuses";"MA-2009";"03/2010";"01/2011";;;;0;22;0;0;25;0;0;21;0;0;25;;;2;1;;5 +"2009_M0001";;"IId,IV";;"GTR_DEF,GNS_DEF";"110-119";;;"Fileyage à poissons tubes";"poissons plats";"PPP2010";"12/2009";"10/2010";0;1;1;1;2;1;1;1;1;1;2;;;;;;1;1;;5 +"2009_M0002";;"IId";;"GTR_DEF,GNS_DEF";"9888-99";;;"Fileyage à sirènes ";"nymphes";"PPP2010";"12/2009";"10/2010";1;1;1;2;3;4;4;4;4;3;3;;;;;;1;1;;10 +"2009_M0003";"TARTANPION";"IV";;"GTR_MOL,GNS";;;;"Fileyages à baleines à bosse";"chameaux";"PPP2010";"12/2009";"10/2010";2;2;2;2;2;2;2;2;2;2;4;;;;;;1;1;;10 "2009_M0004";;"IId";;"PTB_ALG";">=190";">18";;"Chalutage de fond machiavélique";"gadidés; divers";"PPP2010";"12/2009";"10/2010";1;1;1;1;1;1;1;1;1;1;2;;;;;;3;2;;10 "2009_M0005";;"IId";;"PTB_ALG";"70-1100";">1140";;"Plongeur de fond sous-marin";"gadidés; divers";"PPP2010";"12/2009";"10/2010";1;1;1;1;1;1;1;1;1;1;2;;;;;;3;2;;10 -"2009_M0006";"BIS";"I / IV";;"DRB_DEF";"70-1100";">1140";;"Leon";"poissons plats";"PPP2010";"12/2009";"10/2010";0;1;0;1;0;1;2;2;2;2;1;;;;;;3;2;; -"2010_M0004";"TARTANPION";"IId / IV";;"PTB_DEF et PTB_ALG et DRH";;;;"Chalutage de fond";"gadidés; poissons plats; divers";"BLOB";"01/2010";"12/2010";;3;0;0;3;0;0;3;0;0;3;0;0;;;;3;1;; +"2009_M0006";"BIS";"I,IV";;"DRB_DEF";"70-1100";">1140";;"Leon";"poissons plats";"PPP2010";"12/2009";"10/2010";0;1;0;1;0;1;2;2;2;2;1;;;;;;3;2;; +"2010_M0004";"TARTANPION";"IId,IV";;"PTB_DEF,PTB_ALG,DRH";;;;"Chalutage de fond";"gadidés; poissons plats; divers";"BLOB";"01/2010";"12/2010";;3;0;0;3;0;0;3;0;0;3;0;0;;;;3;1;; "2010_M0005";;"IId";;"DRB";;;;"Kamoulox";"merlu; poissons cerises; divers";"MA-2009";"03/2010";"01/2011";;;;0;5;0;0;5;0;0;6;0;0;3;;;3;1;;15 -"2010_M0010";;"I / IV";;"DRB";"70-1100";;;"Kamoulox";"merlu; poissons pommes; divers";"MA-2009";"03/2010";"01/2011";;;;0;5;0;0;5;0;0;6;0;0;3;;;3;1;;15 -"2010_M0011";;"I / IV";;"DRB_ANA";"70-1100";;;"Troubidou";"merlu; poissons cerises";"MA-2009";"03/2010";"01/2011";;;;0;5;0;0;5;0;0;6;0;0;3;;;3;1;; +"2010_M0010";;"I,IV";;"DRB";"70-1100";;;"Kamoulox";"merlu; poissons pommes; divers";"MA-2009";"03/2010";"01/2011";;;;0;5;0;0;5;0;0;6;0;0;3;;;3;1;;15 +"2010_M0011";;"I,IV";;"DRB_ANA";"70-1100";;;"Troubidou";"merlu; poissons cerises";"MA-2009";"03/2010";"01/2011";;;;0;5;0;0;5;0;0;6;0;0;3;;;3;1;;

1 0

r1656 - trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations
by bleny＠users.forge.codelutin.com 13 May '13

13 May '13

Author: bleny Date: 2013-05-13 15:45:20 +0200 (Mon, 13 May 2013) New Revision: 1656 Url: http://forge.codelutin.com/projects/wao/repository/revisions/1656 Log: fixes #2398 always use ',' as separator for multiple values in a single column Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DCF5CodesParserFormatter.java trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/FishingZonesParserFormatter.java Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DCF5CodesParserFormatter.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DCF5CodesParserFormatter.java 2013-05-13 13:27:07 UTC (rev 1655) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DCF5CodesParserFormatter.java 2013-05-13 13:45:20 UTC (rev 1656) @@ -37,7 +37,7 @@ public class DCF5CodesParserFormatter implements ValueParserFormatter<List<DCF5Code>> { - protected static final String DCF5_CODES_SEPARATOR = " et "; + protected static final String DCF5_CODES_SEPARATOR = ", "; @Override public String format(List<DCF5Code> dcf5Codes) { Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/FishingZonesParserFormatter.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/FishingZonesParserFormatter.java 2013-05-13 13:27:07 UTC (rev 1655) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/FishingZonesParserFormatter.java 2013-05-13 13:45:20 UTC (rev 1656) @@ -52,7 +52,7 @@ for (FishingZone fishingZone : fishingZones) { districtCodes.add(fishingZone.getDistrictCode()); } - return StringUtils.join(districtCodes, " / "); + return StringUtils.join(districtCodes, ", "); } @Override @@ -64,7 +64,7 @@ if (StringUtils.isBlank(districtCodes)) { throw new IllegalArgumentException(WaoUtils._("wao.import.sampleRow.failure.fishingZoneMissing")); } else { - String[] codes = districtCodes.split("/"); + String[] codes = districtCodes.split(","); for (String code : codes) { FishingZone fishingZone = indexedFishingZones.get(code.trim()); if (fishingZone == null) {

1 0

r1655 - trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations
by bleny＠users.forge.codelutin.com 13 May '13

13 May '13

Author: bleny Date: 2013-05-13 15:27:07 +0200 (Mon, 13 May 2013) New Revision: 1655 Url: http://forge.codelutin.com/projects/wao/repository/revisions/1655 Log: refs #2398 fix import multiple terrestrial locations on a sample row Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DistrictsParserFormatter.java Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DistrictsParserFormatter.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DistrictsParserFormatter.java 2013-04-23 09:04:36 UTC (rev 1654) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/io/csv2/models/operations/DistrictsParserFormatter.java 2013-05-13 13:27:07 UTC (rev 1655) @@ -74,6 +74,7 @@ if (district == null) { throw new IllegalArgumentException(WaoUtils._("wao.import.contact.failure.wrongDistrict", value)); } + districts.add(district); } return districts; }

1 0