October 2018 - Observe-commits - mailman.cloud.codelutin.com

[Git][ultreiaio/ird-observe][develop-7.x] 6 commits: Petit renommage de libellé - Closes #1099
by Tony CHEMIT 01 Oct '18

01 Oct '18

Tony CHEMIT pushed to branch develop-7.x at ultreiaio / ird-observe Commits: b30b6807 by Tony CHEMIT at 2018-10-01T13:23:36Z Petit renommage de libellé - Closes #1099 - - - - - 2be32d42 by Tony CHEMIT at 2018-10-01T13:24:53Z Fix some memory leaks (some content were not closed properly - - - - - d097f87a by Tony CHEMIT at 2018-10-01T13:28:26Z Extension de fichier du rapport de validation - Closes #1102 - - - - - d09a03ea by Tony CHEMIT at 2018-10-01T13:31:29Z up to version 7.0.0-SNAPSHOT - - - - - 6490f483 by Tony CHEMIT at 2018-10-01T13:37:43Z Widget température °C/°F : message de validation perfectible - Closes #960 (il restait une validation de température au niveau des routes) - - - - - 3ef955bb by Tony CHEMIT at 2018-10-01T14:13:46Z Le calcul des vitesses a eu un raté après avoir fait usage de la fonction "Voulez-vous créer l'activité de fin de veille ?" - See #1100 - - - - - 26 changed files: - client-configuration/pom.xml - client/pom.xml - client/src/main/java/fr/ird/observe/client/ui/admin/validate/ValidateModel.java - client/src/main/java/fr/ird/observe/client/ui/content/ContentUIHandler.java - client/src/main/java/fr/ird/observe/client/ui/content/data/longline/CatchLonglineUIHandler.java - client/src/main/java/fr/ird/observe/client/ui/content/data/longline/GearUseFeaturesLonglineUIHandler.java - client/src/main/java/fr/ird/observe/client/ui/content/data/longline/SetLonglineUIHandler.java - client/src/main/java/fr/ird/observe/client/ui/content/data/longline/TdrUIHandler.java - client/src/main/java/fr/ird/observe/client/ui/content/data/seine/GearUseFeaturesSeineUIHandler.java - client/src/main/java/fr/ird/observe/client/ui/tree/navigation/NavigationTree.java - client/src/main/resources/i18n/client_en_GB.properties - client/src/main/resources/i18n/client_es_ES.properties - client/src/main/resources/i18n/client_fr_FR.properties - dto/pom.xml - observe/pom.xml - persistence/pom.xml - pom.xml - server-configuration/pom.xml - server/pom.xml - services-client/pom.xml - services-local/pom.xml - services/pom.xml - services/src/main/java/fr/ird/observe/services/validation/validators/ActivitySimpleSpeedDtoValidator.java - test/pom.xml - validation/pom.xml - validation/src/main/resources/fr/ird/observe/dto/data/seine/RouteDto-update-error-validation.xml Changes: ===================================== client-configuration/pom.xml ===================================== @@ -26,7 +26,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>client-configuration</artifactId> ===================================== client/pom.xml ===================================== @@ -26,7 +26,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>client</artifactId> ===================================== client/src/main/java/fr/ird/observe/client/ui/admin/validate/ValidateModel.java ===================================== @@ -41,8 +41,8 @@ import fr.ird.observe.services.service.actions.validate.ValidationResultDto; import fr.ird.observe.services.service.actions.validate.ValidationResultDtoMessage; import fr.ird.observe.services.validation.ValidatorDto; import org.apache.commons.lang3.mutable.MutableInt; -import org.apache.logging.log4j.Logger; import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.nuiton.jaxx.runtime.swing.wizard.ext.WizardState; import org.nuiton.validator.NuitonValidatorScope; @@ -75,7 +75,7 @@ public class ValidateModel extends AdminActionModel { static final String PROPERTY_VALIDATION_RESULT = "validationResult"; /** le pattern du fichier de rapport après validation */ - private static final String REPORT_PATTERN = "report-%1$tF--%1$tk-%1$tM-%1$tS.txt"; + private static final String REPORT_PATTERN = "report-%1$tF--%1$tk-%1$tM-%1$tS.html"; /** Logger */ private static final Logger log = LogManager.getLogger(ValidateModel.class); ===================================== client/src/main/java/fr/ird/observe/client/ui/content/ContentUIHandler.java ===================================== @@ -791,11 +791,11 @@ public abstract class ContentUIHandler<E extends IdDto, U extends ContentUI<E, U SwingValidatorUtil.setValidatorChanged(ui, false); } - protected void closeSafeUI() { + public void closeSafeUI() { removeAllMessages(ui); ContentUIModel<E> model = getModel(); - + getUi().getErrorTableModel().clearValidators(); boolean create = model.isCreatingMode(); if (create && model.isEditable()) { // remove node ===================================== client/src/main/java/fr/ird/observe/client/ui/content/data/longline/CatchLonglineUIHandler.java ===================================== @@ -62,8 +62,8 @@ import fr.ird.observe.dto.referential.longline.HealthnessReference; import fr.ird.observe.dto.result.SaveResultDto; import fr.ird.observe.spi.DtoModelHelper; import org.apache.commons.lang3.BooleanUtils; -import org.apache.logging.log4j.Logger; import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.nuiton.jaxx.runtime.context.JAXXContextEntryDef; import org.nuiton.jaxx.runtime.spi.UIHandler; import org.nuiton.jaxx.validator.swing.SwingValidator; @@ -72,7 +72,10 @@ import org.nuiton.jaxx.validator.swing.SwingValidatorMessageTableModel; import org.nuiton.util.DateUtil; import org.nuiton.validator.NuitonValidatorScope; -import javax.swing.*; +import javax.swing.JComponent; +import javax.swing.JOptionPane; +import javax.swing.JTable; +import javax.swing.SwingUtilities; import javax.swing.event.TableModelListener; import javax.swing.table.DefaultTableCellRenderer; import java.awt.Component; @@ -709,7 +712,7 @@ public class CatchLonglineUIHandler extends ContentTableUIHandler<SetLonglineCat } @Override - protected void closeSafeUI() { + public void closeSafeUI() { if (log.isInfoEnabled()) { log.info("CloseUI: " + getModel()); ===================================== client/src/main/java/fr/ird/observe/client/ui/content/data/longline/GearUseFeaturesLonglineUIHandler.java ===================================== @@ -49,8 +49,8 @@ import fr.ird.observe.dto.referential.GearReference; import fr.ird.observe.dto.result.SaveResultDto; import fr.ird.observe.services.service.referential.ReferentialService; import org.apache.commons.lang3.StringUtils; -import org.apache.logging.log4j.Logger; import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.nuiton.jaxx.runtime.spi.UIHandler; import org.nuiton.jaxx.runtime.swing.JTables; import org.nuiton.jaxx.validator.swing.SwingValidatorMessage; @@ -417,7 +417,7 @@ public class GearUseFeaturesLonglineUIHandler extends ContentTableUIHandler<Trip } @Override - protected void closeSafeUI() { + public void closeSafeUI() { if (log.isInfoEnabled()) { log.info("CloseUI: " + getModel()); ===================================== client/src/main/java/fr/ird/observe/client/ui/content/data/longline/SetLonglineUIHandler.java ===================================== @@ -258,7 +258,7 @@ public class SetLonglineUIHandler extends ContentUIHandler<SetLonglineDto, SetLo } @Override - protected void closeSafeUI() { + public void closeSafeUI() { if (log.isInfoEnabled()) { log.info("CloseUI: " + getModel()); ===================================== client/src/main/java/fr/ird/observe/client/ui/content/data/longline/TdrUIHandler.java ===================================== @@ -41,8 +41,8 @@ import fr.ird.observe.dto.referential.SpeciesDto; import fr.ird.observe.dto.referential.SpeciesReference; import fr.ird.observe.dto.referential.longline.SensorBrandReference; import fr.ird.observe.dto.result.SaveResultDto; -import org.apache.logging.log4j.Logger; import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.nuiton.jaxx.runtime.context.JAXXContextEntryDef; import org.nuiton.jaxx.runtime.spi.UIHandler; import org.nuiton.jaxx.validator.swing.SwingValidatorMessage; @@ -233,7 +233,7 @@ public class TdrUIHandler extends ContentTableUIHandler<SetLonglineTdrDto, TdrDt } @Override - protected void closeSafeUI() { + public void closeSafeUI() { if (log.isInfoEnabled()) { log.info("CloseUI: " + getModel()); } ===================================== client/src/main/java/fr/ird/observe/client/ui/content/data/seine/GearUseFeaturesSeineUIHandler.java ===================================== @@ -49,8 +49,8 @@ import fr.ird.observe.dto.referential.GearReference; import fr.ird.observe.dto.result.SaveResultDto; import fr.ird.observe.services.service.referential.ReferentialService; import org.apache.commons.lang3.StringUtils; -import org.apache.logging.log4j.Logger; import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.nuiton.jaxx.runtime.spi.UIHandler; import org.nuiton.jaxx.runtime.swing.JTables; import org.nuiton.jaxx.validator.swing.SwingValidatorMessage; @@ -417,7 +417,7 @@ public class GearUseFeaturesSeineUIHandler extends ContentTableUIHandler<TripSei } @Override - protected void closeSafeUI() { + public void closeSafeUI() { if (log.isInfoEnabled()) { log.info("CloseUI: " + getModel()); ===================================== client/src/main/java/fr/ird/observe/client/ui/tree/navigation/NavigationTree.java ===================================== @@ -25,6 +25,7 @@ package fr.ird.observe.client.ui.tree.navigation; import fr.ird.observe.client.ObserveSwingApplicationContext; import fr.ird.observe.client.db.ClientDataContext; import fr.ird.observe.client.db.ObserveSwingDataSource; +import fr.ird.observe.client.ui.content.ContentUI; import fr.ird.observe.client.ui.tree.navigation.nodes.NavigationTreeNodeSupport; import fr.ird.observe.client.ui.tree.navigation.nodes.ReferenceNavigationTreeNodeSupport; import fr.ird.observe.client.ui.tree.navigation.nodes.RootNavigationTreeNode; @@ -96,14 +97,42 @@ public class NavigationTree extends JXTree { public NavigationTree() { super(new NavigationTreeModel()); setSelectionModel(new DefaultTreeSelectionModel() { +// @Override +// public void setSelectionPath(TreePath path) { +// boolean canChange = skipCheckPreviousContent || (!Objects.equals(path, getSelectionPath()) && ObserveSwingApplicationContext.get().getContentUIManager().closeSelectedContentUI()); +// if (!canChange) { +// // cancel the change of node +// log.warn(String.format("Do not change selection path: %s against: %s", path, getSelectionPath())); +// return; +// } +// if (skipCheckPreviousContent) { +// ContentUI selectedContentUI = ObserveSwingApplicationContext.get().getContentUIManager().getSelectedContentUI(); +// if (selectedContentUI != null) { +// selectedContentUI.getHandler().closeSafeUI(); +// } +// } +// super.setSelectionPath(path); +// } + @Override - public void setSelectionPath(TreePath path) { + public void setSelectionPaths(TreePath[] pPaths) { + if (pPaths.length > 1) { + return; + } + TreePath path = pPaths[0]; boolean canChange = skipCheckPreviousContent || (!Objects.equals(path, getSelectionPath()) && ObserveSwingApplicationContext.get().getContentUIManager().closeSelectedContentUI()); if (!canChange) { // cancel the change of node + log.warn(String.format("Do not change selection path: %s against: %s", path, getSelectionPath())); return; } - super.setSelectionPath(path); + if (skipCheckPreviousContent) { + ContentUI selectedContentUI = ObserveSwingApplicationContext.get().getContentUIManager().getSelectedContentUI(); + if (selectedContentUI != null) { + selectedContentUI.getHandler().closeSafeUI(); + } + } + super.setSelectionPaths(pPaths); } }); setLargeModel(true); ===================================== client/src/main/resources/i18n/client_en_GB.properties ===================================== @@ -1508,7 +1508,7 @@ observe.ui.tree.seine.nonTargetCatch=Non target catch observe.ui.tree.seine.nonTargetCatchRelease=Non target released catches observe.ui.tree.seine.nonTargetSample=Non target samples observe.ui.tree.seine.objectObservedSpecies=Object observed species -observe.ui.tree.seine.objectOperation=Object operations +observe.ui.tree.seine.objectOperation=Beacon observe.ui.tree.seine.objectSchoolEstimate=Object school estimates observe.ui.tree.seine.routes=Routes observe.ui.tree.seine.schoolEstimate=School estimates ===================================== client/src/main/resources/i18n/client_es_ES.properties ===================================== @@ -1508,7 +1508,7 @@ observe.ui.tree.seine.nonTargetCatch=Fauna accesoria conservada o descartada observe.ui.tree.seine.nonTargetCatchRelease=Faunes accessoires libérées \#TODO observe.ui.tree.seine.nonTargetSample=Muestreos de fauna accesoria observe.ui.tree.seine.objectObservedSpecies=Fauna observada -observe.ui.tree.seine.objectOperation=Operaciones sobre el objeto +observe.ui.tree.seine.objectOperation=Baliza observe.ui.tree.seine.objectSchoolEstimate=Estimaciones banco de objetos observe.ui.tree.seine.routes=Rutas observe.ui.tree.seine.schoolEstimate=Estimaciones banco ===================================== client/src/main/resources/i18n/client_fr_FR.properties ===================================== @@ -1508,7 +1508,7 @@ observe.ui.tree.seine.nonTargetCatch=Espèces accessoires conservées ou rejeté observe.ui.tree.seine.nonTargetCatchRelease=Espèces accessoires libérées observe.ui.tree.seine.nonTargetSample=Échantillons espèces accessoires observe.ui.tree.seine.objectObservedSpecies=Faune observée -observe.ui.tree.seine.objectOperation=Opérations sur l'objet +observe.ui.tree.seine.objectOperation=Balise observe.ui.tree.seine.objectSchoolEstimate=Estimations banc objet observe.ui.tree.seine.routes=Routes observe.ui.tree.seine.schoolEstimate=Estimation banc ===================================== dto/pom.xml ===================================== @@ -27,7 +27,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>dto</artifactId> ===================================== observe/pom.xml ===================================== @@ -26,7 +26,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>observe</artifactId> ===================================== persistence/pom.xml ===================================== @@ -26,7 +26,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>persistence</artifactId> ===================================== pom.xml ===================================== @@ -31,7 +31,7 @@ <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> <packaging>pom</packaging> <name>ObServe :: Pom</name> ===================================== server-configuration/pom.xml ===================================== @@ -26,7 +26,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>server-configuration</artifactId> ===================================== server/pom.xml ===================================== @@ -27,7 +27,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>server</artifactId> ===================================== services-client/pom.xml ===================================== @@ -27,7 +27,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>services-client</artifactId> ===================================== services-local/pom.xml ===================================== @@ -27,7 +27,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>services-local</artifactId> ===================================== services/pom.xml ===================================== @@ -27,7 +27,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>services</artifactId> ===================================== services/src/main/java/fr/ird/observe/services/validation/validators/ActivitySimpleSpeedDtoValidator.java ===================================== @@ -29,9 +29,12 @@ import fr.ird.observe.dto.data.seine.ActivitySeineDto; import fr.ird.observe.dto.data.seine.ActivitySeineStubDto; import fr.ird.observe.dto.data.seine.RouteDto; import fr.ird.observe.dto.decoration.DecoratorService; -import org.apache.logging.log4j.Logger; import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; +import java.util.Comparator; +import java.util.LinkedList; +import java.util.List; import java.util.Objects; /** @@ -146,8 +149,24 @@ public class ActivitySimpleSpeedDtoValidator extends FieldValidatorSupport { Objects.requireNonNull(route, " could not find currentRoute in valueStack"); - ActivitySeineStubDto previousActivity = route.getPreviousActivity(activity.getId()); - + ActivitySeineStubDto previousActivity; + + if (activity.isNotPersisted()) { + // since there is no id, can't use the later method, must find out by our self previous activity + List<ActivitySeineStubDto> activitySeine = new LinkedList<>(route.getActivitySeine()); + ActivitySeineStubDto activityStub = new ActivitySeineStubDto(); + activityStub.setTime(activity.getTime()); + activitySeine.add(activityStub); + activitySeine.sort(Comparator.comparing(ActivitySeineStubDto::getTimeSecond)); + int index = activitySeine.indexOf(activityStub); + if (index > 0) { + previousActivity = activitySeine.get(index - 1); + } else { + previousActivity = null; + } + } else { + previousActivity = route.getPreviousActivity(activity.getId()); + } if (previousActivity == null) { // pas d'activity avant, rien à valider ===================================== test/pom.xml ===================================== @@ -26,7 +26,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>test</artifactId> ===================================== validation/pom.xml ===================================== @@ -26,7 +26,7 @@ <parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> - <version>7.0-RC-23-SNAPSHOT</version> + <version>7.0.0-SNAPSHOT</version> </parent> <artifactId>validation</artifactId> ===================================== validation/src/main/resources/fr/ird/observe/dto/data/seine/RouteDto-update-error-validation.xml ===================================== @@ -66,21 +66,6 @@ <message>observe.validation.route.invalid.time##${index}</message> </field-validator> -  - <field-validator type="collectionFieldExpression"> - <param name="mode">ALL</param> - <param name="useSensitiveContext">true</param> - <param name="expression"><![CDATA[ - previous == null || previous.seaSurfaceTemperature == null || current.seaSurfaceTemperature == null - || (previous.seaSurfaceTemperature > current.seaSurfaceTemperature ? - previous.seaSurfaceTemperature - current.seaSurfaceTemperature <= 12.0 - : current.seaSurfaceTemperature - previous.seaSurfaceTemperature <= 12.0 - ) - ]]> - </param> - <message>observe.validation.route.invalid.seaSurfaceTemperature##${index}</message> - </field-validator> -  <field-validator type="collectionFieldExpression"> <param name="mode">ALL</param> View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/compare/1ca1414b0a5f060e0c11aa959a… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/compare/1ca1414b0a5f060e0c11aa959a… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop] 4 commits: Big big refactor around actions in handler, introduce new handler for any case.
by Tony CHEMIT 01 Oct '18

