Author: tchemit Date: 2013-02-10 16:01:31 +0100 (Sun, 10 Feb 2013) New Revision: 384 Url: http://forge.codelutin.com/projects/tutti/repository/revisions/384 Log: improve action api and logs Modified: trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/AbstractTuttiAction.java trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/cruise/SaveCruiseAction.java trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/DeleteProtocolAction.java trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/ExportProtocolAction.java trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/ImportProtocolAction.java trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/program/SaveProgramAction.java trunk/tutti-ui-swing/src/main/resources/i18n/tutti-ui-swing_fr_FR.properties Modified: trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/AbstractTuttiAction.java =================================================================== --- trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/AbstractTuttiAction.java 2013-02-10 15:00:32 UTC (rev 383) +++ trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/AbstractTuttiAction.java 2013-02-10 15:01:31 UTC (rev 384) @@ -27,6 +27,8 @@ import com.google.common.base.Preconditions; import com.google.common.collect.Sets; import fr.ifremer.tutti.service.DecoratorService; +import fr.ifremer.tutti.service.TuttiService; +import fr.ifremer.tutti.ui.swing.config.TuttiApplicationConfig; import jaxx.runtime.SwingUtil; import jaxx.runtime.swing.ErrorDialogUI; import org.apache.commons.logging.Log; @@ -119,7 +121,7 @@ public final void actionPerformed(final ActionEvent event) { if (log.isInfoEnabled()) { - log.info("Will starts action: " + this); + log.info("Task [" + this + "] starting"); } boolean doAction = prepareAction(event); @@ -150,6 +152,8 @@ runAction(false, event); } + } else { + releaseAction(event); } } @@ -170,14 +174,14 @@ this.doAction(event); } catch (Throwable e) { if (log.isErrorEnabled()) { - log.error("Error:" + e.getMessage(), e); + log.error("Task [" + this + "] error: " + e.getMessage(), e); } if (e instanceof Exception) { ErrorDialogUI.showError((Exception) e); } } finally { if (log.isInfoEnabled()) { - log.info("After action: " + this); + log.info("Task [" + this + "] done"); } if (timer != null) { @@ -208,6 +212,14 @@ return handler.getModel(); } + protected <S extends TuttiService> S getService(Class<S> serviceType) { + return getContext().getService(serviceType); + } + + protected TuttiApplicationConfig getConfig() { + return getContext().getConfig(); + } + protected void setMnemonic(int key) { putValue(MNEMONIC_KEY, key); } @@ -243,8 +255,8 @@ if (canceled) { - if (log.isInfoEnabled()) { - log.info("Task [" + action + "] was already canceled, do nothing"); + if (log.isDebugEnabled()) { + log.debug("Task [" + action + "] was already canceled, do nothing"); } } else { @@ -275,8 +287,8 @@ d.pack(); SwingUtil.center(ui, d); - if (log.isInfoEnabled()) { - log.info("Try to open dialog (was canceled ? " + canceled + ")"); + if (log.isDebugEnabled()) { + log.debug("Try to open dialog (was canceled ? " + canceled + ")"); } if (!canceled) { @@ -292,8 +304,8 @@ public boolean cancel() { synchronized (lock) { canceled = true; - if (log.isInfoEnabled()) { - log.info("Task [" + action + "] canceled."); + if (log.isDebugEnabled()) { + log.debug("Task [" + action + "] canceled."); } boolean cancel = super.cancel(); Modified: trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/cruise/SaveCruiseAction.java =================================================================== --- trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/cruise/SaveCruiseAction.java 2013-02-10 15:00:32 UTC (rev 383) +++ trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/cruise/SaveCruiseAction.java 2013-02-10 15:01:31 UTC (rev 384) @@ -66,7 +66,7 @@ TuttiUIContext context = getContext(); EditCruiseUIModel model = getModel(); PersistenceService persistenceService = - context.getService(PersistenceService.class); + getService(PersistenceService.class); Cruise bean = model.toBean(); Modified: trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/DeleteProtocolAction.java =================================================================== --- trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/DeleteProtocolAction.java 2013-02-10 15:00:32 UTC (rev 383) +++ trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/DeleteProtocolAction.java 2013-02-10 15:01:31 UTC (rev 384) @@ -86,7 +86,7 @@ log.info("Delete protocol: " + id); } - PersistenceService service = getContext().getService(PersistenceService.class); + PersistenceService service = getService(PersistenceService.class); service.deleteProtocol(id); getModel().setProtocol(null); Modified: trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/ExportProtocolAction.java =================================================================== --- trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/ExportProtocolAction.java 2013-02-10 15:00:32 UTC (rev 383) +++ trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/ExportProtocolAction.java 2013-02-10 15:01:31 UTC (rev 384) @@ -107,7 +107,7 @@ } // export protocol TuttiProtocolImportExportService service = - getContext().getService(TuttiProtocolImportExportService.class); + getService(TuttiProtocolImportExportService.class); service.exportProtocol(protocol, file); sendMessage("Protocole [" + protocol.getName() + Modified: trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/ImportProtocolAction.java =================================================================== --- trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/ImportProtocolAction.java 2013-02-10 15:00:32 UTC (rev 383) +++ trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/home/ImportProtocolAction.java 2013-02-10 15:01:31 UTC (rev 384) @@ -102,7 +102,7 @@ // import protocol TuttiProtocolImportExportService service = - getContext().getService(TuttiProtocolImportExportService.class); + getService(TuttiProtocolImportExportService.class); TuttiProtocol protocol = service.importProtocol(file); sendMessage("Protocole [" + protocol.getName() + "] lu depuis le fichier."); Modified: trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/program/SaveProgramAction.java =================================================================== --- trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/program/SaveProgramAction.java 2013-02-10 15:00:32 UTC (rev 383) +++ trunk/tutti-ui-swing/src/main/java/fr/ifremer/tutti/ui/swing/content/program/SaveProgramAction.java 2013-02-10 15:01:31 UTC (rev 384) @@ -68,7 +68,9 @@ @Override protected void doAction(ActionEvent event) { TuttiUIContext context = getContext(); - PersistenceService persistenceService = context.getService(PersistenceService.class); + PersistenceService persistenceService = + getService(PersistenceService.class); + EditProgramUIModel model = getModel(); Program bean = model.toBean(); Modified: trunk/tutti-ui-swing/src/main/resources/i18n/tutti-ui-swing_fr_FR.properties =================================================================== --- trunk/tutti-ui-swing/src/main/resources/i18n/tutti-ui-swing_fr_FR.properties 2013-02-10 15:00:32 UTC (rev 383) +++ trunk/tutti-ui-swing/src/main/resources/i18n/tutti-ui-swing_fr_FR.properties 2013-02-10 15:01:31 UTC (rev 384) @@ -3,6 +3,7 @@ tutti.action.about=À propos tutti.action.about.tip=À Propos tutti.action.addSpecies.tip=Ajouter une espèce +tutti.action.addSpeciesProtocol.tip=Ajouter une espèce au protocole tutti.action.attachments=Pièces jointes (%s) tutti.action.attachments.tip=Pièces jointes tutti.action.cancel=Annuler @@ -14,6 +15,8 @@ tutti.action.cancel.editProgram.tip=Annuler l'édition (ou la création) de la série de campagne tutti.action.cancel.editProtocol=Annuler tutti.action.cancel.editProtocol.tip=Annuler l'édition (ou la création) du protocole +tutti.action.cancelEditCatchBatch=Annuler +tutti.action.cancelEditCatchBatch.tip=Annuler l'édition de la capture tutti.action.casino-import=Import Casino tutti.action.chooseProtocolCaracteristicFile=Choisir le fichier des caractéristiques tutti.action.chooseProtocolFile=Choisir le fichier de protocole @@ -37,6 +40,7 @@ tutti.action.editCatches.tip=Saisir des captures tutti.action.editCruise=Éditer tutti.action.editCruise.tip=Éditer la campagne sélectionnée +tutti.action.editFishingOperation.tip=Éditer l'opération de pêche tutti.action.editProgram=Éditer tutti.action.editProgram.tip=Éditer le programme sélectionné tutti.action.editProtocol=Éditer @@ -103,12 +107,16 @@ tutti.action.removeSpecies=Supprimer l'espèce tutti.action.removeSpeciesBatch=Supprimer le lot tutti.action.removeSpeciesBatch.tip=Supprimer le lot courant (celui de la ligne sélectionnée) et tous ces fils +tutti.action.removeSpeciesProtocol=Supprimer l'espèce +tutti.action.removeSpeciesProtocol.tip=Supprimer l'espèce tutti.action.removeSpeciesSubBatch=Supprimer les lots fils tutti.action.removeSpeciesSubBatch.tip=Supprimer tous les lots fils du lot sélectionné tutti.action.renameSpeciesBatch=Corriger l'espèce tutti.action.renameSpeciesBatch.tip=Corriger l'espèce sur tout le lot sélectionné tutti.action.reset.fishingOperationValidState=Réinitialiser tutti.action.save=Enregistrer +tutti.action.saveCatchBatch=Sauver +tutti.action.saveCatchBatch.tip=Sauver la capture tutti.action.saveCruise=Enregistrer tutti.action.saveCruise.tip=Enregistrer la campagne tutti.action.saveFishingOperation=Enregistrer @@ -139,7 +147,9 @@ tutti.config.category.shortcuts.description=Liste des raccourcis clavier tutti.config.category.ui=Interface utilisateur tutti.config.category.ui.description=Options de l'interface utilisateur +tutti.dialog.askSaveBeforeLeaving.createFishingOperation=Vous avez fait des modifications qu vous n'avez pas enregistrées. Voulez-vous les enregistrer ? tutti.dialog.askSaveBeforeLeaving.message=Vous avez fait des modifications qu vous n'avez pas enregistrées. Voulez-vous les enregistrer ? +tutti.dialog.askSaveBeforeLeaving.saveFishingOperation=Vous avez fait des modifications qu vous n'avez pas enregistrées. Voulez-vous les enregistrer ? tutti.dialog.askSaveBeforeLeaving.title=Modifications non enregistrées tutti.dialog.catches.species.computeWeight.error.message=Erreur lors de l'élévation des poids tutti.dialog.catches.species.computeWeight.error.title=Erreur @@ -148,6 +158,13 @@ tutti.duration.format=dj Hh m'm' tutti.file.csv=Extension d'un fichier csv tutti.file.protocol=Extension d'un fichier de protocole Tutti +tutti.flash.information.all.caractristic.exported=Toutes les caractéristiques exportées dans le fichier %s. +tutti.flash.information.caracteristic.imported.in.protocol=Caractéristiques importées dans le protocole depuis le fichier %s. +tutti.flash.information.caractristic.exported.from.protocol=Caractéristiques du protocole exportées dans le fichier %s. +tutti.flash.information.species.add.to.protocol=L'espèce %s a été ajoutée au protocole +tutti.flash.information.species.exported.from.protocol=Espèces du protocole exportées dans le fichier %s. +tutti.flash.information.species.imported.in.protocol=Espèces importées dans le protocole depuis le fichier %s. +tutti.flash.information.species.remove.from.protocol=L'espèce %s a été retirée du protocole tutti.information.import.temporary.referential=Vous pouvez ici importer des référentiels temporaires tutti.label.attachmentEditor.file=Fichier tutti.label.attachmentEditor.fileComment=Commentaire @@ -374,8 +391,8 @@ tutti.table.protocol.species.header.sex.tip=Catégorisation sur le sexe ? tutti.table.protocol.species.header.size=Catégorie Tri tutti.table.protocol.species.header.size.tip=Catégorisation sur la catégorie de tri ? -tutti.table.protocol.species.header.speciesId=Espèce -tutti.table.protocol.species.header.speciesId.tip=Espèce (code permanent - nom scientifique) +tutti.table.protocol.species.header.speciesReferenceTaxonId=Espèce +tutti.table.protocol.species.header.speciesReferenceTaxonId.tip=Espèce (code refTax - nom scientifique) tutti.table.protocol.species.header.speciesSurveyCode=Code tutti.table.protocol.species.header.speciesSurveyCode.tip=Code campagne (non synchronisable) tutti.table.protocol.species.header.weight=Pesée @@ -419,6 +436,7 @@ tutti.title.create.cruise=Créer une nouvelle campagne tutti.title.create.program=Créer une nouvelle série de campagne tutti.title.create.protocol=Créer un nouveau protocol de saisie +tutti.title.createBatch=Créer un lot tutti.title.edit.cruise=Éditer une campagne existante tutti.title.edit.operations=Saisie des opérations de pêches (%s) tutti.title.edit.program=Éditer une série de campagne existante