Author: kmorin Date: 2013-05-17 14:54:48 +0200 (Fri, 17 May 2013) New Revision: 959 Url: http://forge.codelutin.com/projects/tutti/repository/revisions/959 Log: fixes #2444 [DONNEES TEMPORAIRES] Import d'une esp?\195?\168ce temporaire qui existe d?\195?\169j?\195?\160. Modified: trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/TuttiCsvUtil.java trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/GearModel.java trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/SpeciesModel.java trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/TuttiReferentialImportExportService.java trunk/tutti-service/src/main/resources/i18n/tutti-service_en_GB.properties trunk/tutti-service/src/main/resources/i18n/tutti-service_fr_FR.properties Modified: trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/TuttiCsvUtil.java =================================================================== --- trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/TuttiCsvUtil.java 2013-05-17 12:12:39 UTC (rev 958) +++ trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/TuttiCsvUtil.java 2013-05-17 12:54:48 UTC (rev 959) @@ -108,6 +108,14 @@ public void newNotExistingValueColumn(String headerName, String propertyName, Set<String> universe) { newMandatoryColumn(headerName, propertyName, new NotExistingKeyValue(propertyName, universe)); } + + public void newNotExistingValueColumn(String propertyName, Set<String> universe, String errorMessage) { + newNotExistingValueColumn(propertyName, propertyName, universe, errorMessage); + } + + public void newNotExistingValueColumn(String headerName, String propertyName, Set<String> universe, String errorMessage) { + newMandatoryColumn(headerName, propertyName, new NotExistingKeyValue(propertyName, universe, errorMessage)); + } } public static abstract class AbstractTuttiExportModel<M> extends AbstractExportModel<M> { @@ -247,11 +255,21 @@ protected final Set<String> existingValues; + protected final String errorMessage; + public NotExistingKeyValue( String propertyName, Set<String> universe) { + this(propertyName, universe, "tutti.service.csv.parse.entityAlreadyDefined"); + } + + public NotExistingKeyValue( + String propertyName, + Set<String> universe, + String errorMessage) { this.propertyName = propertyName; this.existingValues = universe; + this.errorMessage = errorMessage; } @Override @@ -261,7 +279,7 @@ if (existingValues.contains(value)) { // entity already defined - throw new TuttiTechnicalException(_("tutti.service.csv.parse.entityAlreadyDefined", propertyName, value)); + throw new TuttiTechnicalException(_(errorMessage, propertyName, value)); } } return value; Modified: trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/GearModel.java =================================================================== --- trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/GearModel.java 2013-05-17 12:12:39 UTC (rev 958) +++ trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/GearModel.java 2013-05-17 12:54:48 UTC (rev 959) @@ -31,6 +31,8 @@ import java.util.Set; +import static org.nuiton.i18n.I18n.n_; + /** * Model to import / export {@link Gear} in csv format. * @@ -44,7 +46,7 @@ // import definition - newNotExistingValueColumn(Gear.PROPERTY_NAME, existingGearName); + newNotExistingValueColumn(Gear.PROPERTY_NAME, existingGearName, n_("tutti.service.referential.import.gear.existingName.error")); newMandatoryColumn(Gear.PROPERTY_LABEL); newMandatoryColumn(Gear.PROPERTY_SCIENTIFIC_GEAR, Common.PRIMITIVE_BOOLEAN); Modified: trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/SpeciesModel.java =================================================================== --- trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/SpeciesModel.java 2013-05-17 12:12:39 UTC (rev 958) +++ trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/SpeciesModel.java 2013-05-17 12:54:48 UTC (rev 959) @@ -30,6 +30,8 @@ import java.util.Set; +import static org.nuiton.i18n.I18n.n_; + /** * Model to import / export {@link Species} in csv format. * @@ -43,7 +45,7 @@ // import definition - newNotExistingValueColumn(Species.PROPERTY_NAME, existingSpeciesName); + newNotExistingValueColumn(Species.PROPERTY_NAME, existingSpeciesName, n_("tutti.service.referential.import.species.existingName.error")); // export definition Modified: trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/TuttiReferentialImportExportService.java =================================================================== --- trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/TuttiReferentialImportExportService.java 2013-05-17 12:12:39 UTC (rev 958) +++ trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/TuttiReferentialImportExportService.java 2013-05-17 12:54:48 UTC (rev 959) @@ -46,6 +46,7 @@ import org.apache.commons.logging.LogFactory; import org.nuiton.util.csv.Export; import org.nuiton.util.csv.Import; +import org.nuiton.util.csv.ImportRuntimeException; import java.io.BufferedWriter; import java.io.File; @@ -106,8 +107,13 @@ IOUtils.closeQuietly(importer); } reader.close(); + } catch (IOException e) { throw new IOException(_("tutti.service.referential.import.species.error", file), e); + + } catch (ImportRuntimeException e) { + throw new TuttiTechnicalException(e.getCause().getMessage()); + } finally { IOUtils.closeQuietly(reader); @@ -242,6 +248,10 @@ reader.close(); } catch (IOException e) { throw new IOException(_("tutti.service.referential.import.gears.error", file), e); + + } catch (ImportRuntimeException e) { + throw new TuttiTechnicalException(e.getCause().getMessage()); + } finally { IOUtils.closeQuietly(reader); Modified: trunk/tutti-service/src/main/resources/i18n/tutti-service_en_GB.properties =================================================================== --- trunk/tutti-service/src/main/resources/i18n/tutti-service_en_GB.properties 2013-05-17 12:12:39 UTC (rev 958) +++ trunk/tutti-service/src/main/resources/i18n/tutti-service_en_GB.properties 2013-05-17 12:54:48 UTC (rev 959) @@ -167,9 +167,11 @@ tutti.service.referential.export.person.error= tutti.service.referential.export.species.error= tutti.service.referential.export.vessel.error= +tutti.service.referential.import.gear.existingName.error= tutti.service.referential.import.gears.error= tutti.service.referential.import.persons.error= tutti.service.referential.import.species.error= +tutti.service.referential.import.species.existingName.error= tutti.service.referential.import.vessels.error= tutti.service.referential.import.vessels.existingValue.error= tutti.service.sumatra.export.error= Modified: trunk/tutti-service/src/main/resources/i18n/tutti-service_fr_FR.properties =================================================================== --- trunk/tutti-service/src/main/resources/i18n/tutti-service_fr_FR.properties 2013-05-17 12:12:39 UTC (rev 958) +++ trunk/tutti-service/src/main/resources/i18n/tutti-service_fr_FR.properties 2013-05-17 12:54:48 UTC (rev 959) @@ -137,9 +137,11 @@ tutti.service.referential.export.person.error=Erreur lors de l'export des personnes dans le fichier %s tutti.service.referential.export.species.error=Erreur lors de l'export des espèces dans le fichier %s tutti.service.referential.export.vessel.error=Erreur lors de l'export des navires dans le fichier %s +tutti.service.referential.import.gear.existingName.error=L'engin '%2$s' n'a pas été ajouté au référentiel car il est déjà présent tutti.service.referential.import.gears.error=Erreur lors de l'import des engins du fichier %s tutti.service.referential.import.persons.error=Erreur lors de l'import des personnes du fichier %s tutti.service.referential.import.species.error=Erreur lors de l'import des espèces du fichier %s +tutti.service.referential.import.species.existingName.error=L'espèce '%2$s' n'a pas été ajoutée au référentiel car elle est déjà présente tutti.service.referential.import.vessels.error=Erreur lors de l'import des navires du fichier %s tutti.service.referential.import.vessels.existingValue.error=Un navire avec le nom %1s et l'immatriculation %2s existe déjà dans le référentiel tutti.service.sumatra.export.error=Erreur lors de l'export Sumatra dans le fichier %s