December 2013 - Tutti-commits - mailman.cloud.codelutin.com

r1421 - in trunk: tutti-persistence/src/test/resources tutti-service/src/test/resources tutti-ui-swing/src/main/filtered-resources
by tchemit＠users.forge.codelutin.com 03 Dec '13

03 Dec '13

Author: tchemit Date: 2013-12-03 11:14:03 +0100 (Tue, 03 Dec 2013) New Revision: 1421 Url: http://forge.codelutin.com/projects/tutti/repository/revisions/1421 Log: improve log levels Modified: trunk/tutti-persistence/src/test/resources/log4j.properties trunk/tutti-service/src/test/resources/log4j.properties trunk/tutti-ui-swing/src/main/filtered-resources/log4j.properties Modified: trunk/tutti-persistence/src/test/resources/log4j.properties =================================================================== --- trunk/tutti-persistence/src/test/resources/log4j.properties 2013-12-02 18:42:53 UTC (rev 1420) +++ trunk/tutti-persistence/src/test/resources/log4j.properties 2013-12-03 10:14:03 UTC (rev 1421) @@ -39,4 +39,6 @@ log4j.logger.fr.ifremer.adagio.core=ERROR log4j.logger.fr.ifremer.tutti=INFO +#See https://forum.hibernate.org/viewtopic.php?p=2404391 +log4j.logger.org.hibernate.engine.StatefulPersistenceContext.ProxyWarnLog=ERROR #log4j.logger.org.nuiton=INFO Modified: trunk/tutti-service/src/test/resources/log4j.properties =================================================================== --- trunk/tutti-service/src/test/resources/log4j.properties 2013-12-02 18:42:53 UTC (rev 1420) +++ trunk/tutti-service/src/test/resources/log4j.properties 2013-12-03 10:14:03 UTC (rev 1421) @@ -29,4 +29,7 @@ log4j.appender.A1.layout.ConversionPattern=%d %-4r [%t] %-5p %c %x - %m%n log4j.logger.fr.ifremer.tutti=INFO -log4j.logger.org.nuiton=INFO \ No newline at end of file +#log4j.logger.org.nuiton=INFO + +#See https://forum.hibernate.org/viewtopic.php?p=2404391 +log4j.logger.org.hibernate.engine.StatefulPersistenceContext.ProxyWarnLog=ERROR \ No newline at end of file Modified: trunk/tutti-ui-swing/src/main/filtered-resources/log4j.properties =================================================================== --- trunk/tutti-ui-swing/src/main/filtered-resources/log4j.properties 2013-12-02 18:42:53 UTC (rev 1420) +++ trunk/tutti-ui-swing/src/main/filtered-resources/log4j.properties 2013-12-03 10:14:03 UTC (rev 1421) @@ -27,17 +27,14 @@ # Console output log4j.appender.stdout=org.apache.log4j.ConsoleAppender log4j.appender.stdout.layout=org.apache.log4j.PatternLayout -#log4j.appender.stdout.layout.ConversionPattern=%d{ISO8601} %5p [%20t] (%c:%L) - %m%n log4j.appender.stdout.layout.ConversionPattern=%d{ISO8601} %5p (%c:%L) - %m%n # tutti levels -log4j.logger.fr.ifremer.tutti=INFO -log4j.logger.fr.ifremer.shared=INFO -#log4j.logger.fr.ifremer.tutti.ui.swing.util=WARN -#log4j.logger.fr.ifremer.tutti.ui.swing.util.action=INFO -#log4j.logger.org.hibernate.SQL=DEBUG -#log4j.logger.org.hibernate.hql.ast.QueryTranslatorImpl=DEBUG +log4j.logger.fr.ifremer=INFO +#See https://forum.hibernate.org/viewtopic.php?p=2404391 +log4j.logger.org.hibernate.engine.StatefulPersistenceContext.ProxyWarnLog=ERROR + log4j.appender.file=org.apache.log4j.RollingFileAppender log4j.appender.file.file=${tutti.log.file} log4j.appender.file.MaxFileSize=10MB @@ -46,3 +43,4 @@ log4j.appender.file.layout=org.apache.log4j.PatternLayout #log4j.appender.file.layout.ConversionPattern=%d{ISO8601} %5p [%20t] %c - %m%n log4j.appender.file.layout.ConversionPattern=%d{ISO8601} %5p %c - %m%n +

1 0

r1420 - in trunk: tutti-persistence/src/main/java/fr/ifremer/tutti/persistence/service tutti-persistence/src/main/resources tutti-service/src/main/java/fr/ifremer/tutti/service/pupitri tutti-service/src/test/java/fr/ifremer/tutti/service/pupitri
by tchemit＠users.forge.codelutin.com 02 Dec '13

02 Dec '13