01 Oct '18

Tony CHEMIT pushed to branch develop at ultreiaio / ird-observe Commits: dd220ec5 by Tony CHEMIT at 2018-10-01T11:23:15Z Big big refactor around actions in handler, introduce new handler for any case. Now UI are no more responsible of any actions except of loading - start and stop editing :) Yes now we have more classes, but no more fear about whre does goes our code... - - - - - 18dec528 by Tony CHEMIT at 2018-10-01T12:37:21Z update pom - - - - - 96138fd7 by Tony CHEMIT at 2018-10-01T12:40:02Z [RAPPORTS] Les 4 requêtes sur espèces cibles ne fonctionnent pas - Closes #1101 - - - - - 7e6fa47b by Tony CHEMIT at 2018-10-01T12:42:50Z update to geotools 20.0 - - - - - 30 changed files: - client/pom.xml - client/src/main/java/fr/ird/observe/client/ObserveSwingApplicationContext.java - client/src/main/java/fr/ird/observe/client/db/ObserveSwingDataSource.java - client/src/main/java/fr/ird/observe/client/navigation/RouteCloseCallback.java - client/src/main/java/fr/ird/observe/client/ui/ObserveKeyStrokes.java - client/src/main/java/fr/ird/observe/client/ui/ObserveMainUI.jaxx - client/src/main/java/fr/ird/observe/client/ui/ObserveMainUIInitializer.java - client/src/main/java/fr/ird/observe/client/ui/actions/UIActionSupport.java - − client/src/main/java/fr/ird/observe/client/ui/actions/content/CloseAndCreateUIAction.java - − client/src/main/java/fr/ird/observe/client/ui/actions/content/DeleteDataUIAction.java - − client/src/main/java/fr/ird/observe/client/ui/actions/content/DeleteReferenceUIAction.java - − client/src/main/java/fr/ird/observe/client/ui/actions/content/SaveEditUIAction.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/AbstractContentUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/AbstractContentUIAction.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/ResetEditUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/ResetEditUIAction.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/ShowTechnicalInformationsAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/ShowTechnicalInformationsAction.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/edit/DeleteEditDataUIActionSupport.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/edit/SaveEditDataUIActionSupport.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/list/CloseOpenDataFromListUIAction.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/list/CreateOpenDataFromListUIAction.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/move/MoveMultipleDataUIActionSupport.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/list/MoveMultipleDataUIActionSupport.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/move/MoveMultipleTripUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/list/MoveMultipleTripUIActionSupport.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/ReOpenUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/list/OpenOpenDataFromListUIAction.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/CloseOpenUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/CloseOpenDataUIAction.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/CreateOpenDataUIAction.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/DeleteOpenDataUIActionSupport.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/move/MoveSingleDataUIActionSupport.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/MoveSingleDataUIActionSupport.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/move/MoveSingleTripUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/MoveSingleTripUIActionSupport.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/OpenOpenDataUIAction.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/SaveOpenDataUIActionSupport.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/simple/SaveSimpleDataUIActionSupport.java The diff was not included because it is too large. View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/compare/59d01fca3942724d14eaebb6f4… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/compare/59d01fca3942724d14eaebb6f4… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop-7.x] 4 commits: can not select tab if not enabled
by Tony CHEMIT 01 Oct '18

