January 2023 - Observe-commits - mailman.cloud.codelutin.com

[Git][ultreiaio/ird-observe][release/9.0.23] [jgitflow-maven-plugin]updating poms for branch'release/9.0.23' with non-snapshot versions[skip ci]
by Tony CHEMIT (＠tchemit) 05 Jan '23

05 Jan '23

Tony CHEMIT pushed to branch release/9.0.23 at ultreiaio / ird-observe Commits: 00fa1f1f by Tony Chemit at 2023-01-05T17:34:23+01:00 [jgitflow-maven-plugin]updating poms for branch'release/9.0.23' with non-snapshot versions[skip ci] - - - - - 30 changed files: - client/configuration/pom.xml - client/core/pom.xml - client/datasource/actions/pom.xml - client/datasource/editor/api-test/pom.xml - client/datasource/editor/api/pom.xml - client/datasource/editor/common/pom.xml - client/datasource/editor/ll/pom.xml - client/datasource/editor/pom.xml - client/datasource/editor/ps/pom.xml - client/datasource/editor/spi/pom.xml - client/datasource/pom.xml - client/pom.xml - client/runner/pom.xml - core/api/dto-decoration/pom.xml - core/api/dto-validation/pom.xml - core/api/dto/pom.xml - core/api/pom.xml - core/api/services/pom.xml - core/api/tree/pom.xml - core/api/validation/pom.xml - core/persistence/decoration/pom.xml - core/persistence/java/pom.xml - core/persistence/pom.xml - core/persistence/report/pom.xml - core/persistence/resources/pom.xml - core/persistence/test/pom.xml - core/pom.xml - core/services/client/pom.xml - core/services/i18n/pom.xml - core/services/local/pom.xml The diff was not included because it is too large. View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/00fa1f1f6a02d89b439f05f45… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/00fa1f1f6a02d89b439f05f45… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][master-9.0.x] 70 commits: [jgitflow-maven-plugin]updating poms for 9.0.23-SNAPSHOT development[skip ci]
by Tony CHEMIT (＠tchemit) 05 Jan '23

05 Jan '23

Tony CHEMIT pushed to branch master-9.0.x at ultreiaio / ird-observe Commits: fe5e9fbc by Tony Chemit at 2022-12-12T19:33:42+01:00 [jgitflow-maven-plugin]updating poms for 9.0.23-SNAPSHOT development[skip ci] - - - - - 7fe3e3c3 by Tony Chemit at 2022-12-12T20:24:50+01:00 [jgitflow-maven-plugin]updating develop poms to master versions to avoid merge conflicts[skip ci] - - - - - f4ae314d by Tony Chemit at 2022-12-12T20:24:52+01:00 [jgitflow-maven-plugin]merging 'master-9.0.x' into 'develop-9.0.x' - - - - - 2328882e by Tony Chemit at 2022-12-12T20:24:55+01:00 [jgitflow-maven-plugin]Updating develop poms back to pre merge state[skip ci] - - - - - 08ad9278 by Tony Chemit at 2022-12-16T18:51:03+01:00 Persistence - Add BusinessProject inside the PersistenceBusinessProject (to be able to transform reference type to dto type) - - - - - a0fc9666 by Tony Chemit at 2022-12-16T18:51:19+01:00 Report - improve DataMatrixDimension and DataMatrixPoint code (they are immutable objects). - - - - - c3acbf65 by Tony Chemit at 2022-12-16T18:51:19+01:00 Report - introduce ReportRequestExecutor.isVariableUsed method to use the same code (we previously had two different code for this. - - - - - 3dc171b4 by Tony Chemit at 2022-12-16T18:51:19+01:00 Report - add TimeLog instance in ReportRequestExecutor to avoid override in DefaultReportRequestExecutor some methods to just add TimeLog. - - - - - 800b253b by Tony Chemit at 2022-12-16T18:51:19+01:00 Report - move ReportRequest.extractParams to ReportRequestExecutor (better place, and the ReportRequest is part of the report model, it should contains no logic at all) - - - - - f98d4f2b by Tony Chemit at 2022-12-16T18:51:19+01:00 Report - clean ReportRequest code - - - - - 864f2970 by Tony Chemit at 2022-12-16T18:51:19+01:00 Report - introduce ReportVariableValueAware for report variable with type that are not a full entity dto (to replace the very weak code used for speciesFeate.discard) - - - - - 1e543c56 by Tony Chemit at 2022-12-16T18:51:19+01:00 Report API - Code review and some nice new features : * improve ReportVariable (have now ReportVariableSupport base class then ReportVariable and ReportRepeatVariable as implementation) * improve code of ReportRequestExecutor * now we can manage to populate variables with dependencies on other variables - - - - - 4fcc15ca by Tony Chemit at 2022-12-16T18:51:19+01:00 Report UI - Code review and some nice new features : * review handler and model code (was pretty a mess (thanks to K.M and S.B...) * variables can now depend on others \o/ - - - - - 05b6509d by Tony Chemit at 2022-12-16T18:51:19+01:00 Report - Introduce SpeciesFateDiscardModeDto to replace the discard (as boolean) in reports : * psObservationCatchWeightByGroupAndSpeciesFateDiscardPerAssociation * psObservationCatchTotalCountByGroupAndSpeciesFateDiscardPerAssociation - - - - - 7c98149a by Tony Chemit at 2022-12-16T18:51:19+01:00 Report - Improve psObservationCatch report (show only speciesGroup with data and species within the selected group really used in data) - - - - - 66df35e8 by Tony Chemit at 2022-12-16T18:51:19+01:00 Report - Improve psObservationLengthsDistribution report (make variable sizeMeasureType depends on selected variable species) - - - - - 606c3fc2 by Tony Chemit at 2022-12-16T18:51:53+01:00 Merge branch 'feature/issue_2577' into develop-9.0.x Améliorer les variables dans les rapports pour proposer les variables suivantes en filtrant sur la variable précédente - Closes #2577 - - - - - 9911e717 by Tony Chemit at 2022-12-23T07:44:53+01:00 update pom - - - - - def9e61f by Tony Chemit at 2022-12-23T07:52:23+01:00 Introduce toolkit-api-services Service - use DataMatrix from java-util Review how to load DifferentialMetaModel (generates it and remove it from BusinessProject) - - - - - 07b8da59 by Tony Chemit at 2022-12-23T07:53:56+01:00 Fix commons-collections license issue - - - - - 385bb2b7 by Tony Chemit at 2022-12-23T07:53:56+01:00 Report API - Introduce toolkit-api-report module - - - - - 98885a37 by Tony Chemit at 2022-12-23T07:53:56+01:00 Report UI - Review code and use last services report API - - - - - 7da90227 by Tony Chemit at 2022-12-23T07:53:56+01:00 Validation - Introduce toolkit-api-validation and core-api-dto-resources - - - - - a6a144c0 by Tony Chemit at 2022-12-23T07:57:46+01:00 Introduce toolkit-api-datasource and toolkit-api-server modules - - - - - 1c8eb7ff by Tony Chemit at 2022-12-23T07:57:46+01:00 Introduce CleanTemporaryFilesTask (application level for client or server) - Closes ultreiaio/config#38 - - - - - 5245ff3c by Tony Chemit at 2022-12-23T07:57:46+01:00 Introduce ConfigHelper - Closes ultreiaio/config#39 - - - - - 7df0e0ec by Tony Chemit at 2022-12-23T07:57:46+01:00 Introduce toolkit-api-decoration , core-api-dto-decoration, core-api-dto-validation - - - - - 847af88a by Tony Chemit at 2022-12-23T07:57:46+01:00 Introduce core-persistence-decoration and move persistence fixtures to model module - - - - - 447d44e4 by Tony Chemit at 2022-12-23T07:57:46+01:00 Introduce new maven modules http-spi and http-spi-processor (split of http-maven-plugin) - See ultreiaio/http#57 - - - - - 2db2d26e by Tony Chemit at 2022-12-23T07:57:46+01:00 remove provided dependencies in pom - - - - - d945b8c1 by Tony Chemit at 2022-12-23T07:57:46+01:00 introduce module toolkit-api-i18n, fix some i18n keys, clean pom dependencies - - - - - 70070b1a by Tony Chemit at 2022-12-23T09:04:29+01:00 optimize i18n - - - - - 194323af by Tony Chemit at 2022-12-23T09:04:29+01:00 Be able to add (or not!) the date on Generated annotation by DecoratorDefinitionJavaFileBuilder - Closes ultreiaio/decorator#13 - - - - - dd09ffa7 by Tony Chemit at 2022-12-23T09:30:01+01:00 Decorator - let's generate them in module sources directory (to make i18n possible) - - - - - 52760025 by Tony Chemit at 2022-12-23T10:21:32+01:00 update pom - - - - - 7cbd7e7d by Tony Chemit at 2022-12-23T10:38:33+01:00 optimize i18n - - - - - 0a531588 by Tony Chemit at 2022-12-28T16:53:27+01:00 Services tck - improve logs of loaded fixtures - - - - - d1271a87 by Tony Chemit at 2022-12-28T17:01:32+01:00 Merge branch 'feature/split_toolkit_api' into develop-9.0.x - - - - - 5ae21984 by Tony Chemit at 2022-12-29T05:10:01+01:00 test - move assertFixture to ObserveFixtures - - - - - 5ad2ce48 by Tony Chemit at 2022-12-29T08:13:55+01:00 test - move asserts to ToolkitFixtures - - - - - faf0528c by Tony Chemit at 2022-12-29T08:13:55+01:00 client config - clean code - - - - - d9bb23a5 by Tony Chemit at 2022-12-29T08:21:30+01:00 Report Api - low log level on ReportRequestExecutor - - - - - 4e561b18 by Tony Chemit at 2022-12-29T08:21:38+01:00 Report Api - Add comment on variables and repeat variables - - - - - 1f68ec39 by Tony Chemit at 2022-12-29T08:21:38+01:00 Report Api - Add comment on request - - - - - 5d6b021b by Tony Chemit at 2022-12-29T08:21:39+01:00 Report Api - Introduce new syntax builder - - - - - bb3b18d5 by Tony Chemit at 2022-12-29T08:21:39+01:00 Report Api - Introduce migrate report file tool - - - - - 299e0317 by Tony Chemit at 2022-12-29T08:27:59+01:00 Report Api - Introduce core-persistence-report module and use new report syntax - - - - - 0cbd36ae by Tony Chemit at 2022-12-29T08:27:59+01:00 Report Test - Improve them and test legacy and default syntax - - - - - b6e1143a by Tony Chemit at 2022-12-29T08:46:50+01:00 Merge branch 'feature/issue_2578' into develop-9.0.x Revoir la syntaxe des rapports - Closes #2578 - - - - - 0d8cc4c6 by Tony Chemit at 2023-01-03T12:49:20+01:00 Impossible de créer une route en 9.0.22 - Closes #2579 - - - - - 8a8f2a7c by Tony Chemit at 2023-01-03T13:03:36+01:00 Happy new yar commit \o/ - - - - - d8016878 by Tony Chemit at 2023-01-03T14:29:48+01:00 Lors de la suppression gloable d'une capture utilisée sur le formulaire espèces sensibles, la demande de confirmation n'est pas proposée (mais elle l'est bien si l'espèce est utilisée dans un échantillon) - Closes #2580 - - - - - cf5d1c5b by Tony Chemit at 2023-01-03T18:35:10+01:00 Imperfection de la validation sur une activité de pêche - Closes #2581 - - - - - 5662c2b9 by Tony Chemit at 2023-01-03T18:35:10+01:00 Après création d'une activité, le nœud captures n'apparaît pas - Closes #2582 - - - - - 395dc4e4 by Tony Chemit at 2023-01-03T18:35:10+01:00 Les activités ne sont pas disponibles lors de la création d'un échantillon (mais disponible en mode mis à jour) - Closes #2584 - - - - - a4c18864 by Tony Chemit at 2023-01-03T18:35:10+01:00 Problème sur la sauvegarde d'un échantillon (si on lui associe des activités) - Closes #2585 - - - - - 1cec9150 by Tony Chemit at 2023-01-03T19:37:33+01:00 update pom - - - - - fc4ee4b9 by Tony Chemit at 2023-01-04T15:11:19+01:00 Amélioration du texte d'information quand on ne peut pas encore remplir le marché local - closes #2586 - - - - - 6836cd1c by Tony Chemit at 2023-01-04T16:03:51+01:00 Ajout sur le formulaire d'une information sur la saisie possible ou non du marché local à partir du port d'arrivée sélectionné - closes #2590 - - - - - 49d90e71 by Tony Chemit at 2023-01-04T17:03:48+01:00 Mauvaise persistence des lots au niveau des sondages - closes #2591 - - - - - e18a2ccd by Tony Chemit at 2023-01-04T17:28:06+01:00 Si on quitte le formulaire marée modifié, aucune alerte n'est proposée pour enregistrer les modifications - closes #2587 - - - - - b8e6082e by Tony Chemit at 2023-01-04T17:29:00+01:00 Sur un formulaire tableau modifié, la demande confirmation est demandée deux fois si la première fois, on dit annuler - closes #2589 - - - - - 09569172 by Tony Chemit at 2023-01-04T17:56:56+01:00 Sur le formulaire lot, le champs commentaire n'est pas lié à un commentaire de la marée - closes #2592 - - - - - 90eb070a by Tony Chemit at 2023-01-05T13:39:00+01:00 Update changelog for v.9.0.23 [skip CI] - - - - - 29d23042 by Tony Chemit at 2023-01-05T13:39:00+01:00 Report API - Review it and make json compliant - - - - - 37f45b04 by Tony Chemit at 2023-01-05T13:39:00+01:00 Améliorations de libellés de l'arbre - Closes #2595 - - - - - 902babdd by Tony Chemit at 2023-01-05T14:18:01+01:00 update pom - - - - - 3487d61a by Tony Chemit at 2023-01-05T17:34:00+01:00 [jgitflow-maven-plugin]updating poms for 9.0.23 branch with snapshot versions[skip ci] - - - - - 00fa1f1f by Tony Chemit at 2023-01-05T17:34:23+01:00 [jgitflow-maven-plugin]updating poms for branch'release/9.0.23' with non-snapshot versions[skip ci] - - - - - bdcac517 by Tony Chemit at 2023-01-05T17:54:40+01:00 [jgitflow-maven-plugin]merging 'release/9.0.23' into 'master-9.0.x' - - - - - 30 changed files: - CHANGELOG.md - client/configuration/pom.xml - − client/configuration/src/license/THIRD-PARTY.properties - client/configuration/src/main/assembly/dcp-presets.xml - client/configuration/src/main/config/Client.ini - client/configuration/src/main/filtered-resources/log4j2.xml - client/configuration/src/main/filtered-resources/observe-log4j2.xml - client/configuration/src/main/i18n/templates/about_en_GB.ftl - client/configuration/src/main/i18n/templates/about_es_ES.ftl - client/configuration/src/main/i18n/templates/about_fr_FR.ftl - client/configuration/src/main/java/fr/ird/observe/client/ObServeSwingSessionConfig.java - client/configuration/src/main/java/fr/ird/observe/client/ObserveSwingSession.java - client/configuration/src/main/java/fr/ird/observe/client/configuration/ClientConfig.java - client/configuration/src/main/java/fr/ird/observe/client/configuration/ClientConfigFinder.java - client/configuration/src/main/java/fr/ird/observe/client/configuration/ClientResources.java - client/configuration/src/main/java/fr/ird/observe/client/configuration/SeineBycatchObservedSystemConfigConverter.java - client/configuration/src/test/java/fr/ird/observe/client/configuration/ClientConfigTest.java - client/core/pom.xml - client/core/src/license/THIRD-PARTY.properties - + client/core/src/main/i18n/getters/java-enumeration.getter - client/core/src/main/i18n/templates/ConfigurationModel_en_GB.ftl - client/core/src/main/i18n/templates/ConfigurationModel_es_ES.ftl - client/core/src/main/i18n/templates/ConfigurationModel_fr_FR.ftl - toolkit/api/src/main/i18n/templates/DataSourceInitModel_en_GB.ftl → client/core/src/main/i18n/templates/DataSourceInitModel_en_GB.ftl - toolkit/api/src/main/i18n/templates/DataSourceInitModel_es_ES.ftl → client/core/src/main/i18n/templates/DataSourceInitModel_es_ES.ftl - toolkit/api/src/main/i18n/templates/DataSourceInitModel_fr_FR.ftl → client/core/src/main/i18n/templates/DataSourceInitModel_fr_FR.ftl - client/core/src/main/i18n/templates/InitStorageModel_en_GB.ftl - client/core/src/main/i18n/templates/InitStorageModel_es_ES.ftl - client/core/src/main/i18n/templates/InitStorageModel_fr_FR.ftl - client/core/src/main/i18n/templates/ObserveSwingDataSource_en_GB.ftl The diff was not included because it is too large. View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/c2573c84fe1fb2459081840f… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/c2573c84fe1fb2459081840f… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop-9.0.x] 6 commits: [jgitflow-maven-plugin]updating poms for 9.0.23 branch with snapshot versions[skip ci]
by Tony CHEMIT (＠tchemit) 05 Jan '23

05 Jan '23

Tony CHEMIT pushed to branch develop-9.0.x at ultreiaio / ird-observe Commits: 3487d61a by Tony Chemit at 2023-01-05T17:34:00+01:00 [jgitflow-maven-plugin]updating poms for 9.0.23 branch with snapshot versions[skip ci] - - - - - 00fa1f1f by Tony Chemit at 2023-01-05T17:34:23+01:00 [jgitflow-maven-plugin]updating poms for branch'release/9.0.23' with non-snapshot versions[skip ci] - - - - - bdcac517 by Tony Chemit at 2023-01-05T17:54:40+01:00 [jgitflow-maven-plugin]merging 'release/9.0.23' into 'master-9.0.x' - - - - - 8f2257d0 by Tony Chemit at 2023-01-05T17:54:46+01:00 [jgitflow-maven-plugin]updating develop poms to master versions to avoid merge conflicts[skip ci] - - - - - 195671d2 by Tony Chemit at 2023-01-05T17:54:47+01:00 [jgitflow-maven-plugin]merging 'master-9.0.x' into 'develop-9.0.x' - - - - - 2bffe65c by Tony Chemit at 2023-01-05T17:54:48+01:00 [jgitflow-maven-plugin]Updating develop poms back to pre merge state[skip ci] - - - - - 0 changed files: Changes: View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/625e7e7bac7c43ed40ef77e5… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/625e7e7bac7c43ed40ef77e5… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe] Pushed new branch release/9.0.23
by Tony CHEMIT (＠tchemit) 05 Jan '23

05 Jan '23

Tony CHEMIT pushed new branch release/9.0.23 at ultreiaio / ird-observe -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/tree/release/9.0.23 You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop-9.0.x] [jgitflow-maven-plugin]updating poms for 9.0.24-SNAPSHOT development[skip ci]
by Tony CHEMIT (＠tchemit) 05 Jan '23