Author: tchemit Date: 2013-12-02 19:42:53 +0100 (Mon, 02 Dec 2013) New Revision: 1420 Url: http://forge.codelutin.com/projects/tutti/repository/revisions/1420 Log: fixes #3898: [CAPTURE] probl?\195?\168me ?\195?\160 l'import pupitri sur lot cat?\195?\169goris?\195?\169 ?\195?\160 p ou g Modified: trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/persistence/service/TuttiEnumerationFile.java trunk/tutti-persistence/src/main/resources/tutti-db-enumerations.properties trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/pupitri/PupitriCatch.java trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/pupitri/PupitriImportExportService.java trunk/tutti-service/src/test/java/fr/ifremer/tutti/service/pupitri/PupitriImportExportServiceTest.java Modified: trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/persistence/service/TuttiEnumerationFile.java =================================================================== --- trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/persistence/service/TuttiEnumerationFile.java 2013-12-02 09:36:16 UTC (rev 1419) +++ trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/persistence/service/TuttiEnumerationFile.java 2013-12-02 18:42:53 UTC (rev 1420) @@ -146,6 +146,9 @@ @Value("${QualitativeValueId.SEX_FEMALE}") public final Integer QUALITATIVE_SEX_FEMALE_ID = null; + @Value("${QualitativeValueId.SEX_UNDEFINED}") + public final Integer QUALITATIVE_SEX_UNDEFINED_ID = null; + @Value("${QualitativeValueId.SIZE_SMALL}") public final Integer QUALITATIVE_SIZE_SMALL_ID = null; Modified: trunk/tutti-persistence/src/main/resources/tutti-db-enumerations.properties =================================================================== --- trunk/tutti-persistence/src/main/resources/tutti-db-enumerations.properties 2013-12-02 09:36:16 UTC (rev 1419) +++ trunk/tutti-persistence/src/main/resources/tutti-db-enumerations.properties 2013-12-02 18:42:53 UTC (rev 1420) @@ -317,6 +317,7 @@ # Catégorie Sex PmfmId.SEX=196 +QualitativeValueId.SEX_UNDEFINED=299 QualitativeValueId.SEX_MALE=300 QualitativeValueId.SEX_FEMALE=301 Modified: trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/pupitri/PupitriCatch.java =================================================================== --- trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/pupitri/PupitriCatch.java 2013-12-02 09:36:16 UTC (rev 1419) +++ trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/pupitri/PupitriCatch.java 2013-12-02 18:42:53 UTC (rev 1420) @@ -25,7 +25,12 @@ */ import com.google.common.collect.Maps; +import fr.ifremer.tutti.persistence.entities.TuttiEntities; +import fr.ifremer.tutti.persistence.entities.referential.Caracteristic; +import fr.ifremer.tutti.persistence.entities.referential.CaracteristicQualitativeValue; import fr.ifremer.tutti.persistence.entities.referential.Species; +import fr.ifremer.tutti.persistence.service.TuttiEnumerationFile; +import org.apache.commons.lang3.mutable.MutableFloat; import java.io.Serializable; import java.util.Map; @@ -39,13 +44,73 @@ private static final long serialVersionUID = 1L; public static enum Signs { - DEFAULT("0"), - MALE("1"), - FEMALE("2"), - SMALL("P"), - MEDIUM("M"), - BIG("G"); + DEFAULT("0") { + @Override + public Integer getCategory(TuttiEnumerationFile enumerationFile) { + return enumerationFile.PMFM_ID_SEX; + } + @Override + public Integer getQualitativeValueId(TuttiEnumerationFile enumerationFile) { + return enumerationFile.QUALITATIVE_SEX_UNDEFINED_ID; + } + }, + MALE("1") { + @Override + public Integer getCategory(TuttiEnumerationFile enumerationFile) { + return enumerationFile.PMFM_ID_SEX; + } + + @Override + public Integer getQualitativeValueId(TuttiEnumerationFile enumerationFile) { + return enumerationFile.QUALITATIVE_SEX_MALE_ID; + } + }, + FEMALE("2") { + @Override + public Integer getCategory(TuttiEnumerationFile enumerationFile) { + return enumerationFile.PMFM_ID_SEX; + } + + @Override + public Integer getQualitativeValueId(TuttiEnumerationFile enumerationFile) { + return enumerationFile.QUALITATIVE_SEX_FEMALE_ID; + } + }, + SMALL("P") { + @Override + public Integer getCategory(TuttiEnumerationFile enumerationFile) { + return enumerationFile.PMFM_ID_SIZE_CATEGORY; + } + + @Override + public Integer getQualitativeValueId(TuttiEnumerationFile enumerationFile) { + return enumerationFile.QUALITATIVE_SIZE_SMALL_ID; + } + }, + MEDIUM("M") { + @Override + public Integer getCategory(TuttiEnumerationFile enumerationFile) { + return enumerationFile.PMFM_ID_SIZE_CATEGORY; + } + + @Override + public Integer getQualitativeValueId(TuttiEnumerationFile enumerationFile) { + return enumerationFile.QUALITATIVE_SIZE_MEDIUM_ID; + } + }, + BIG("G") { + @Override + public Integer getCategory(TuttiEnumerationFile enumerationFile) { + return enumerationFile.PMFM_ID_SIZE_CATEGORY; + } + + @Override + public Integer getQualitativeValueId(TuttiEnumerationFile enumerationFile) { + return enumerationFile.QUALITATIVE_SIZE_MEDIUM_ID; + } + }; + private String sign; Signs(String sign) { @@ -62,13 +127,25 @@ } return result; } + + public abstract Integer getCategory(TuttiEnumerationFile enumerationFile); + + public abstract Integer getQualitativeValueId(TuttiEnumerationFile enumerationFile); + + public void registerSign(TuttiEnumerationFile enumerationFile, + Caracteristic caracteristic, + Map<Signs, CaracteristicQualitativeValue> map) { + Integer valueId = getQualitativeValueId(enumerationFile); + CaracteristicQualitativeValue result = TuttiEntities.getQualitativeValue(caracteristic, valueId); + map.put(this, result); + } } protected Species species; protected boolean sorted; - protected Map<Signs, Float> weightBySign = Maps.newHashMap(); + protected Map<Signs, MutableFloat> weightBySign = Maps.newHashMap(); public PupitriCatch(Species species, boolean sorted) { this.species = species; @@ -83,18 +160,18 @@ return sorted; } - public Map<Signs, Float> getWeightBySign() { + public Map<Signs, MutableFloat> getWeightBySign() { return weightBySign; } public void addToSign(String sign, Float weight) { Signs s = Signs.getSign(sign); - Float f = weightBySign.get(s); + MutableFloat f = weightBySign.get(s); if (f == null) { - f = 0f; + f = new MutableFloat(); + weightBySign.put(s, f); } - f += weight; - weightBySign.put(s, f); + f.add(weight); } @Override Modified: trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/pupitri/PupitriImportExportService.java =================================================================== --- trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/pupitri/PupitriImportExportService.java 2013-12-02 09:36:16 UTC (rev 1419) +++ trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/pupitri/PupitriImportExportService.java 2013-12-02 18:42:53 UTC (rev 1420) @@ -30,6 +30,7 @@ import com.google.common.collect.ArrayListMultimap; import com.google.common.collect.ListMultimap; import com.google.common.collect.Lists; +import com.google.common.collect.Maps; import com.google.common.collect.Multimaps; import com.google.common.collect.Sets; import com.google.common.io.Files; @@ -56,10 +57,12 @@ import org.apache.commons.io.FileUtils; import org.apache.commons.io.IOUtils; import org.apache.commons.lang3.StringUtils; +import org.apache.commons.lang3.mutable.MutableFloat; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.nuiton.csv.Export; import org.nuiton.csv.Import; +import org.nuiton.decorator.Decorator; import java.io.File; import java.io.IOException; @@ -69,6 +72,7 @@ import java.util.Collections; import java.util.Date; import java.util.List; +import java.util.Map; import java.util.Set; import static org.nuiton.i18n.I18n._; @@ -82,6 +86,8 @@ private static final Log log = LogFactory.getLog(PupitriImportExportService.class); + protected DecoratorService decoratorService; + protected PersistenceService persistenceService; protected TuttiDataContext dataContext; @@ -90,25 +96,31 @@ protected CaracteristicQualitativeValue unsortedCaracteristic; - protected CaracteristicQualitativeValue maleCaracteristic; +// protected CaracteristicQualitativeValue maleCaracteristic; +// +// protected CaracteristicQualitativeValue femaleCaracteristic; +// +// protected CaracteristicQualitativeValue smallCaracteristic; +// +// protected CaracteristicQualitativeValue mediumCaracteristic; +// +// protected CaracteristicQualitativeValue bigCaracteristic; - protected CaracteristicQualitativeValue femaleCaracteristic; + protected Map<PupitriCatch.Signs, CaracteristicQualitativeValue> signsToCaracteristicValue; - protected CaracteristicQualitativeValue smallCaracteristic; - - protected CaracteristicQualitativeValue mediumCaracteristic; - - protected CaracteristicQualitativeValue bigCaracteristic; - @Override public void setServiceContext(TuttiServiceContext context) { super.setServiceContext(context); persistenceService = getService(PersistenceService.class); + decoratorService = getService(DecoratorService.class); dataContext = context.getDataContext(); TuttiEnumerationFile enumerationFile = persistenceService.getEnumerationFile(); + + signsToCaracteristicValue = Maps.newEnumMap(PupitriCatch.Signs.class); + { // sorted/unsorted caracteristic Caracteristic caracteristic = persistenceService.getSortedUnsortedCaracteristic(); @@ -120,17 +132,20 @@ { // sex category Caracteristic caracteristic = persistenceService.getSexCaracteristic(); - maleCaracteristic = TuttiEntities.getQualitativeValue(caracteristic, enumerationFile.QUALITATIVE_SEX_MALE_ID); - femaleCaracteristic = TuttiEntities.getQualitativeValue(caracteristic, enumerationFile.QUALITATIVE_SEX_FEMALE_ID); + PupitriCatch.Signs.DEFAULT.registerSign(enumerationFile, caracteristic, signsToCaracteristicValue); + PupitriCatch.Signs.FEMALE.registerSign(enumerationFile, caracteristic, signsToCaracteristicValue); + PupitriCatch.Signs.MALE.registerSign(enumerationFile, caracteristic, signsToCaracteristicValue); } { // size category Caracteristic caracteristic = persistenceService.getSizeCategoryCaracteristic(); - smallCaracteristic = TuttiEntities.getQualitativeValue(caracteristic, enumerationFile.QUALITATIVE_SIZE_SMALL_ID); - mediumCaracteristic = TuttiEntities.getQualitativeValue(caracteristic, enumerationFile.QUALITATIVE_SIZE_MEDIUM_ID); - bigCaracteristic = TuttiEntities.getQualitativeValue(caracteristic, enumerationFile.QUALITATIVE_SIZE_BIG_ID); + + PupitriCatch.Signs.SMALL.registerSign(enumerationFile, caracteristic, signsToCaracteristicValue); + PupitriCatch.Signs.MEDIUM.registerSign(enumerationFile, caracteristic, signsToCaracteristicValue); + PupitriCatch.Signs.BIG.registerSign(enumerationFile, caracteristic, signsToCaracteristicValue); } } + /** * @param trunkFile incoming trunk file * @param carrouselFile incoming carroussle file @@ -257,6 +272,7 @@ try { + //FIXME tchemit-2013-12-02 Explain what does it do ? // get the map of species by survey code ListMultimap<String, Species> speciesBySurveyCode = ArrayListMultimap.create(); @@ -348,66 +364,127 @@ } TuttiEnumerationFile enumerationFile = persistenceService.getEnumerationFile(); - + Decorator<Species> speciesDecorator = decoratorService.getDecoratorByType(Species.class); for (PupitriCatch pupitriCatch : catches) { - Float catchWeight = pupitriCatch.getWeightBySign().get(PupitriCatch.Signs.DEFAULT); + Species species = pupitriCatch.getSpecies(); CaracteristicQualitativeValue cqv = pupitriCatch.isSorted() ? sortedCaracteristic : unsortedCaracteristic; - Species species = pupitriCatch.getSpecies(); + boolean splitSpecies = isSplitSpecies(pupitriCatch); - SpeciesBatch batch = createSpeciesBatch(operation, - species, - catchWeight, - enumerationFile.PMFM_ID_SORTED_UNSORTED, - cqv); + Map<PupitriCatch.Signs, MutableFloat> weightBySign = + pupitriCatch.getWeightBySign(); - batch = persistenceService.createSpeciesBatch(batch, null); + if (splitSpecies) { - // if the batch is splitted - if (catchWeight == null) { - Integer categoryId = null; - for (PupitriCatch.Signs s : pupitriCatch.getWeightBySign().keySet()) { - CaracteristicQualitativeValue splitCqv = null; - switch (s) { - case MALE: - categoryId = enumerationFile.PMFM_ID_SEX; - splitCqv = maleCaracteristic; - break; + // create a top batch + a son for each sign + if (log.isInfoEnabled()) { + log.info("Create a categorized batches for species " + speciesDecorator.toString(species)); + } + SpeciesBatch batch = createSpeciesBatch(operation, + species, + null, + enumerationFile.PMFM_ID_SORTED_UNSORTED, + cqv); - case FEMALE: - categoryId = enumerationFile.PMFM_ID_SEX; - splitCqv = femaleCaracteristic; - break; + batch = persistenceService.createSpeciesBatch(batch, null); - case SMALL: - categoryId = enumerationFile.PMFM_ID_SIZE_CATEGORY; - splitCqv = smallCaracteristic; - break; + String parentBatchId = batch.getId(); - case MEDIUM: - categoryId = enumerationFile.PMFM_ID_SIZE_CATEGORY; - splitCqv = mediumCaracteristic; - break; + for (Map.Entry<PupitriCatch.Signs, MutableFloat> entry : weightBySign.entrySet()) { - case BIG: - categoryId = enumerationFile.PMFM_ID_SIZE_CATEGORY; - splitCqv = bigCaracteristic; - break; - } + PupitriCatch.Signs signs = entry.getKey(); + float catchWeight = entry.getValue().floatValue(); + Integer categoryId = signs.getCategory(enumerationFile); + CaracteristicQualitativeValue splitCqv = signsToCaracteristicValue.get(signs); SpeciesBatch childBatch = createSpeciesBatch( operation, species, - pupitriCatch.getWeightBySign().get(s), + catchWeight, categoryId, splitCqv ); - persistenceService.createSpeciesBatch(childBatch, batch.getId()); + + persistenceService.createSpeciesBatch(childBatch, parentBatchId); } + + } else { + + // create a unique batch with sum all weights as simple weight + if (log.isInfoEnabled()) { + log.info("Create a unique batch for species " + speciesDecorator.toString(species)); + } + float totalWeight = 0f; + for (MutableFloat weight : weightBySign.values()) { + totalWeight += weight.floatValue(); + } + + SpeciesBatch batch = createSpeciesBatch(operation, + species, + totalWeight, + enumerationFile.PMFM_ID_SORTED_UNSORTED, + cqv); + + persistenceService.createSpeciesBatch(batch, null); } + +// MutableFloat catchWeight = weightBySign.get(PupitriCatch.Signs.DEFAULT); +// +// +// SpeciesBatch batch = createSpeciesBatch(operation, +// species, +// catchWeight.floatValue(), +// enumerationFile.PMFM_ID_SORTED_UNSORTED, +// cqv); +// +// batch = persistenceService.createSpeciesBatch(batch, null); +// +// // if the batch is splitted +// if (catchWeight == null) { +// Integer categoryId = null; +// for (PupitriCatch.Signs s : weightBySign.keySet()) { +// CaracteristicQualitativeValue splitCqv = null; +// switch (s) { +// case MALE: +// categoryId = enumerationFile.PMFM_ID_SEX; +// splitCqv = maleCaracteristic; +// break; +// +// case FEMALE: +// categoryId = enumerationFile.PMFM_ID_SEX; +// splitCqv = femaleCaracteristic; +// break; +// +// case SMALL: +// categoryId = enumerationFile.PMFM_ID_SIZE_CATEGORY; +// splitCqv = smallCaracteristic; +// break; +// +// case MEDIUM: +// categoryId = enumerationFile.PMFM_ID_SIZE_CATEGORY; +// splitCqv = mediumCaracteristic; +// break; +// +// case BIG: +// categoryId = enumerationFile.PMFM_ID_SIZE_CATEGORY; +// splitCqv = bigCaracteristic; +// break; +// } +// +// SpeciesBatch childBatch = createSpeciesBatch( +// operation, +// species, +// weightBySign.get(s).floatValue(), +// categoryId, +// splitCqv +// ); +// +// persistenceService.createSpeciesBatch(childBatch, batch.getId()); +// } +// } } carrouselImporter.close(); @@ -426,7 +503,73 @@ return result; } + public static final Set<PupitriCatch.Signs> DEFAULT_SIGNS = Sets.newHashSet( + PupitriCatch.Signs.DEFAULT + ); + public static final Set<PupitriCatch.Signs> SEX_SIGNS = Sets.newHashSet( + PupitriCatch.Signs.DEFAULT, + PupitriCatch.Signs.MALE, + PupitriCatch.Signs.FEMALE + ); + + public static final Set<PupitriCatch.Signs> SIZE_SIGNS = Sets.newHashSet( + PupitriCatch.Signs.SMALL, + PupitriCatch.Signs.MEDIUM, + PupitriCatch.Signs.BIG + ); + + /** + * Check that the given species catch can be split. + * For the moment accept for a same catch : + * <ul> + * <li>Signs.DEFAULT, Signs.MALE, Signs.FEMALE</li> + * <li>Signs.SMALL, Signs.MEDIUM, Signs.BIG</li> + * </ul> + * See http://forge.codelutin.com/issues/3898 + * + * @param speciesCatch catch to cehck + * @return {@code true} if species catch is safe, {@code false} otherwise. + * @since 3.0-rc-2 + */ + private boolean isSplitSpecies(PupitriCatch speciesCatch) { + boolean result; + Map<PupitriCatch.Signs, MutableFloat> weightBySign = + speciesCatch.getWeightBySign(); + Set<PupitriCatch.Signs> signs = + Sets.newHashSet(weightBySign.keySet()); + + if (DEFAULT_SIGNS.equals(signs)) { + + // only a default sign, no split + result = false; + } else { + + // remove all sex signs + boolean contains = signs.removeAll(SEX_SIGNS); + if (contains) { + + // check there only sex signs + result = signs.isEmpty(); + } else { + + // remove all size signs + contains = signs.removeAll(SIZE_SIGNS); + if (contains) { + + // check there only size signs + result = signs.isEmpty(); + } else { + + // in all other cases, not a safe + result = false; + } + } + } + return result; + } + + public void exportSpecies(List<Species> species, File target) { SpeciesRowModel speciesCsvModel = new SpeciesRowModel(';'); Modified: trunk/tutti-service/src/test/java/fr/ifremer/tutti/service/pupitri/PupitriImportExportServiceTest.java =================================================================== --- trunk/tutti-service/src/test/java/fr/ifremer/tutti/service/pupitri/PupitriImportExportServiceTest.java 2013-12-02 09:36:16 UTC (rev 1419) +++ trunk/tutti-service/src/test/java/fr/ifremer/tutti/service/pupitri/PupitriImportExportServiceTest.java 2013-12-02 18:42:53 UTC (rev 1420) @@ -28,6 +28,8 @@ import fr.ifremer.tutti.persistence.entities.data.CatchBatch; import fr.ifremer.tutti.persistence.entities.data.FishingOperation; import fr.ifremer.tutti.persistence.entities.data.SpeciesBatch; +import fr.ifremer.tutti.persistence.entities.referential.Species; +import fr.ifremer.tutti.service.DecoratorService; import fr.ifremer.tutti.service.PersistenceService; import fr.ifremer.tutti.service.ServiceDbResource; import fr.ifremer.tutti.service.TuttiServiceContext; @@ -35,6 +37,7 @@ import org.junit.Before; import org.junit.ClassRule; import org.junit.Test; +import org.nuiton.decorator.Decorator; import java.io.File; import java.io.IOException; @@ -63,6 +66,8 @@ protected PersistenceService persistenceService; + protected DecoratorService decoratorService; + protected ServiceDbResource.DataContext dataContext; @Before @@ -72,6 +77,8 @@ persistenceService = serviceContext.getService(PersistenceService.class); + decoratorService = serviceContext.getService(DecoratorService.class); + dbResource.openDataContext(); service = serviceContext.getService(PupitriImportExportService.class); @@ -99,14 +106,23 @@ BatchContainer<SpeciesBatch> rootSpeciesBatchAfter = persistenceService.getRootSpeciesBatch(operation.getId(), null); Assert.assertEquals(18, rootSpeciesBatchAfter.sizeChildren()); - int index = 0; + Decorator<Species> speciesDecorator = decoratorService.getDecoratorByType(Species.class); + +// int index = 0; for (SpeciesBatch speciesBatch : rootSpeciesBatchAfter.getChildren()) { - int nbChildren = 0; - if (index == 11) { + int nbChildren; + // species 17392 has two childs (MALE - FEMALE) + if (17392 == speciesBatch.getSpecies().getIdAsInt()) { nbChildren = 2; + } else { + nbChildren = 0; + } - Assert.assertEquals("SpeciesBatch at index " + (index++) + " should have " + nbChildren + " but had " + speciesBatch.sizeChildBatchs(), nbChildren, speciesBatch.sizeChildBatchs()); +// if (index == 11) { +// nbChildren = 2; +// } + Assert.assertEquals("SpeciesBatch with species " + speciesDecorator.toString(speciesBatch.getSpecies()) + " should have with " + nbChildren + " but had " + speciesBatch.sizeChildBatchs(), nbChildren, speciesBatch.sizeChildBatchs()); } } }

