Author: chatellier Date: 2009-12-07 15:46:49 +0000 (Mon, 07 Dec 2009) New Revision: 2805 Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/IsisConfig.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/aspect/RuleAspect.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorage.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationStorage.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulatorStorage.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/entities/PopulationSeasonInfoImpl.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/equation/PopulationMeanWeight.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/map/IsisMapBean.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/MetierMonitor.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/PopulationMonitor.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/SSHSimulatorLauncher.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/SimulationService.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/package-info.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/config/SSHLauncherConfigAction.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/script/ScriptAction.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/script/model/ScriptTreeModel.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/EditorHelper.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/EquationContinuousPanelUI.jaxx isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/FactorWizardUI.jaxx isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/widget/editor/ParameterTableCellEditor.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/util/ssh/SSHUtils.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/util/ssh/package-info.java Log: Global javadoc fix (@return warning, etc...) Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/IsisConfig.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/IsisConfig.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/IsisConfig.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -124,8 +124,9 @@ } /** - * le nombre global ex: 3.2.0.0 - * @return + * Le nombre global ex: 3.2.0.0 + * + * @return full version */ static public String getVersion() { String result = version.toString(); @@ -134,7 +135,8 @@ /** * La version de la base ex: 3.2 - * @return + * + * @return database version */ public static Version getDatabaseVersion() { return databaseVersion; @@ -142,7 +144,8 @@ /** * La version de l'api de programmation ex: 3.2.0 - * @return + * + * @return api version */ public static Version getApiVersion() { return apiVersion; @@ -189,7 +192,8 @@ /** * Retourne le repertoire racine de toutes les donnees (script, simulation * region, ...) - * @return + * + * @return database directory */ public File getDatabaseDirectory() { File result = getOptionAsFile(Option.DATABASE_DIRECTORY.key); @@ -268,8 +272,9 @@ } /** - * Retourne l'encoding a utiliser pour les fichiers textes - * @return + * Retourne l'encoding a utiliser pour les fichiers textes. + * + * @return encoding to use */ public String getEncoding() { String result = getOption(Option.ENCODING.key); @@ -277,8 +282,9 @@ } /** - * Retourne le serveur SMTP a utiliser pour l'envoie de mail - * @return + * Retourne le serveur SMTP a utiliser pour l'envoie de mail. + * + * @return smtp server address */ public String getSmtpServer() { String result = getOption(Option.SMTP_SERVER.key); @@ -286,8 +292,9 @@ } /** - * Retourne le nom usuel de l'utilisateur - * @return + * Retourne le nom usuel de l'utilisateur. + * + * @return username */ public String getUserName() { String result = getOption(Option.USER_NAME.key); @@ -295,8 +302,9 @@ } /** - * Retourne l'email de l'utilisateur - * @return + * Retourne l'email de l'utilisateur. + * + * @return user email */ public String getUserMail() { String result = getOption(Option.USER_MAIL.key); Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/aspect/RuleAspect.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/aspect/RuleAspect.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/aspect/RuleAspect.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -103,8 +103,7 @@ * Effectue l'appel reel en calculant le temps pris. * * @param jp join point - * @param state - * @param aspectized class return object + * @param state state * @throws Throwable */ protected Object makeTimedCall(JoinPoint jp, State state) throws Throwable { Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorage.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorage.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/ResultStorage.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -123,19 +123,20 @@ /** * Retourne le nom de tous les resultats disponibles le nom est constitué - * de la date et du nom du resultat - * @return + * de la date et du nom du resultat. + * + * @return available results */ protected Set<String> getAvailableResult() { if (availableResult == null) { availableResult = new HashSet<String>(); try { - TopiaContext tx = null; - boolean mustClose = false; + TopiaContext tx = null; + boolean mustClose = false; - if (simulation == SimulationContext.get().getSimulationStorage()) { - tx = SimulationContext.get().getDbResult(); - } + if (simulation == SimulationContext.get().getSimulationStorage()) { + tx = SimulationContext.get().getDbResult(); + } if (tx == null) { // not in simulation, create transaction @@ -163,15 +164,17 @@ * Verifie si un resultat est disponible pour une date donnée * @param date * @param name - * @return + * @return {@code true} if result is available */ protected boolean isAvailableResult(Date date, String name) { String key = date.getDate() + ":" + name; boolean result = getAvailableResult().contains(key); return result; } + /** - * Ajoute un resultat comme etant disponible pour une date donnée + * Ajoute un resultat comme etant disponible pour une date donnée. + * * @param date * @param name */ @@ -181,9 +184,12 @@ } /** - * Permet de savoir si lorsque l'on ajoutera ce resultat, il sera - * sauvé ou non. - */ + * Permet de savoir si lorsque l'on ajoutera ce resultat, il sera + * sauvé ou non. + * + * @param name result name + * @return {@code true} if result is enabled + */ public boolean isEnabled(String name){ name = name.trim(); if (enabledResult == null) { @@ -267,12 +273,12 @@ protected void doAddResult(Date date, String name, MatrixND mat) throws IsisFishException{ try { - TopiaContext tx = null; - boolean mustClose = false; + TopiaContext tx = null; + boolean mustClose = false; - if (simulation == SimulationContext.get().getSimulationStorage()) { - tx = SimulationContext.get().getDbResult(); - } + if (simulation == SimulationContext.get().getSimulationStorage()) { + tx = SimulationContext.get().getDbResult(); + } if (tx == null) { // not in simulation, create transaction tx = simulation.getStorage().beginTransaction(); @@ -290,9 +296,9 @@ protected void doAddResult(Date date, String name, MatrixND mat, TopiaContext tx) throws IsisFishException{ // si la matrice n'a pas de semantique on refuse - for(int i=0; i<mat.getNbDim(); i++){ + for (int i=0; i<mat.getNbDim(); i++) { // la semantique n'est pas bonne des qu'il y a un null dedans - if(mat.getSemantics(i).contains(null)){ + if (mat.getSemantics(i).contains(null)) { throw new SimulationException("Erreur le résultat que vous souhaitez enregistrer n'a pas d'information convenable pour la dimension: " + i + " " + mat.getDimensionName(i) ); } } @@ -316,12 +322,12 @@ public void addActiveRule(Date date, Rule rule) throws IsisFishException { try { - TopiaContext tx = null; - boolean mustClose = false; + TopiaContext tx = null; + boolean mustClose = false; - if (simulation == SimulationContext.get().getSimulationStorage()) { - tx = SimulationContext.get().getDbResult(); - } + if (simulation == SimulationContext.get().getSimulationStorage()) { + tx = SimulationContext.get().getDbResult(); + } if (tx == null) { // not in simulation, create transaction tx = simulation.getStorage().beginTransaction(); @@ -343,21 +349,20 @@ } /** - * Retourne la liste de tous les résultats. Si le résultat est categorisé - * par une population alors le nom de la population est automatiquement - * ajouté au nom du résultat - */ - @SuppressWarnings("unchecked") - public List<String> getResultName(){ + * Retourne la liste de tous les résultats. Si le résultat est categorisé + * par une population alors le nom de la population est automatiquement + * ajouté au nom du résultat. + */ + public List<String> getResultName() { List<String> result = null; try { - TopiaContext tx = null; - boolean mustClose = false; + TopiaContext tx = null; + boolean mustClose = false; - if (simulation == SimulationContext.get().getSimulationStorage()) { - tx = SimulationContext.get().getDbResult(); - } + if (simulation == SimulationContext.get().getSimulationStorage()) { + tx = SimulationContext.get().getDbResult(); + } if (tx == null) { // not in simulation, create transaction tx = simulation.getStorage().beginTransaction(); @@ -381,14 +386,15 @@ } return result; } + /** - * Retourne la matrice stocke pour un pas de temps - * @param date le pas de temps que l'on souhaite - * @param pop la population pour lequelle on souhaite le resultat - * @param name le nom des resultats dont on veut la matrice - * @return La matrice demandée ou null si aucune matrice ne correspond a - * la demande. - */ + * Retourne la matrice stocke pour un pas de temps + * @param date le pas de temps que l'on souhaite + * @param pop la population pour lequelle on souhaite le resultat + * @param name le nom des resultats dont on veut la matrice + * @return La matrice demandée ou null si aucune matrice ne correspond a + * la demande. + */ public MatrixND getMatrix(Date date, Population pop, String name) { String newName = name + " " + pop; return getMatrix(date, newName); @@ -398,12 +404,12 @@ MatrixND mat = getInCache(date, name); if (mat == null && isAvailableResult(date, name)) { try { - TopiaContext tx = null; - boolean mustClose = false; + TopiaContext tx = null; + boolean mustClose = false; - if (simulation == SimulationContext.get().getSimulationStorage()) { - tx = SimulationContext.get().getDbResult(); - } + if (simulation == SimulationContext.get().getSimulationStorage()) { + tx = SimulationContext.get().getDbResult(); + } if (tx == null) { // not in simulation, create transaction tx = simulation.getStorage().beginTransaction(); @@ -428,14 +434,15 @@ } /** - * Retourne la matrice stocke pour un pas de temps - * @param date le pas de temps que l'on souhaite - * @param name le nom des resultats dont on veut la matrice - * @param tx TopiaContext a utiliser pour recuperer les resultats et donc les semantiques - * @return La matrice demandée ou null si aucune matrice ne correspond a - * la demande. - */ - public MatrixND getMatrix(Date date, String name, TopiaContext tx){ + * Retourne la matrice stocke pour un pas de temps. + * + * @param date le pas de temps que l'on souhaite + * @param name le nom des resultats dont on veut la matrice + * @param tx TopiaContext a utiliser pour recuperer les resultats et donc les semantiques + * @return La matrice demandée ou {@code null} si aucune matrice ne correspond a + * la demande. + */ + public MatrixND getMatrix(Date date, String name, TopiaContext tx) { MatrixND mat = getInCache(date, name); if (mat == null && isAvailableResult(date, name)) { try { @@ -459,7 +466,7 @@ * @param pop la population pour lequel on souhaite la matrice * @param name le nom des resultats dont on veut une matrice globale. */ - public MatrixND getMatrix(Population pop, String name){ + public MatrixND getMatrix(Population pop, String name) { String newName = name + " " + pop; return getMatrix(newName); } @@ -468,15 +475,15 @@ * Retourne une matrice contenant tous les pas de temps. * @param name le nom des resultats dont on veut une matrice globale. */ - public MatrixND getMatrix(String name){ + public MatrixND getMatrix(String name) { MatrixND resultMat = null; try { - TopiaContext tx = null; - boolean mustClose = false; + TopiaContext tx = null; + boolean mustClose = false; - if (simulation == SimulationContext.get().getSimulationStorage()) { - tx = SimulationContext.get().getDbResult(); - } + if (simulation == SimulationContext.get().getSimulationStorage()) { + tx = SimulationContext.get().getDbResult(); + } if (tx == null) { // not in simulation, create transaction tx = simulation.getStorage().beginTransaction(); @@ -500,12 +507,11 @@ } /** - * Retourne une matrice contenant tous les pas de temps. - * @param name le nom des resultats dont on veut une matrice globale. - * @param tx TopiaContext a utiliser pour recuperer les resultats et donc les semantiques - */ - @SuppressWarnings("unchecked") - public MatrixND getMatrix(String name, TopiaContext tx){ + * Retourne une matrice contenant tous les pas de temps. + * @param name le nom des resultats dont on veut une matrice globale. + * @param tx TopiaContext a utiliser pour recuperer les resultats et donc les semantiques + */ + public MatrixND getMatrix(String name, TopiaContext tx) { log.debug("Get result: " + name); MatrixND resultMat = getInCache(name); @@ -525,7 +531,7 @@ } // s'il n'y pas de resultat, on retourne null - if(results == null || results.size() == 0){ + if (results == null || results.size() == 0) { return null; } @@ -536,12 +542,12 @@ List<Date> dates = new ArrayList<Date>(); Date date = new Date(0); dates.add(date); - while(date.before(lastDate)){ + while (date.before(lastDate)) { date = date.next(); dates.add(date); } - if(log.isTraceEnabled()) { + if (log.isTraceEnabled()) { log.trace("List des dates: "+ dates); } @@ -551,7 +557,7 @@ // recuperation des noms des dimensions String [] dimNames = new String[1 + mat.getNbDim()]; dimNames[0] = _("isisfish.common.date"); - for(int i=1; i<dimNames.length; i++){ + for (int i=1; i<dimNames.length; i++) { dimNames[i] = mat.getDimensionName(i-1); } @@ -560,30 +566,28 @@ List [] sem = new List[1 + mat.getNbDim()]; sem[0] = dates; - for(int i=1; i<sem.length; i++){ + for (int i=1; i<sem.length; i++) { sem[i] = new HashList(); } - for(Result result: results){ + for (Result result: results) { MatrixND mattmp = result.getMatrix(); - if(log.isTraceEnabled()) { + if (log.isTraceEnabled()) { log.trace("Ajout de la semantics: "+ Arrays.asList(mattmp.getSemantics())); } - - for(int s=0; s<mattmp.getNbDim(); s++){ + for (int s=0; s<mattmp.getNbDim(); s++) { sem[s+1].addAll(mattmp.getSemantics(s)); } } - if(log.isTraceEnabled()) { + if (log.isTraceEnabled()) { log.trace("La semantique final est: "+ Arrays.asList(sem)); } // creation de la matrice resultat resultMat = MatrixFactory.getInstance().create(name, sem, dimNames); - // recuperation du resultat pour chaque date de la simulation, de Date(0) à lastDate for(Result result: results){ Date d = result.getResultDate(); @@ -608,7 +612,9 @@ } /** - * @return + * Get last simulation date. + * + * @return last simulation date */ public Date getLastDate() { int monthNumber = simulation.getParameter().getNumberOfYear() * Month.NUMBER_OF_MONTH; @@ -616,9 +622,10 @@ return result; } - /* (non-Javadoc) + /* * @see fr.ifremer.isisfish.simulator.SimulationResultListener#addResult(fr.ifremer.isisfish.simulator.SimulationContext, fr.ifremer.isisfish.types.Date, java.lang.String, org.nuiton.math.matrix.MatrixND) */ + @Override public void addResult(SimulationContext context, Date date, String name, MatrixND mat) throws IsisFishException { try { doAddResult(date, name, mat, context.getDbResult()); @@ -627,9 +634,10 @@ } } - /* (non-Javadoc) + /* * @see fr.ifremer.isisfish.simulator.SimulationResultGetter#getMatrix(fr.ifremer.isisfish.simulator.SimulationContext, fr.ifremer.isisfish.types.Date, java.lang.String) */ + @Override public MatrixND getMatrix(SimulationContext context, Date date, String name) { MatrixND result = null; try { @@ -642,9 +650,10 @@ return result; } - /* (non-Javadoc) + /* * @see fr.ifremer.isisfish.simulator.SimulationResultGetter#getMatrix(fr.ifremer.isisfish.simulator.SimulationContext, java.lang.String) */ + @Override public MatrixND getMatrix(SimulationContext context, String name) { MatrixND result = null; try { @@ -657,15 +666,17 @@ return result; } - /* (non-Javadoc) + /* * @see fr.ifremer.isisfish.simulator.SimulationListener#afterSimulation(fr.ifremer.isisfish.simulator.SimulationContext) */ + @Override public void afterSimulation(SimulationContext context) { } - /* (non-Javadoc) + /* * @see fr.ifremer.isisfish.simulator.SimulationListener#beforeSimulation(fr.ifremer.isisfish.simulator.SimulationContext) */ + @Override public void beforeSimulation(SimulationContext context) { } Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationStorage.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationStorage.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationStorage.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -146,10 +146,11 @@ } /** - * Retourne le repertoire de stockage des exports de resulatat d'une + * Retourne le repertoire de stockage des exports de resultat d'une * simulation. + * * @param root le repertoire de stockage de la simulation - * @return + * @return result export directory */ public static File getResultExportDirectory(File root) { File result = new File(root, RESULT_EXPORT_DIRECTORY); @@ -160,9 +161,10 @@ } /** - * Retourne le fichier de stockage des parametres de la simulation + * Retourne le fichier de stockage des parametres de la simulation. + * * @param root le repertoire de stockage de la simulation - * @return + * @return parameter file */ public static File getSimulationParametersFile(File root) { File result = new File(root, PARAMETERS_FILENAME); @@ -170,9 +172,10 @@ } /** - * Retourne le fichier de stockage des inforations de la simulation + * Retourne le fichier de stockage des inforations de la simulation. + * * @param root le repertoire de stockage de la simulation - * @return + * @return information file */ public static File getSimulationInformationFile(File root) { File result = new File(root, INFORMATION_FILENAME); @@ -184,7 +187,7 @@ * de la simulation. * * @param root le repertoire de stockage de la simulation - * @return + * @return mexico xml file */ public static File getSimulationResultXmlFile(File root) { File result = new File(root, RESULT_XML_FILENAME); @@ -194,7 +197,7 @@ /** * Retourne le fichier de stockage des parametres de la simulation courante. * - * @return + * @return parameter file */ protected File getSimulationParametersFile() { File result = getSimulationParametersFile(getDirectory()); @@ -215,8 +218,9 @@ /** * Retourne le fichier de stockage du fichier de control de la simulation. * Ce fichier conserve l'etat de la simulation, cela permet a un processus - * externe de connaitre l'etat d'une simulation distante - * @return + * externe de connaitre l'etat d'une simulation distante. + * + * @return simulation control file */ protected File getSimulationControlFile() { if (simulationControlFile == null) { @@ -228,8 +232,9 @@ /** * Retourne le fichier de stockage du fichier de control de la simulation. * Ce fichier conserve l'etat de la simulation, cela permet a un processus - * externe de connaitre l'etat d'une simulation distante - * @return + * externe de connaitre l'etat d'une simulation distante. + * + * @return simulation control file */ protected static File getSimulationControlFile(String id) { File result = new File(getSimulationDirectory(id), CONTROL_FILENAME); @@ -256,7 +261,7 @@ } /** - * Reli le fichier contenant les infos de SimulationControl, en excluant + * Relit le fichier contenant les infos de SimulationControl, en excluant * certain champs. * * @param id l'identifiant de la simulation a lire @@ -424,8 +429,9 @@ } /** - * Retourne le nom du fichier contenant les informations de simulation - * @return + * Retourne le nom du fichier contenant les informations de simulation. + * + * @return information file */ protected File getInformationFile() { File result = new File(getDirectory(), INFORMATION_FILENAME); @@ -433,7 +439,9 @@ } /** - * @return Returns the information. + * Get simulation informations. + * + * @return simulation information */ public SimulationInformation getInformation() { if (information == null) { @@ -443,7 +451,9 @@ } /** - * @return le ResultStorage a utiliser pour cette simulation + * Get simulation {@link ResultStorage}. + * + * @return le {@link ResultStorage} a utiliser pour cette simulation */ public ResultStorage getResultStorage() { if (resultStorage == null) { @@ -453,7 +463,8 @@ } /** - * Indique si la simulation demandée existe ou non + * Indique si la simulation demandée existe ou non. + * * @param name le nom de la simulation à tester * @return true si la simulation existe déjà */ @@ -465,7 +476,8 @@ } /** - * Indique si la simulation demandée existe ou non + * Indique si la simulation demandée existe ou non. + * * @param name le nom de la simulation à tester * @return true si la simulation existe déjà */ @@ -475,6 +487,7 @@ /** * Retourne la simulation demandée. + * * @param name name of the required simulation * @return la simulation souhaitée ou null si la simulation n'existe pas */ @@ -530,7 +543,8 @@ } /** - * Retourne la liste des noms de toutes les régions disponible en local + * Retourne la liste des noms de toutes les régions disponible en local. + * * @return la liste des noms de toutes les régions disponible en local */ public static List<String> getSimulationNames() { @@ -540,7 +554,8 @@ /** * Retourne la liste des noms de toutes les régions disponible en local qui - * ne sont pas encore sur le serveur CVS + * ne sont pas encore sur le serveur VCS. + * * @return liste de noms de simulations */ public static List<String> getNewSimulationNames() { @@ -551,9 +566,10 @@ /** * Retourne la liste des noms de toutes les régions disponible sur le - * serveur CVS + * serveur VCS. + * * @return la liste des noms de toutes les régions disponible sur le - * serveur CVS. Si le serveur n'est pas disponible la liste retournée + * serveur VCS. Si le serveur n'est pas disponible la liste retournée * est vide. */ public static List<String> getRemoteSimulationNames() { @@ -563,7 +579,8 @@ /** * Retourne la liste des noms de toutes les régions disponible sur le - * serveur CVS qui ne sont pas encore en local + * serveur VCS qui ne sont pas encore en local. + * * @return liste de noms de simulations */ public static List<String> getNewRemoteSimulationNames() { @@ -573,9 +590,10 @@ } /** - * Retourne la {@link FisheryRegion} associee a cette simulation - * @param context - * @return + * Retourne la {@link FisheryRegion} associee a cette simulation. + * + * @param context context + * @return simulation's {@link FisheryRegion} * @throws StorageException */ public static FisheryRegion getFisheryRegion(TopiaContext context) @@ -810,23 +828,6 @@ } /* - * @see fr.ifremer.isisfish.datastore.DataStorage#prepare() - * - @Override - protected void prepare() { - super.prepare(); - - // force parameter save - // because they can have unsaved modification - if (log.isDebugEnabled()) { - log.debug("Save simulation parameters"); - } - setParameter(parameter); - - this methods sometime force save of empty parameters - }*/ - - /* * @see fr.ifremer.isisfish.datastore.DataStorage#createZip(java.io.File) */ @Override Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulatorStorage.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulatorStorage.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulatorStorage.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -47,9 +47,12 @@ /** Template freemarker pour les scripts. */ public static final String SIMULATOR_TEMPLATE = "templates/script/simulator.ftl"; - + + /** Simulators cache. */ + static private Map<String, SimulatorStorage> simulatorsCache = (Map<String, SimulatorStorage>) new ReferenceMap(); + /** - * Constructeur + * Constructeur. * * @param rootSrc */ @@ -68,7 +71,7 @@ /** * Retourne une nouvelle instance de la regle. Compile le fichier si besoin * - * @return + * @return new rule instance * @throws IsisFishException */ public Simulator getNewSimulatorInstance() throws IsisFishException { @@ -145,8 +148,4 @@ result.removeAll(getSimulatorNames()); return result; } - - @SuppressWarnings("unchecked") - static private Map<String, SimulatorStorage> simulatorsCache = (Map<String, SimulatorStorage>) new ReferenceMap(); - } Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -345,7 +345,7 @@ /** * Fire a change event to all registred listeners. * - * For example, to notify, that a {@link JavaStorage} file has been added... + * For example, to notify, that a {@link JavaSourceStorage} file has been added... * * @param event event to fire */ Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/entities/PopulationSeasonInfoImpl.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/entities/PopulationSeasonInfoImpl.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/entities/PopulationSeasonInfoImpl.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -224,7 +224,7 @@ /** * Create not initialized no spacialized change group matrix. * - * @return + * @return not initialized no spacialized change group matrix */ public MatrixND createNoSpacializedChangeGroupMatrix() { Population pop = getPopulation(); @@ -263,7 +263,7 @@ /** * Create not initialized spacialized change group matrix. * - * @return + * @return not initialized spacialized change group matrix */ protected MatrixND createSpacializedChangeGroupMatrix() { Population pop = getPopulation(); Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/equation/PopulationMeanWeight.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/equation/PopulationMeanWeight.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/equation/PopulationMeanWeight.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -17,37 +17,32 @@ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. *##%*/ -/* * - * PopulationMeanWeight.java - * - * Created: 23 août 2006 11:14:46 - * - * @author poussin - * @version $Revision$ - * - * Last update: $Date$ - * by : $Author$ - */ - package fr.ifremer.isisfish.equation; import fr.ifremer.isisfish.entities.PopulationGroup; import fr.ifremer.isisfish.simulator.SimulationContext; import fr.ifremer.isisfish.util.Args; - /** + * PopulationMeanWeight equation. + * + * Created: 23 août 2006 11:14:46 + * * @author poussin + * @version $Revision$ * + * Last update: $Date$ + * by : $Author$ */ - public interface PopulationMeanWeight { /** + * Compute population mean weight. * - * @param context + * @param context simulation context * @param group le groupe dont on souhaite le poids moyen - * @return + * @return population mean weight + * @throws Exception */ @Args({"context", "group"}) public double compute(SimulationContext context, PopulationGroup group) throws Exception; Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/map/IsisMapBean.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/map/IsisMapBean.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/map/IsisMapBean.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -502,7 +502,7 @@ * @param fillColor fill color * * @return vpf layer - * @see vpfLayer + * @see VPFLayer */ protected Layer getVPFLayer(String layerId, String mapFile, String lineColor, String fillColor) { Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/MetierMonitor.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/MetierMonitor.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/MetierMonitor.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -116,11 +116,11 @@ /** * Metier is forbidden is can be found in {@link #forbiddenMetier} or in - * {@link #forbiddenMetierMonth} + * {@link #forbiddenMetierMonth}. * * @param metier * @param month - * @return + * @return {@code true} if metier forbidden */ public boolean isForbidden(Metier metier, Month month) { boolean result = forbiddenMetier.contains(metier); @@ -223,7 +223,7 @@ * @param metier * @param firstMonth * @param lastMonth - * @return + * @return limited period */ public boolean isLimitedPeriode(Metier metier, Month firstMonth, Month lastMonth) { for(Month i = firstMonth; i.compareTo(lastMonth) <= 0; i = i.next()) { @@ -254,7 +254,7 @@ /** * @param metier * @param i - * @return + * @return proportion month */ private double computeProportionMonth(Metier metier, Month month) { double prop = 0; @@ -280,7 +280,7 @@ /** * @param metier * @param nbLicences - * @return + * @return proportion max total */ private double computeProportionMaxTotal(Metier metier, int nbLicences) { int nbBoat = 0; Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/PopulationMonitor.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/PopulationMonitor.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/PopulationMonitor.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -104,9 +104,10 @@ } /** - * Return current biomass for species - * @param species - * @return + * Return current biomass for species. + * + * @param species species + * @return species biomass */ public double getBiomass(Species species) { double result = 0; @@ -119,9 +120,10 @@ } /** - * Return current biomass for population - * @param pop - * @return + * Return current biomass for population. + * + * @param pop population + * @return population biomass */ public double getBiomass(Population pop) { double result = 0; @@ -140,9 +142,10 @@ } /** - * Return current numbers for population - * @param pop - * @return + * Return current numbers for population. + * + * @param pop population + * @return population numbers */ public MatrixND getN(Population pop) { MatrixND result = Ns.get(pop); @@ -162,7 +165,9 @@ } /** - * @return Returns the reproduction. + * Return reproduction. + * + * @return the reproduction. */ public MatrixND getReproduction(Date date, Population pop) { return this.reproductions.get(pop).get(date); @@ -203,14 +208,15 @@ } /** - * @todo les reproductions qui sont trop veille pour encore servir + * + * TODO les reproductions qui sont trop veille pour encore servir * doivent être supprimées. c-a-d date < currentDate - etalement - gap between repro recru * Une autre facon de faire est de supprimer les repro == 0 car normalement * toute la repro doit etre utilisé au bout d'un certain temps * * @param date * @param pop - * @return + * @return population recruitment */ public MatrixND getRecruitment(Date date, Population pop) { MatrixND matEtalement = pop.getRecruitmentDistribution(); @@ -306,6 +312,7 @@ } /** + * @param pop population * @param catchPerStrategyMet */ public void holdCatch(Population pop, MatrixND catchPerStrategyMet) { @@ -327,8 +334,10 @@ } /** - * @param pop - * @return + * Get population catch. + * + * @param pop population + * @return population catch */ public MatrixND getCatch(Population pop) { MatrixND result = catchs.get(pop); @@ -336,8 +345,10 @@ } /** - * @param pop - * @return + * Get population hold catch. + * + * @param pop population + * @return population hold catch */ public MatrixND getHoldCatch(Population pop) { MatrixND result = holdCatchs.get(pop); @@ -345,8 +356,10 @@ } /** - * @param pop - * @return + * Population total hold catch. + * + * @param pop population + * @return population total hold catch */ public double getTotalHoldCatch(Population pop) { double result = totalHoldCatch; @@ -354,7 +367,7 @@ } /** - * raz capture cumulée de toutes les pops + * RAZ capture cumulée de toutes les pops. */ public void clearCatch() { catchs.clear(); @@ -392,7 +405,7 @@ discards.get(pop).put(date, tmp); } } - + /** * FIXME: discards n'est jamais vider, on le conserve pour toutes les annees * il serait bon de faire un peu le menage de temps en temps. @@ -413,6 +426,4 @@ } } } - - } Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/SSHSimulatorLauncher.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/SSHSimulatorLauncher.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/SSHSimulatorLauncher.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -1109,7 +1109,7 @@ /** * Get remote directory absolute path. * - * Don't use {@link File.separator} here, caparmor is always unix. + * Don't use {@link java.io.File.separator} here, caparmor is always unix. * * @return remote temp directory path */ Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/SimulationService.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/SimulationService.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/SimulationService.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -232,8 +232,9 @@ } /** - * Retourne la liste de tous les {@link SimulatorLauncher} disponible - * @return + * Retourne la liste de tous les {@link SimulatorLauncher} disponible. + * + * @return {@link SimulatorLauncher}s */ protected Set<SimulatorLauncher> getSimulatorLaunchers() { return executors.keySet(); Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/package-info.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/package-info.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/simulator/launcher/package-info.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -37,9 +37,9 @@ * * <h2>Principe general</h2> * <p> - * Les simulations sont soumises au {@link SimulationService} via sa methode - * submit. Un objet {@link SimulationJob} est alors cree et ajoute a la liste - * des simulations presentes ({@link SimulationService#getJobs}). Si la + * Les simulations sont soumises au {@link fr.ifremer.isisfish.simulator.launcher.SimulationService} via sa methode + * submit. Un objet {@link fr.ifremer.isisfish.simulator.launcher.SimulationJob} est alors cree et ajoute a la liste + * des simulations presentes ({@link fr.ifremer.isisfish.simulator.launcher.SimulationService#getJobs()}). Si la * simulation est une simple simulation ou une simulation avec plan d'analyse * dependant, elle est alors directement ajoutee a la queue de simulation * (simulation a faire). Si @@ -52,8 +52,9 @@ * * <p> * Lorsqu'un thread recupere un job dans la queue, il leve un event {@link - * SimulationListener#start}, la simulation est alors ajoutee a la liste des - * simulations visibles si elle ne l'etait pas encore. + * fr.ifremer.isisfish.simulator.launcher.SimulationServiceListener#simulationStart(SimulationService, SimulationJob)}, + * la simulation est alors ajoutee a la liste des simulations visibles si elle + * ne l'etait pas encore. * </p> * * <p> @@ -62,60 +63,65 @@ * </p> * * <p> - * Le {@link SimulationService#autoLaunch} permet d'indique si le service est + * Le {@link fr.ifremer.isisfish.simulator.launcher.SimulationService#autoLaunch} permet d'indique si le service est * actif ou non. S'il n'est pas actif, il accepte les simulations mais ne les * execute pas (elles sont en attente). S'il est actif alors les differents - * {@link SimulationExecutor}) prenent les jobs de la queue pour faire les + * {@link fr.ifremer.isisfish.simulator.launcher.SimulationExecutor}) prenent les jobs de la queue pour faire les * simulations. * </p> * * <h2>SimulationExecutor</h2> * <p> - * Lors de sa creation le {@link SimulationService} a initialise different - * {@link SimulationExecutor} en fonction de la configuration. Ces {@link - * SimulationExecutor} sont responsable de l'execution des simulations de la - * queue. Chaque {@link SimulationExecutor} a un {@link SimulatorLauncher} - * qu'il utilise si la simulation n'a pas encore de {@link SimulatorLauncher} + * Lors de sa creation le {@link fr.ifremer.isisfish.simulator.launcher.SimulationService} a initialise different + * {@link fr.ifremer.isisfish.simulator.launcher.SimulationExecutor} en fonction de la configuration. Ces {@link + * fr.ifremer.isisfish.simulator.launcher.SimulationExecutor} sont responsable de l'execution des simulations de la + * queue. Chaque {@link fr.ifremer.isisfish.simulator.launcher.SimulationExecutor} a un + * {@link fr.ifremer.isisfish.simulator.launcher.SimulatorLauncher} qu'il utilise + * si la simulation n'a pas encore de {@link SimulatorLauncher} * d'assigne. * </p> * <p> - * Un {@link SimulationExecutor} peut etre mis en pause puis relance. Lorsqu'il - * est en pause, il termine les simulations en cours mais n'en reprend pas de - * nouvelle. Cela permet d'arrete un {@link SimulationExecutor} particulier + * Un {@link fr.ifremer.isisfish.simulator.launcher.SimulationExecutor} peut etre + * mis en pause puis relance. Lorsqu'il est en pause, il termine les simulations + * en cours mais n'en reprend pas de nouvelle. Cela permet d'arrete un + * {@link fr.ifremer.isisfish.simulator.launcher.SimulationExecutor} particulier * sans devoir arreter tout le service de simulation. * </p> - * <p>Si un {@link SimulationExecutor} prend un job ayant deja un {@link - * SimulatorLauncher} d'assigne, il utilise alors ce launcher plutot que le - * sien. Ce choix est derangeant lorsque l'on souhaite utilise un nombre de - * thread limite pour un launcher particulier, mais il est le plus raisonnable - * car l'autre possibilite est que le job soit resoumis au {@link - * SimulationService} jusqu'a ce que le bon {@link SimulationExecutor} le + * <p>Si un {@link fr.ifremer.isisfish.simulator.launcher.SimulationExecutor} + * prend un job ayant deja un {@link fr.ifremer.isisfish.simulator.launcher.SimulatorLauncher} + * d'assigne, il utilise alors ce launcher plutot que le sien. Ce choix est derangeant + * lorsque l'on souhaite utilise un nombre de thread limite pour un launcher particulier, + * mais il est le plus raisonnable car l'autre possibilite est que le job soit + * resoumis au {@link fr.ifremer.isisfish.simulator.launcher.SimulationService} + * jusqu'a ce que le bon {@link fr.ifremer.isisfish.simulator.launcher.SimulationExecutor} le * prenne pour l'executer. On risque dans ce cas d'arriver a une forte - * consommation CPU si le seul {@link SimulationExecutor} disponible ne gere - * pas les jobs en queue. + * consommation CPU si le seul {@link fr.ifremer.isisfish.simulator.launcher.SimulationExecutor} + * disponible ne gere pas les jobs en queue. * </p> * * <h2>SimulationJob</h2> * * <p> * Le simulation Job encapsule l'appel pour que les implantantations des {@link - * SimulatorLauncher} soit la plus simple possible. Il gere les simulations - * avec plan dependant, les exports depandes par l'utilisateur, ainsi que - * l'effacement des simulations si seul les exports interessait l'utilisateur. + * fr.ifremer.isisfish.simulator.launcher.SimulatorLauncher} soit la plus simple + * possible. Il gere les simulations avec plan dependant, les exports depandes + * par l'utilisateur, ainsi que l'effacement des simulations si seul les exports + * interessait l'utilisateur. * </p> * * <p> - * Si le job n'arrive pas a utilise le {@link SimulatorLauncher} il en notifie - * le {@link SimulationService} qui resoumet le job dans la queue pour qu'un - * autre {@link SimulationExecutor} prenne ce job. Si trop d'erreurs sont - * notifiees pour un meme {@link SimulatorLauncher}, le {@link - * SimulatorService} prend alors la decision d'arreter l'executor associe. + * Si le job n'arrive pas a utilise le {@link fr.ifremer.isisfish.simulator.launcher.SimulatorLauncher} + * il en notifie le {@link fr.ifremer.isisfish.simulator.launcher.SimulationService} + * qui resoumet le job dans la queue pour qu'un autre {@link fr.ifremer.isisfish.simulator.launcher.SimulationExecutor} + * prenne ce job. Si trop d'erreurs sont notifiees pour un meme {@link fr.ifremer.isisfish.simulator.launcher.SimulatorLauncher}, + * le {@link fr.ifremer.isisfish.simulator.launcher.SimulationService} prend alors la decision d'arreter l'executor associe. * </p> * <p> * Pour les simulations ou l'utilisateur avait fixe un {@link - * SimulatorLauncher} particulier en cas de notification d'erreur au {@link - * SimulationService} ce {@link SimulatorLauncher} n'est plus pris en compte et - * n'importe quel {@link SimulatorLauncher} peut faire cette simulation. + * fr.ifremer.isisfish.simulator.launcher.SimulatorLauncher} particulier en cas + * de notification d'erreur au {@link fr.ifremer.isisfish.simulator.launcher.SimulationService} + * ce {@link fr.ifremer.isisfish.simulator.launcher.SimulatorLauncher} n'est plus pris en compte et + * n'importe quel {@link fr.ifremer.isisfish.simulator.launcher.SimulatorLauncher} peut faire cette simulation. * </p> */ package fr.ifremer.isisfish.simulator.launcher; Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/config/SSHLauncherConfigAction.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/config/SSHLauncherConfigAction.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/config/SSHLauncherConfigAction.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -375,7 +375,7 @@ /** * Clear specified directory. * - * @param session opened session + * @param path path to clear */ protected void clearFolder(String path) { Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/script/ScriptAction.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/script/ScriptAction.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/script/ScriptAction.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -508,9 +508,7 @@ } /** - * Save script, and disply commit UI. - * - * @return + * Save script, and display commit UI. */ public void commitScript() { Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/script/model/ScriptTreeModel.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/script/model/ScriptTreeModel.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/script/model/ScriptTreeModel.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -183,7 +183,7 @@ /** * Filter input file array and return only sorted collection with only directories. * - * @param files file array + * @param filesArray file array * @return sorted list */ protected List<File> getVersionnableSortedFiles(File[] filesArray) { Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/EditorHelper.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/EditorHelper.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/EditorHelper.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -292,7 +292,7 @@ * * @param component component * @param entity entity - * @return + * @return bound editor */ protected static JTextField getNewBoundEditor(JComponent component, TopiaEntity entity) { String fieldName = getMethod(component); @@ -379,6 +379,10 @@ public static JComponent getEditorWithValue(JComponent component, Object value) { JComponent result = null; + if (log.isDebugEnabled()) { + log.debug("Get editor for " + component + " with value = " + value); + } + try { result = component.getClass().newInstance(); @@ -688,7 +692,7 @@ * * @param entity * @param fieldName - * @return + * @return method return value */ protected static Object getMethodValue(TopiaEntity entity, String fieldName) { Object result = null; Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/EquationContinuousPanelUI.jaxx =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/EquationContinuousPanelUI.jaxx 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/EquationContinuousPanelUI.jaxx 2009-12-07 15:46:49 UTC (rev 2805) @@ -220,7 +220,7 @@ } }); } -protected void add(){ +protected void add() { EquationContinuousDomain domain = new EquationContinuousDomain(); // fixe NPE in table model renderer domain.setCoefficient(1.0); @@ -228,7 +228,7 @@ newButton(); setTable(); } -protected void remove(){ +protected void remove() { int i = params.getSelectedRow(); if (i != -1){ datas.remove(i); Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/FactorWizardUI.jaxx =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/FactorWizardUI.jaxx 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/sensitivity/FactorWizardUI.jaxx 2009-12-07 15:46:49 UTC (rev 2805) @@ -26,7 +26,7 @@ <Boolean id='existingValue' javaBean='false'/> - <script><![CDATA[ + <script><![CDATA[ import org.apache.commons.lang.StringUtils; import org.nuiton.topia.persistence.TopiaEntity; import fr.ifremer.isisfish.ui.simulator.SimulAction; @@ -228,7 +228,7 @@ </cell> </row> <row> - <cell fill='both' columns='4' weightx='1' weighty='0.5'> + <cell columns='4' fill='both' weightx='1' weighty='0.5'> <JPanel id='continuousPanelContainer' layout='{new BorderLayout()}'/> </cell> </row> Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/widget/editor/ParameterTableCellEditor.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/widget/editor/ParameterTableCellEditor.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/ui/widget/editor/ParameterTableCellEditor.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -224,12 +224,12 @@ /** * Get type for value. * - * Must be overriden (value can be null). + * Must be overridden (value can be null). * * @param value * @param row * @param column - * @return + * @return type for value */ protected Class<?> getType(Object value, int row, int column) { // default to value's class Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/util/ssh/SSHUtils.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/util/ssh/SSHUtils.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/util/ssh/SSHUtils.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -508,8 +508,9 @@ * Open sftp channel. * * @param session - * @return + * @return sftp channel * @throws JSchException + * @see ChannelSftp */ protected static ChannelSftp openSftpChannel(Session session) throws JSchException { ChannelSftp channel = (ChannelSftp) session.openChannel("sftp"); Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/util/ssh/package-info.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/util/ssh/package-info.java 2009-12-07 15:46:13 UTC (rev 2804) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/util/ssh/package-info.java 2009-12-07 15:46:49 UTC (rev 2805) @@ -1,5 +1,5 @@ /** - * Utility classes for {@link SSHSimulatorLauncher}. + * Utility classes for {@link fr.ifremer.isisfish.simulator.launcher.SSHSimulatorLauncher}. * * Contains : * - SSH utility code