- Isis-fish-commits - mailman.cloud.codelutin.com

r4059 - trunk
by echatellier＠users.forge.codelutin.com 23 Jul '14

23 Jul '14

Author: echatellier Date: 2014-07-23 09:52:41 +0200 (Wed, 23 Jul 2014) New Revision: 4059 Url: http://forge.codelutin.com/projects/isis-fish/repository/revisions/4059 Log: Update commons-logging (fix ThreadContext introspection) Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2014-07-23 07:51:53 UTC (rev 4058) +++ trunk/pom.xml 2014-07-23 07:52:41 UTC (rev 4059) @@ -215,6 +215,7 @@ <dependency> <groupId>commons-logging</groupId> <artifactId>commons-logging</artifactId> + <version>1.2</version> </dependency> <dependency>

1 0

r4058 - trunk/src/main/resources/i18n
by echatellier＠users.forge.codelutin.com 23 Jul '14

23 Jul '14

Author: echatellier Date: 2014-07-23 09:51:53 +0200 (Wed, 23 Jul 2014) New Revision: 4058 Url: http://forge.codelutin.com/projects/isis-fish/repository/revisions/4058 Log: Fix removal of 'Zones' translation. Modified: trunk/src/main/resources/i18n/isis-fish_en_GB.properties trunk/src/main/resources/i18n/isis-fish_fr_FR.properties Modified: trunk/src/main/resources/i18n/isis-fish_en_GB.properties =================================================================== --- trunk/src/main/resources/i18n/isis-fish_en_GB.properties 2014-07-22 21:00:31 UTC (rev 4057) +++ trunk/src/main/resources/i18n/isis-fish_en_GB.properties 2014-07-23 07:51:53 UTC (rev 4058) @@ -69,7 +69,7 @@ Use\ branches,\ switch\ not\ needed= User\ restart\ simulation\ %s= User\ stop\ simulation\ %s= -Zones= +Zones=Zones an\ alternative\ way\ to\ specify\ the\ scale.= could\ not\ close\ reader\ %1$s=could not close reader %1$s could\ not\ found\ %s= @@ -531,7 +531,7 @@ isisfish.input.tree.strategies=Strategies isisfish.input.tree.triptypes=Trip types isisfish.input.tree.vesseltypes=Vessel types -isisfish.input.tree.zones= +isisfish.input.tree.zones=Zones isisfish.launch.anonymous=read only isisfish.launch.email=email isisfish.launch.firstname=firstname @@ -729,7 +729,7 @@ isisfish.population.recruitment=Recruitment isisfish.population.reproduction=Reproduction isisfish.population.season=Season -isisfish.population.zones= +isisfish.population.zones=Zones isisfish.populationBasics.confirmCreateGroups=Warning, running groups creation wizard will delete all existing groups and objects which depend on it.\nDo you want to re-create groups ? isisfish.populationBasics.geographicID=Geographic identifier isisfish.populationBasics.groupMax=Max group @@ -808,7 +808,7 @@ isisfish.populationZones.selectPopulationAreas=Select the population areas isisfish.populationZones.selectRecruitmentAreas=Select the recruitment areas isisfish.populationZones.selectSpawningAreas=Select the spawning areas -isisfish.populationZones.title= +isisfish.populationZones.title=Zones isisfish.port.cell=Port cell isisfish.port.comments=Comments isisfish.port.name=Name Modified: trunk/src/main/resources/i18n/isis-fish_fr_FR.properties =================================================================== --- trunk/src/main/resources/i18n/isis-fish_fr_FR.properties 2014-07-22 21:00:31 UTC (rev 4057) +++ trunk/src/main/resources/i18n/isis-fish_fr_FR.properties 2014-07-23 07:51:53 UTC (rev 4058) @@ -69,7 +69,7 @@ Use\ branches,\ switch\ not\ needed= User\ restart\ simulation\ %s= User\ stop\ simulation\ %s= -Zones= +Zones=Zones an\ alternative\ way\ to\ specify\ the\ scale.= could\ not\ close\ reader\ %1$s=could not close reader %1$s could\ not\ found\ %s= @@ -531,7 +531,7 @@ isisfish.input.tree.strategies=Stratégies isisfish.input.tree.triptypes=Types de marée isisfish.input.tree.vesseltypes=Types de navire -isisfish.input.tree.zones= +isisfish.input.tree.zones=Zones isisfish.launch.anonymous=lecture seule isisfish.launch.email=courriel isisfish.launch.firstname=nom @@ -724,7 +724,7 @@ isisfish.population.recruitment=Recrutement isisfish.population.reproduction=Reproduction isisfish.population.season=Saison -isisfish.population.zones= +isisfish.population.zones=Zones isisfish.populationBasics.confirmCreateGroups=Attention, la re-création des groupes supprimera les groupes existants et tous les objets qui en dépendent.\nÊtes vous sur de vouloir re-créer les groupes ? isisfish.populationBasics.geographicID=Identifiant géographique isisfish.populationBasics.groupMax=Group max @@ -803,7 +803,7 @@ isisfish.populationZones.selectPopulationAreas=Sélectionnez les zones des populations isisfish.populationZones.selectRecruitmentAreas=Sélectionnez les zones de recrutement isisfish.populationZones.selectSpawningAreas=Sélectionnez les zones de reproduction -isisfish.populationZones.title= +isisfish.populationZones.title=Zones isisfish.port.cell=Maille du port isisfish.port.comments=Commentaires isisfish.port.name=Nom

1 0

r4057 - in trunk/src/main: java/fr/ifremer/isisfish/simulator resources/i18n
by bpoussin＠users.forge.codelutin.com 22 Jul '14

22 Jul '14