1 0

r1419 - trunk/src/conception/specifications
by lkaufmann＠users.forge.codelutin.com 02 Dec '13

02 Dec '13

Author: lkaufmann Date: 2013-12-02 10:36:16 +0100 (Mon, 02 Dec 2013) New Revision: 1419 Url: http://forge.codelutin.com/projects/tutti/repository/revisions/1419 Log: Specification v0.3 update : add document version notice Modified: trunk/src/conception/specifications/AllegroCampagne-Specifications.odt trunk/src/conception/specifications/AllegroCampagne-Specifications.pdf Modified: trunk/src/conception/specifications/AllegroCampagne-Specifications.odt =================================================================== (Binary files differ) Modified: trunk/src/conception/specifications/AllegroCampagne-Specifications.pdf =================================================================== (Binary files differ)

1 0

r1418 - in trunk: src/conception/specifications/Fonctionnalites-transversales tutti-persistence/src/main/java/fr/ifremer/tutti
by tchemit＠users.forge.codelutin.com 01 Dec '13

01 Dec '13

Author: tchemit Date: 2013-12-01 22:18:22 +0100 (Sun, 01 Dec 2013) New Revision: 1418 Url: http://forge.codelutin.com/projects/tutti/repository/revisions/1418 Log: add headers + svn properties Modified: trunk/src/conception/specifications/Fonctionnalites-transversales/Taxonomie.txt trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfiguration.java trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfigurationOption.java trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfigurationProvider.java Property changes on: trunk/src/conception/specifications/Fonctionnalites-transversales/Taxonomie.txt ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native Modified: trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfiguration.java =================================================================== --- trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfiguration.java 2013-12-01 21:17:47 UTC (rev 1417) +++ trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfiguration.java 2013-12-01 21:18:22 UTC (rev 1418) @@ -1,5 +1,29 @@ package fr.ifremer.tutti; +/* + * #%L + * Tutti :: Persistence + * $Id$ + * $HeadURL$ + * %% + * Copyright (C) 2012 - 2013 Ifremer + * %% + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public + * License along with this program. If not, see + * <http://www.gnu.org/licenses/gpl-3.0.html>. + * #L% + */ + import com.google.common.base.Charsets; import com.google.common.base.Predicate; import com.google.common.collect.Sets; Property changes on: trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfiguration.java ___________________________________________________________________ Modified: svn:keywords - Author Date Id Revision + Author Date Id Revision HeadURL Modified: trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfigurationOption.java =================================================================== --- trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfigurationOption.java 2013-12-01 21:17:47 UTC (rev 1417) +++ trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfigurationOption.java 2013-12-01 21:18:22 UTC (rev 1418) @@ -1,5 +1,29 @@ package fr.ifremer.tutti; +/* + * #%L + * Tutti :: Persistence + * $Id$ + * $HeadURL$ + * %% + * Copyright (C) 2012 - 2013 Ifremer + * %% + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public + * License along with this program. If not, see + * <http://www.gnu.org/licenses/gpl-3.0.html>. + * #L% + */ + import fr.ifremer.shared.application.type.CoordinateEditorType; import fr.ifremer.shared.application.type.WeightUnit; import fr.ifremer.tutti.persistence.entities.data.SampleCategoryModel; Property changes on: trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfigurationOption.java ___________________________________________________________________ Modified: svn:keywords - Author Date Id Revision + Author Date Id Revision HeadURL Modified: trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfigurationProvider.java =================================================================== --- trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfigurationProvider.java 2013-12-01 21:17:47 UTC (rev 1417) +++ trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfigurationProvider.java 2013-12-01 21:18:22 UTC (rev 1418) @@ -1,5 +1,29 @@ package fr.ifremer.tutti; +/* + * #%L + * Tutti :: Persistence + * $Id$ + * $HeadURL$ + * %% + * Copyright (C) 2012 - 2013 Ifremer + * %% + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public + * License along with this program. If not, see + * <http://www.gnu.org/licenses/gpl-3.0.html>. + * #L% + */ + import org.nuiton.config.ApplicationConfigProvider; import org.nuiton.config.ConfigActionDef; import org.nuiton.config.ConfigOptionDef; Property changes on: trunk/tutti-persistence/src/main/java/fr/ifremer/tutti/TuttiConfigurationProvider.java ___________________________________________________________________ Modified: svn:keywords - Author Date Id Revision + Author Date Id Revision HeadURL