01 Oct '18

Tony CHEMIT pushed to branch develop-7.x at ultreiaio / ird-observe Commits: e356f4d8 by Tony CHEMIT at 2018-10-01T11:38:49Z can not select tab if not enabled - - - - - 56d4e718 by Tony CHEMIT at 2018-10-01T12:25:21Z [RAPPORTS] Les 4 requêtes sur espèces cibles ne fonctionnent pas - Closes #1101 - - - - - e88f041e by Tony CHEMIT at 2018-10-01T12:34:37Z update to geotools 20.0 - - - - - 1ca1414b by Tony CHEMIT at 2018-10-01T12:34:43Z update pom - - - - - 7 changed files: - client/pom.xml - client/src/main/java/fr/ird/observe/client/ui/actions/content/GoToTabUIActionSupport.java - client/src/main/java/fr/ird/observe/client/ui/util/tripMap/ObserveMapPaneLegendItem.java - client/src/main/java/fr/ird/observe/client/ui/util/tripMap/ObserverMapPanLegendDrawer.java - client/src/main/java/fr/ird/observe/client/ui/util/tripMap/TripMapContentBuilder.java - pom.xml - services/src/main/resources/observe-reports.properties Changes: ===================================== client/pom.xml ===================================== @@ -240,6 +240,14 @@ <groupId>it.geosolutions.jaiext.utilities</groupId> <artifactId>jt-utilities</artifactId> </exclusion> + <exclusion> + <groupId>it.geosolutions.jaiext.shadedrelief</groupId> + <artifactId>jt-shadedrelief</artifactId> + </exclusion> + <exclusion> + <groupId>it.geosolutions.jaiext.scale2</groupId> + <artifactId>jt-scale2</artifactId> + </exclusion> <exclusion> <groupId>org.jaitools</groupId> <artifactId>jt-zonalstats</artifactId> @@ -289,6 +297,11 @@ <artifactId>gt-epsg-hsql</artifactId> <version>${lib.version.geoTools}</version> </dependency> + <dependency> + <groupId>org.locationtech.jts</groupId> + <artifactId>jts-core</artifactId> + <version>1.16.0</version> + </dependency> </dependencies> </dependencyManagement> @@ -598,7 +611,7 @@ </dependency> <dependency> - <groupId>com.vividsolutions</groupId> + <groupId>org.locationtech.jts</groupId> <artifactId>jts-core</artifactId> </dependency> @@ -643,7 +656,6 @@ <dependency> <groupId>org.jetbrains</groupId> <artifactId>annotations</artifactId> - <version>16.0.1</version> </dependency> </dependencies> ===================================== client/src/main/java/fr/ird/observe/client/ui/actions/content/GoToTabUIActionSupport.java ===================================== @@ -51,12 +51,9 @@ public abstract class GoToTabUIActionSupport extends AbstractContentUIAction { @Override protected void actionPerformed(ContentUI<?, ?> contentUI) { JTabbedPane mainTabbedPane = (JTabbedPane) contentUI.getObjectById(tabbedPaneName); - if (mainTabbedPane == null) { - - // rien à faire - return; + if (mainTabbedPane != null && mainTabbedPane.isEnabledAt(tabIndex)) { + mainTabbedPane.setSelectedIndex(tabIndex); } - mainTabbedPane.setSelectedIndex(tabIndex); } @Override ===================================== client/src/main/java/fr/ird/observe/client/ui/util/tripMap/ObserveMapPaneLegendItem.java ===================================== @@ -10,20 +10,20 @@ package fr.ird.observe.client.ui.util.tripMap; * it under the terms of the GNU General Public License as * published by the Free Software Foundation, either version 3 of the * License, or (at your option) any later version. - * + * * This program is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. - * + * * You should have received a copy of the GNU General Public * License along with this program. If not, see * <http://www.gnu.org/licenses/gpl-3.0.html>. * #L% */ -import com.vividsolutions.jts.geom.Coordinate; import org.geotools.styling.Style; +import org.locationtech.jts.geom.Coordinate; import org.opengis.feature.simple.SimpleFeature; import java.awt.Color; @@ -37,7 +37,15 @@ public class ObserveMapPaneLegendItem { protected static final int LEGEND_SYMBOL_WIDTH = 50; protected static final int LEGEND_MARGIN = 5; protected static final Color LEGEND_BACKGROUND = new Color(1f, 1f, 1f, 0.8f); + protected SimpleFeature simpleFeature; + protected Style style; + protected String label; + public ObserveMapPaneLegendItem(SimpleFeature simpleFeature, Style style, String label) { + this.simpleFeature = simpleFeature; + this.style = style; + this.label = label; + } public static Coordinate[] lineCoordinates() { Coordinate[] coordinates = new Coordinate[2]; @@ -50,19 +58,6 @@ public class ObserveMapPaneLegendItem { return new Coordinate(LEGEND_SYMBOL_WIDTH / 2, LEGEND_ITEM_HEIGHT / 2); } - - protected SimpleFeature simpleFeature; - - protected Style style; - - protected String label; - - public ObserveMapPaneLegendItem(SimpleFeature simpleFeature, Style style, String label) { - this.simpleFeature = simpleFeature; - this.style = style; - this.label = label; - } - public SimpleFeature getSimpleFeature() { return simpleFeature; } ===================================== client/src/main/java/fr/ird/observe/client/ui/util/tripMap/ObserverMapPanLegendDrawer.java ===================================== @@ -10,25 +10,25 @@ package fr.ird.observe.client.ui.util.tripMap; * it under the terms of the GNU General Public License as * published by the Free Software Foundation, either version 3 of the * License, or (at your option) any later version. - * + * * This program is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. - * + * * You should have received a copy of the GNU General Public * License along with this program. If not, see * <http://www.gnu.org/licenses/gpl-3.0.html>. * #L% */ -import com.vividsolutions.jts.geom.Geometry; import org.geotools.geometry.jts.LiteShape; import org.geotools.legend.Drawer; import org.geotools.styling.LineSymbolizer; import org.geotools.styling.Rule; import org.geotools.styling.SLD; import org.geotools.styling.Symbolizer; +import org.locationtech.jts.geom.Geometry; import org.opengis.feature.simple.SimpleFeature; import java.awt.BasicStroke; ===================================== client/src/main/java/fr/ird/observe/client/ui/util/tripMap/TripMapContentBuilder.java ===================================== @@ -10,12 +10,12 @@ package fr.ird.observe.client.ui.util.tripMap; * it under the terms of the GNU General Public License as * published by the Free Software Foundation, either version 3 of the * License, or (at your option) any later version. - * + * * This program is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. - * + * * You should have received a copy of the GNU General Public * License along with this program. If not, see * <http://www.gnu.org/licenses/gpl-3.0.html>. @@ -25,11 +25,6 @@ package fr.ird.observe.client.ui.util.tripMap; import com.google.common.collect.ImmutableSet; import com.google.common.collect.Lists; import com.google.common.collect.Sets; -import com.vividsolutions.jts.geom.Coordinate; -import com.vividsolutions.jts.geom.GeometryFactory; -import com.vividsolutions.jts.geom.LineString; -import com.vividsolutions.jts.geom.Point; -import com.vividsolutions.jts.geom.Polygon; import fr.ird.observe.client.ObserveSwingTechnicalException; import fr.ird.observe.dto.I18nEnumHelper; import fr.ird.observe.dto.data.TripMapPoint; @@ -56,6 +51,11 @@ import org.geotools.styling.StyleFactory; import org.geotools.styling.StyledLayer; import org.geotools.styling.StyledLayerDescriptor; import org.geotools.styling.UserLayer; +import org.locationtech.jts.geom.Coordinate; +import org.locationtech.jts.geom.GeometryFactory; +import org.locationtech.jts.geom.LineString; +import org.locationtech.jts.geom.Point; +import org.locationtech.jts.geom.Polygon; import org.opengis.feature.simple.SimpleFeature; import org.opengis.feature.simple.SimpleFeatureType; import org.opengis.referencing.FactoryException; @@ -80,7 +80,7 @@ public class TripMapContentBuilder { public static final String TRIP_LONGLINE_FISHING_ZONE_LAYER_NAME = "Trip longline zone"; public static final String TRIP_LONGLINE_LINE_LAYER_NAME = "Trip longline line"; public static final String TRIP_POINTS_LAYER_NAME = "Trip points"; - + public static final SimpleDateFormat DATE_FORMAT = new SimpleDateFormat("dd/MM"); protected static final Set<TripMapPointType> TRIP_MAP_POINT_TYPES_IN_LEGEND = ImmutableSet.of( TripMapPointType.seineActivityInHarbour, TripMapPointType.seineActivityWithFreeSchoolType, @@ -92,12 +92,9 @@ public class TripMapContentBuilder { TripMapPointType.longlineActivityWithHaulingEnd, TripMapPointType.longlineActivityWithStation, TripMapPointType.longlineActivityWithInteraction); - - public static final SimpleDateFormat DATE_FORMAT = new SimpleDateFormat("dd/MM"); - - protected StyledLayerDescriptor styledLayerDescriptor; protected final MapContent mapContent; protected final List<ObserveMapPaneLegendItem> legendItems; + protected StyledLayerDescriptor styledLayerDescriptor; public TripMapContentBuilder() { mapContent = new MapContent(); @@ -132,7 +129,7 @@ public class TripMapContentBuilder { public void addTripLine(List<TripMapPoint> tripMapPoints) throws FactoryException { - GeometryFactory geometryFactory = JTSFactoryFinder.getGeometryFactory(); + org.locationtech.jts.geom.GeometryFactory geometryFactory = JTSFactoryFinder.getGeometryFactory(); SimpleFeatureTypeBuilder lineFeatureTypeBuilder = new SimpleFeatureTypeBuilder(); lineFeatureTypeBuilder.setName("lineBuilder"); ===================================== pom.xml ===================================== @@ -26,7 +26,7 @@ <parent> <groupId>io.ultreia.maven</groupId> <artifactId>pom</artifactId> - <version>2018.58</version> + <version>2018.59</version> </parent> <groupId>fr.ird.observe</groupId> ===================================== services/src/main/resources/observe-reports.properties ===================================== @@ -728,7 +728,7 @@ Join r.activitySeine a \ Join a.setSeine.targetSample ts \ Join ts.targetLength tl \ Where t.id In :tripId \ -And tl.sizeMeasureType = 1 \ +And tl.sizeMeasureType = 'fr.ird.observe.entities.referentiel.SizeMeasureType#1433499466774#0.529249255312607' \ And ts.discarded = false report.targetStoredDistributionLengthsLD1.request.1=0,0|row| \ Select tl.length , Sum(tl.count) \ @@ -739,7 +739,7 @@ Join a.setSeine.targetSample ts \ Join ts.targetLength tl \ Where t.id In :tripId \ And tl.species.id = :speciesId \ - And tl.sizeMeasureType = 1 \ + And tl.sizeMeasureType = 'fr.ird.observe.entities.referentiel.SizeMeasureType#1433499466774#0.529249255312607' \ And ts.discarded = false \ Group By tl.length \ Order By tl.length @@ -759,7 +759,7 @@ Join r.activitySeine a \ Join a.setSeine.targetSample ts \ Join ts.targetLength tl \ Where t.id In :tripId \ -And tl.sizeMeasureType = 1 \ +And tl.sizeMeasureType = 'fr.ird.observe.entities.referentiel.SizeMeasureType#1433499466774#0.529249255312607' \ And ts.discarded = true report.targetDiscardedDistributionLengthsLD1.request.1=0,0|row| \ Select tl.length , Sum(tl.count) \ @@ -770,7 +770,7 @@ Join a.setSeine.targetSample ts \ Join ts.targetLength tl \ Where t.id In :tripId \ And tl.species.id = :speciesId \ - And tl.sizeMeasureType = 1\ + And tl.sizeMeasureType = 'fr.ird.observe.entities.referentiel.SizeMeasureType#1433499466774#0.529249255312607'\ And ts.discarded = true \ Group By tl.length \ Order By tl.length @@ -790,7 +790,7 @@ Join r.activitySeine a \ Join a.setSeine.targetSample ts \ Join ts.targetLength tl \ Where t.id In :tripId \ -And tl.sizeMeasureType = 2 \ +And tl.sizeMeasureType = 'fr.ird.observe.entities.referentiel.SizeMeasureType#1433499465700#0.0902433863375336' \ And ts.discarded = false report.targetStoredDistributionLengthsLF.request.1=0,0|row| \ Select tl.length , Sum(tl.count) \ @@ -801,7 +801,7 @@ Join a.setSeine.targetSample ts \ Join ts.targetLength tl \ Where t.id In :tripId \ And tl.species.id = :speciesId \ - And tl.sizeMeasureType = 2 \ + And tl.sizeMeasureType = 'fr.ird.observe.entities.referentiel.SizeMeasureType#1433499465700#0.0902433863375336' \ And ts.discarded = false \ Group By tl.length \ Order By tl.length @@ -821,7 +821,7 @@ Join r.activitySeine a \ Join a.setSeine.targetSample ts \ Join ts.targetLength tl \ Where t.id In :tripId \ -And tl.sizeMeasureType = 2 \ +And tl.sizeMeasureType = 'fr.ird.observe.entities.referentiel.SizeMeasureType#1433499465700#0.0902433863375336' \ And ts.discarded = true report.targetDiscardedDistributionLengthsLF.request.1=0,0|row| \ Select tl.length , Sum(tl.count) \ @@ -832,7 +832,7 @@ Join a.setSeine.targetSample ts \ Join ts.targetLength tl \ Where t.id In :tripId \ And tl.species.id = :speciesId \ - And tl.sizeMeasureType = 2 \ + And tl.sizeMeasureType = 'fr.ird.observe.entities.referentiel.SizeMeasureType#1433499465700#0.0902433863375336' \ And ts.discarded = true \ Group By tl.length \ Order By tl.length View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/compare/26c0d1842ae7c2d93bccd12478… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/compare/26c0d1842ae7c2d93bccd12478… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop] Big big refactor around actions in handler, introduce new handler for any case.
by Tony CHEMIT 01 Oct '18