05 Jan '23

Tony CHEMIT pushed to branch develop-9.0.x at ultreiaio / ird-observe Commits: 625e7e7b by Tony Chemit at 2023-01-05T17:34:02+01:00 [jgitflow-maven-plugin]updating poms for 9.0.24-SNAPSHOT development[skip ci] - - - - - 30 changed files: - client/configuration/pom.xml - client/core/pom.xml - client/datasource/actions/pom.xml - client/datasource/editor/api-test/pom.xml - client/datasource/editor/api/pom.xml - client/datasource/editor/common/pom.xml - client/datasource/editor/ll/pom.xml - client/datasource/editor/pom.xml - client/datasource/editor/ps/pom.xml - client/datasource/editor/spi/pom.xml - client/datasource/pom.xml - client/pom.xml - client/runner/pom.xml - core/api/dto-decoration/pom.xml - core/api/dto-validation/pom.xml - core/api/dto/pom.xml - core/api/pom.xml - core/api/services/pom.xml - core/api/tree/pom.xml - core/api/validation/pom.xml - core/persistence/decoration/pom.xml - core/persistence/java/pom.xml - core/persistence/pom.xml - core/persistence/report/pom.xml - core/persistence/resources/pom.xml - core/persistence/test/pom.xml - core/pom.xml - core/services/client/pom.xml - core/services/i18n/pom.xml - core/services/local/pom.xml The diff was not included because it is too large. View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/625e7e7bac7c43ed40ef77e54… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/625e7e7bac7c43ed40ef77e54… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop-9.0.x] update pom
by Tony CHEMIT (＠tchemit) 05 Jan '23

05 Jan '23

Tony CHEMIT pushed to branch develop-9.0.x at ultreiaio / ird-observe Commits: 902babdd by Tony Chemit at 2023-01-05T14:18:01+01:00 update pom - - - - - 1 changed file: - pom.xml Changes: ===================================== pom.xml ===================================== @@ -23,7 +23,7 @@ <parent> <groupId>io.ultreia.maven</groupId> <artifactId>pom</artifactId> - <version>2023.1</version> + <version>2023.2</version> </parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/902babdd590f6b8a42e68f2f6… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/902babdd590f6b8a42e68f2f6… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop] update pom
by Tony CHEMIT (＠tchemit) 05 Jan '23

05 Jan '23

Tony CHEMIT pushed to branch develop at ultreiaio / ird-observe Commits: bb4a6a16 by Tony Chemit at 2023-01-05T15:21:59+01:00 update pom - - - - - 1 changed file: - pom.xml Changes: ===================================== pom.xml ===================================== @@ -23,7 +23,7 @@ <parent> <groupId>io.ultreia.maven</groupId> <artifactId>pom</artifactId> - <version>2023.1</version> + <version>2023.2</version> </parent> <groupId>fr.ird.observe</groupId> <artifactId>ird-observe</artifactId> View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/bb4a6a16847ed80fb65d79ea7… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/bb4a6a16847ed80fb65d79ea7… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop] 3 commits: Update changelog for v.9.0.23 [skip CI]
by Tony CHEMIT (＠tchemit) 05 Jan '23

05 Jan '23