Author: bpoussin Date: 2014-07-22 23:00:31 +0200 (Tue, 22 Jul 2014) New Revision: 4057 Url: http://forge.codelutin.com/projects/isis-fish/repository/revisions/4057 Log: catch Exception to prevent NPE if optimisation class doesn't exist localy Modified: trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterImpl.java trunk/src/main/resources/i18n/isis-fish_en_GB.properties trunk/src/main/resources/i18n/isis-fish_fr_FR.properties Modified: trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterImpl.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterImpl.java 2014-07-22 20:30:17 UTC (rev 4056) +++ trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterImpl.java 2014-07-22 21:00:31 UTC (rev 4057) @@ -903,7 +903,7 @@ // 0 = only single optimization StorageHelper.populateStorageParams(0, getRegion().getStorage(), optimization, propertiesParameters, "optimization"); - } catch (IsisFishException eee) { + } catch (Exception eee) { optimization = null; if (log.isWarnEnabled()) { log.warn("Can't find optimization: " + optimization, eee); Modified: trunk/src/main/resources/i18n/isis-fish_en_GB.properties =================================================================== --- trunk/src/main/resources/i18n/isis-fish_en_GB.properties 2014-07-22 20:30:17 UTC (rev 4056) +++ trunk/src/main/resources/i18n/isis-fish_en_GB.properties 2014-07-22 21:00:31 UTC (rev 4057) @@ -531,7 +531,7 @@ isisfish.input.tree.strategies=Strategies isisfish.input.tree.triptypes=Trip types isisfish.input.tree.vesseltypes=Vessel types -isisfish.input.tree.zones=Zones +isisfish.input.tree.zones= isisfish.launch.anonymous=read only isisfish.launch.email=email isisfish.launch.firstname=firstname @@ -729,7 +729,7 @@ isisfish.population.recruitment=Recruitment isisfish.population.reproduction=Reproduction isisfish.population.season=Season -isisfish.population.zones=Zones +isisfish.population.zones= isisfish.populationBasics.confirmCreateGroups=Warning, running groups creation wizard will delete all existing groups and objects which depend on it.\nDo you want to re-create groups ? isisfish.populationBasics.geographicID=Geographic identifier isisfish.populationBasics.groupMax=Max group @@ -808,7 +808,7 @@ isisfish.populationZones.selectPopulationAreas=Select the population areas isisfish.populationZones.selectRecruitmentAreas=Select the recruitment areas isisfish.populationZones.selectSpawningAreas=Select the spawning areas -isisfish.populationZones.title=Zones +isisfish.populationZones.title= isisfish.port.cell=Port cell isisfish.port.comments=Comments isisfish.port.name=Name Modified: trunk/src/main/resources/i18n/isis-fish_fr_FR.properties =================================================================== --- trunk/src/main/resources/i18n/isis-fish_fr_FR.properties 2014-07-22 20:30:17 UTC (rev 4056) +++ trunk/src/main/resources/i18n/isis-fish_fr_FR.properties 2014-07-22 21:00:31 UTC (rev 4057) @@ -531,7 +531,7 @@ isisfish.input.tree.strategies=Stratégies isisfish.input.tree.triptypes=Types de marée isisfish.input.tree.vesseltypes=Types de navire -isisfish.input.tree.zones=Zones +isisfish.input.tree.zones= isisfish.launch.anonymous=lecture seule isisfish.launch.email=courriel isisfish.launch.firstname=nom @@ -724,7 +724,7 @@ isisfish.population.recruitment=Recrutement isisfish.population.reproduction=Reproduction isisfish.population.season=Saison -isisfish.population.zones=Zones +isisfish.population.zones= isisfish.populationBasics.confirmCreateGroups=Attention, la re-création des groupes supprimera les groupes existants et tous les objets qui en dépendent.\nÊtes vous sur de vouloir re-créer les groupes ? isisfish.populationBasics.geographicID=Identifiant géographique isisfish.populationBasics.groupMax=Group max @@ -803,7 +803,7 @@ isisfish.populationZones.selectPopulationAreas=Sélectionnez les zones des populations isisfish.populationZones.selectRecruitmentAreas=Sélectionnez les zones de recrutement isisfish.populationZones.selectSpawningAreas=Sélectionnez les zones de reproduction -isisfish.populationZones.title=Zones +isisfish.populationZones.title= isisfish.port.cell=Maille du port isisfish.port.comments=Commentaires isisfish.port.name=Nom

1 0

r4056 - trunk/src/main/java/fr/ifremer/isisfish/simulator
by bpoussin＠users.forge.codelutin.com 22 Jul '14

22 Jul '14

Author: bpoussin Date: 2014-07-22 22:30:17 +0200 (Tue, 22 Jul 2014) New Revision: 4056 Url: http://forge.codelutin.com/projects/isis-fish/repository/revisions/4056 Log: bug fixes in getDiscard now return discard, before return all time null :( (error between getValue and getKey :() Modified: trunk/src/main/java/fr/ifremer/isisfish/simulator/PopulationMonitor.java Modified: trunk/src/main/java/fr/ifremer/isisfish/simulator/PopulationMonitor.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/simulator/PopulationMonitor.java 2014-07-22 20:02:09 UTC (rev 4055) +++ trunk/src/main/java/fr/ifremer/isisfish/simulator/PopulationMonitor.java 2014-07-22 20:30:17 UTC (rev 4056) @@ -333,14 +333,14 @@ MatrixND holdCatch = holdCatchs.get(pop); if (holdCatch == null) { - holdCatch = MatrixFactory.getInstance().create(catchPerStrategyMet); + holdCatch = catchPerStrategyMet.copy(); holdCatchs.put(pop, holdCatch); } else { holdCatch.add(catchPerStrategyMet); } // compute total - for (MatrixIterator i = catchPerStrategyMet.iterator(); i.next();) { + for (MatrixIterator i = catchPerStrategyMet.iteratorNotZero(); i.next();) { this.totalHoldCatch += i.getValue(); } // this.totalHoldCatch += catchPerStrategyMet.sumAll(); @@ -402,7 +402,7 @@ // seul le discard de l'annee en cours est disponible, sinon on doit // aller le chercher dans les resultats Map.Entry<TimeStep, MatrixND> discard = discards.get(pop); - if (discard != null && discard.getValue().equals(step)) { + if (discard != null && discard.getKey().equals(step)) { result = discard.getValue(); } return result;

1 0

r4055 - trunk/src/main/java/fr/ifremer/isisfish/util
by bpoussin＠users.forge.codelutin.com 22 Jul '14

22 Jul '14

