This is an automated email from the git hooks/post-receive script. New commit to branch develop in repository tutti. See http://git.codelutin.com/tutti.git commit 1e1f975122b46f4d2ca656f6daf5adb8e15e3e08 Author: Tony CHEMIT <chemit@codelutin.com> Date: Mon Nov 24 17:51:07 2014 +0100 fixes #6096 (amélioration du message lors que les références sont utilisées) --- .../service/referential/ReferentialTemporaryGearService.java | 8 +++++++- .../service/referential/ReferentialTemporaryPersonService.java | 8 +++++++- .../service/referential/ReferentialTemporarySpeciesService.java | 8 +++++++- .../service/referential/ReferentialTemporaryVesselService.java | 9 ++++++++- .../src/main/resources/i18n/tutti-service_fr_FR.properties | 8 ++++---- 5 files changed, 33 insertions(+), 8 deletions(-) diff --git a/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryGearService.java b/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryGearService.java index 1909181..c41c27e 100644 --- a/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryGearService.java +++ b/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryGearService.java @@ -8,6 +8,7 @@ import fr.ifremer.tutti.persistence.entities.TuttiEntities; import fr.ifremer.tutti.persistence.entities.referential.Gear; import fr.ifremer.tutti.persistence.entities.referential.Gears; import fr.ifremer.tutti.service.AbstractTuttiService; +import fr.ifremer.tutti.service.DecoratorService; import fr.ifremer.tutti.service.PersistenceService; import fr.ifremer.tutti.service.TuttiServiceContext; import org.apache.commons.io.IOUtils; @@ -44,10 +45,13 @@ public class ReferentialTemporaryGearService extends AbstractTuttiService { protected PersistenceService persistenceService; + protected DecoratorService decoratorService; + @Override public void setServiceContext(TuttiServiceContext context) { super.setServiceContext(context); persistenceService = getService(PersistenceService.class); + decoratorService = getService(DecoratorService.class); } public ReferentialImportResult<Gear> importTemporaryGear(File file) throws IOException { @@ -218,7 +222,9 @@ public class ReferentialTemporaryGearService extends AbstractTuttiService { // Suppression if (persistenceService.isTemporaryGearUsed(id)) { - throw new IllegalArgumentException(t("tutti.service.referential.import.species.used.error", id)); + + String gearRef = id + " : " + decoratorService.getDecoratorByType(Gear.class).toString(gear); + throw new IllegalArgumentException(t("tutti.service.referential.import.gear.used.error", gearRef)); } requestResult.addIdToDelete(id); diff --git a/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryPersonService.java b/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryPersonService.java index 003dc34..f8628f9 100644 --- a/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryPersonService.java +++ b/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryPersonService.java @@ -8,6 +8,7 @@ import fr.ifremer.tutti.persistence.entities.TuttiEntities; import fr.ifremer.tutti.persistence.entities.referential.Person; import fr.ifremer.tutti.persistence.entities.referential.Persons; import fr.ifremer.tutti.service.AbstractTuttiService; +import fr.ifremer.tutti.service.DecoratorService; import fr.ifremer.tutti.service.PersistenceService; import fr.ifremer.tutti.service.TuttiServiceContext; import org.apache.commons.io.IOUtils; @@ -43,10 +44,13 @@ public class ReferentialTemporaryPersonService extends AbstractTuttiService { protected PersistenceService persistenceService; + protected DecoratorService decoratorService; + @Override public void setServiceContext(TuttiServiceContext context) { super.setServiceContext(context); persistenceService = getService(PersistenceService.class); + decoratorService = getService(DecoratorService.class); } public ReferentialImportResult<Person> importTemporaryPerson(File file) throws IOException { @@ -205,7 +209,9 @@ public class ReferentialTemporaryPersonService extends AbstractTuttiService { // Suppression if (persistenceService.isTemporaryPersonUsed(id)) { - throw new IllegalArgumentException(t("tutti.service.referential.import.person.used.error", id)); + + String personRef = id + " : " + decoratorService.getDecoratorByType(Person.class).toString(person); + throw new IllegalArgumentException(t("tutti.service.referential.import.person.used.error", personRef)); } requestResult.addIdToDelete(id); diff --git a/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporarySpeciesService.java b/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporarySpeciesService.java index 2d4d180..af8650d 100644 --- a/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporarySpeciesService.java +++ b/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporarySpeciesService.java @@ -8,6 +8,7 @@ import fr.ifremer.tutti.persistence.entities.TuttiEntities; import fr.ifremer.tutti.persistence.entities.referential.Species; import fr.ifremer.tutti.persistence.entities.referential.Speciess; import fr.ifremer.tutti.service.AbstractTuttiService; +import fr.ifremer.tutti.service.DecoratorService; import fr.ifremer.tutti.service.PersistenceService; import fr.ifremer.tutti.service.TuttiServiceContext; import org.apache.commons.io.IOUtils; @@ -43,10 +44,13 @@ public class ReferentialTemporarySpeciesService extends AbstractTuttiService { protected PersistenceService persistenceService; + protected DecoratorService decoratorService; + @Override public void setServiceContext(TuttiServiceContext context) { super.setServiceContext(context); persistenceService = getService(PersistenceService.class); + decoratorService = getService(DecoratorService.class); } public ReferentialImportResult<Species> importTemporarySpecies(File file) { @@ -231,7 +235,9 @@ public class ReferentialTemporarySpeciesService extends AbstractTuttiService { // Suppression if (persistenceService.isTemporarySpeciesUsed(referenceTaxonId)) { - throw new IllegalArgumentException(t("tutti.service.referential.import.species.used.error", id)); + + String speciesRef = id + " : " + decoratorService.getDecoratorByType(Species.class, DecoratorService.WITH_SURVEY_CODE).toString(species); + throw new IllegalArgumentException(t("tutti.service.referential.import.species.used.error", speciesRef)); } requestResult.addIdToDelete(referenceTaxonId); diff --git a/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryVesselService.java b/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryVesselService.java index ae27251..c5026d5 100644 --- a/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryVesselService.java +++ b/tutti-service/src/main/java/fr/ifremer/tutti/service/referential/ReferentialTemporaryVesselService.java @@ -8,6 +8,7 @@ import fr.ifremer.tutti.persistence.entities.TuttiEntities; import fr.ifremer.tutti.persistence.entities.referential.Vessel; import fr.ifremer.tutti.persistence.entities.referential.Vessels; import fr.ifremer.tutti.service.AbstractTuttiService; +import fr.ifremer.tutti.service.DecoratorService; import fr.ifremer.tutti.service.PersistenceService; import fr.ifremer.tutti.service.TuttiServiceContext; import org.apache.commons.io.IOUtils; @@ -43,10 +44,13 @@ public class ReferentialTemporaryVesselService extends AbstractTuttiService { protected PersistenceService persistenceService; + protected DecoratorService decoratorService; + @Override public void setServiceContext(TuttiServiceContext context) { super.setServiceContext(context); persistenceService = getService(PersistenceService.class); + decoratorService = getService(DecoratorService.class); } public ReferentialImportResult<Vessel> importTemporaryVessel(File file) throws IOException { @@ -214,6 +218,7 @@ public class ReferentialTemporaryVesselService extends AbstractTuttiService { Vessel vessel = requestResult.getExistingEntityById(id); if (vessel == null) { + throw new IllegalArgumentException(t("tutti.service.referential.import.vessel.notExistingId.error", id)); } @@ -222,7 +227,9 @@ public class ReferentialTemporaryVesselService extends AbstractTuttiService { // Suppression if (persistenceService.isTemporaryVesselUsed(id)) { - throw new IllegalArgumentException(t("tutti.service.referential.import.vessel.used.error", id)); + + String vesselRef = id + " :" + decoratorService.getDecoratorByType(Vessel.class).toString(vessel); + throw new IllegalArgumentException(t("tutti.service.referential.import.vessel.used.error", vesselRef)); } requestResult.addIdToDelete(id); diff --git a/tutti-service/src/main/resources/i18n/tutti-service_fr_FR.properties b/tutti-service/src/main/resources/i18n/tutti-service_fr_FR.properties index 1e4416b..c607b3b 100644 --- a/tutti-service/src/main/resources/i18n/tutti-service_fr_FR.properties +++ b/tutti-service/src/main/resources/i18n/tutti-service_fr_FR.properties @@ -206,7 +206,7 @@ tutti.service.referential.import.gear.existingName.error=L'engin '%s' n'a pas é tutti.service.referential.import.gear.idNotNegative.error=L'identifiant d'un engin temporaire doit être négatif \: %s tutti.service.referential.import.gear.noName.error=Le nom de l'engin est obligatoire tutti.service.referential.import.gear.notExistingId.error=L'identifiant '%s' n'a pas été trouvé parmi les engins temporaires -tutti.service.referential.import.gear.used.error=L'engin temporaire %s est utilisé dans l'application. Vous devez le remplacer pour pouvoir le supprimer. +tutti.service.referential.import.gear.used.error=L'engin temporaire '%s' est utilisé dans l'application. Vous devez le remplacer pour pouvoir le supprimer. tutti.service.referential.import.gears.error=Erreur lors de l'import des engins du fichier %s tutti.service.referential.import.person.add.noName.error=Le nom de la personne (prénom ou nom) est obligatoire tutti.service.referential.import.person.cannotDeleteNotExisting.error=La personne temporaire d'identifiant %s n'extsite pas, impossible de la supprimer @@ -215,7 +215,7 @@ tutti.service.referential.import.person.existingName.error=Une personne avec ce tutti.service.referential.import.person.idNotNegative.error=L'identifiant d'une personne temporaire doit être négatif \: %s tutti.service.referential.import.person.noName.error=Le nom de la personne (prénom ou nom) est obligatoire tutti.service.referential.import.person.notExistingId.error=L'identifiant '%s' n'a pas été trouvé parmi les personnes temporaires -tutti.service.referential.import.person.used.error=La personne temporaire %s est utilisée dans l'application. Vous devez la remplacer pour pouvoir la supprimer. +tutti.service.referential.import.person.used.error=La personne temporaire '%s' est utilisée dans l'application. Vous devez la remplacer pour pouvoir la supprimer. tutti.service.referential.import.persons.error=Erreur lors de l'import des personnes du fichier %s tutti.service.referential.import.species.add.noName.error=Le nom pour l'espèce %s est obligatoire. tutti.service.referential.import.species.cannotDeleteNotExisting.error=L'espèce temporaire d'identifiant %s n'extsite pas, impossible de la supprimer @@ -225,7 +225,7 @@ tutti.service.referential.import.species.existingName.error=L'espèce '%s' n'a p tutti.service.referential.import.species.idNotNegative.error=L'identifiant d'une espèce temporaire doit être négatif \: %s tutti.service.referential.import.species.noName.error=Le nom pour l'espèce %s est obligatoire. tutti.service.referential.import.species.notExistingId.error=L'identifiant '%s' n'a pas été trouvé parmi les espèces temporaires -tutti.service.referential.import.species.used.error=L'espèce temporaire %s est utilisée dans l'application. Vous devez la remplacer pour pouvoir la supprimer. +tutti.service.referential.import.species.used.error=L'espèce temporaire '%s' est utilisée dans l'application. Vous devez la remplacer pour pouvoir la supprimer. tutti.service.referential.import.vessel.add.noRegistrationCode.error=L'immatriculation est obligatoire tutti.service.referential.import.vessel.cannotDeleteNotExisting.error=Le navire temporaire d'identifiant %s n'extsite pas, impossible de la supprimer tutti.service.referential.import.vessel.cannotDeleteWithoutId.error=L'identifiant est obligatoire pour la suppression @@ -233,7 +233,7 @@ tutti.service.referential.import.vessel.codePrefixMissing.error=L'identifiant d' tutti.service.referential.import.vessel.existingRegistrationCode.error=Le navire '%s' n'a pas été ajouté au référentiel car il est déjà présent tutti.service.referential.import.vessel.noRegistrationCode.error=L'immatriculation est obligatoire tutti.service.referential.import.vessel.notExistingId.error=L'identifiant '%s' n'a pas été trouvé parmi les navires temporaires -tutti.service.referential.import.vessel.used.error=Le navire temporaire %s est utilisé dans l'application. Vous devez le remplacer pour pouvoir le supprimer. +tutti.service.referential.import.vessel.used.error=Le navire temporaire '%s' est utilisé dans l'application. Vous devez le remplacer pour pouvoir le supprimer. 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 -- To stop receiving notification emails like this one, please contact codelutin.com SCM administrator <admin+scm@codelutin.com>.