Tony CHEMIT pushed to branch develop at ultreiaio / ird-observe Commits: b7395468 by Tony Chemit at 2023-01-05T13:40:05+01:00 Update changelog for v.9.0.23 [skip CI] - - - - - ab590996 by Tony Chemit at 2023-01-05T13:40:05+01:00 Report API - Review it and make json compliant - - - - - e74f13ef by Tony Chemit at 2023-01-05T13:40:05+01:00 Améliorations de libellés de l'arbre - Closes #2595 - - - - - 18 changed files: - CHANGELOG.md - client/datasource/actions/src/main/java/fr/ird/observe/client/datasource/actions/report/ReportModel.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUINavigationHandler.java - core/services/i18n/src/main/i18n/translations/services_en_GB.properties - core/services/i18n/src/main/i18n/translations/services_fr_FR.properties - toolkit/api-report/src/main/java/fr/ird/observe/report/Report.java - toolkit/api-report/src/main/java/fr/ird/observe/report/ReportRepeatVariable.java - toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariable.java - toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariableSupport.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/DefaultReportDefinitionsBuilder.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportOperationDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportRepeatVariableDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableSupportDefinition.java → toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinitionSupport.java - toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportRepeatVariableAdapter.java - toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableAdapter.java - toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableSupportAdapter.java Changes: ===================================== CHANGELOG.md ===================================== @@ -1,16 +1,33 @@ # ObServe changelog * Author [Tony Chemit](mailto:dev@tchemit.fr) - * Last generated at 2022-12-12 19:31. + * Last generated at 2023-01-05 13:38. -## Version [9.1.0-RC-1](https://gitlab.com/ultreiaio/ird-observe/-/milestones/243) +## Version [9.0.23](https://gitlab.com/ultreiaio/ird-observe/-/milestones/253) -**Closed at 2022-11-25.** +**Closed at 2023-01-04.** +### Download +* [Client (observe-9.0.23-client.zip)](https://repo1.maven.org/maven2/fr/ird/observe/o…) +* [Serveur (observe-9.0.23.war)](https://repo1.maven.org/maven2/fr/ird/observe/observe/…) +* [Serveur (observe-9.0.23-server.zip)](https://repo1.maven.org/maven2/fr/ird/observe/o…) ### Issues - * [[Type::Evolution 2218]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2218) **tms_version va-t-elle rester dans le schéma public ?** (Thanks to Tony CHEMIT) (Reported by Pascal Cauquil) - * [[Type::Evolution 2478]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2478) **Réusinage du plan de cuve** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2579]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2579) **Impossible de créer une route en 9.0.22** (Thanks to Tony CHEMIT) (Reported by Pascal Cauquil) + * [[Type::Anomalie 2580]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2580) **Lors de la suppression gloable d'une capture utilisée sur le formulaire espèces sensibles, la demande de confirmation n'est pas proposée (mais elle l'est bien si l'espèce est utilisée dans un échantillon)** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2581]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2581) **Imperfection de la validation sur une activité de pêche** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2582]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2582) **Après création d'une activité, le nœud captures n'apparaît pas** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2584]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2584) **Les activités ne sont pas disponibles lors de la création d'un échantillon (mais disponible en mode mis à jour)** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2585]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2585) **Problème sur la sauvegarde d'un échantillon (si on lui associe des activités)** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2587]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2587) **Si on quitte le formulaire marée modifié, aucune alerte n'est proposée pour enregistrer les modifications** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2589]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2589) **Sur un formulaire tableau modifié, la demande confirmation est demandée deux fois si la première fois, on dit annuler** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2591]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2591) **Mauvaise persistence des lots au niveau des sondages** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2592]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2592) **Sur le formulaire lot, le champs commentaire n'est pas lié à un commentaire de la marée** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Evolution 2577]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2577) **Améliorer les variables dans les rapports pour proposer les variables suivantes en filtrant sur la variable précédente** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Evolution 2578]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2578) **Revoir la syntaxe des rapports** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Evolution 2586]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2586) **Amélioration du texte d'information quand on ne peut pas encore remplir le marché local** (Thanks to Pascal Cauquil) (Reported by Tony CHEMIT) + * [[Type::Evolution 2590]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2590) **Ajout sur le formulaire d'une information sur la saisie possible ou non du marché local à partir du port d'arrivée sélectionné** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Evolution 2595]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2595) **Améliorations de libellés de l'arbre** (Thanks to Pascal Cauquil) (Reported by Pascal Cauquil) ## Version [9.0.22](https://gitlab.com/ultreiaio/ird-observe/-/milestones/252) ===================================== client/datasource/actions/src/main/java/fr/ird/observe/client/datasource/actions/report/ReportModel.java ===================================== @@ -342,7 +342,7 @@ public class ReportModel extends AdminActionModel { } public ReportDefinition getSelectedReportDefinition() { - return selectedReport == null ? null : selectedReport.getDefinition(); + return selectedReport == null ? null : selectedReport.definition(); } public void setSelectedReport(Report selectedReport) { ===================================== client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUINavigationHandler.java ===================================== @@ -64,7 +64,7 @@ public abstract class ContentTableUINavigationHandler<N extends ContentTableUINa @Override public String getText() { - String result = getNode().getScope().getI18nTranslation("type"); + String result = getNode().getScope().getI18nTranslation("title"); Long count = getNode().getInitializer().getCount(); if (count != null) { result += String.format(" (%d)", count); ===================================== core/services/i18n/src/main/i18n/translations/services_en_GB.properties ===================================== @@ -862,13 +862,13 @@ observe.data.ll.observation.SetDetailComposition.section.delete.message=Confirm observe.data.ll.observation.SetDetailComposition.sectionTemplate.delete=Confirm to delete current template. observe.data.ll.observation.SetDetailComposition.sections=Sections observe.data.ll.observation.SetDetailComposition.title.section.cant.use.firstTemplate=Can't use the first template while generating -observe.data.ll.observation.SetDetailComposition.type=Detailed compositions +observe.data.ll.observation.SetDetailComposition.type=Detailed composition observe.data.ll.observation.SetGlobalComposition.baitsCompositionTab=Baits observe.data.ll.observation.SetGlobalComposition.branchlinesCompositionTab=Branchlines observe.data.ll.observation.SetGlobalComposition.floatlinesCompositionTab=Floatlines observe.data.ll.observation.SetGlobalComposition.hooksCompositionTab=Hooks observe.data.ll.observation.SetGlobalComposition.mitigationTypeTab=Mitigation -observe.data.ll.observation.SetGlobalComposition.type=Global compositions +observe.data.ll.observation.SetGlobalComposition.type=Global composition observe.data.ll.observation.SetTdr.type=Fishing operation tdr observe.data.ll.observation.SizeMeasure.size=Size (cm) observe.data.ll.observation.SizeMeasure.size.short=Size (cm) @@ -996,7 +996,7 @@ observe.data.ps.localmarket.Batch.count.short=Count observe.data.ps.localmarket.Batch.count.validation.required=Count or weight must be filled (but not both). observe.data.ps.localmarket.Batch.date=Date observe.data.ps.localmarket.Batch.title=Batches -observe.data.ps.localmarket.Batch.type=Batches +observe.data.ps.localmarket.Batch.type=Batch observe.data.ps.localmarket.Batch.validation.date.after.currentPsCommonTrip.endDate=Date (%2$s) must be after trip end date (%1$s). observe.data.ps.localmarket.Batch.validation.date.after.currentPsCommonTrip.startDate=Date (%2$s) must be after trip start date (%1$s). observe.data.ps.localmarket.Batch.validation.date.delay.too.long=Delay in days with trip end date is %d and can not be more than %s days. @@ -1351,7 +1351,7 @@ observe.data.ps.observation.ObjectObservedSpecies.type=Object observed species observe.data.ps.observation.ObjectSchoolEstimate.action.create=Create a new estimate observe.data.ps.observation.ObjectSchoolEstimate.action.save=Insert this estimate observe.data.ps.observation.ObjectSchoolEstimate.action.save.tip=Insert this estimate -observe.data.ps.observation.ObjectSchoolEstimate.title=School estimate +observe.data.ps.observation.ObjectSchoolEstimate.title=Object School estimate observe.data.ps.observation.ObjectSchoolEstimate.totalWeight=Weight (in t) observe.data.ps.observation.ObjectSchoolEstimate.totalWeight.short=Total weight observe.data.ps.observation.ObjectSchoolEstimate.type=Object school estimate ===================================== core/services/i18n/src/main/i18n/translations/services_fr_FR.properties ===================================== @@ -471,8 +471,8 @@ observe.data.GearUseFeatures.gear=Équipement observe.data.GearUseFeatures.gear.short=Équipement observe.data.GearUseFeatures.number=Nombre observe.data.GearUseFeatures.number.short=Nombre -observe.data.GearUseFeatures.title=Équipement du bateau -observe.data.GearUseFeatures.type=Équipement du bateau +observe.data.GearUseFeatures.title=Équipements du navire +observe.data.GearUseFeatures.type=Équipement du navire observe.data.GearUseFeaturesMeasurement.type=Caractéristique d'un équipement observe.data.InlineData.action.add=Nouvelle entrée observe.data.InlineData.action.add.tip=Ajouter une nouvelle donnée de type «%s» @@ -714,7 +714,7 @@ observe.data.ll.logbook.SetGlobalComposition.branchlinesCompositionTab=Avançons observe.data.ll.logbook.SetGlobalComposition.floatlinesCompositionTab=Orins observe.data.ll.logbook.SetGlobalComposition.hooksCompositionTab=Hameçons observe.data.ll.logbook.SetGlobalComposition.mitigationTypeTab=Atténuations -observe.data.ll.logbook.SetGlobalComposition.type=Compositions globales +observe.data.ll.logbook.SetGlobalComposition.type=Composition globale observe.data.ll.observation.Activity.action.create=Activité suivante observe.data.ll.observation.Activity.action.move=Changer de marée observe.data.ll.observation.Activity.action.move.choose.parent.message=À quelle marée voulez-vous associer les activités sélectionnées ? @@ -729,8 +729,8 @@ observe.data.ll.observation.Activity.sensorUsedStat=Nombre de capteurs utilisés observe.data.ll.observation.Activity.setStat=Nombre d'opérations de pêche observe.data.ll.observation.Activity.title=Activité observe.data.ll.observation.Activity.type=Activité -observe.data.ll.observation.ActivityEncounter.type=Rencontre -observe.data.ll.observation.ActivitySensorUsed.type=Capteur +observe.data.ll.observation.ActivityEncounter.type=Rencontres +observe.data.ll.observation.ActivitySensorUsed.type=Capteurs utilisés observe.data.ll.observation.BaitsComposition.action.create=Nouvel appât observe.data.ll.observation.BaitsComposition.action.save=Insérer cet appât observe.data.ll.observation.BaitsComposition.action.save.tip=Insérer cet appât @@ -798,7 +798,7 @@ observe.data.ll.observation.SensorUsed.action.create=Nouveau capteur observe.data.ll.observation.SensorUsed.action.save=Insérer ce capteur observe.data.ll.observation.SensorUsed.action.save.tip=Insérer ce capteur observe.data.ll.observation.SensorUsed.dataLocation.short=Hébergement -observe.data.ll.observation.SensorUsed.title=Capteurs +observe.data.ll.observation.SensorUsed.title=Capteurs utilisés observe.data.ll.observation.SensorUsed.type=Capteur utilisé observe.data.ll.observation.Set.action.add=Ajouter l'opération de pêche observe.data.ll.observation.Set.action.add.tip=Ajouter l'opération de pêche associée à cette activité @@ -862,13 +862,13 @@ observe.data.ll.observation.SetDetailComposition.section.delete.message=Confirme observe.data.ll.observation.SetDetailComposition.sectionTemplate.delete=Confirmer la suppression du patron sélectionné. observe.data.ll.observation.SetDetailComposition.sections=Sections observe.data.ll.observation.SetDetailComposition.title.section.cant.use.firstTemplate=Impossible d'utiliser le premier modèle lors de la génération -observe.data.ll.observation.SetDetailComposition.type=Compositions détaillées +observe.data.ll.observation.SetDetailComposition.type=Composition détaillée observe.data.ll.observation.SetGlobalComposition.baitsCompositionTab=Appâts observe.data.ll.observation.SetGlobalComposition.branchlinesCompositionTab=Avançons observe.data.ll.observation.SetGlobalComposition.floatlinesCompositionTab=Orins observe.data.ll.observation.SetGlobalComposition.hooksCompositionTab=Hameçons observe.data.ll.observation.SetGlobalComposition.mitigationTypeTab=Atténuations -observe.data.ll.observation.SetGlobalComposition.type=Compositions globales +observe.data.ll.observation.SetGlobalComposition.type=Composition globale observe.data.ll.observation.SetTdr.type=Enregistreur de profondeur observe.data.ll.observation.SizeMeasure.size=Mesure (cm) observe.data.ll.observation.SizeMeasure.size.short=Mesure (cm) @@ -883,7 +883,7 @@ observe.data.ll.observation.Tdr.keyDataTab=Données clef observe.data.ll.observation.Tdr.localisationTab=Localisation observe.data.ll.observation.Tdr.speciesTab=Espèces associées observe.data.ll.observation.Tdr.timestampTab=Horodatage -observe.data.ll.observation.Tdr.title=Enregistreur de profondeur +observe.data.ll.observation.Tdr.title=Enregistreurs de profondeur observe.data.ll.observation.Tdr.type=Enregistreur de profondeur observe.data.ll.observation.Tdr.validation.deploymentEnd.after.fishingEnd=L'horodatage de fin de déploiement (%2$s) doit être après le début de celui de fin de pêche (%1$s). observe.data.ll.observation.Tdr.validation.fishingEnd.after.fishingStart=L'horodatage de fin de pêche (%2$s) doit être après le début de celui de début de pêche (%1$s). @@ -1351,7 +1351,7 @@ observe.data.ps.observation.ObjectObservedSpecies.type=Faune observée observe.data.ps.observation.ObjectSchoolEstimate.action.create=Nouvelle estimation observe.data.ps.observation.ObjectSchoolEstimate.action.save=Insérer cette estimation observe.data.ps.observation.ObjectSchoolEstimate.action.save.tip=Insérer la nouvelle estimation dans la table -observe.data.ps.observation.ObjectSchoolEstimate.title=Estimation banc +observe.data.ps.observation.ObjectSchoolEstimate.title=Estimation banc objet observe.data.ps.observation.ObjectSchoolEstimate.totalWeight=Poids (en t) observe.data.ps.observation.ObjectSchoolEstimate.totalWeight.short=Poids total observe.data.ps.observation.ObjectSchoolEstimate.type=Estimation banc objet ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/Report.java ===================================== @@ -59,11 +59,11 @@ public final class Report implements Serializable, JsonAware { /** * Cache of variables of this report indexed by their name. */ - private final transient Map<String, ReportVariable<?>> variableByName; + private transient Map<String, ReportVariable<?>> variableByName; /** * Cache of repeat variables of this report indexed by their name. */ - private final transient Map<String, ReportRepeatVariable<?>> repeatVariableByName; + private transient Map<String, ReportRepeatVariable<?>> repeatVariableByName; public Report(ReportDefinition definition) { this.definition = Objects.requireNonNull(definition); @@ -82,82 +82,90 @@ public final class Report implements Serializable, JsonAware { ReportRepeatVariableDefinition<?> definitionRepeatVariable = definitionRepeatVariables[i]; this.repeatVariables[i] = definitionRepeatVariable.toVariable(); } - this.variableByName = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariableSupport::getName, Function.identity()))); - this.repeatVariableByName = Collections.unmodifiableMap(Arrays.stream(repeatVariables).collect(Collectors.toMap(ReportVariableSupport::getName, Function.identity()))); } - public ReportDefinition getDefinition() { + public ReportDefinition definition() { return definition; } + public String getModelType() { + return definition().getModelType(); + } + public String getId() { - return getDefinition().getId(); + return definition().getId(); } public String getName() { - return getDefinition().getName(); + return definition().getName(); } public String getDescription() { - return getDefinition().getDescription(); + return definition().getDescription(); } public int getRows() { - return getDefinition().getRows(); + return definition().getRows(); } public int getColumns() { - return getDefinition().getColumns(); + return definition().getColumns(); } public String[] getColumnHeaders() { - return getDefinition().getColumnHeaders(); + return definition().getColumnHeaders(); } public String[] getRowHeaders() { - return getDefinition().getRowHeaders(); + return definition().getRowHeaders(); } public ReportRequestDefinition[] getRequests() { - return getDefinition().getRequests(); + return definition().getRequests(); } public ReportOperationDefinition[] getOperations() { - return getDefinition().getOperations(); + return definition().getOperations(); } - public ReportVariable<?>[] getVariables() { - return variables; + public Map<String, Set<String>> getVariableDependencies() { + return definition().getVariableDependencies(); } - public Map<String, Set<String>> getVariableDependencies() { - return getDefinition().getVariableDependencies(); + public ReportVariable<?>[] getVariables() { + return variables; } public Map<String, ReportVariable<?>> getVariableByName() { + if (variableByName == null) { + this.variableByName = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariable::getName, Function.identity()))); + } return variableByName; } public ReportVariable<?> getVariable(String variableName) { - return variableByName.get(variableName); + return getVariableByName().get(variableName); } public ReportRepeatVariable<?>[] getRepeatVariables() { return repeatVariables; } - public String getModelType() { - return getDefinition().getModelType(); - } - - @Override - public String toString() { - return getDefinition().toString(); + public Map<String, ReportRepeatVariable<?>> getRepeatVariableByName() { + if (repeatVariableByName == null) { + this.repeatVariableByName = Collections.unmodifiableMap(Arrays.stream(repeatVariables).collect(Collectors.toMap(ReportRepeatVariable::getName, Function.identity()))); + } + return repeatVariableByName; } @SuppressWarnings("unchecked") public <V> ReportRepeatVariable<V> getRepeatVariable(String name) { - return (ReportRepeatVariable<V>) repeatVariableByName.get(name); + return (ReportRepeatVariable<V>) getRepeatVariableByName().get(name); + } + + @Override + public String toString() { + return definition().toString(); } /** @@ -246,12 +254,13 @@ public final class Report implements Serializable, JsonAware { * @param dependencyVariableName the dependency variable name that has changed */ public void unloadVariablesIfNecessary(String dependencyVariableName) { + Map<String, Set<String>> variableDependencyNamesMap = getVariableDependencies(); for (ReportVariable<?> variable : variables) { if (!variable.isValuesLoaded()) { // this variable is not loaded (so can not unload it) continue; } - Set<String> variableDependencyNames = getVariableDependencies().get(variable.getName()); + Set<String> variableDependencyNames = variableDependencyNamesMap.get(variable.getName()); if (variableDependencyNames == null) { // this variable has no dependency continue; @@ -262,4 +271,5 @@ public final class Report implements Serializable, JsonAware { } } } + } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/ReportRepeatVariable.java ===================================== @@ -44,22 +44,16 @@ import java.util.stream.Collectors; * @author Tony Chemit - dev(a)tchemit.fr * @since 9.0.23 */ -public final class ReportRepeatVariable<V> extends ReportVariableSupport<V> { +public final class ReportRepeatVariable<V> extends ReportVariableSupport<V, ReportRepeatVariableDefinition<V>> { private static final long serialVersionUID = 1L; - private final ReportRepeatVariableDefinition<V> definition; public ReportRepeatVariable(ReportRepeatVariableDefinition<V> definition) { - this.definition = definition; - } - - @Override - public ReportRepeatVariableDefinition<V> getDefinition() { - return definition; + super(definition); } public boolean isAddNullValue() { - return getDefinition().isAddNullValue(); + return definition().isAddNullValue(); } @Override @@ -72,7 +66,7 @@ public final class ReportRepeatVariable<V> extends ReportVariableSupport<V> { public List<String> computeIndexList() { //FIXME This is already done by setValues method? Why using a "null" instead of null value? - Set<V> tmp = new LinkedHashSet<>(values); + Set<V> tmp = new LinkedHashSet<>(getValues()); boolean useNullValue = tmp.remove(null); List<String> result = tmp.stream().map(s -> s instanceof ToolkitId ? ((ToolkitId) s).getId() : String.valueOf(s)).collect(Collectors.toList()); if (useNullValue || isAddNullValue()) { ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariable.java ===================================== @@ -30,25 +30,19 @@ import fr.ird.observe.report.definition.ReportVariableDefinition; * @author Tony Chemit - dev(a)tchemit.fr * @since 1.7 */ -public final class ReportVariable<V> extends ReportVariableSupport<V> { +public final class ReportVariable<V> extends ReportVariableSupport<V, ReportVariableDefinition<V>> { public final static String PROPERTY_SELECTED_VALUE = "selectedValue"; private static final long serialVersionUID = 1L; - private final ReportVariableDefinition<V> definition; /** * Selected value for this variable, that will be used to execute report. */ private V selectedValue; public ReportVariable(ReportVariableDefinition<V> definition) { - this.definition = definition; - } - - @Override - public ReportVariableDefinition<V> getDefinition() { - return definition; + super(definition); } public V getSelectedValue() { @@ -64,7 +58,7 @@ public final class ReportVariable<V> extends ReportVariableSupport<V> { } public boolean isValuesLoadedAndSelectedValueFilled() { - return isValuesLoaded() && getSelectedValue() != null; + return isValuesLoaded() && isSelectedValueFilled(); } @Override ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariableSupport.java ===================================== @@ -22,7 +22,7 @@ package fr.ird.observe.report; * #L% */ -import fr.ird.observe.report.definition.ReportVariableSupportDefinition; +import fr.ird.observe.report.definition.ReportVariableDefinitionSupport; import io.ultreia.java4all.util.json.JsonAware; import java.io.Serializable; @@ -39,27 +39,38 @@ import java.util.Set; * @see ReportRepeatVariable * @since 9.0.22 */ -public abstract class ReportVariableSupport<V> implements Serializable, JsonAware { +public abstract class ReportVariableSupport<V, D extends ReportVariableDefinitionSupport<V>> implements Serializable, JsonAware { + public final static String PROPERTY_DEFINITION = "definition"; public final static String PROPERTY_VALUES = "values"; /** - * Found values to this variable. + * Definition of this variable. */ - protected Set<V> values; + private final D definition; + /** + * Values loaded for this variable. + */ + private Set<V> values; - public abstract ReportVariableSupportDefinition<V> getDefinition(); + protected ReportVariableSupport(D definition) { + this.definition = definition; + } + + public final D definition() { + return definition; + } public final String getName() { - return getDefinition().getName(); + return definition().getName(); } public final Class<V> getType() { - return getDefinition().getType(); + return definition().getType(); } public final String getRequest() { - return getDefinition().getRequest(); + return definition().getRequest(); } public final Set<V> getValues() { @@ -82,12 +93,12 @@ public abstract class ReportVariableSupport<V> implements Serializable, JsonAwar public final boolean equals(Object o) { if (this == o) return true; if (o == null || !getClass().equals(o.getClass())) return false; - ReportVariableSupport<?> that = (ReportVariableSupport<?>) o; - return Objects.equals(getDefinition(), that.getDefinition()); + ReportVariableSupport<?, ?> that = (ReportVariableSupport<?, ?>) o; + return Objects.equals(definition(), that.definition()); } @Override public final int hashCode() { - return Objects.hash(getDefinition()); + return Objects.hash(definition()); } } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/DefaultReportDefinitionsBuilder.java ===================================== @@ -139,7 +139,7 @@ public class DefaultReportDefinitionsBuilder implements ReportDefinitionsBuilder String[] rowHeaders = loadHeaders(reportName, REPORT_ROWS, reportProperties); String[] columnHeaders = loadHeaders(reportName, REPORT_COLUMNS, reportProperties); List<ReportVariableDefinition<?>> variables = getVariables(reportName, reportProperties); - Set<String> variablesNames = variables.stream().map(ReportVariableSupportDefinition::getName).collect(Collectors.toSet()); + Set<String> variablesNames = variables.stream().map(ReportVariableDefinition::getName).collect(Collectors.toSet()); List<ReportRepeatVariableDefinition<?>> repeatVariables = getRepeatVariables(reportName, reportProperties, variablesNames); ReportRequestDefinition[] requests = getRequests(reportName, reportProperties); List<ReportOperationDefinition> operations = getOperations(reportName, reportProperties); ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportDefinition.java ===================================== @@ -94,7 +94,7 @@ public final class ReportDefinition implements Serializable, JsonAware { /** * Cache of dependencies on variables (key is variable name, value is the required variable names to be able to populate the variable). */ - private final transient Map<String, Set<String>> variableDependencies; + private transient Map<String, Set<String>> variableDependencies; public ReportDefinition(String modelType, String id, @@ -116,8 +116,6 @@ public final class ReportDefinition implements Serializable, JsonAware { this.operations = operations; this.variables = variables; this.repeatVariables = repeatVariables; - Map<String, ReportVariableDefinition<?>> variableByName = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariableDefinition::getName, Function.identity()))); - this.variableDependencies = computeVariableDependencies(variableByName); } public String getModelType() { @@ -149,6 +147,9 @@ public final class ReportDefinition implements Serializable, JsonAware { } public Map<String, Set<String>> getVariableDependencies() { + if (variableDependencies == null) { + this.variableDependencies = computeVariableDependencies(); + } return variableDependencies; } @@ -181,8 +182,8 @@ public final class ReportDefinition implements Serializable, JsonAware { return t(name); } - static Map<String, Set<String>> computeVariableDependencies(Map<String, ReportVariableDefinition<?>> variableByNames) { - + private Map<String, Set<String>> computeVariableDependencies() { + Map<String, ReportVariableDefinition<?>> variableByNames = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariableDefinition::getName, Function.identity()))); Set<String> variableNames = variableByNames.keySet(); Map<String, Set<String>> variableDependencies = new TreeMap<>(); for (Map.Entry<String, ReportVariableDefinition<?>> entry : variableByNames.entrySet()) { ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportOperationDefinition.java ===================================== @@ -38,6 +38,9 @@ public final class ReportOperationDefinition implements JsonAware { * Name of the operation, this is the fully qualified name of the ReportOperationConsumer. */ private final String name; + /** + * Optional parameters of this operation. + */ private final String parameters; public static ReportOperationDefinition of(String name) { ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportRepeatVariableDefinition.java ===================================== @@ -30,9 +30,7 @@ import fr.ird.observe.report.ReportRepeatVariable; * @author Tony Chemit - dev(a)tchemit.fr * @since 9.0.23 */ -public final class ReportRepeatVariableDefinition<V> extends ReportVariableSupportDefinition<V> { - - public final static String PROPERTY_ADD_NULL_VALUE = "addNullValue"; +public final class ReportRepeatVariableDefinition<V> extends ReportVariableDefinitionSupport<V> { private static final long serialVersionUID = 1L; ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinition.java ===================================== @@ -33,7 +33,7 @@ import fr.ird.observe.report.ReportVariable; * @author Tony Chemit - dev(a)tchemit.fr * @since 2.0.23 */ -public final class ReportVariableDefinition<V> extends ReportVariableSupportDefinition<V> { +public final class ReportVariableDefinition<V> extends ReportVariableDefinitionSupport<V> { public ReportVariableDefinition(String name, Class<V> type, String request, String comment) { super(name, type, request, comment); ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableSupportDefinition.java → toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinitionSupport.java ===================================== @@ -37,13 +37,7 @@ import java.util.Objects; * @author Tony Chemit - dev(a)tchemit.fr * @since 2.0.23 */ -public abstract class ReportVariableSupportDefinition<V> implements Serializable, JsonAware { - - public final static String PROPERTY_NAME = "name"; - public final static String PROPERTY_TYPE = "type"; - public final static String PROPERTY_REQUEST = "request"; - public final static String PROPERTY_COMMENT = "comment"; - +public abstract class ReportVariableDefinitionSupport<V> implements Serializable, JsonAware { /** * Name of the variable. */ @@ -61,7 +55,7 @@ public abstract class ReportVariableSupportDefinition<V> implements Serializable */ private final String comment; - public ReportVariableSupportDefinition(String name, Class<V> type, String request, String comment) { + public ReportVariableDefinitionSupport(String name, Class<V> type, String request, String comment) { this.name = Objects.requireNonNull(name); this.type = Objects.requireNonNull(type); this.request = Objects.requireNonNull(request); @@ -88,13 +82,13 @@ public abstract class ReportVariableSupportDefinition<V> implements Serializable return comment != null; } - public abstract ReportVariableSupport<?> toVariable(); + public abstract ReportVariableSupport<?, ?> toVariable(); @Override public final boolean equals(Object o) { if (this == o) return true; if (o == null || !getClass().equals(o.getClass())) return false; - ReportVariableSupportDefinition<?> that = (ReportVariableSupportDefinition<?>) o; + ReportVariableDefinitionSupport<?> that = (ReportVariableDefinitionSupport<?>) o; return Objects.equals(name, that.name); } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportRepeatVariableAdapter.java ===================================== @@ -24,10 +24,8 @@ package fr.ird.observe.report.json; import com.google.auto.service.AutoService; import com.google.gson.JsonDeserializationContext; -import com.google.gson.JsonDeserializer; -import com.google.gson.JsonElement; import com.google.gson.JsonObject; -import com.google.gson.JsonParseException; +import com.google.gson.reflect.TypeToken; import fr.ird.observe.report.ReportRepeatVariable; import fr.ird.observe.report.definition.ReportRepeatVariableDefinition; import io.ultreia.java4all.util.json.JsonAdapter; @@ -41,7 +39,7 @@ import java.lang.reflect.Type; * @since 9.0.22 */ @AutoService(JsonAdapter.class) -public class ReportRepeatVariableAdapter extends ReportVariableSupportAdapter implements JsonDeserializer<ReportRepeatVariable<?>> { +public class ReportRepeatVariableAdapter<V> extends ReportVariableSupportAdapter<V, ReportRepeatVariableDefinition<V>, ReportRepeatVariable<V>> { @Override public Class<?> type() { @@ -49,22 +47,15 @@ public class ReportRepeatVariableAdapter extends ReportVariableSupportAdapter im } @Override - public ReportRepeatVariable<?> deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) throws JsonParseException { - JsonObject jsonObject = json.getAsJsonObject(); - - String name = getVariableName(context, jsonObject); - Class<?> type = getVariableType(context, jsonObject); - String request = getVariableRequest(context, jsonObject); - String comment = getVariableComment(context, jsonObject); - boolean addNullValue = getAddNullValue(context, jsonObject); - ReportRepeatVariableDefinition<?> definition = new ReportRepeatVariableDefinition<>(name, type, request, comment, addNullValue); - ReportRepeatVariable<?> result = new ReportRepeatVariable<>(definition); - - loadVariableValues(context, jsonObject, result); - return result; + protected Type definitionType() { + return new TypeToken<ReportRepeatVariableDefinition<V>>() { + }.getType(); } - protected boolean getAddNullValue(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportRepeatVariableDefinition.PROPERTY_ADD_NULL_VALUE), boolean.class); + @Override + protected ReportRepeatVariable<V> deserialize(JsonDeserializationContext context, JsonObject jsonObject, ReportRepeatVariableDefinition<V> definition) { + ReportRepeatVariable<V> result = new ReportRepeatVariable<>(definition); + deserializeValues(context, jsonObject, result); + return result; } } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableAdapter.java ===================================== @@ -25,10 +25,9 @@ package fr.ird.observe.report.json; import com.google.auto.service.AutoService; import com.google.gson.JsonDeserializationContext; -import com.google.gson.JsonDeserializer; import com.google.gson.JsonElement; import com.google.gson.JsonObject; -import com.google.gson.JsonParseException; +import com.google.gson.reflect.TypeToken; import fr.ird.observe.report.ReportVariable; import fr.ird.observe.report.definition.ReportVariableDefinition; import io.ultreia.java4all.util.json.JsonAdapter; @@ -39,7 +38,7 @@ import java.lang.reflect.Type; * @author Tony Chemit - dev(a)tchemit.fr */ @AutoService(JsonAdapter.class) -public class ReportVariableAdapter extends ReportVariableSupportAdapter implements JsonDeserializer<ReportVariable<?>> { +public class ReportVariableAdapter<V> extends ReportVariableSupportAdapter<V, ReportVariableDefinition<V>, ReportVariable<V>> { @Override public Class<?> type() { @@ -47,27 +46,22 @@ public class ReportVariableAdapter extends ReportVariableSupportAdapter implemen } @Override - public ReportVariable<?> deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) throws JsonParseException { - JsonObject jsonObject = json.getAsJsonObject(); - - String name = getVariableName(context, jsonObject); - Class<?> type = getVariableType(context, jsonObject); - String request = getVariableRequest(context, jsonObject); - String comment = getVariableComment(context, jsonObject); - - ReportVariableDefinition<?> definition = new ReportVariableDefinition<>(name, type, request, comment); - ReportVariable<?> result = new ReportVariable<>(definition); + protected Type definitionType() { + return new TypeToken<ReportVariableDefinition<V>>() { + }.getType(); + } - loadVariableValues(context, jsonObject, result); - loadSelectedValue(context, jsonObject, result); + @Override + protected ReportVariable<V> deserialize(JsonDeserializationContext context, JsonObject jsonObject, ReportVariableDefinition<V> definition) { + ReportVariable<V> result = new ReportVariable<>(definition); + deserializeValues(context, jsonObject, result); + deserializeSelectedValue(context, jsonObject, result); return result; } - protected <V> void loadSelectedValue(JsonDeserializationContext context, JsonObject jsonObject, ReportVariable<V> variable) { + protected void deserializeSelectedValue(JsonDeserializationContext context, JsonObject jsonObject, ReportVariable<V> variable) { JsonElement json = jsonObject.get(ReportVariable.PROPERTY_SELECTED_VALUE); - if (json != null) { - V selectedValue = context.deserialize(json, variable.getType()); - variable.setSelectedValue(selectedValue); - } + V selectedValue = deserializeValue(context, variable.getType(), json); + variable.setSelectedValue(selectedValue); } } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableSupportAdapter.java ===================================== @@ -22,13 +22,17 @@ package fr.ird.observe.report.json; * #L% */ +import com.google.gson.JsonArray; import com.google.gson.JsonDeserializationContext; +import com.google.gson.JsonDeserializer; import com.google.gson.JsonElement; import com.google.gson.JsonObject; +import com.google.gson.JsonParseException; import fr.ird.observe.report.ReportVariableSupport; -import fr.ird.observe.report.definition.ReportVariableSupportDefinition; +import fr.ird.observe.report.definition.ReportVariableDefinitionSupport; import io.ultreia.java4all.util.json.JsonAdapter; +import java.lang.reflect.Type; import java.util.HashSet; import java.util.Set; @@ -40,34 +44,38 @@ import java.util.Set; * @author Tony Chemit - dev(a)tchemit.fr * @since 9.0.22 */ -public abstract class ReportVariableSupportAdapter implements JsonAdapter { +public abstract class ReportVariableSupportAdapter<V, D extends ReportVariableDefinitionSupport<V>, R extends ReportVariableSupport<V, D>> implements JsonAdapter, JsonDeserializer<R> { - protected String getVariableName(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_NAME), String.class); - } + protected abstract Type definitionType(); - protected Class<?> getVariableType(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_TYPE), Class.class); - } + protected abstract R deserialize(JsonDeserializationContext context, JsonObject jsonObject, D definition); - protected String getVariableRequest(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_REQUEST), String.class); + @Override + public final R deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) throws JsonParseException { + JsonObject jsonObject = json.getAsJsonObject(); + D definition = deserializeDefinition(context, jsonObject); + return deserialize(context, jsonObject, definition); } - protected String getVariableComment(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_COMMENT), String.class); + protected D deserializeDefinition(JsonDeserializationContext context, JsonObject jsonObject) { + return context.deserialize(jsonObject.get(ReportVariableSupport.PROPERTY_DEFINITION), definitionType()); } - protected <V> void loadVariableValues(JsonDeserializationContext context, JsonObject jsonObject, ReportVariableSupport<V> variable) { + protected void deserializeValues(JsonDeserializationContext context, JsonObject jsonObject, R variable) { JsonElement valuesJson = jsonObject.get(ReportVariableSupport.PROPERTY_VALUES); if (valuesJson != null) { - Set<V> values = new HashSet<>(); Class<V> type = variable.getType(); - for (JsonElement valueJson : valuesJson.getAsJsonArray()) { - V value = context.deserialize(valueJson, type); + JsonArray jsonArray = valuesJson.getAsJsonArray(); + Set<V> values = new HashSet<>(jsonArray.size()); + for (JsonElement valueJson : jsonArray) { + V value = deserializeValue(context, type, valueJson); values.add(value); } variable.setValues(values); } } + + protected V deserializeValue(JsonDeserializationContext context, Class<V> variableType, JsonElement valueJson) { + return valueJson == null ? null : context.deserialize(valueJson, variableType); + } } View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/d17023ba996938b5925ae992… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/d17023ba996938b5925ae992… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop-9.0.x] 3 commits: Update changelog for v.9.0.23 [skip CI]
by Tony CHEMIT (＠tchemit) 05 Jan '23