Author: bpoussin Date: 2014-07-22 22:02:09 +0200 (Tue, 22 Jul 2014) New Revision: 4055 Url: http://forge.codelutin.com/projects/isis-fish/repository/revisions/4055 Log: change mark size to 2, 1 doesn't work when file is created under windows (when EOF is reached mark fail with 1) Modified: trunk/src/main/java/fr/ifremer/isisfish/util/SimpleParser.java Modified: trunk/src/main/java/fr/ifremer/isisfish/util/SimpleParser.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/util/SimpleParser.java 2014-07-22 19:54:22 UTC (rev 4054) +++ trunk/src/main/java/fr/ifremer/isisfish/util/SimpleParser.java 2014-07-22 20:02:09 UTC (rev 4055) @@ -43,6 +43,9 @@ /** to use log facility, just put in your code: log.info(\"...\"); */ static private Log log = LogFactory.getLog(SimpleParser.class); + // MARK_SIZE = 1 doesn't work when EOF is reached + static final private int MARK_SIZE = 2; + protected boolean intern; protected BufferedReader in; protected StringBuilder sb = new StringBuilder(); @@ -83,7 +86,7 @@ char c = (char)i; if (c == '\r') { // try to eat \n after \r - in.mark(1); + in.mark(MARK_SIZE); c = (char)in.read(); if (c != '\n') { in.reset(); @@ -102,7 +105,7 @@ if (i != -1) { // test if not eof - in.mark(1); + in.mark(MARK_SIZE); i = in.read(); in.reset(); }

1 0

r4054 - in trunk/src/main: java/fr/ifremer/isisfish/datastore java/fr/ifremer/isisfish/simulator java/fr/ifremer/isisfish/ui/result java/fr/ifremer/isisfish/util resources/i18n
by bpoussin＠users.forge.codelutin.com 22 Jul '14

22 Jul '14