01 Oct '18

Tony CHEMIT pushed to branch develop at ultreiaio / ird-observe Commits: 59d01fca by Tony CHEMIT at 2018-10-01T02:17:04Z Big big refactor around actions in handler, introduce new handler for any case. Now UI are no more responsible of any actions except of loading - start and stop editing :) Yes now we have more classes, but no more fear about whre does goes our code... - - - - - 30 changed files: - client/pom.xml - client/src/main/java/fr/ird/observe/client/ObserveSwingApplicationContext.java - client/src/main/java/fr/ird/observe/client/db/ObserveSwingDataSource.java - client/src/main/java/fr/ird/observe/client/navigation/RouteCloseCallback.java - client/src/main/java/fr/ird/observe/client/ui/ObserveKeyStrokes.java - client/src/main/java/fr/ird/observe/client/ui/ObserveMainUI.jaxx - client/src/main/java/fr/ird/observe/client/ui/ObserveMainUIInitializer.java - client/src/main/java/fr/ird/observe/client/ui/actions/UIActionSupport.java - − client/src/main/java/fr/ird/observe/client/ui/actions/content/CloseAndCreateUIAction.java - − client/src/main/java/fr/ird/observe/client/ui/actions/content/DeleteDataUIAction.java - − client/src/main/java/fr/ird/observe/client/ui/actions/content/DeleteReferenceUIAction.java - − client/src/main/java/fr/ird/observe/client/ui/actions/content/SaveEditUIAction.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/AbstractContentUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/AbstractContentUIAction.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/ResetEditUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/ResetEditUIAction.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/ShowTechnicalInformationsAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/ShowTechnicalInformationsAction.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/edit/DeleteEditDataUIActionSupport.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/edit/SaveEditDataUIActionSupport.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/list/CloseOpenDataFromListUIAction.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/list/CreateOpenDataFromListUIAction.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/move/MoveMultipleDataUIActionSupport.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/list/MoveMultipleDataUIActionSupport.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/move/MoveMultipleTripUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/list/MoveMultipleTripUIActionSupport.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/ReOpenUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/list/OpenOpenDataFromListUIAction.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/CloseOpenUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/CloseOpenDataUIAction.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/CreateOpenDataUIAction.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/DeleteOpenDataUIActionSupport.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/move/MoveSingleDataUIActionSupport.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/MoveSingleDataUIActionSupport.java - client/src/main/java/fr/ird/observe/client/ui/actions/content/move/MoveSingleTripUIAction.java → client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/MoveSingleTripUIActionSupport.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/OpenOpenDataUIAction.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/open/SaveOpenDataUIActionSupport.java - + client/src/main/java/fr/ird/observe/client/ui/actions/content/api/data/simple/SaveSimpleDataUIActionSupport.java The diff was not included because it is too large. View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/commit/59d01fca3942724d14eaebb6f4c… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/commit/59d01fca3942724d14eaebb6f4c… You're receiving this email because of your account on gitlab.com.

1 0