05 Jan '23

Tony CHEMIT pushed to branch develop-9.0.x at ultreiaio / ird-observe Commits: 90eb070a by Tony Chemit at 2023-01-05T13:39:00+01:00 Update changelog for v.9.0.23 [skip CI] - - - - - 29d23042 by Tony Chemit at 2023-01-05T13:39:00+01:00 Report API - Review it and make json compliant - - - - - 37f45b04 by Tony Chemit at 2023-01-05T13:39:00+01:00 Améliorations de libellés de l'arbre - Closes #2595 - - - - - 18 changed files: - CHANGELOG.md - client/datasource/actions/src/main/java/fr/ird/observe/client/datasource/actions/report/ReportModel.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUINavigationHandler.java - core/services/i18n/src/main/i18n/translations/services_en_GB.properties - core/services/i18n/src/main/i18n/translations/services_fr_FR.properties - toolkit/api-report/src/main/java/fr/ird/observe/report/Report.java - toolkit/api-report/src/main/java/fr/ird/observe/report/ReportRepeatVariable.java - toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariable.java - toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariableSupport.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/DefaultReportDefinitionsBuilder.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportOperationDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportRepeatVariableDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableSupportDefinition.java → toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinitionSupport.java - toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportRepeatVariableAdapter.java - toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableAdapter.java - toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableSupportAdapter.java Changes: ===================================== CHANGELOG.md ===================================== @@ -1,16 +1,33 @@ # ObServe changelog * Author [Tony Chemit](mailto:dev@tchemit.fr) - * Last generated at 2022-12-12 19:31. + * Last generated at 2023-01-05 13:38. -## Version [9.1.0-RC-1](https://gitlab.com/ultreiaio/ird-observe/-/milestones/243) +## Version [9.0.23](https://gitlab.com/ultreiaio/ird-observe/-/milestones/253) -**Closed at 2022-11-25.** +**Closed at 2023-01-04.** +### Download +* [Client (observe-9.0.23-client.zip)](https://repo1.maven.org/maven2/fr/ird/observe/o…) +* [Serveur (observe-9.0.23.war)](https://repo1.maven.org/maven2/fr/ird/observe/observe/…) +* [Serveur (observe-9.0.23-server.zip)](https://repo1.maven.org/maven2/fr/ird/observe/o…) ### Issues - * [[Type::Evolution 2218]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2218) **tms_version va-t-elle rester dans le schéma public ?** (Thanks to Tony CHEMIT) (Reported by Pascal Cauquil) - * [[Type::Evolution 2478]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2478) **Réusinage du plan de cuve** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2579]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2579) **Impossible de créer une route en 9.0.22** (Thanks to Tony CHEMIT) (Reported by Pascal Cauquil) + * [[Type::Anomalie 2580]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2580) **Lors de la suppression gloable d'une capture utilisée sur le formulaire espèces sensibles, la demande de confirmation n'est pas proposée (mais elle l'est bien si l'espèce est utilisée dans un échantillon)** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2581]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2581) **Imperfection de la validation sur une activité de pêche** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2582]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2582) **Après création d'une activité, le nœud captures n'apparaît pas** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2584]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2584) **Les activités ne sont pas disponibles lors de la création d'un échantillon (mais disponible en mode mis à jour)** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2585]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2585) **Problème sur la sauvegarde d'un échantillon (si on lui associe des activités)** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2587]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2587) **Si on quitte le formulaire marée modifié, aucune alerte n'est proposée pour enregistrer les modifications** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2589]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2589) **Sur un formulaire tableau modifié, la demande confirmation est demandée deux fois si la première fois, on dit annuler** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2591]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2591) **Mauvaise persistence des lots au niveau des sondages** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Anomalie 2592]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2592) **Sur le formulaire lot, le champs commentaire n'est pas lié à un commentaire de la marée** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Evolution 2577]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2577) **Améliorer les variables dans les rapports pour proposer les variables suivantes en filtrant sur la variable précédente** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Evolution 2578]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2578) **Revoir la syntaxe des rapports** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Evolution 2586]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2586) **Amélioration du texte d'information quand on ne peut pas encore remplir le marché local** (Thanks to Pascal Cauquil) (Reported by Tony CHEMIT) + * [[Type::Evolution 2590]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2590) **Ajout sur le formulaire d'une information sur la saisie possible ou non du marché local à partir du port d'arrivée sélectionné** (Thanks to Tony CHEMIT) (Reported by Tony CHEMIT) + * [[Type::Evolution 2595]](https://gitlab.com/ultreiaio/ird-observe/-/issues/2595) **Améliorations de libellés de l'arbre** (Thanks to Pascal Cauquil) (Reported by Pascal Cauquil) ## Version [9.0.22](https://gitlab.com/ultreiaio/ird-observe/-/milestones/252) ===================================== client/datasource/actions/src/main/java/fr/ird/observe/client/datasource/actions/report/ReportModel.java ===================================== @@ -342,7 +342,7 @@ public class ReportModel extends AdminActionModel { } public ReportDefinition getSelectedReportDefinition() { - return selectedReport == null ? null : selectedReport.getDefinition(); + return selectedReport == null ? null : selectedReport.definition(); } public void setSelectedReport(Report selectedReport) { ===================================== client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUINavigationHandler.java ===================================== @@ -64,7 +64,7 @@ public abstract class ContentTableUINavigationHandler<N extends ContentTableUINa @Override public String getText() { - String result = getNode().getScope().getI18nTranslation("type"); + String result = getNode().getScope().getI18nTranslation("title"); Long count = getNode().getInitializer().getCount(); if (count != null) { result += String.format(" (%d)", count); ===================================== core/services/i18n/src/main/i18n/translations/services_en_GB.properties ===================================== @@ -862,13 +862,13 @@ observe.data.ll.observation.SetDetailComposition.section.delete.message=Confirm observe.data.ll.observation.SetDetailComposition.sectionTemplate.delete=Confirm to delete current template. observe.data.ll.observation.SetDetailComposition.sections=Sections observe.data.ll.observation.SetDetailComposition.title.section.cant.use.firstTemplate=Can't use the first template while generating -observe.data.ll.observation.SetDetailComposition.type=Detailed compositions +observe.data.ll.observation.SetDetailComposition.type=Detailed composition observe.data.ll.observation.SetGlobalComposition.baitsCompositionTab=Baits observe.data.ll.observation.SetGlobalComposition.branchlinesCompositionTab=Branchlines observe.data.ll.observation.SetGlobalComposition.floatlinesCompositionTab=Floatlines observe.data.ll.observation.SetGlobalComposition.hooksCompositionTab=Hooks observe.data.ll.observation.SetGlobalComposition.mitigationTypeTab=Mitigation -observe.data.ll.observation.SetGlobalComposition.type=Global compositions +observe.data.ll.observation.SetGlobalComposition.type=Global composition observe.data.ll.observation.SetTdr.type=Fishing operation tdr observe.data.ll.observation.SizeMeasure.size=Size (cm) observe.data.ll.observation.SizeMeasure.size.short=Size (cm) @@ -994,7 +994,7 @@ observe.data.ps.localmarket.Batch.count.short=Count observe.data.ps.localmarket.Batch.count.validation.required=Count or weight must be filled (but not both). observe.data.ps.localmarket.Batch.date=Date observe.data.ps.localmarket.Batch.title=Batches -observe.data.ps.localmarket.Batch.type=Batches +observe.data.ps.localmarket.Batch.type=Batch observe.data.ps.localmarket.Batch.validation.date.after.currentPsCommonTrip.endDate=Date (%2$s) must be after trip end date (%1$s). observe.data.ps.localmarket.Batch.validation.date.after.currentPsCommonTrip.startDate=Date (%2$s) must be after trip start date (%1$s). observe.data.ps.localmarket.Batch.validation.date.delay.too.long=Delay in days with trip end date is %d and can not be more than %s days. @@ -1308,7 +1308,7 @@ observe.data.ps.observation.ObjectObservedSpecies.type=Object observed species observe.data.ps.observation.ObjectSchoolEstimate.action.create=Create a new estimate observe.data.ps.observation.ObjectSchoolEstimate.action.save=Insert this estimate observe.data.ps.observation.ObjectSchoolEstimate.action.save.tip=Insert this estimate -observe.data.ps.observation.ObjectSchoolEstimate.title=School estimate +observe.data.ps.observation.ObjectSchoolEstimate.title=Object School estimate observe.data.ps.observation.ObjectSchoolEstimate.totalWeight=Weight (in t) observe.data.ps.observation.ObjectSchoolEstimate.totalWeight.short=Total weight observe.data.ps.observation.ObjectSchoolEstimate.type=Object school estimate ===================================== core/services/i18n/src/main/i18n/translations/services_fr_FR.properties ===================================== @@ -471,8 +471,8 @@ observe.data.GearUseFeatures.gear=Équipement observe.data.GearUseFeatures.gear.short=Équipement observe.data.GearUseFeatures.number=Nombre observe.data.GearUseFeatures.number.short=Nombre -observe.data.GearUseFeatures.title=Équipement du bateau -observe.data.GearUseFeatures.type=Équipement du bateau +observe.data.GearUseFeatures.title=Équipements du navire +observe.data.GearUseFeatures.type=Équipement du navire observe.data.GearUseFeaturesMeasurement.type=Caractéristique d'un équipement observe.data.InlineData.action.add=Nouvelle entrée observe.data.InlineData.action.add.tip=Ajouter une nouvelle donnée de type «%s» @@ -714,7 +714,7 @@ observe.data.ll.logbook.SetGlobalComposition.branchlinesCompositionTab=Avançons observe.data.ll.logbook.SetGlobalComposition.floatlinesCompositionTab=Orins observe.data.ll.logbook.SetGlobalComposition.hooksCompositionTab=Hameçons observe.data.ll.logbook.SetGlobalComposition.mitigationTypeTab=Atténuations -observe.data.ll.logbook.SetGlobalComposition.type=Compositions globales +observe.data.ll.logbook.SetGlobalComposition.type=Composition globale observe.data.ll.observation.Activity.action.create=Activité suivante observe.data.ll.observation.Activity.action.move=Changer de marée observe.data.ll.observation.Activity.action.move.choose.parent.message=À quelle marée voulez-vous associer les activités sélectionnées ? @@ -729,8 +729,8 @@ observe.data.ll.observation.Activity.sensorUsedStat=Nombre de capteurs utilisés observe.data.ll.observation.Activity.setStat=Nombre d'opérations de pêche observe.data.ll.observation.Activity.title=Activité observe.data.ll.observation.Activity.type=Activité -observe.data.ll.observation.ActivityEncounter.type=Rencontre -observe.data.ll.observation.ActivitySensorUsed.type=Capteur +observe.data.ll.observation.ActivityEncounter.type=Rencontres +observe.data.ll.observation.ActivitySensorUsed.type=Capteurs utilisés observe.data.ll.observation.BaitsComposition.action.create=Nouvel appât observe.data.ll.observation.BaitsComposition.action.save=Insérer cet appât observe.data.ll.observation.BaitsComposition.action.save.tip=Insérer cet appât @@ -798,7 +798,7 @@ observe.data.ll.observation.SensorUsed.action.create=Nouveau capteur observe.data.ll.observation.SensorUsed.action.save=Insérer ce capteur observe.data.ll.observation.SensorUsed.action.save.tip=Insérer ce capteur observe.data.ll.observation.SensorUsed.dataLocation.short=Hébergement -observe.data.ll.observation.SensorUsed.title=Capteurs +observe.data.ll.observation.SensorUsed.title=Capteurs utilisés observe.data.ll.observation.SensorUsed.type=Capteur utilisé observe.data.ll.observation.Set.action.add=Ajouter l'opération de pêche observe.data.ll.observation.Set.action.add.tip=Ajouter l'opération de pêche associée à cette activité @@ -862,13 +862,13 @@ observe.data.ll.observation.SetDetailComposition.section.delete.message=Confirme observe.data.ll.observation.SetDetailComposition.sectionTemplate.delete=Confirmer la suppression du patron sélectionné. observe.data.ll.observation.SetDetailComposition.sections=Sections observe.data.ll.observation.SetDetailComposition.title.section.cant.use.firstTemplate=Impossible d'utiliser le premier modèle lors de la génération -observe.data.ll.observation.SetDetailComposition.type=Compositions détaillées +observe.data.ll.observation.SetDetailComposition.type=Composition détaillée observe.data.ll.observation.SetGlobalComposition.baitsCompositionTab=Appâts observe.data.ll.observation.SetGlobalComposition.branchlinesCompositionTab=Avançons observe.data.ll.observation.SetGlobalComposition.floatlinesCompositionTab=Orins observe.data.ll.observation.SetGlobalComposition.hooksCompositionTab=Hameçons observe.data.ll.observation.SetGlobalComposition.mitigationTypeTab=Atténuations -observe.data.ll.observation.SetGlobalComposition.type=Compositions globales +observe.data.ll.observation.SetGlobalComposition.type=Composition globale observe.data.ll.observation.SetTdr.type=Enregistreur de profondeur observe.data.ll.observation.SizeMeasure.size=Mesure (cm) observe.data.ll.observation.SizeMeasure.size.short=Mesure (cm) @@ -883,7 +883,7 @@ observe.data.ll.observation.Tdr.keyDataTab=Données clef observe.data.ll.observation.Tdr.localisationTab=Localisation observe.data.ll.observation.Tdr.speciesTab=Espèces associées observe.data.ll.observation.Tdr.timestampTab=Horodatage -observe.data.ll.observation.Tdr.title=Enregistreur de profondeur +observe.data.ll.observation.Tdr.title=Enregistreurs de profondeur observe.data.ll.observation.Tdr.type=Enregistreur de profondeur observe.data.ll.observation.Tdr.validation.deploymentEnd.after.fishingEnd=L'horodatage de fin de déploiement (%2$s) doit être après le début de celui de fin de pêche (%1$s). observe.data.ll.observation.Tdr.validation.fishingEnd.after.fishingStart=L'horodatage de fin de pêche (%2$s) doit être après le début de celui de début de pêche (%1$s). @@ -1308,7 +1308,7 @@ observe.data.ps.observation.ObjectObservedSpecies.type=Faune observée observe.data.ps.observation.ObjectSchoolEstimate.action.create=Nouvelle estimation observe.data.ps.observation.ObjectSchoolEstimate.action.save=Insérer cette estimation observe.data.ps.observation.ObjectSchoolEstimate.action.save.tip=Insérer la nouvelle estimation dans la table -observe.data.ps.observation.ObjectSchoolEstimate.title=Estimation banc +observe.data.ps.observation.ObjectSchoolEstimate.title=Estimation banc objet observe.data.ps.observation.ObjectSchoolEstimate.totalWeight=Poids (en t) observe.data.ps.observation.ObjectSchoolEstimate.totalWeight.short=Poids total observe.data.ps.observation.ObjectSchoolEstimate.type=Estimation banc objet ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/Report.java ===================================== @@ -59,11 +59,11 @@ public final class Report implements Serializable, JsonAware { /** * Cache of variables of this report indexed by their name. */ - private final transient Map<String, ReportVariable<?>> variableByName; + private transient Map<String, ReportVariable<?>> variableByName; /** * Cache of repeat variables of this report indexed by their name. */ - private final transient Map<String, ReportRepeatVariable<?>> repeatVariableByName; + private transient Map<String, ReportRepeatVariable<?>> repeatVariableByName; public Report(ReportDefinition definition) { this.definition = Objects.requireNonNull(definition); @@ -82,82 +82,90 @@ public final class Report implements Serializable, JsonAware { ReportRepeatVariableDefinition<?> definitionRepeatVariable = definitionRepeatVariables[i]; this.repeatVariables[i] = definitionRepeatVariable.toVariable(); } - this.variableByName = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariableSupport::getName, Function.identity()))); - this.repeatVariableByName = Collections.unmodifiableMap(Arrays.stream(repeatVariables).collect(Collectors.toMap(ReportVariableSupport::getName, Function.identity()))); } - public ReportDefinition getDefinition() { + public ReportDefinition definition() { return definition; } + public String getModelType() { + return definition().getModelType(); + } + public String getId() { - return getDefinition().getId(); + return definition().getId(); } public String getName() { - return getDefinition().getName(); + return definition().getName(); } public String getDescription() { - return getDefinition().getDescription(); + return definition().getDescription(); } public int getRows() { - return getDefinition().getRows(); + return definition().getRows(); } public int getColumns() { - return getDefinition().getColumns(); + return definition().getColumns(); } public String[] getColumnHeaders() { - return getDefinition().getColumnHeaders(); + return definition().getColumnHeaders(); } public String[] getRowHeaders() { - return getDefinition().getRowHeaders(); + return definition().getRowHeaders(); } public ReportRequestDefinition[] getRequests() { - return getDefinition().getRequests(); + return definition().getRequests(); } public ReportOperationDefinition[] getOperations() { - return getDefinition().getOperations(); + return definition().getOperations(); } - public ReportVariable<?>[] getVariables() { - return variables; + public Map<String, Set<String>> getVariableDependencies() { + return definition().getVariableDependencies(); } - public Map<String, Set<String>> getVariableDependencies() { - return getDefinition().getVariableDependencies(); + public ReportVariable<?>[] getVariables() { + return variables; } public Map<String, ReportVariable<?>> getVariableByName() { + if (variableByName == null) { + this.variableByName = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariable::getName, Function.identity()))); + } return variableByName; } public ReportVariable<?> getVariable(String variableName) { - return variableByName.get(variableName); + return getVariableByName().get(variableName); } public ReportRepeatVariable<?>[] getRepeatVariables() { return repeatVariables; } - public String getModelType() { - return getDefinition().getModelType(); - } - - @Override - public String toString() { - return getDefinition().toString(); + public Map<String, ReportRepeatVariable<?>> getRepeatVariableByName() { + if (repeatVariableByName == null) { + this.repeatVariableByName = Collections.unmodifiableMap(Arrays.stream(repeatVariables).collect(Collectors.toMap(ReportRepeatVariable::getName, Function.identity()))); + } + return repeatVariableByName; } @SuppressWarnings("unchecked") public <V> ReportRepeatVariable<V> getRepeatVariable(String name) { - return (ReportRepeatVariable<V>) repeatVariableByName.get(name); + return (ReportRepeatVariable<V>) getRepeatVariableByName().get(name); + } + + @Override + public String toString() { + return definition().toString(); } /** @@ -246,12 +254,13 @@ public final class Report implements Serializable, JsonAware { * @param dependencyVariableName the dependency variable name that has changed */ public void unloadVariablesIfNecessary(String dependencyVariableName) { + Map<String, Set<String>> variableDependencyNamesMap = getVariableDependencies(); for (ReportVariable<?> variable : variables) { if (!variable.isValuesLoaded()) { // this variable is not loaded (so can not unload it) continue; } - Set<String> variableDependencyNames = getVariableDependencies().get(variable.getName()); + Set<String> variableDependencyNames = variableDependencyNamesMap.get(variable.getName()); if (variableDependencyNames == null) { // this variable has no dependency continue; @@ -262,4 +271,5 @@ public final class Report implements Serializable, JsonAware { } } } + } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/ReportRepeatVariable.java ===================================== @@ -44,22 +44,16 @@ import java.util.stream.Collectors; * @author Tony Chemit - dev(a)tchemit.fr * @since 9.0.23 */ -public final class ReportRepeatVariable<V> extends ReportVariableSupport<V> { +public final class ReportRepeatVariable<V> extends ReportVariableSupport<V, ReportRepeatVariableDefinition<V>> { private static final long serialVersionUID = 1L; - private final ReportRepeatVariableDefinition<V> definition; public ReportRepeatVariable(ReportRepeatVariableDefinition<V> definition) { - this.definition = definition; - } - - @Override - public ReportRepeatVariableDefinition<V> getDefinition() { - return definition; + super(definition); } public boolean isAddNullValue() { - return getDefinition().isAddNullValue(); + return definition().isAddNullValue(); } @Override @@ -72,7 +66,7 @@ public final class ReportRepeatVariable<V> extends ReportVariableSupport<V> { public List<String> computeIndexList() { //FIXME This is already done by setValues method? Why using a "null" instead of null value? - Set<V> tmp = new LinkedHashSet<>(values); + Set<V> tmp = new LinkedHashSet<>(getValues()); boolean useNullValue = tmp.remove(null); List<String> result = tmp.stream().map(s -> s instanceof ToolkitId ? ((ToolkitId) s).getId() : String.valueOf(s)).collect(Collectors.toList()); if (useNullValue || isAddNullValue()) { ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariable.java ===================================== @@ -30,25 +30,19 @@ import fr.ird.observe.report.definition.ReportVariableDefinition; * @author Tony Chemit - dev(a)tchemit.fr * @since 1.7 */ -public final class ReportVariable<V> extends ReportVariableSupport<V> { +public final class ReportVariable<V> extends ReportVariableSupport<V, ReportVariableDefinition<V>> { public final static String PROPERTY_SELECTED_VALUE = "selectedValue"; private static final long serialVersionUID = 1L; - private final ReportVariableDefinition<V> definition; /** * Selected value for this variable, that will be used to execute report. */ private V selectedValue; public ReportVariable(ReportVariableDefinition<V> definition) { - this.definition = definition; - } - - @Override - public ReportVariableDefinition<V> getDefinition() { - return definition; + super(definition); } public V getSelectedValue() { @@ -64,7 +58,7 @@ public final class ReportVariable<V> extends ReportVariableSupport<V> { } public boolean isValuesLoadedAndSelectedValueFilled() { - return isValuesLoaded() && getSelectedValue() != null; + return isValuesLoaded() && isSelectedValueFilled(); } @Override ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariableSupport.java ===================================== @@ -22,7 +22,7 @@ package fr.ird.observe.report; * #L% */ -import fr.ird.observe.report.definition.ReportVariableSupportDefinition; +import fr.ird.observe.report.definition.ReportVariableDefinitionSupport; import io.ultreia.java4all.util.json.JsonAware; import java.io.Serializable; @@ -39,27 +39,38 @@ import java.util.Set; * @see ReportRepeatVariable * @since 9.0.22 */ -public abstract class ReportVariableSupport<V> implements Serializable, JsonAware { +public abstract class ReportVariableSupport<V, D extends ReportVariableDefinitionSupport<V>> implements Serializable, JsonAware { + public final static String PROPERTY_DEFINITION = "definition"; public final static String PROPERTY_VALUES = "values"; /** - * Found values to this variable. + * Definition of this variable. */ - protected Set<V> values; + private final D definition; + /** + * Values loaded for this variable. + */ + private Set<V> values; - public abstract ReportVariableSupportDefinition<V> getDefinition(); + protected ReportVariableSupport(D definition) { + this.definition = definition; + } + + public final D definition() { + return definition; + } public final String getName() { - return getDefinition().getName(); + return definition().getName(); } public final Class<V> getType() { - return getDefinition().getType(); + return definition().getType(); } public final String getRequest() { - return getDefinition().getRequest(); + return definition().getRequest(); } public final Set<V> getValues() { @@ -82,12 +93,12 @@ public abstract class ReportVariableSupport<V> implements Serializable, JsonAwar public final boolean equals(Object o) { if (this == o) return true; if (o == null || !getClass().equals(o.getClass())) return false; - ReportVariableSupport<?> that = (ReportVariableSupport<?>) o; - return Objects.equals(getDefinition(), that.getDefinition()); + ReportVariableSupport<?, ?> that = (ReportVariableSupport<?, ?>) o; + return Objects.equals(definition(), that.definition()); } @Override public final int hashCode() { - return Objects.hash(getDefinition()); + return Objects.hash(definition()); } } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/DefaultReportDefinitionsBuilder.java ===================================== @@ -139,7 +139,7 @@ public class DefaultReportDefinitionsBuilder implements ReportDefinitionsBuilder String[] rowHeaders = loadHeaders(reportName, REPORT_ROWS, reportProperties); String[] columnHeaders = loadHeaders(reportName, REPORT_COLUMNS, reportProperties); List<ReportVariableDefinition<?>> variables = getVariables(reportName, reportProperties); - Set<String> variablesNames = variables.stream().map(ReportVariableSupportDefinition::getName).collect(Collectors.toSet()); + Set<String> variablesNames = variables.stream().map(ReportVariableDefinition::getName).collect(Collectors.toSet()); List<ReportRepeatVariableDefinition<?>> repeatVariables = getRepeatVariables(reportName, reportProperties, variablesNames); ReportRequestDefinition[] requests = getRequests(reportName, reportProperties); List<ReportOperationDefinition> operations = getOperations(reportName, reportProperties); ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportDefinition.java ===================================== @@ -94,7 +94,7 @@ public final class ReportDefinition implements Serializable, JsonAware { /** * Cache of dependencies on variables (key is variable name, value is the required variable names to be able to populate the variable). */ - private final transient Map<String, Set<String>> variableDependencies; + private transient Map<String, Set<String>> variableDependencies; public ReportDefinition(String modelType, String id, @@ -116,8 +116,6 @@ public final class ReportDefinition implements Serializable, JsonAware { this.operations = operations; this.variables = variables; this.repeatVariables = repeatVariables; - Map<String, ReportVariableDefinition<?>> variableByName = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariableDefinition::getName, Function.identity()))); - this.variableDependencies = computeVariableDependencies(variableByName); } public String getModelType() { @@ -149,6 +147,9 @@ public final class ReportDefinition implements Serializable, JsonAware { } public Map<String, Set<String>> getVariableDependencies() { + if (variableDependencies == null) { + this.variableDependencies = computeVariableDependencies(); + } return variableDependencies; } @@ -181,8 +182,8 @@ public final class ReportDefinition implements Serializable, JsonAware { return t(name); } - static Map<String, Set<String>> computeVariableDependencies(Map<String, ReportVariableDefinition<?>> variableByNames) { - + private Map<String, Set<String>> computeVariableDependencies() { + Map<String, ReportVariableDefinition<?>> variableByNames = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariableDefinition::getName, Function.identity()))); Set<String> variableNames = variableByNames.keySet(); Map<String, Set<String>> variableDependencies = new TreeMap<>(); for (Map.Entry<String, ReportVariableDefinition<?>> entry : variableByNames.entrySet()) { ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportOperationDefinition.java ===================================== @@ -38,6 +38,9 @@ public final class ReportOperationDefinition implements JsonAware { * Name of the operation, this is the fully qualified name of the ReportOperationConsumer. */ private final String name; + /** + * Optional parameters of this operation. + */ private final String parameters; public static ReportOperationDefinition of(String name) { ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportRepeatVariableDefinition.java ===================================== @@ -30,9 +30,7 @@ import fr.ird.observe.report.ReportRepeatVariable; * @author Tony Chemit - dev(a)tchemit.fr * @since 9.0.23 */ -public final class ReportRepeatVariableDefinition<V> extends ReportVariableSupportDefinition<V> { - - public final static String PROPERTY_ADD_NULL_VALUE = "addNullValue"; +public final class ReportRepeatVariableDefinition<V> extends ReportVariableDefinitionSupport<V> { private static final long serialVersionUID = 1L; ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinition.java ===================================== @@ -33,7 +33,7 @@ import fr.ird.observe.report.ReportVariable; * @author Tony Chemit - dev(a)tchemit.fr * @since 2.0.23 */ -public final class ReportVariableDefinition<V> extends ReportVariableSupportDefinition<V> { +public final class ReportVariableDefinition<V> extends ReportVariableDefinitionSupport<V> { public ReportVariableDefinition(String name, Class<V> type, String request, String comment) { super(name, type, request, comment); ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableSupportDefinition.java → toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinitionSupport.java ===================================== @@ -37,13 +37,7 @@ import java.util.Objects; * @author Tony Chemit - dev(a)tchemit.fr * @since 2.0.23 */ -public abstract class ReportVariableSupportDefinition<V> implements Serializable, JsonAware { - - public final static String PROPERTY_NAME = "name"; - public final static String PROPERTY_TYPE = "type"; - public final static String PROPERTY_REQUEST = "request"; - public final static String PROPERTY_COMMENT = "comment"; - +public abstract class ReportVariableDefinitionSupport<V> implements Serializable, JsonAware { /** * Name of the variable. */ @@ -61,7 +55,7 @@ public abstract class ReportVariableSupportDefinition<V> implements Serializable */ private final String comment; - public ReportVariableSupportDefinition(String name, Class<V> type, String request, String comment) { + public ReportVariableDefinitionSupport(String name, Class<V> type, String request, String comment) { this.name = Objects.requireNonNull(name); this.type = Objects.requireNonNull(type); this.request = Objects.requireNonNull(request); @@ -88,13 +82,13 @@ public abstract class ReportVariableSupportDefinition<V> implements Serializable return comment != null; } - public abstract ReportVariableSupport<?> toVariable(); + public abstract ReportVariableSupport<?, ?> toVariable(); @Override public final boolean equals(Object o) { if (this == o) return true; if (o == null || !getClass().equals(o.getClass())) return false; - ReportVariableSupportDefinition<?> that = (ReportVariableSupportDefinition<?>) o; + ReportVariableDefinitionSupport<?> that = (ReportVariableDefinitionSupport<?>) o; return Objects.equals(name, that.name); } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportRepeatVariableAdapter.java ===================================== @@ -24,10 +24,8 @@ package fr.ird.observe.report.json; import com.google.auto.service.AutoService; import com.google.gson.JsonDeserializationContext; -import com.google.gson.JsonDeserializer; -import com.google.gson.JsonElement; import com.google.gson.JsonObject; -import com.google.gson.JsonParseException; +import com.google.gson.reflect.TypeToken; import fr.ird.observe.report.ReportRepeatVariable; import fr.ird.observe.report.definition.ReportRepeatVariableDefinition; import io.ultreia.java4all.util.json.JsonAdapter; @@ -41,7 +39,7 @@ import java.lang.reflect.Type; * @since 9.0.22 */ @AutoService(JsonAdapter.class) -public class ReportRepeatVariableAdapter extends ReportVariableSupportAdapter implements JsonDeserializer<ReportRepeatVariable<?>> { +public class ReportRepeatVariableAdapter<V> extends ReportVariableSupportAdapter<V, ReportRepeatVariableDefinition<V>, ReportRepeatVariable<V>> { @Override public Class<?> type() { @@ -49,22 +47,15 @@ public class ReportRepeatVariableAdapter extends ReportVariableSupportAdapter im } @Override - public ReportRepeatVariable<?> deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) throws JsonParseException { - JsonObject jsonObject = json.getAsJsonObject(); - - String name = getVariableName(context, jsonObject); - Class<?> type = getVariableType(context, jsonObject); - String request = getVariableRequest(context, jsonObject); - String comment = getVariableComment(context, jsonObject); - boolean addNullValue = getAddNullValue(context, jsonObject); - ReportRepeatVariableDefinition<?> definition = new ReportRepeatVariableDefinition<>(name, type, request, comment, addNullValue); - ReportRepeatVariable<?> result = new ReportRepeatVariable<>(definition); - - loadVariableValues(context, jsonObject, result); - return result; + protected Type definitionType() { + return new TypeToken<ReportRepeatVariableDefinition<V>>() { + }.getType(); } - protected boolean getAddNullValue(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportRepeatVariableDefinition.PROPERTY_ADD_NULL_VALUE), boolean.class); + @Override + protected ReportRepeatVariable<V> deserialize(JsonDeserializationContext context, JsonObject jsonObject, ReportRepeatVariableDefinition<V> definition) { + ReportRepeatVariable<V> result = new ReportRepeatVariable<>(definition); + deserializeValues(context, jsonObject, result); + return result; } } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableAdapter.java ===================================== @@ -25,10 +25,9 @@ package fr.ird.observe.report.json; import com.google.auto.service.AutoService; import com.google.gson.JsonDeserializationContext; -import com.google.gson.JsonDeserializer; import com.google.gson.JsonElement; import com.google.gson.JsonObject; -import com.google.gson.JsonParseException; +import com.google.gson.reflect.TypeToken; import fr.ird.observe.report.ReportVariable; import fr.ird.observe.report.definition.ReportVariableDefinition; import io.ultreia.java4all.util.json.JsonAdapter; @@ -39,7 +38,7 @@ import java.lang.reflect.Type; * @author Tony Chemit - dev(a)tchemit.fr */ @AutoService(JsonAdapter.class) -public class ReportVariableAdapter extends ReportVariableSupportAdapter implements JsonDeserializer<ReportVariable<?>> { +public class ReportVariableAdapter<V> extends ReportVariableSupportAdapter<V, ReportVariableDefinition<V>, ReportVariable<V>> { @Override public Class<?> type() { @@ -47,27 +46,22 @@ public class ReportVariableAdapter extends ReportVariableSupportAdapter implemen } @Override - public ReportVariable<?> deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) throws JsonParseException { - JsonObject jsonObject = json.getAsJsonObject(); - - String name = getVariableName(context, jsonObject); - Class<?> type = getVariableType(context, jsonObject); - String request = getVariableRequest(context, jsonObject); - String comment = getVariableComment(context, jsonObject); - - ReportVariableDefinition<?> definition = new ReportVariableDefinition<>(name, type, request, comment); - ReportVariable<?> result = new ReportVariable<>(definition); + protected Type definitionType() { + return new TypeToken<ReportVariableDefinition<V>>() { + }.getType(); + } - loadVariableValues(context, jsonObject, result); - loadSelectedValue(context, jsonObject, result); + @Override + protected ReportVariable<V> deserialize(JsonDeserializationContext context, JsonObject jsonObject, ReportVariableDefinition<V> definition) { + ReportVariable<V> result = new ReportVariable<>(definition); + deserializeValues(context, jsonObject, result); + deserializeSelectedValue(context, jsonObject, result); return result; } - protected <V> void loadSelectedValue(JsonDeserializationContext context, JsonObject jsonObject, ReportVariable<V> variable) { + protected void deserializeSelectedValue(JsonDeserializationContext context, JsonObject jsonObject, ReportVariable<V> variable) { JsonElement json = jsonObject.get(ReportVariable.PROPERTY_SELECTED_VALUE); - if (json != null) { - V selectedValue = context.deserialize(json, variable.getType()); - variable.setSelectedValue(selectedValue); - } + V selectedValue = deserializeValue(context, variable.getType(), json); + variable.setSelectedValue(selectedValue); } } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableSupportAdapter.java ===================================== @@ -22,13 +22,17 @@ package fr.ird.observe.report.json; * #L% */ +import com.google.gson.JsonArray; import com.google.gson.JsonDeserializationContext; +import com.google.gson.JsonDeserializer; import com.google.gson.JsonElement; import com.google.gson.JsonObject; +import com.google.gson.JsonParseException; import fr.ird.observe.report.ReportVariableSupport; -import fr.ird.observe.report.definition.ReportVariableSupportDefinition; +import fr.ird.observe.report.definition.ReportVariableDefinitionSupport; import io.ultreia.java4all.util.json.JsonAdapter; +import java.lang.reflect.Type; import java.util.HashSet; import java.util.Set; @@ -40,34 +44,38 @@ import java.util.Set; * @author Tony Chemit - dev(a)tchemit.fr * @since 9.0.22 */ -public abstract class ReportVariableSupportAdapter implements JsonAdapter { +public abstract class ReportVariableSupportAdapter<V, D extends ReportVariableDefinitionSupport<V>, R extends ReportVariableSupport<V, D>> implements JsonAdapter, JsonDeserializer<R> { - protected String getVariableName(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_NAME), String.class); - } + protected abstract Type definitionType(); - protected Class<?> getVariableType(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_TYPE), Class.class); - } + protected abstract R deserialize(JsonDeserializationContext context, JsonObject jsonObject, D definition); - protected String getVariableRequest(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_REQUEST), String.class); + @Override + public final R deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) throws JsonParseException { + JsonObject jsonObject = json.getAsJsonObject(); + D definition = deserializeDefinition(context, jsonObject); + return deserialize(context, jsonObject, definition); } - protected String getVariableComment(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_COMMENT), String.class); + protected D deserializeDefinition(JsonDeserializationContext context, JsonObject jsonObject) { + return context.deserialize(jsonObject.get(ReportVariableSupport.PROPERTY_DEFINITION), definitionType()); } - protected <V> void loadVariableValues(JsonDeserializationContext context, JsonObject jsonObject, ReportVariableSupport<V> variable) { + protected void deserializeValues(JsonDeserializationContext context, JsonObject jsonObject, R variable) { JsonElement valuesJson = jsonObject.get(ReportVariableSupport.PROPERTY_VALUES); if (valuesJson != null) { - Set<V> values = new HashSet<>(); Class<V> type = variable.getType(); - for (JsonElement valueJson : valuesJson.getAsJsonArray()) { - V value = context.deserialize(valueJson, type); + JsonArray jsonArray = valuesJson.getAsJsonArray(); + Set<V> values = new HashSet<>(jsonArray.size()); + for (JsonElement valueJson : jsonArray) { + V value = deserializeValue(context, type, valueJson); values.add(value); } variable.setValues(values); } } + + protected V deserializeValue(JsonDeserializationContext context, Class<V> variableType, JsonElement valueJson) { + return valueJson == null ? null : context.deserialize(valueJson, variableType); + } } View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/a4f58b7468ccf53188522b75… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/a4f58b7468ccf53188522b75… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop] 2 commits: Report API - Review it and make json compliant
by Tony CHEMIT (＠tchemit) 05 Jan '23

05 Jan '23

Tony CHEMIT pushed to branch develop at ultreiaio / ird-observe Commits: 74be88a7 by Tony Chemit at 2023-01-05T13:36:50+01:00 Report API - Review it and make json compliant - - - - - d17023ba by Tony Chemit at 2023-01-05T13:36:54+01:00 Améliorations de libellés de l'arbre - Closes #2595 - - - - - 17 changed files: - client/datasource/actions/src/main/java/fr/ird/observe/client/datasource/actions/report/ReportModel.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUINavigationHandler.java - core/services/i18n/src/main/i18n/translations/services_en_GB.properties - core/services/i18n/src/main/i18n/translations/services_fr_FR.properties - toolkit/api-report/src/main/java/fr/ird/observe/report/Report.java - toolkit/api-report/src/main/java/fr/ird/observe/report/ReportRepeatVariable.java - toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariable.java - toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariableSupport.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/DefaultReportDefinitionsBuilder.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportOperationDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportRepeatVariableDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinition.java - toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableSupportDefinition.java → toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinitionSupport.java - toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportRepeatVariableAdapter.java - toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableAdapter.java - toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableSupportAdapter.java Changes: ===================================== client/datasource/actions/src/main/java/fr/ird/observe/client/datasource/actions/report/ReportModel.java ===================================== @@ -342,7 +342,7 @@ public class ReportModel extends AdminActionModel { } public ReportDefinition getSelectedReportDefinition() { - return selectedReport == null ? null : selectedReport.getDefinition(); + return selectedReport == null ? null : selectedReport.definition(); } public void setSelectedReport(Report selectedReport) { ===================================== client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUINavigationHandler.java ===================================== @@ -64,7 +64,7 @@ public abstract class ContentTableUINavigationHandler<N extends ContentTableUINa @Override public String getText() { - String result = getNode().getScope().getI18nTranslation("type"); + String result = getNode().getScope().getI18nTranslation("title"); Long count = getNode().getInitializer().getCount(); if (count != null) { result += String.format(" (%d)", count); ===================================== core/services/i18n/src/main/i18n/translations/services_en_GB.properties ===================================== @@ -862,13 +862,13 @@ observe.data.ll.observation.SetDetailComposition.section.delete.message=Confirm observe.data.ll.observation.SetDetailComposition.sectionTemplate.delete=Confirm to delete current template. observe.data.ll.observation.SetDetailComposition.sections=Sections observe.data.ll.observation.SetDetailComposition.title.section.cant.use.firstTemplate=Can't use the first template while generating -observe.data.ll.observation.SetDetailComposition.type=Detailed compositions +observe.data.ll.observation.SetDetailComposition.type=Detailed composition observe.data.ll.observation.SetGlobalComposition.baitsCompositionTab=Baits observe.data.ll.observation.SetGlobalComposition.branchlinesCompositionTab=Branchlines observe.data.ll.observation.SetGlobalComposition.floatlinesCompositionTab=Floatlines observe.data.ll.observation.SetGlobalComposition.hooksCompositionTab=Hooks observe.data.ll.observation.SetGlobalComposition.mitigationTypeTab=Mitigation -observe.data.ll.observation.SetGlobalComposition.type=Global compositions +observe.data.ll.observation.SetGlobalComposition.type=Global composition observe.data.ll.observation.SetTdr.type=Fishing operation tdr observe.data.ll.observation.SizeMeasure.size=Size (cm) observe.data.ll.observation.SizeMeasure.size.short=Size (cm) @@ -996,7 +996,7 @@ observe.data.ps.localmarket.Batch.count.short=Count observe.data.ps.localmarket.Batch.count.validation.required=Count or weight must be filled (but not both). observe.data.ps.localmarket.Batch.date=Date observe.data.ps.localmarket.Batch.title=Batches -observe.data.ps.localmarket.Batch.type=Batches +observe.data.ps.localmarket.Batch.type=Batch observe.data.ps.localmarket.Batch.validation.date.after.currentPsCommonTrip.endDate=Date (%2$s) must be after trip end date (%1$s). observe.data.ps.localmarket.Batch.validation.date.after.currentPsCommonTrip.startDate=Date (%2$s) must be after trip start date (%1$s). observe.data.ps.localmarket.Batch.validation.date.delay.too.long=Delay in days with trip end date is %d and can not be more than %s days. @@ -1351,7 +1351,7 @@ observe.data.ps.observation.ObjectObservedSpecies.type=Object observed species observe.data.ps.observation.ObjectSchoolEstimate.action.create=Create a new estimate observe.data.ps.observation.ObjectSchoolEstimate.action.save=Insert this estimate observe.data.ps.observation.ObjectSchoolEstimate.action.save.tip=Insert this estimate -observe.data.ps.observation.ObjectSchoolEstimate.title=School estimate +observe.data.ps.observation.ObjectSchoolEstimate.title=Object School estimate observe.data.ps.observation.ObjectSchoolEstimate.totalWeight=Weight (in t) observe.data.ps.observation.ObjectSchoolEstimate.totalWeight.short=Total weight observe.data.ps.observation.ObjectSchoolEstimate.type=Object school estimate ===================================== core/services/i18n/src/main/i18n/translations/services_fr_FR.properties ===================================== @@ -471,8 +471,8 @@ observe.data.GearUseFeatures.gear=Équipement observe.data.GearUseFeatures.gear.short=Équipement observe.data.GearUseFeatures.number=Nombre observe.data.GearUseFeatures.number.short=Nombre -observe.data.GearUseFeatures.title=Équipement du bateau -observe.data.GearUseFeatures.type=Équipement du bateau +observe.data.GearUseFeatures.title=Équipements du navire +observe.data.GearUseFeatures.type=Équipement du navire observe.data.GearUseFeaturesMeasurement.type=Caractéristique d'un équipement observe.data.InlineData.action.add=Nouvelle entrée observe.data.InlineData.action.add.tip=Ajouter une nouvelle donnée de type «%s» @@ -714,7 +714,7 @@ observe.data.ll.logbook.SetGlobalComposition.branchlinesCompositionTab=Avançons observe.data.ll.logbook.SetGlobalComposition.floatlinesCompositionTab=Orins observe.data.ll.logbook.SetGlobalComposition.hooksCompositionTab=Hameçons observe.data.ll.logbook.SetGlobalComposition.mitigationTypeTab=Atténuations -observe.data.ll.logbook.SetGlobalComposition.type=Compositions globales +observe.data.ll.logbook.SetGlobalComposition.type=Composition globale observe.data.ll.observation.Activity.action.create=Activité suivante observe.data.ll.observation.Activity.action.move=Changer de marée observe.data.ll.observation.Activity.action.move.choose.parent.message=À quelle marée voulez-vous associer les activités sélectionnées ? @@ -729,8 +729,8 @@ observe.data.ll.observation.Activity.sensorUsedStat=Nombre de capteurs utilisés observe.data.ll.observation.Activity.setStat=Nombre d'opérations de pêche observe.data.ll.observation.Activity.title=Activité observe.data.ll.observation.Activity.type=Activité -observe.data.ll.observation.ActivityEncounter.type=Rencontre -observe.data.ll.observation.ActivitySensorUsed.type=Capteur +observe.data.ll.observation.ActivityEncounter.type=Rencontres +observe.data.ll.observation.ActivitySensorUsed.type=Capteurs utilisés observe.data.ll.observation.BaitsComposition.action.create=Nouvel appât observe.data.ll.observation.BaitsComposition.action.save=Insérer cet appât observe.data.ll.observation.BaitsComposition.action.save.tip=Insérer cet appât @@ -798,7 +798,7 @@ observe.data.ll.observation.SensorUsed.action.create=Nouveau capteur observe.data.ll.observation.SensorUsed.action.save=Insérer ce capteur observe.data.ll.observation.SensorUsed.action.save.tip=Insérer ce capteur observe.data.ll.observation.SensorUsed.dataLocation.short=Hébergement -observe.data.ll.observation.SensorUsed.title=Capteurs +observe.data.ll.observation.SensorUsed.title=Capteurs utilisés observe.data.ll.observation.SensorUsed.type=Capteur utilisé observe.data.ll.observation.Set.action.add=Ajouter l'opération de pêche observe.data.ll.observation.Set.action.add.tip=Ajouter l'opération de pêche associée à cette activité @@ -862,13 +862,13 @@ observe.data.ll.observation.SetDetailComposition.section.delete.message=Confirme observe.data.ll.observation.SetDetailComposition.sectionTemplate.delete=Confirmer la suppression du patron sélectionné. observe.data.ll.observation.SetDetailComposition.sections=Sections observe.data.ll.observation.SetDetailComposition.title.section.cant.use.firstTemplate=Impossible d'utiliser le premier modèle lors de la génération -observe.data.ll.observation.SetDetailComposition.type=Compositions détaillées +observe.data.ll.observation.SetDetailComposition.type=Composition détaillée observe.data.ll.observation.SetGlobalComposition.baitsCompositionTab=Appâts observe.data.ll.observation.SetGlobalComposition.branchlinesCompositionTab=Avançons observe.data.ll.observation.SetGlobalComposition.floatlinesCompositionTab=Orins observe.data.ll.observation.SetGlobalComposition.hooksCompositionTab=Hameçons observe.data.ll.observation.SetGlobalComposition.mitigationTypeTab=Atténuations -observe.data.ll.observation.SetGlobalComposition.type=Compositions globales +observe.data.ll.observation.SetGlobalComposition.type=Composition globale observe.data.ll.observation.SetTdr.type=Enregistreur de profondeur observe.data.ll.observation.SizeMeasure.size=Mesure (cm) observe.data.ll.observation.SizeMeasure.size.short=Mesure (cm) @@ -883,7 +883,7 @@ observe.data.ll.observation.Tdr.keyDataTab=Données clef observe.data.ll.observation.Tdr.localisationTab=Localisation observe.data.ll.observation.Tdr.speciesTab=Espèces associées observe.data.ll.observation.Tdr.timestampTab=Horodatage -observe.data.ll.observation.Tdr.title=Enregistreur de profondeur +observe.data.ll.observation.Tdr.title=Enregistreurs de profondeur observe.data.ll.observation.Tdr.type=Enregistreur de profondeur observe.data.ll.observation.Tdr.validation.deploymentEnd.after.fishingEnd=L'horodatage de fin de déploiement (%2$s) doit être après le début de celui de fin de pêche (%1$s). observe.data.ll.observation.Tdr.validation.fishingEnd.after.fishingStart=L'horodatage de fin de pêche (%2$s) doit être après le début de celui de début de pêche (%1$s). @@ -1351,7 +1351,7 @@ observe.data.ps.observation.ObjectObservedSpecies.type=Faune observée observe.data.ps.observation.ObjectSchoolEstimate.action.create=Nouvelle estimation observe.data.ps.observation.ObjectSchoolEstimate.action.save=Insérer cette estimation observe.data.ps.observation.ObjectSchoolEstimate.action.save.tip=Insérer la nouvelle estimation dans la table -observe.data.ps.observation.ObjectSchoolEstimate.title=Estimation banc +observe.data.ps.observation.ObjectSchoolEstimate.title=Estimation banc objet observe.data.ps.observation.ObjectSchoolEstimate.totalWeight=Poids (en t) observe.data.ps.observation.ObjectSchoolEstimate.totalWeight.short=Poids total observe.data.ps.observation.ObjectSchoolEstimate.type=Estimation banc objet ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/Report.java ===================================== @@ -59,11 +59,11 @@ public final class Report implements Serializable, JsonAware { /** * Cache of variables of this report indexed by their name. */ - private final transient Map<String, ReportVariable<?>> variableByName; + private transient Map<String, ReportVariable<?>> variableByName; /** * Cache of repeat variables of this report indexed by their name. */ - private final transient Map<String, ReportRepeatVariable<?>> repeatVariableByName; + private transient Map<String, ReportRepeatVariable<?>> repeatVariableByName; public Report(ReportDefinition definition) { this.definition = Objects.requireNonNull(definition); @@ -82,82 +82,90 @@ public final class Report implements Serializable, JsonAware { ReportRepeatVariableDefinition<?> definitionRepeatVariable = definitionRepeatVariables[i]; this.repeatVariables[i] = definitionRepeatVariable.toVariable(); } - this.variableByName = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariableSupport::getName, Function.identity()))); - this.repeatVariableByName = Collections.unmodifiableMap(Arrays.stream(repeatVariables).collect(Collectors.toMap(ReportVariableSupport::getName, Function.identity()))); } - public ReportDefinition getDefinition() { + public ReportDefinition definition() { return definition; } + public String getModelType() { + return definition().getModelType(); + } + public String getId() { - return getDefinition().getId(); + return definition().getId(); } public String getName() { - return getDefinition().getName(); + return definition().getName(); } public String getDescription() { - return getDefinition().getDescription(); + return definition().getDescription(); } public int getRows() { - return getDefinition().getRows(); + return definition().getRows(); } public int getColumns() { - return getDefinition().getColumns(); + return definition().getColumns(); } public String[] getColumnHeaders() { - return getDefinition().getColumnHeaders(); + return definition().getColumnHeaders(); } public String[] getRowHeaders() { - return getDefinition().getRowHeaders(); + return definition().getRowHeaders(); } public ReportRequestDefinition[] getRequests() { - return getDefinition().getRequests(); + return definition().getRequests(); } public ReportOperationDefinition[] getOperations() { - return getDefinition().getOperations(); + return definition().getOperations(); } - public ReportVariable<?>[] getVariables() { - return variables; + public Map<String, Set<String>> getVariableDependencies() { + return definition().getVariableDependencies(); } - public Map<String, Set<String>> getVariableDependencies() { - return getDefinition().getVariableDependencies(); + public ReportVariable<?>[] getVariables() { + return variables; } public Map<String, ReportVariable<?>> getVariableByName() { + if (variableByName == null) { + this.variableByName = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariable::getName, Function.identity()))); + } return variableByName; } public ReportVariable<?> getVariable(String variableName) { - return variableByName.get(variableName); + return getVariableByName().get(variableName); } public ReportRepeatVariable<?>[] getRepeatVariables() { return repeatVariables; } - public String getModelType() { - return getDefinition().getModelType(); - } - - @Override - public String toString() { - return getDefinition().toString(); + public Map<String, ReportRepeatVariable<?>> getRepeatVariableByName() { + if (repeatVariableByName == null) { + this.repeatVariableByName = Collections.unmodifiableMap(Arrays.stream(repeatVariables).collect(Collectors.toMap(ReportRepeatVariable::getName, Function.identity()))); + } + return repeatVariableByName; } @SuppressWarnings("unchecked") public <V> ReportRepeatVariable<V> getRepeatVariable(String name) { - return (ReportRepeatVariable<V>) repeatVariableByName.get(name); + return (ReportRepeatVariable<V>) getRepeatVariableByName().get(name); + } + + @Override + public String toString() { + return definition().toString(); } /** @@ -246,12 +254,13 @@ public final class Report implements Serializable, JsonAware { * @param dependencyVariableName the dependency variable name that has changed */ public void unloadVariablesIfNecessary(String dependencyVariableName) { + Map<String, Set<String>> variableDependencyNamesMap = getVariableDependencies(); for (ReportVariable<?> variable : variables) { if (!variable.isValuesLoaded()) { // this variable is not loaded (so can not unload it) continue; } - Set<String> variableDependencyNames = getVariableDependencies().get(variable.getName()); + Set<String> variableDependencyNames = variableDependencyNamesMap.get(variable.getName()); if (variableDependencyNames == null) { // this variable has no dependency continue; @@ -262,4 +271,5 @@ public final class Report implements Serializable, JsonAware { } } } + } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/ReportRepeatVariable.java ===================================== @@ -44,22 +44,16 @@ import java.util.stream.Collectors; * @author Tony Chemit - dev(a)tchemit.fr * @since 9.0.23 */ -public final class ReportRepeatVariable<V> extends ReportVariableSupport<V> { +public final class ReportRepeatVariable<V> extends ReportVariableSupport<V, ReportRepeatVariableDefinition<V>> { private static final long serialVersionUID = 1L; - private final ReportRepeatVariableDefinition<V> definition; public ReportRepeatVariable(ReportRepeatVariableDefinition<V> definition) { - this.definition = definition; - } - - @Override - public ReportRepeatVariableDefinition<V> getDefinition() { - return definition; + super(definition); } public boolean isAddNullValue() { - return getDefinition().isAddNullValue(); + return definition().isAddNullValue(); } @Override @@ -72,7 +66,7 @@ public final class ReportRepeatVariable<V> extends ReportVariableSupport<V> { public List<String> computeIndexList() { //FIXME This is already done by setValues method? Why using a "null" instead of null value? - Set<V> tmp = new LinkedHashSet<>(values); + Set<V> tmp = new LinkedHashSet<>(getValues()); boolean useNullValue = tmp.remove(null); List<String> result = tmp.stream().map(s -> s instanceof ToolkitId ? ((ToolkitId) s).getId() : String.valueOf(s)).collect(Collectors.toList()); if (useNullValue || isAddNullValue()) { ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariable.java ===================================== @@ -30,25 +30,19 @@ import fr.ird.observe.report.definition.ReportVariableDefinition; * @author Tony Chemit - dev(a)tchemit.fr * @since 1.7 */ -public final class ReportVariable<V> extends ReportVariableSupport<V> { +public final class ReportVariable<V> extends ReportVariableSupport<V, ReportVariableDefinition<V>> { public final static String PROPERTY_SELECTED_VALUE = "selectedValue"; private static final long serialVersionUID = 1L; - private final ReportVariableDefinition<V> definition; /** * Selected value for this variable, that will be used to execute report. */ private V selectedValue; public ReportVariable(ReportVariableDefinition<V> definition) { - this.definition = definition; - } - - @Override - public ReportVariableDefinition<V> getDefinition() { - return definition; + super(definition); } public V getSelectedValue() { @@ -64,7 +58,7 @@ public final class ReportVariable<V> extends ReportVariableSupport<V> { } public boolean isValuesLoadedAndSelectedValueFilled() { - return isValuesLoaded() && getSelectedValue() != null; + return isValuesLoaded() && isSelectedValueFilled(); } @Override ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/ReportVariableSupport.java ===================================== @@ -22,7 +22,7 @@ package fr.ird.observe.report; * #L% */ -import fr.ird.observe.report.definition.ReportVariableSupportDefinition; +import fr.ird.observe.report.definition.ReportVariableDefinitionSupport; import io.ultreia.java4all.util.json.JsonAware; import java.io.Serializable; @@ -39,27 +39,38 @@ import java.util.Set; * @see ReportRepeatVariable * @since 9.0.22 */ -public abstract class ReportVariableSupport<V> implements Serializable, JsonAware { +public abstract class ReportVariableSupport<V, D extends ReportVariableDefinitionSupport<V>> implements Serializable, JsonAware { + public final static String PROPERTY_DEFINITION = "definition"; public final static String PROPERTY_VALUES = "values"; /** - * Found values to this variable. + * Definition of this variable. */ - protected Set<V> values; + private final D definition; + /** + * Values loaded for this variable. + */ + private Set<V> values; - public abstract ReportVariableSupportDefinition<V> getDefinition(); + protected ReportVariableSupport(D definition) { + this.definition = definition; + } + + public final D definition() { + return definition; + } public final String getName() { - return getDefinition().getName(); + return definition().getName(); } public final Class<V> getType() { - return getDefinition().getType(); + return definition().getType(); } public final String getRequest() { - return getDefinition().getRequest(); + return definition().getRequest(); } public final Set<V> getValues() { @@ -82,12 +93,12 @@ public abstract class ReportVariableSupport<V> implements Serializable, JsonAwar public final boolean equals(Object o) { if (this == o) return true; if (o == null || !getClass().equals(o.getClass())) return false; - ReportVariableSupport<?> that = (ReportVariableSupport<?>) o; - return Objects.equals(getDefinition(), that.getDefinition()); + ReportVariableSupport<?, ?> that = (ReportVariableSupport<?, ?>) o; + return Objects.equals(definition(), that.definition()); } @Override public final int hashCode() { - return Objects.hash(getDefinition()); + return Objects.hash(definition()); } } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/DefaultReportDefinitionsBuilder.java ===================================== @@ -139,7 +139,7 @@ public class DefaultReportDefinitionsBuilder implements ReportDefinitionsBuilder String[] rowHeaders = loadHeaders(reportName, REPORT_ROWS, reportProperties); String[] columnHeaders = loadHeaders(reportName, REPORT_COLUMNS, reportProperties); List<ReportVariableDefinition<?>> variables = getVariables(reportName, reportProperties); - Set<String> variablesNames = variables.stream().map(ReportVariableSupportDefinition::getName).collect(Collectors.toSet()); + Set<String> variablesNames = variables.stream().map(ReportVariableDefinition::getName).collect(Collectors.toSet()); List<ReportRepeatVariableDefinition<?>> repeatVariables = getRepeatVariables(reportName, reportProperties, variablesNames); ReportRequestDefinition[] requests = getRequests(reportName, reportProperties); List<ReportOperationDefinition> operations = getOperations(reportName, reportProperties); ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportDefinition.java ===================================== @@ -94,7 +94,7 @@ public final class ReportDefinition implements Serializable, JsonAware { /** * Cache of dependencies on variables (key is variable name, value is the required variable names to be able to populate the variable). */ - private final transient Map<String, Set<String>> variableDependencies; + private transient Map<String, Set<String>> variableDependencies; public ReportDefinition(String modelType, String id, @@ -116,8 +116,6 @@ public final class ReportDefinition implements Serializable, JsonAware { this.operations = operations; this.variables = variables; this.repeatVariables = repeatVariables; - Map<String, ReportVariableDefinition<?>> variableByName = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariableDefinition::getName, Function.identity()))); - this.variableDependencies = computeVariableDependencies(variableByName); } public String getModelType() { @@ -149,6 +147,9 @@ public final class ReportDefinition implements Serializable, JsonAware { } public Map<String, Set<String>> getVariableDependencies() { + if (variableDependencies == null) { + this.variableDependencies = computeVariableDependencies(); + } return variableDependencies; } @@ -181,8 +182,8 @@ public final class ReportDefinition implements Serializable, JsonAware { return t(name); } - static Map<String, Set<String>> computeVariableDependencies(Map<String, ReportVariableDefinition<?>> variableByNames) { - + private Map<String, Set<String>> computeVariableDependencies() { + Map<String, ReportVariableDefinition<?>> variableByNames = Collections.unmodifiableMap(Arrays.stream(variables).collect(Collectors.toMap(ReportVariableDefinition::getName, Function.identity()))); Set<String> variableNames = variableByNames.keySet(); Map<String, Set<String>> variableDependencies = new TreeMap<>(); for (Map.Entry<String, ReportVariableDefinition<?>> entry : variableByNames.entrySet()) { ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportOperationDefinition.java ===================================== @@ -38,6 +38,9 @@ public final class ReportOperationDefinition implements JsonAware { * Name of the operation, this is the fully qualified name of the ReportOperationConsumer. */ private final String name; + /** + * Optional parameters of this operation. + */ private final String parameters; public static ReportOperationDefinition of(String name) { ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportRepeatVariableDefinition.java ===================================== @@ -30,9 +30,7 @@ import fr.ird.observe.report.ReportRepeatVariable; * @author Tony Chemit - dev(a)tchemit.fr * @since 9.0.23 */ -public final class ReportRepeatVariableDefinition<V> extends ReportVariableSupportDefinition<V> { - - public final static String PROPERTY_ADD_NULL_VALUE = "addNullValue"; +public final class ReportRepeatVariableDefinition<V> extends ReportVariableDefinitionSupport<V> { private static final long serialVersionUID = 1L; ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinition.java ===================================== @@ -33,7 +33,7 @@ import fr.ird.observe.report.ReportVariable; * @author Tony Chemit - dev(a)tchemit.fr * @since 2.0.23 */ -public final class ReportVariableDefinition<V> extends ReportVariableSupportDefinition<V> { +public final class ReportVariableDefinition<V> extends ReportVariableDefinitionSupport<V> { public ReportVariableDefinition(String name, Class<V> type, String request, String comment) { super(name, type, request, comment); ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableSupportDefinition.java → toolkit/api-report/src/main/java/fr/ird/observe/report/definition/ReportVariableDefinitionSupport.java ===================================== @@ -37,13 +37,7 @@ import java.util.Objects; * @author Tony Chemit - dev(a)tchemit.fr * @since 2.0.23 */ -public abstract class ReportVariableSupportDefinition<V> implements Serializable, JsonAware { - - public final static String PROPERTY_NAME = "name"; - public final static String PROPERTY_TYPE = "type"; - public final static String PROPERTY_REQUEST = "request"; - public final static String PROPERTY_COMMENT = "comment"; - +public abstract class ReportVariableDefinitionSupport<V> implements Serializable, JsonAware { /** * Name of the variable. */ @@ -61,7 +55,7 @@ public abstract class ReportVariableSupportDefinition<V> implements Serializable */ private final String comment; - public ReportVariableSupportDefinition(String name, Class<V> type, String request, String comment) { + public ReportVariableDefinitionSupport(String name, Class<V> type, String request, String comment) { this.name = Objects.requireNonNull(name); this.type = Objects.requireNonNull(type); this.request = Objects.requireNonNull(request); @@ -88,13 +82,13 @@ public abstract class ReportVariableSupportDefinition<V> implements Serializable return comment != null; } - public abstract ReportVariableSupport<?> toVariable(); + public abstract ReportVariableSupport<?, ?> toVariable(); @Override public final boolean equals(Object o) { if (this == o) return true; if (o == null || !getClass().equals(o.getClass())) return false; - ReportVariableSupportDefinition<?> that = (ReportVariableSupportDefinition<?>) o; + ReportVariableDefinitionSupport<?> that = (ReportVariableDefinitionSupport<?>) o; return Objects.equals(name, that.name); } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportRepeatVariableAdapter.java ===================================== @@ -24,10 +24,8 @@ package fr.ird.observe.report.json; import com.google.auto.service.AutoService; import com.google.gson.JsonDeserializationContext; -import com.google.gson.JsonDeserializer; -import com.google.gson.JsonElement; import com.google.gson.JsonObject; -import com.google.gson.JsonParseException; +import com.google.gson.reflect.TypeToken; import fr.ird.observe.report.ReportRepeatVariable; import fr.ird.observe.report.definition.ReportRepeatVariableDefinition; import io.ultreia.java4all.util.json.JsonAdapter; @@ -41,7 +39,7 @@ import java.lang.reflect.Type; * @since 9.0.22 */ @AutoService(JsonAdapter.class) -public class ReportRepeatVariableAdapter extends ReportVariableSupportAdapter implements JsonDeserializer<ReportRepeatVariable<?>> { +public class ReportRepeatVariableAdapter<V> extends ReportVariableSupportAdapter<V, ReportRepeatVariableDefinition<V>, ReportRepeatVariable<V>> { @Override public Class<?> type() { @@ -49,22 +47,15 @@ public class ReportRepeatVariableAdapter extends ReportVariableSupportAdapter im } @Override - public ReportRepeatVariable<?> deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) throws JsonParseException { - JsonObject jsonObject = json.getAsJsonObject(); - - String name = getVariableName(context, jsonObject); - Class<?> type = getVariableType(context, jsonObject); - String request = getVariableRequest(context, jsonObject); - String comment = getVariableComment(context, jsonObject); - boolean addNullValue = getAddNullValue(context, jsonObject); - ReportRepeatVariableDefinition<?> definition = new ReportRepeatVariableDefinition<>(name, type, request, comment, addNullValue); - ReportRepeatVariable<?> result = new ReportRepeatVariable<>(definition); - - loadVariableValues(context, jsonObject, result); - return result; + protected Type definitionType() { + return new TypeToken<ReportRepeatVariableDefinition<V>>() { + }.getType(); } - protected boolean getAddNullValue(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportRepeatVariableDefinition.PROPERTY_ADD_NULL_VALUE), boolean.class); + @Override + protected ReportRepeatVariable<V> deserialize(JsonDeserializationContext context, JsonObject jsonObject, ReportRepeatVariableDefinition<V> definition) { + ReportRepeatVariable<V> result = new ReportRepeatVariable<>(definition); + deserializeValues(context, jsonObject, result); + return result; } } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableAdapter.java ===================================== @@ -25,10 +25,9 @@ package fr.ird.observe.report.json; import com.google.auto.service.AutoService; import com.google.gson.JsonDeserializationContext; -import com.google.gson.JsonDeserializer; import com.google.gson.JsonElement; import com.google.gson.JsonObject; -import com.google.gson.JsonParseException; +import com.google.gson.reflect.TypeToken; import fr.ird.observe.report.ReportVariable; import fr.ird.observe.report.definition.ReportVariableDefinition; import io.ultreia.java4all.util.json.JsonAdapter; @@ -39,7 +38,7 @@ import java.lang.reflect.Type; * @author Tony Chemit - dev(a)tchemit.fr */ @AutoService(JsonAdapter.class) -public class ReportVariableAdapter extends ReportVariableSupportAdapter implements JsonDeserializer<ReportVariable<?>> { +public class ReportVariableAdapter<V> extends ReportVariableSupportAdapter<V, ReportVariableDefinition<V>, ReportVariable<V>> { @Override public Class<?> type() { @@ -47,27 +46,22 @@ public class ReportVariableAdapter extends ReportVariableSupportAdapter implemen } @Override - public ReportVariable<?> deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) throws JsonParseException { - JsonObject jsonObject = json.getAsJsonObject(); - - String name = getVariableName(context, jsonObject); - Class<?> type = getVariableType(context, jsonObject); - String request = getVariableRequest(context, jsonObject); - String comment = getVariableComment(context, jsonObject); - - ReportVariableDefinition<?> definition = new ReportVariableDefinition<>(name, type, request, comment); - ReportVariable<?> result = new ReportVariable<>(definition); + protected Type definitionType() { + return new TypeToken<ReportVariableDefinition<V>>() { + }.getType(); + } - loadVariableValues(context, jsonObject, result); - loadSelectedValue(context, jsonObject, result); + @Override + protected ReportVariable<V> deserialize(JsonDeserializationContext context, JsonObject jsonObject, ReportVariableDefinition<V> definition) { + ReportVariable<V> result = new ReportVariable<>(definition); + deserializeValues(context, jsonObject, result); + deserializeSelectedValue(context, jsonObject, result); return result; } - protected <V> void loadSelectedValue(JsonDeserializationContext context, JsonObject jsonObject, ReportVariable<V> variable) { + protected void deserializeSelectedValue(JsonDeserializationContext context, JsonObject jsonObject, ReportVariable<V> variable) { JsonElement json = jsonObject.get(ReportVariable.PROPERTY_SELECTED_VALUE); - if (json != null) { - V selectedValue = context.deserialize(json, variable.getType()); - variable.setSelectedValue(selectedValue); - } + V selectedValue = deserializeValue(context, variable.getType(), json); + variable.setSelectedValue(selectedValue); } } ===================================== toolkit/api-report/src/main/java/fr/ird/observe/report/json/ReportVariableSupportAdapter.java ===================================== @@ -22,13 +22,17 @@ package fr.ird.observe.report.json; * #L% */ +import com.google.gson.JsonArray; import com.google.gson.JsonDeserializationContext; +import com.google.gson.JsonDeserializer; import com.google.gson.JsonElement; import com.google.gson.JsonObject; +import com.google.gson.JsonParseException; import fr.ird.observe.report.ReportVariableSupport; -import fr.ird.observe.report.definition.ReportVariableSupportDefinition; +import fr.ird.observe.report.definition.ReportVariableDefinitionSupport; import io.ultreia.java4all.util.json.JsonAdapter; +import java.lang.reflect.Type; import java.util.HashSet; import java.util.Set; @@ -40,34 +44,38 @@ import java.util.Set; * @author Tony Chemit - dev(a)tchemit.fr * @since 9.0.22 */ -public abstract class ReportVariableSupportAdapter implements JsonAdapter { +public abstract class ReportVariableSupportAdapter<V, D extends ReportVariableDefinitionSupport<V>, R extends ReportVariableSupport<V, D>> implements JsonAdapter, JsonDeserializer<R> { - protected String getVariableName(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_NAME), String.class); - } + protected abstract Type definitionType(); - protected Class<?> getVariableType(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_TYPE), Class.class); - } + protected abstract R deserialize(JsonDeserializationContext context, JsonObject jsonObject, D definition); - protected String getVariableRequest(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_REQUEST), String.class); + @Override + public final R deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) throws JsonParseException { + JsonObject jsonObject = json.getAsJsonObject(); + D definition = deserializeDefinition(context, jsonObject); + return deserialize(context, jsonObject, definition); } - protected String getVariableComment(JsonDeserializationContext context, JsonObject jsonObject) { - return context.deserialize(jsonObject.get(ReportVariableSupportDefinition.PROPERTY_COMMENT), String.class); + protected D deserializeDefinition(JsonDeserializationContext context, JsonObject jsonObject) { + return context.deserialize(jsonObject.get(ReportVariableSupport.PROPERTY_DEFINITION), definitionType()); } - protected <V> void loadVariableValues(JsonDeserializationContext context, JsonObject jsonObject, ReportVariableSupport<V> variable) { + protected void deserializeValues(JsonDeserializationContext context, JsonObject jsonObject, R variable) { JsonElement valuesJson = jsonObject.get(ReportVariableSupport.PROPERTY_VALUES); if (valuesJson != null) { - Set<V> values = new HashSet<>(); Class<V> type = variable.getType(); - for (JsonElement valueJson : valuesJson.getAsJsonArray()) { - V value = context.deserialize(valueJson, type); + JsonArray jsonArray = valuesJson.getAsJsonArray(); + Set<V> values = new HashSet<>(jsonArray.size()); + for (JsonElement valueJson : jsonArray) { + V value = deserializeValue(context, type, valueJson); values.add(value); } variable.setValues(values); } } + + protected V deserializeValue(JsonDeserializationContext context, Class<V> variableType, JsonElement valueJson) { + return valueJson == null ? null : context.deserialize(valueJson, variableType); + } } View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/f7fe96e96019244798bc1f60… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/f7fe96e96019244798bc1f60… You're receiving this email because of your account on gitlab.com.

1 0