Author: bpoussin Date: 2014-07-22 21:54:22 +0200 (Tue, 22 Jul 2014) New Revision: 4054 Url: http://forge.codelutin.com/projects/isis-fish/repository/revisions/4054 Log: rewrite resume to use only String representation (no object instanciation) this prevent error when plan, export, rule or other doesn't exist localy Added: trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterPropertiesHelper.java Modified: trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterImpl.java trunk/src/main/java/fr/ifremer/isisfish/ui/result/MatrixSummaryRenderer.java trunk/src/main/java/fr/ifremer/isisfish/util/StringConverter.java trunk/src/main/java/fr/ifremer/isisfish/util/TopiaEntityConverter.java trunk/src/main/resources/i18n/isis-fish_en_GB.properties trunk/src/main/resources/i18n/isis-fish_fr_FR.properties Modified: trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java 2014-07-22 09:01:49 UTC (rev 4053) +++ trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java 2014-07-22 19:54:22 UTC (rev 4054) @@ -43,6 +43,7 @@ import java.util.Set; import org.apache.commons.io.IOUtils; +import org.apache.commons.lang.StringUtils; import org.apache.commons.lang3.time.DurationFormatUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; @@ -175,14 +176,14 @@ // general information String info = getInformation(); - if (info != null && !"".equals(info)) { - result += "Information:\n" + info + "\n"; + if (StringUtils.isNotBlank(info)) { + result += "\nInformation:\n\t" + info + "\n"; } // Statistic String v = getStatistic(); if (v != null) { - result += "Statistic:\n" + v + "\n"; + result += "\nStatistic:\n" + v + "\n"; } // Optimisation usage Modified: trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterImpl.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterImpl.java 2014-07-22 09:01:49 UTC (rev 4053) +++ trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterImpl.java 2014-07-22 19:54:22 UTC (rev 4054) @@ -75,6 +75,10 @@ import fr.ifremer.isisfish.rule.Rule; import fr.ifremer.isisfish.rule.RuleHelper; import fr.ifremer.isisfish.simulator.sensitivity.SensitivityAnalysis; +import fr.ifremer.isisfish.util.ConverterUtil; +import fr.ifremer.isisfish.util.StringConverter; +import fr.ifremer.isisfish.util.TopiaEntityConverter; +import org.apache.commons.beanutils.ConvertUtilsBean; import static org.nuiton.i18n.I18n.n; @@ -90,7 +94,7 @@ public class SimulationParameterImpl implements SimulationParameter { /** Logger for this class. */ - private static Log log = LogFactory.getLog(SimulationParameterImpl.class); + final private static Log log = LogFactory.getLog(SimulationParameterImpl.class); /** Remember last read properties. */ protected Properties propertiesParameters; @@ -310,10 +314,10 @@ List groups = pop.getPopulationGroup(); List zones = pop.getPopulationZone(); - if (groups.size() == 0) { + if (groups.isEmpty()) { groups.add("No group"); } - if (zones.size() == 0) { + if (zones.isEmpty()) { zones.add("No zone"); } List[] semantics = new List[] { groups, zones }; @@ -373,10 +377,10 @@ PopulationDAO populationDAO = IsisFishDAOHelper .getPopulationDAO(tx); - String[] populationList = propertiesParameters.getProperty( - "populations", "").split(","); + String[] populationList = SimulationParameterPropertiesHelper + .getPopulationNames(propertiesParameters); for (String name : populationList) { - if (name != null && !"".equals(name)) { + if (StringUtils.isNotEmpty(name)) { try { Population population = populationDAO.findByName(name); populations.add(population); @@ -448,7 +452,8 @@ } StrategyDAO strategyDAO = IsisFishDAOHelper.getStrategyDAO(tx); - String[] strategyList = propertiesParameters.getProperty("strategies", "").split(","); + String[] strategyList = SimulationParameterPropertiesHelper + .getStrategieNames(propertiesParameters); for (String name : strategyList) { if (name != null && !"".equals(name)) { try { @@ -617,7 +622,8 @@ } // rules - String[] ruleList = propertiesParameters.getProperty("rules", "").split(","); + String[] ruleList = SimulationParameterPropertiesHelper + .getRuleNames(propertiesParameters); int ruleIndex = 0; for (String name : ruleList) { if (name != null && !"".equals(name)) { @@ -685,7 +691,7 @@ useCache = Boolean.TRUE; } } - return useCache.booleanValue(); + return useCache; } @Override @@ -758,11 +764,11 @@ numberOfSensitivitySimulation = Integer.valueOf(propertiesParameters.getProperty( "numberOfSensitivitySimulation", "-1")); } else { - numberOfSensitivitySimulation = Integer.valueOf(-1); + numberOfSensitivitySimulation = -1; } } - return numberOfSensitivitySimulation.intValue(); + return numberOfSensitivitySimulation; } /* @@ -925,7 +931,7 @@ useOptimization = Boolean.FALSE; } } - return useOptimization.booleanValue(); + return useOptimization; } @Override @@ -976,7 +982,7 @@ if (propertiesParameters != null) { try { // On verifie tout d'abord que l'on ai pas dans une simulation - // si on y es, on utilise le context static non null du thread local + // si on y est, on utilise le context static non null du thread local // Resoud les lazy exceptions des parametres des regles boolean mustClose = false; TopiaContext tx = SimulationContext.get().getDB(); @@ -989,6 +995,7 @@ // rules String[] optimizationExportList = propertiesParameters.getProperty("optimizationexports", "").split(","); + ConvertUtilsBean beanUtils = ConverterUtil.getConverter(tx); int optimizationExportIndex = 0; for (String name : optimizationExportList) { if (name != null && !"".equals(name)) { @@ -997,7 +1004,7 @@ Observation observation = null; String observationId = propertiesParameters.getProperty("optimizationobservation." + optimizationExportIndex); if (StringUtils.isNotBlank(observationId)) { - observation = (Observation)tx.findByTopiaId(observationId); + observation = (Observation)beanUtils.convert(observationId, TopiaEntity.class); } optimizationExportsObservations.put(export, observation); } catch (IsisFishException eee) { @@ -1066,11 +1073,11 @@ if (propertiesParameters != null) { simulationPlanNumber = Integer.valueOf(propertiesParameters.getProperty("simulationPlanNumber", "-1")); } else { - simulationPlanNumber = Integer.valueOf(-1); + simulationPlanNumber = -1; } } - return simulationPlanNumber.intValue(); + return simulationPlanNumber; } /* @@ -1092,7 +1099,7 @@ } } - return sensitivityAnalysisOnlyKeepFirst.booleanValue(); + return sensitivityAnalysisOnlyKeepFirst; } @Override @@ -1111,10 +1118,10 @@ if (propertiesParameters != null) { numberOfYear = Integer.valueOf(propertiesParameters.getProperty("numberOfYear", "1")); } else { - numberOfYear = Integer.valueOf(1); + numberOfYear = 1; } } - return numberOfYear.intValue(); + return numberOfYear; } /* @@ -1140,7 +1147,7 @@ } } - return usePreScript.booleanValue(); + return usePreScript; } /* @@ -1596,7 +1603,7 @@ */ @Override public String toString() { - StringBuffer result = new StringBuffer(); + StringBuilder result = new StringBuilder(); result.append( t("isisfish.params.toString.simulation.done", getIsisFishVersion())).append('\n'); @@ -1738,13 +1745,13 @@ result.append(getPreScript()); result.append('\n'); result.append(t("isisfish.params.toString.simul.logger.level", - simulLogLevel)); + getSimulLogLevel())); result.append('\n'); result.append(t("isisfish.params.toString.script.logger.level", - scriptLogLevel)); + getScriptLogLevel())); result.append('\n'); result.append(t("isisfish.params.toString.lib.logger.level", - libLogLevel)); + getLibLogLevel())); result.append('\n'); return result.toString(); } @@ -1893,6 +1900,7 @@ if (optimizationExportsObservations != null) { String optimizationExportsList = ""; int optimizationExportIndex = 0; + ConvertUtilsBean beanUtils = ConverterUtil.getConverter(null); for (Map.Entry<Export, Observation> exportObservationEntry : optimizationExportsObservations.entrySet()) { Export export = exportObservationEntry.getKey(); optimizationExportsList += ExportStorage.getName(export) + ","; @@ -1900,7 +1908,7 @@ // add in props observation id export index Observation observation = exportObservationEntry.getValue(); if (observation != null) { - result.setProperty("optimizationobservation." + optimizationExportIndex, observation.getTopiaId()); + result.setProperty("optimizationobservation." + optimizationExportIndex, beanUtils.convert(observation)); } optimizationExportIndex++; } Added: trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterPropertiesHelper.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterPropertiesHelper.java (rev 0) +++ trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterPropertiesHelper.java 2014-07-22 19:54:22 UTC (rev 4054) @@ -0,0 +1,347 @@ +package fr.ifremer.isisfish.simulator; + + +import java.util.HashMap; +import java.util.Map; +import java.util.Properties; +import org.apache.commons.lang3.StringUtils; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; +import org.nuiton.util.VersionUtil; + +import static org.nuiton.i18n.I18n.t; + +/** + * Class utilitaire permettant de travailler avec les parametres sous forme de + * properties + * + * @author poussin + * @version $Revision$ + * + * Last update: $Date$ + * by : $Author$ + */ +public class SimulationParameterPropertiesHelper { + + /** to use log facility, just put in your code: log.info(\"...\"); */ + final static private Log log = LogFactory.getLog(SimulationParameterPropertiesHelper.class); + + final public static String ISIS_FISH_VERSION_KEY = "isisFishVersion"; + final public static String SIMULATOR_NAME_KEY = "simulatorName"; + final public static String POPULATIONS_KEY = "populations"; + final public static String RULES_KEY = "rules"; + final public static String PLANS_KEY = "plans"; + final public static String STRATEGIES_KEY = "strategies"; + final public static String DESCRIPTION_KEY = "description"; + final public static String REGION_NAME_KEY = "regionName"; + final public static String USE_SIMULATION_PLAN_KEY = "useSimulationPlan"; + final public static String SIMULATION_PLAN_NUMBER_KEY = "simulationPlanNumber"; + final public static String USE_OPTIMIZATION_KEY = "useOptimization"; + final public static String OPTIMIZATION_KEY = "optimization"; + final public static String OBJECTIVE_KEY = "objective"; + final public static String OPTIMIZATION_EXPORTS_KEY = "optimizationexports"; + final public static String USE_CACHE_KEY = "useCache"; + final public static String NUMBER_OF_YEAR_KEY = "numberOfYear"; + final public static String GENERATED_PRE_SCRIPT_KEY = "generatedPreScript"; + final public static String PRE_SCRIPT_KEY = "preScript"; + final public static String SIMUL_LOG_LEVEL_KEY = "simulLogLevel"; + final public static String SCRIPT_LOG_LEVEL_KEY = "simulLogLevel"; + final public static String LIB_LOG_LEVEL_KEY = "simulLogLevel"; + + public static String getIsisFishVersion(Properties prop) { + String result = prop.getProperty(ISIS_FISH_VERSION_KEY, ""); + return result; + } + + public static String getSimulatorName(Properties prop) { + String result = prop.getProperty(SIMULATOR_NAME_KEY, "DefaultSimulator.java"); + return result; + } + + public static String getDescription(Properties prop) { + String result = prop.getProperty(DESCRIPTION_KEY, ""); + return result; + } + + public static boolean getUseSimulationPlan(Properties prop) { + boolean result = Boolean.valueOf(prop.getProperty(USE_SIMULATION_PLAN_KEY, "false")); + return result; + } + + public static int getSimulationPlanNumber(Properties prop) { + int result = Integer.valueOf(prop.getProperty(SIMULATION_PLAN_NUMBER_KEY, "-1")); + return result; + } + + public static boolean getUseCache(Properties prop) { + boolean result; + if (VersionUtil.smallerThan(getIsisFishVersion(prop), "4.3.0.0")) { + result = Boolean.valueOf(prop.getProperty(USE_OPTIMIZATION_KEY, "true")); + } else { + result = Boolean.valueOf(prop.getProperty(USE_CACHE_KEY, "true")); + } + return result; + } + + public static int getNumberOfYear(Properties prop) { + int result = Integer.valueOf(prop.getProperty(NUMBER_OF_YEAR_KEY, "1")); + return result; + } + + public static String getRegionName(Properties prop) { + String result = prop.getProperty(REGION_NAME_KEY, ""); + return result; + } + + public static String[] getPopulationNames(Properties prop) { + String[] result = prop.getProperty(POPULATIONS_KEY, "").split(","); + return result; + } + + public static String[] getStrategieNames(Properties prop) { + String[] result = prop.getProperty(STRATEGIES_KEY, "").split(","); + return result; + } + + public static String[] getRuleNames(Properties prop) { + String[] result = prop.getProperty(RULES_KEY, "").split(","); + return result; + } + + public static String[] getSimulationPlanNames(Properties prop) { + String[] result = prop.getProperty(PLANS_KEY, "").split(","); + return result; + } + + private static boolean getUseOptimization(Properties prop) { + boolean result; + // en version < 4.3, le parametre était nommé 'useOptimization' + // qualifiant le cache, il a été renommé ensuite + if (VersionUtil.smallerThan(getIsisFishVersion(prop), "4.3.0.0")) { + result = Boolean.FALSE; + } else { + result = Boolean.valueOf(prop.getProperty(USE_OPTIMIZATION_KEY, "false")); + } + return result; + } + + public static String getOptimizationName(Properties prop) { + String result = prop.getProperty(OPTIMIZATION_KEY, ""); + return result; + } + + public static String getObjectiveName(Properties prop) { + String result = prop.getProperty(OBJECTIVE_KEY, ""); + return result; + } + + public static String[] getOptimizationExportNames(Properties prop) { + String[] result = prop.getProperty(OPTIMIZATION_EXPORTS_KEY, "").split(","); + return result; + } + + public static String getOptimizationObservationName(Properties prop, int index) { + String result = prop.getProperty("optimizationobservation" + "." + index, ""); + if (result.matches("[\\w.]+#[\\d.]+#[\\d.]+:(.+)")) { + result = result.replaceFirst("[\\w.]+#[\\d.]+#[\\d.]+:(.+)", "$1"); + } + + return result; + } + + public static String getGeneratedPreScript(Properties prop) { + String result = prop.getProperty(GENERATED_PRE_SCRIPT_KEY, ""); + return result; + } + + public static String getPreScript(Properties prop) { + String result = prop.getProperty(PRE_SCRIPT_KEY, ""); + return result; + } + + public static String getSimulLogLevel(Properties prop) { + String result = prop.getProperty(SIMUL_LOG_LEVEL_KEY, "info"); + return result; + } + + public static String getScriptLogLevel(Properties prop) { + String result = prop.getProperty(SCRIPT_LOG_LEVEL_KEY, "info"); + return result; + } + + public static String getLibLogLevel(Properties prop) { + String result = prop.getProperty(LIB_LOG_LEVEL_KEY, "info"); + return result; + } + + /** + * Retourne les parametres sous forme de String pour une regles. Les regles + * sont numeroter dans l'ordre dans lequel on les recupere via #getRuleNames(). + * + * @param prop + * @param prefix can be 'rule', or other + * @param index l'indice de la regle dont on veut les parametre en String + * @return + */ + public static Map<String, String> getParamAsString(Properties prop, String prefix, int index) { + Map<String, String> result = new HashMap<String, String>(); + if (prop != null) { + String paramTag = prefix + "." + index + ".parameter."; + int paramTagLength = paramTag.length(); + + for (String key : prop.stringPropertyNames()) { + if (StringUtils.startsWith(key, paramTag)) { + String paramName = key.substring(paramTagLength); + String value = prop.getProperty(key); + // get only string representation of entities + if (value.matches("[\\w.]+#[\\d.]+#[\\d.]+:(.+)")) { + value = value.replaceFirst("[\\w.]+#[\\d.]+#[\\d.]+:(.+)", "$1"); + } + result.put(paramName, value); + } + } + } + + return result; + } + + public static String toString(Properties prop) { + StringBuilder result = new StringBuilder(); + result.append( + t("isisfish.params.toString.simulation.done", + getIsisFishVersion(prop))).append(" (").append(getSimulatorName(prop)).append(")").append('\n'); + result.append("--------------------\n"); + result.append(getDescription(prop)).append('\n'); + result.append("--------------------\n"); + + if (getUseSimulationPlan(prop)) { + result.append(t("isisfish.params.toString.plan.number", + getSimulationPlanNumber(prop))); + result.append("\n"); + } + + result.append("\n"); + + boolean useCache = getUseCache(prop); + if (useCache) { + result.append(t("isisfish.params.toString.use.cache", + useCache)); + result.append("\n"); + } + + result.append(t("isisfish.params.toString.fishery", getRegionName(prop))); + result.append("\n\n"); + result.append(t("isisfish.params.toString.number.years", + getNumberOfYear(prop))); + result.append("\n\n"); + + // startegies + String[] strs = getStrategieNames(prop); + result.append(t("isisfish.params.toString.strategies")); + String sep = ""; + for (String str : strs) { + result.append(sep).append(str); + sep = ","; + } + result.append("\n\n"); + + // populations + result.append(t("isisfish.params.toString.populations")); + sep = ""; + for (String pop : getPopulationNames(prop)) { + result.append(sep).append(pop); + sep = ","; + } + result.append("\n\n"); + + // rules + int cpt = 0; + for (String name : getRuleNames(prop)) { + result.append(t("isisfish.params.toString.rule", name)); + result.append('\n'); + for (Map.Entry<String, String> e : getParamAsString(prop, "rule", cpt++).entrySet()) { + result.append("\t").append(e.getKey()).append(" : ").append(e.getValue()).append("\n"); + } + result.append("\n"); + + } + result.append("\n\n"); + + // simulation plans + cpt = 0; + for (String plan : getSimulationPlanNames(prop)) { + result.append(t("isisfish.params.toString.plan", plan)); + result.append('\n'); + for (Map.Entry<String, String> e : getParamAsString(prop, "plan", cpt++).entrySet()) { + result.append("\t").append(e.getKey()).append(" : ").append(e.getValue()).append("\n"); + } + result.append('\n'); + + } + + boolean useOptimization = getUseOptimization(prop); + + if (useOptimization) { + result.append(t("isisfish.params.toString.use.optimization", + useOptimization)); + + String objective = getObjectiveName(prop); + if (StringUtils.isNotBlank(objective)) { + result.append(t("isisfish.params.toString.objective", objective)); + result.append('\n'); + for (Map.Entry<String, String> e : getParamAsString(prop, OBJECTIVE_KEY, 0).entrySet()) { + result.append("\t").append(e.getKey()).append(" : ").append(e.getValue()).append("\n"); + } + result.append('\n'); + } + + String optimization = getOptimizationName(prop); + if (StringUtils.isNotBlank(optimization)) { + result.append(t("isisfish.params.toString.optimization", optimization)); + result.append('\n'); + for (Map.Entry<String, String> e : getParamAsString(prop, OPTIMIZATION_KEY, 0).entrySet()) { + result.append("\t").append(e.getKey()).append(" : ").append(e.getValue()).append("\n"); + } + result.append('\n'); + } + + cpt = 0; + for (String export : getOptimizationExportNames(prop)) { + String observation = getOptimizationObservationName(prop, cpt++); + + result.append(t("isisfish.params.toString.optimizationExport", export)); + if (StringUtils.isNotBlank(observation)) { + result.append("\t->").append(t("isisfish.params.toString.optimizationObservation", observation)); + } + result.append('\n'); + } + } + + String genScript = getGeneratedPreScript(prop); + if (StringUtils.isNotBlank(genScript)) { + result.append('\n'); + result.append(t("isisfish.params.toString.script.generatedpresimulation")) + .append(":\n"); + result.append(genScript); + } + String prescript = getPreScript(prop); + if (StringUtils.isNotBlank(prescript)) { + result.append('\n'); + result.append(t("isisfish.params.toString.script.presimulation")) + .append(":\n"); + result.append(prescript); + } + result.append('\n'); + result.append(t("isisfish.params.toString.simul.logger.level", + getSimulLogLevel(prop))); + result.append('\n'); + result.append(t("isisfish.params.toString.script.logger.level", + getScriptLogLevel(prop))); + result.append('\n'); + result.append(t("isisfish.params.toString.lib.logger.level", + getLibLogLevel(prop))); + result.append('\n'); + return result.toString(); + } + +} Property changes on: trunk/src/main/java/fr/ifremer/isisfish/simulator/SimulationParameterPropertiesHelper.java ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Modified: trunk/src/main/java/fr/ifremer/isisfish/ui/result/MatrixSummaryRenderer.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/ui/result/MatrixSummaryRenderer.java 2014-07-22 09:01:49 UTC (rev 4053) +++ trunk/src/main/java/fr/ifremer/isisfish/ui/result/MatrixSummaryRenderer.java 2014-07-22 19:54:22 UTC (rev 4054) @@ -27,7 +27,6 @@ import static org.nuiton.i18n.I18n.n; import java.awt.Component; -import java.util.Iterator; import javax.swing.Icon; import javax.swing.JScrollPane; @@ -40,7 +39,9 @@ import fr.ifremer.isisfish.datastore.ResultStorage; import fr.ifremer.isisfish.datastore.SimulationStorage; import fr.ifremer.isisfish.simulator.SimulationParameter; +import fr.ifremer.isisfish.simulator.SimulationParameterPropertiesHelper; import fr.ifremer.isisfish.types.TimeStep; +import java.util.Properties; /** * Simulation summary renderer. @@ -79,11 +80,12 @@ if (cachedComponent == null) { SimulationParameter sp = simulationStorage.getParameter(); - StringBuffer resume = new StringBuffer(sp.toString()); + Properties prop = sp.toProperties(); + String params = SimulationParameterPropertiesHelper.toString(prop); + StringBuilder resume = new StringBuilder(params); resume.append("\n"); - for(Iterator<String> i = resultStorage.getResultName().iterator(); i.hasNext();) { - String name = (String)i.next(); + for(String name : resultStorage.getResultName()) { try { // TODO EC20090717 ca marche plus ca Modified: trunk/src/main/java/fr/ifremer/isisfish/util/StringConverter.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/util/StringConverter.java 2014-07-22 09:01:49 UTC (rev 4053) +++ trunk/src/main/java/fr/ifremer/isisfish/util/StringConverter.java 2014-07-22 19:54:22 UTC (rev 4054) @@ -58,7 +58,8 @@ Object result = null; if (o != null) { if (o instanceof TopiaEntity) { - result = ((TopiaEntity) o).getTopiaId(); + // put after ':' string entity representation + result = ((TopiaEntity) o).getTopiaId() + ":" + o.toString(); } else if (o instanceof TimeStep) { result = String.valueOf(((TimeStep) o).getStep()); } else if (o instanceof Month) { Modified: trunk/src/main/java/fr/ifremer/isisfish/util/TopiaEntityConverter.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/util/TopiaEntityConverter.java 2014-07-22 09:01:49 UTC (rev 4053) +++ trunk/src/main/java/fr/ifremer/isisfish/util/TopiaEntityConverter.java 2014-07-22 19:54:22 UTC (rev 4054) @@ -29,6 +29,7 @@ import org.apache.commons.beanutils.ConversionException; import org.apache.commons.beanutils.Converter; +import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.hibernate.HibernateException; @@ -69,6 +70,8 @@ if (value instanceof TopiaEntity) { result = value; } else if (value instanceof String) { + // after ':' is entity string representation + value = StringUtils.substringBefore((String)value, ":"); result = getTopiaEntity((String) value); } else { throw new ConversionException("Can't convert '" + value + "' to " + type.getName()); Modified: trunk/src/main/resources/i18n/isis-fish_en_GB.properties =================================================================== --- trunk/src/main/resources/i18n/isis-fish_en_GB.properties 2014-07-22 09:01:49 UTC (rev 4053) +++ trunk/src/main/resources/i18n/isis-fish_en_GB.properties 2014-07-22 19:54:22 UTC (rev 4054) @@ -697,20 +697,20 @@ isisfish.params.simulationName=Simulation name isisfish.params.stategiesAndPopulations=Strategies and populations isisfish.params.title=Parameters -isisfish.params.toString.fishery=Fishery\: %1$s\n\n +isisfish.params.toString.fishery=Fishery\: %1$s isisfish.params.toString.lib.logger.level=Level of libraries' logger \: %1$s -isisfish.params.toString.number.years=Number of years\: %1$s\n\n +isisfish.params.toString.number.years=Number of years\: %1$s isisfish.params.toString.objective= isisfish.params.toString.optimization= isisfish.params.toString.optimizationExport= isisfish.params.toString.optimizationObservation= isisfish.params.toString.plan=Plan\: %1$s -isisfish.params.toString.plan.number=Simulation plan sequence number\: %1$s\n\n +isisfish.params.toString.plan.number=Simulation plan sequence number\: %1$s isisfish.params.toString.populations=Populations \: isisfish.params.toString.rule=Rule\: %1$s -isisfish.params.toString.script.generatedpresimulation= +isisfish.params.toString.script.generatedpresimulation=Generated script isisfish.params.toString.script.logger.level=Level of scripts' logger \: %1$s -isisfish.params.toString.script.presimulation=Script de presimulation +isisfish.params.toString.script.presimulation=Pre-simulation script isisfish.params.toString.simul.logger.level=Level of simulator's logger \: %1$s isisfish.params.toString.simulation.done=Simulation done with \: %1$s isisfish.params.toString.strategies=Strategies \: Modified: trunk/src/main/resources/i18n/isis-fish_fr_FR.properties =================================================================== --- trunk/src/main/resources/i18n/isis-fish_fr_FR.properties 2014-07-22 09:01:49 UTC (rev 4053) +++ trunk/src/main/resources/i18n/isis-fish_fr_FR.properties 2014-07-22 19:54:22 UTC (rev 4054) @@ -692,18 +692,18 @@ isisfish.params.simulationName=Nom de la simulation isisfish.params.stategiesAndPopulations=Stratégies et populations isisfish.params.title=Paramètres -isisfish.params.toString.fishery=Pêcherie\: %s\n\n +isisfish.params.toString.fishery=Pêcherie\: %s isisfish.params.toString.lib.logger.level=niveau du logger de librairies \: %s -isisfish.params.toString.number.years=Nombre d'années \: %s\n\n +isisfish.params.toString.number.years=Nombre d'années \: %s isisfish.params.toString.objective=Fonction d'objectif \: %s isisfish.params.toString.optimization=Méthode d'optimisation \: %s isisfish.params.toString.optimizationExport=Export d'optimisation \: %s isisfish.params.toString.optimizationObservation=Observation d'optimisation \: %s isisfish.params.toString.plan=Plan\: %s -isisfish.params.toString.plan.number=Numéro de séquence dans le plan \: %s\n\n +isisfish.params.toString.plan.number=Numéro de séquence dans le plan \: %s isisfish.params.toString.populations=Populations \: isisfish.params.toString.rule=Règle\: %s -isisfish.params.toString.script.generatedpresimulation= +isisfish.params.toString.script.generatedpresimulation=Script généré isisfish.params.toString.script.logger.level=niveau du logger de scripts \: %1$s isisfish.params.toString.script.presimulation=Script de presimulation isisfish.params.toString.simul.logger.level=niveau du logger de simulateur \: %1$s