1 0

r1417 - trunk/tutti-persistence/src/main/resources
by tchemit＠users.forge.codelutin.com 01 Dec '13

01 Dec '13

Author: tchemit Date: 2013-12-01 22:17:47 +0100 (Sun, 01 Dec 2013) New Revision: 1417 Url: http://forge.codelutin.com/projects/tutti/repository/revisions/1417 Log: do not use hibernate second level cache Added: trunk/tutti-persistence/src/main/resources/ehcache.xml Modified: trunk/tutti-persistence/src/main/resources/applicationContext-service-tutti.xml trunk/tutti-persistence/src/main/resources/tutti-db-conf.properties Modified: trunk/tutti-persistence/src/main/resources/applicationContext-service-tutti.xml =================================================================== --- trunk/tutti-persistence/src/main/resources/applicationContext-service-tutti.xml 2013-12-01 21:15:43 UTC (rev 1416) +++ trunk/tutti-persistence/src/main/resources/applicationContext-service-tutti.xml 2013-12-01 21:17:47 UTC (rev 1417) @@ -40,127 +40,106 @@ class="fr.ifremer.tutti.persistence.service.TuttiEnumerationFile"/>  -  +    - <bean id="tuttiAbstractCache" abstract="true" class="org.springframework.cache.ehcache.EhCacheFactoryBean"> - <property name="cacheManager" ref="ehcache"/> - <property name="maxElementsInMemory" value="5000"/> - <property name="eternal" value="false"/> - <property name="timeToLive" value="300"/> - <property name="timeToIdle" value="300"/> - <property name="overflowToDisk" value="false"/> - <property name="diskPersistent" value="false"/> - <property name="diskExpiryThreadIntervalSeconds" value="300"/> - </bean> - -  - <bean id="tuttiAbstractEternalCache" abstract="true" parent="tuttiAbstractCache"> - <property name="eternal" value="true"/> - <property name="overflowToDisk" value="true"/> - <property name="maxElementsOnDisk" value="5000"/> - <property name="diskPersistent" value="true"/> -  - <property name="timeToLive" value="0"/> - <property name="timeToIdle" value="0"/> - <property name="diskExpiryThreadIntervalSeconds" value="0"/> - </bean> - - <bean id="tuttiProgramsCache" parent="tuttiAbstractEternalCache"> + + <bean id="tuttiProgramsCache" parent="abstractEternalCache"> <property name="cacheName" value="programs" /> <property name="maxElementsInMemory" value="1"/> <property name="maxElementsOnDisk" value="1"/> </bean> - <bean id="tuttiProgramZonesCache" parent="tuttiAbstractEternalCache"> + <bean id="tuttiProgramZonesCache" parent="abstractEternalCache"> <property name="cacheName" value="programZones" /> <property name="maxElementsInMemory" value="1"/> <property name="maxElementsOnDisk" value="1"/> </bean> - <bean id="countriesCache" parent="tuttiAbstractEternalCache"> + <bean id="countriesCache" parent="abstractEternalCache"> <property name="cacheName" value="countries" /> <property name="maxElementsInMemory" value="1"/> <property name="maxElementsOnDisk" value="1"/> </bean> - <bean id="harboursCache" parent="tuttiAbstractEternalCache"> + <bean id="harboursCache" parent="abstractEternalCache"> <property name="cacheName" value="harbours" /> <property name="maxElementsInMemory" value="1"/> <property name="maxElementsOnDisk" value="1"/> </bean> - <bean id="tuttiPmfmsCache" parent="tuttiAbstractEternalCache"> + <bean id="tuttiPmfmsCache" parent="abstractEternalCache"> <property name="cacheName" value="pmfms" /> </bean> - <bean id="tuttiPmfmsWithProtectedCache" parent="tuttiAbstractEternalCache"> + + <bean id="tuttiPmfmsWithProtectedCache" parent="abstractEternalCache"> <property name="cacheName" value="pmfmsWithProtected" /> </bean> - <bean id="tuttiPmfmByIdCache" parent="tuttiAbstractEternalCache"> + + <bean id="tuttiPmfmByIdCache" parent="abstractEternalCache"> <property name="cacheName" value="pmfmById" /> </bean> - <bean id="tuttiObjectTypeCache" parent="tuttiAbstractEternalCache"> + <bean id="tuttiObjectTypeCache" parent="abstractEternalCache"> <property name="cacheName" value="objectTypes" /> </bean> - <bean id="tuttiObjectTypeByCodeCache" parent="tuttiAbstractEternalCache"> + <bean id="tuttiObjectTypeByCodeCache" parent="abstractEternalCache"> <property name="cacheName" value="objectTypeByCode" /> </bean> - <bean id="tuttiFishingVesselsCache" parent="tuttiAbstractEternalCache"> + <bean id="tuttiFishingVesselsCache" parent="abstractEternalCache"> <property name="cacheName" value="fishingVessels" /> <property name="maxElementsInMemory" value="1"/> <property name="maxElementsOnDisk" value="1"/> </bean> - <bean id="tuttiFishingVesselByCodeCache" parent="tuttiAbstractEternalCache"> + + <bean id="tuttiFishingVesselByCodeCache" parent="abstractEternalCache"> <property name="cacheName" value="vesselByCode" /> <property name="maxElementsInMemory" value="40000"/> <property name="maxElementsOnDisk" value="40000"/> </bean> - <bean id="tuttiSpeciesCache" parent="tuttiAbstractEternalCache"> + <bean id="tuttiSpeciesCache" parent="abstractEternalCache"> <property name="cacheName" value="species" /> <property name="maxElementsInMemory" value="1"/> <property name="maxElementsOnDisk" value="1"/> </bean> - <bean id="tuttiReferentSpeciesCache" parent="tuttiAbstractEternalCache"> + <bean id="tuttiReferentSpeciesCache" parent="abstractEternalCache"> <property name="cacheName" value="referentSpecies" /> <property name="maxElementsInMemory" value="1"/> <property name="maxElementsOnDisk" value="1"/> </bean> - <bean id="tuttiReferentSpeciesByIdCache" parent="tuttiAbstractEternalCache"> + + <bean id="tuttiReferentSpeciesByIdCache" parent="abstractEternalCache"> <property name="cacheName" value="referentSpeciesById" /> <property name="maxElementsInMemory" value="20000"/> <property name="maxElementsOnDisk" value="20000"/> </bean> - <bean id="tuttiReferentSpeciesByIdVenacularCache" parent="tuttiAbstractEternalCache"> + + <bean id="tuttiReferentSpeciesByIdVenacularCache" parent="abstractEternalCache"> <property name="cacheName" value="referentSpeciesByIdVernacular" /> <property name="maxElementsInMemory" value="20000"/> <property name="maxElementsOnDisk" value="20000"/> </bean> - <bean id="tuttiGearsCache" parent="tuttiAbstractEternalCache"> + <bean id="tuttiGearsCache" parent="abstractEternalCache"> <property name="cacheName" value="gears" /> <property name="maxElementsInMemory" value="1"/> <property name="maxElementsOnDisk" value="1"/> </bean> - <bean id="tuttiPersonsCache" parent="tuttiAbstractEternalCache"> + <bean id="tuttiPersonsCache" parent="abstractEternalCache"> <property name="cacheName" value="persons" /> <property name="maxElementsInMemory" value="1"/> <property name="maxElementsOnDisk" value="1"/> </bean> - <bean id="tuttiPersonByIdCache" parent="tuttiAbstractEternalCache"> + <bean id="tuttiPersonByIdCache" parent="abstractEternalCache"> <property name="cacheName" value="personById" /> <property name="maxElementsInMemory" value="1000"/> <property name="maxElementsOnDisk" value="1000"/> </bean> - -  - + </beans> \ No newline at end of file Added: trunk/tutti-persistence/src/main/resources/ehcache.xml =================================================================== --- trunk/tutti-persistence/src/main/resources/ehcache.xml (rev 0) +++ trunk/tutti-persistence/src/main/resources/ehcache.xml 2013-12-01 21:17:47 UTC (rev 1417) @@ -0,0 +1,80 @@ +<?xml version="1.0" encoding="UTF-8"?> + + +<ehcache xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" + xsi:noNamespaceSchemaLocation="http://ehcache.org/ehcache.xsd" + updateCheck="false"> + +  +  + +  + <defaultCache + maxElementsInMemory="10000" + eternal="false" + timeToIdleSeconds="120" + timeToLiveSeconds="120" + overflowToDisk="true"/> + + <cache name="org.hibernate.cache.UpdateTimestampsCache" + eternal="false" + maxElementsInMemory="10000" + timeToIdleSeconds="300" + timeToLiveSeconds="300" + overflowToDisk="false"/> + <cache name="org.hibernate.cache.StandardQueryCache" + eternal="false" + maxElementsInMemory="10000" + timeToIdleSeconds="300" + timeToLiveSeconds="300" + overflowToDisk="false"/> + +</ehcache> Property changes on: trunk/tutti-persistence/src/main/resources/ehcache.xml ___________________________________________________________________ Added: svn:mime-type + text/xml Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native Modified: trunk/tutti-persistence/src/main/resources/tutti-db-conf.properties =================================================================== --- trunk/tutti-persistence/src/main/resources/tutti-db-conf.properties 2013-12-01 21:15:43 UTC (rev 1416) +++ trunk/tutti-persistence/src/main/resources/tutti-db-conf.properties 2013-12-01 21:17:47 UTC (rev 1417) @@ -43,6 +43,7 @@ hibernate.show_sql=${tutti.persistence.hibernate.showSql} hibernate.format_sql=${tutti.persistence.hibernate.formatSql} hibernate.use_sql_comments=${tutti.persistence.hibernate.useSqlComment} +hibernate.use_second_level_cache=false hibernate.generate_statistics=false hibernate.query.substitutions=true 1, false 0 hibernate.hbm2ddl.auto=none

1 0

r1416 - trunk/tutti-service/src/main/java/fr/ifremer/tutti/service
by tchemit＠users.forge.codelutin.com 01 Dec '13

01 Dec '13

Author: tchemit Date: 2013-12-01 22:15:43 +0100 (Sun, 01 Dec 2013) New Revision: 1416 Url: http://forge.codelutin.com/projects/tutti/repository/revisions/1416 Log: do not call driver after it was closed Modified: trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/PersistenceService.java Modified: trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/PersistenceService.java =================================================================== --- trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/PersistenceService.java 2013-11-29 16:47:13 UTC (rev 1415) +++ trunk/tutti-service/src/main/java/fr/ifremer/tutti/service/PersistenceService.java 2013-12-01 21:15:43 UTC (rev 1416) @@ -638,14 +638,16 @@ long t0 = TimeLog.getTime(); + String driverName = driver.getImplementationName(); + if (log.isDebugEnabled()) { - log.debug("Closing persistence Service..."); + log.debug("Closing persistence Service *" + driverName + "*..."); } IOUtils.closeQuietly(driver); TIME_LOG.log(t0, "Persistence driver *" + - driver.getImplementationName() + "* closed."); + driverName + "* closed."); } //------------------------------------------------------------------------//

1 0