1 0

r4053 - trunk/src/main/java/fr/ifremer/isisfish/ui/util
by bpoussin＠users.forge.codelutin.com 22 Jul '14

22 Jul '14

Author: bpoussin Date: 2014-07-22 11:01:49 +0200 (Tue, 22 Jul 2014) New Revision: 4053 Url: http://forge.codelutin.com/projects/isis-fish/repository/revisions/4053 Log: with normal stackTrace is more readeable Modified: trunk/src/main/java/fr/ifremer/isisfish/ui/util/ErrorHelper.java Modified: trunk/src/main/java/fr/ifremer/isisfish/ui/util/ErrorHelper.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/ui/util/ErrorHelper.java 2014-07-21 11:02:35 UTC (rev 4052) +++ trunk/src/main/java/fr/ifremer/isisfish/ui/util/ErrorHelper.java 2014-07-22 09:01:49 UTC (rev 4053) @@ -68,8 +68,11 @@ public static void showErrorDialog(Component parent, String message, Throwable cause) { JXErrorPane pane = new JXErrorPane(); + + String details = "<pre>\n" + getDetails(cause) + "\n</pre>"; + ErrorInfo info = new ErrorInfo(t("isisfish.error.errorpane.title"), - t("isisfish.error.errorpane.htmlmessage", message), null, null, + t("isisfish.error.errorpane.htmlmessage", message), details, null, cause, null, null); pane.setErrorInfo(info); if (cause != null) { @@ -97,6 +100,18 @@ showErrorDialog(null, message, cause); } + private static String getDetails(Throwable cause) { + String result = ""; + if (cause != null) { + StringWriter out = new StringWriter(); + PrintWriter writer = new PrintWriter(out); + cause.printStackTrace(writer); + out.flush(); + result = out.toString(); + } + return result; + } + /* * @see org.jdesktop.swingx.error.ErrorReporter#reportError(org.jdesktop.swingx.error.ErrorInfo) */ @@ -137,10 +152,7 @@ } // message exception - StringWriter out = new StringWriter(); - PrintWriter writer = new PrintWriter(out); - errorInfo.getErrorException().printStackTrace(writer); - message += formatMessage("Exception", out.toString()); + message += formatMessage("Exception", getDetails(errorInfo.getErrorException())); // TODO i18n files are iso encoded :( email.setContent(message, "text/plain; charset=ISO-8859-9");

1 0

r4052 - trunk/src/main/java/fr/ifremer/isisfish/entities
by bpoussin＠users.forge.codelutin.com 21 Jul '14

21 Jul '14

Author: bpoussin Date: 2014-07-21 13:02:35 +0200 (Mon, 21 Jul 2014) New Revision: 4052 Url: http://forge.codelutin.com/projects/isis-fish/repository/revisions/4052 Log: fixes #5464 - NPE in method StrategyMonthInfoImpl#getMetierWithProportion() Modified: trunk/src/main/java/fr/ifremer/isisfish/entities/StrategyMonthInfoImpl.java Modified: trunk/src/main/java/fr/ifremer/isisfish/entities/StrategyMonthInfoImpl.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/entities/StrategyMonthInfoImpl.java 2014-07-21 11:01:58 UTC (rev 4051) +++ trunk/src/main/java/fr/ifremer/isisfish/entities/StrategyMonthInfoImpl.java 2014-07-21 11:02:35 UTC (rev 4052) @@ -39,6 +39,7 @@ import fr.ifremer.isisfish.IsisFishRuntimeException; import fr.ifremer.isisfish.datastore.RegionStorage; import fr.ifremer.isisfish.datastore.StorageException; +import java.util.Collections; import org.nuiton.math.matrix.MatrixIterator; /** @@ -73,9 +74,14 @@ @Override public List<Metier> getMetierWithProportion() { MatrixND mat = super.getProportionMetier(); - List<Metier> result = new ArrayList<Metier>(mat.getDim(0)); - for (MatrixIterator i=mat.iteratorNotZero(); i.next();) { - result.add((Metier)i.getSemanticsCoordinates()[0]); + List<Metier> result; + if (mat == null) { + result = Collections.EMPTY_LIST; + } else { + result = new ArrayList<Metier>(mat.getDim(0)); + for (MatrixIterator i=mat.iteratorNotZero(); i.next();) { + result.add((Metier)i.getSemanticsCoordinates()[0]); + } } return result; }

1 0

r4051 - in trunk/src/main/java/fr/ifremer/isisfish: datastore ui/result
by bpoussin＠users.forge.codelutin.com 21 Jul '14

21 Jul '14

Author: bpoussin Date: 2014-07-21 13:01:58 +0200 (Mon, 21 Jul 2014) New Revision: 4051 Url: http://forge.codelutin.com/projects/isis-fish/repository/revisions/4051 Log: add @Override and fixme comment Modified: trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorageAbstract.java trunk/src/main/java/fr/ifremer/isisfish/ui/result/ResultHandler.java Modified: trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorageAbstract.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorageAbstract.java 2014-07-21 10:35:11 UTC (rev 4050) +++ trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorageAbstract.java 2014-07-21 11:01:58 UTC (rev 4051) @@ -207,6 +207,7 @@ * return list of all result name available * @return */ + @Override abstract public List<String> getResultName(); /** Modified: trunk/src/main/java/fr/ifremer/isisfish/ui/result/ResultHandler.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/ui/result/ResultHandler.java 2014-07-21 10:35:11 UTC (rev 4050) +++ trunk/src/main/java/fr/ifremer/isisfish/ui/result/ResultHandler.java 2014-07-21 11:01:58 UTC (rev 4051) @@ -218,6 +218,10 @@ resultSimulationFrame.getMatrixViewerContainer().add(matrixViewerPanel, BorderLayout.CENTER); if (model.getSize() > 0) { + // FIXME poussin 20140721 est-ce bien intelligent de forcer l'ouverture + // du 1er resultat trouver. On prend du temps pour ca, alors que l'utilisateur + // veut potentiellement regarder autre chose. Demander au utilisateur si on peut + // ne pas ouvrir de resultat par defaut. model.setSelectedItem(model.getElementAt(0)); } } catch (TopiaException ex) {

1 0

r4050 - trunk/src/main/java/fr/ifremer/isisfish/datastore
by bpoussin＠users.forge.codelutin.com 21 Jul '14

21 Jul '14

Author: bpoussin Date: 2014-07-21 12:35:11 +0200 (Mon, 21 Jul 2014) New Revision: 4050 Url: http://forge.codelutin.com/projects/isis-fish/repository/revisions/4050 Log: if no result return null for getMatrix(name, tx) Modified: trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorageAbstract.java Modified: trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorageAbstract.java =================================================================== --- trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorageAbstract.java 2014-07-20 20:26:44 UTC (rev 4049) +++ trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorageAbstract.java 2014-07-21 10:35:11 UTC (rev 4050) @@ -558,6 +558,11 @@ } } + // s'il n'y pas de resultat, on retourne null + if (sample == null) { + return null; + } + // creation des dimensions String[] dimNames = new String[1 + sample.getDimCount()]; dimNames[0] = t("isisfish.common.date");

1 0