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

[Git][ultreiaio/ird-observe][develop-9.0.x] 19 commits: Review trip i18n comment
by Tony CHEMIT (＠tchemit) 02 Nov '22

02 Nov '22

Tony CHEMIT pushed to branch develop-9.0.x at ultreiaio / ird-observe Commits: 2adc5bee by Tony Chemit at 2022-11-02T15:41:38+01:00 Review trip i18n comment - - - - - a022dda3 by Tony Chemit at 2022-11-02T15:42:25+01:00 Model - Introduce notBlank tagvalue - - - - - 73e29dd6 by Tony Chemit at 2022-11-02T15:42:28+01:00 Dto - Add autoTrim method (generated dto will fill this method if necessary) - - - - - 65bff72e by Tony Chemit at 2022-11-02T15:42:32+01:00 Dto (generator) - Generate autoTrim method if necessary - - - - - 012579fb by Tony Chemit at 2022-11-02T15:42:36+01:00 Validation - Introduce notBlankString validator - - - - - b6107bf1 by Tony Chemit at 2022-11-02T15:42:39+01:00 Validation (generator) - Let's generate notBlankString from model - - - - - 24e8fae5 by Tony Chemit at 2022-11-02T15:42:42+01:00 GUI - Better TagValues usage in UI mojo - - - - - 5fd44d0d by Tony Chemit at 2022-11-02T15:42:44+01:00 GUI - Review SavePredicate API to always check if states is valid before all (this is a side effect to auto-trim before save) - - - - - c077d163 by Tony Chemit at 2022-11-02T15:42:47+01:00 GUI (generator) - clean one - - - - - 09818749 by Tony Chemit at 2022-11-02T15:42:51+01:00 GUI - Add auto-trim on content save process - - - - - 6bff57f9 by Tony Chemit at 2022-11-02T15:42:56+01:00 Model (referential) - add notBlank tagvalue - - - - - 4e081a84 by Tony Chemit at 2022-11-02T15:43:00+01:00 Validation (referential) - Update validation definitions - - - - - cb3e9cb1 by Tony Chemit at 2022-11-02T15:43:03+01:00 Validation (referential) - Update validation tests fixtures - - - - - b4c38d89 by Tony Chemit at 2022-11-02T15:43:06+01:00 Validation (referential) - Update Public API doc - - - - - e59114d3 by Tony Chemit at 2022-11-02T15:43:09+01:00 Model (data) - add notBlank tagvalue - - - - - 805a466d by Tony Chemit at 2022-11-02T15:43:12+01:00 Validation (data) - Update validation definitions - - - - - f448e7ea by Tony Chemit at 2022-11-02T15:43:16+01:00 Validation (data) - Update validation tests fixtures - - - - - bf0692db by Tony Chemit at 2022-11-02T15:43:18+01:00 Validation (data) - Update Public API doc - - - - - 8b522f7b by Tony Chemit at 2022-11-02T15:43:28+01:00 Merge branch 'feature/issue_1544' into develop-9.0.x See #1544 - - - - - 30 changed files: - + client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/actions/save/SavePredicateSupport.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/simple/ContentSimpleUIModelStatesSupport.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUIModelStates.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUITableModel.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/TableSavePredicate.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/actions/entry/SaveTableEntry.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/referential/ReferentialSavePredicate.java - client/datasource/editor/ll/src/main/java/fr/ird/observe/client/datasource/editor/ll/data/logbook/ActivityUISavePredicate.java - client/datasource/editor/ll/src/main/java/fr/ird/observe/client/datasource/editor/ll/data/observation/ActivityUISavePredicate.java - client/datasource/editor/ll/src/main/java/fr/ird/observe/client/datasource/editor/ll/data/observation/SetDetailCompositionUISavePredicate.java - client/datasource/editor/ps/src/main/java/fr/ird/observe/client/datasource/editor/ps/data/logbook/ActivityUISavePredicate.java - client/datasource/editor/ps/src/main/java/fr/ird/observe/client/datasource/editor/ps/data/observation/ActivityUISavePredicate.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/GenerateContentUIMojo.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/UiMojoRunnable.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/table/GenerateContentTableUITableModel.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/referential/GenerateContentReferentialUI.java - core/api/services/src/main/resources/META-INF/i18n/Observe-labels.properties - core/api/services/src/main/resources/validators.xml - core/api/validation/src/main/filtered-resources/META-INF/validators/validation.json - core/api/validation/src/main/i18n/getters/eugene.getter - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/GearUseFeaturesDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/GearUseFeaturesDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/GearUseFeaturesMeasurementDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/GearUseFeaturesMeasurementDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/TripDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/TripDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/TripGearUseFeaturesDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/TripGearUseFeaturesDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/landing/LandingDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/landing/LandingDto-update-error-validation.xml The diff was not included because it is too large. View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/0a6062e6628b1fff74b62dce… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/0a6062e6628b1fff74b62dce… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop-9.0.x] Deleted 18 commits: Model - Introduce notBlank tagvalue (and apply them to referential)
by Tony CHEMIT (＠tchemit) 02 Nov '22

02 Nov '22

Tony CHEMIT pushed to branch develop-9.0.x at ultreiaio / ird-observe WARNING: The push did not contain any new commits, but force pushed to delete the commits and changes below. Deleted commits: 70a13cf7 by Tony Chemit at 2022-11-01T13:35:48+01:00 Model - Introduce notBlank tagvalue (and apply them to referential) - - - - - 85204998 by Tony Chemit at 2022-11-01T13:35:48+01:00 Validation - Introduce notBlankString validator - - - - - 3d3c2763 by Tony Chemit at 2022-11-01T13:36:00+01:00 Validation - Update referential validation definitions - - - - - 48bd7b1f by Tony Chemit at 2022-11-01T13:36:00+01:00 Validation - Update validation tests fixtures - - - - - dc1d1c07 by Tony Chemit at 2022-11-01T13:36:00+01:00 GUI - Better TagValues usage in UI mojo - - - - - d788a95a by Tony Chemit at 2022-11-01T13:36:00+01:00 GUI - Review SavePredicate API to always check if states is valid before all (this is a side effect to auto-trim before save) - - - - - 0b261e0c by Tony Chemit at 2022-11-01T13:36:00+01:00 GUI - Add auto-trim on content save process - - - - - 422324dd by Tony Chemit at 2022-11-01T13:36:00+01:00 Model - Introduce notBlank tagvalue (and apply them to data) - - - - - 9c2cb9cd by Tony Chemit at 2022-11-01T13:36:00+01:00 Validation - Update data validation definitions - - - - - 87bd3579 by Tony Chemit at 2022-11-01T13:36:00+01:00 Validation - Update data validation tests fixtures - - - - - b929d061 by Tony Chemit at 2022-11-01T14:03:19+01:00 Merge branch 'feature/issue_1544' into develop-9.0.x - - - - - 5f8affc8 by Tony Chemit at 2022-11-01T14:37:36+01:00 Validation - Update Public referential API doc - - - - - dee21e4d by Tony Chemit at 2022-11-02T12:18:16+01:00 Validation - Update Public data API doc - - - - - 42d7047a by Tony Chemit at 2022-11-02T12:18:16+01:00 GUI - Add auto-trim on content save process - - - - - a6591407 by Tony Chemit at 2022-11-02T12:18:16+01:00 GUI - Review SavePredicate API to always check if states is valid before all (this is a side effect to auto-trim before save) - - - - - cfb5a4e4 by Tony Chemit at 2022-11-02T12:18:16+01:00 Model - Introduce notBlank tagvalue (and apply them to data) - - - - - 47af6833 by Tony Chemit at 2022-11-02T12:18:16+01:00 Validation - Update data validation definitions - - - - - ce8765bb by Tony Chemit at 2022-11-02T12:18:16+01:00 Review trip i18n comment - - - - - 30 changed files: - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/ContentUIModelStates.java - + client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/actions/save/SavePredicateSupport.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/simple/ContentSimpleUIModelStatesSupport.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUIModelStates.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUITableModel.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/TableSavePredicate.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/actions/entry/SaveTableEntry.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/referential/ReferentialSavePredicate.java - client/datasource/editor/ll/src/main/java/fr/ird/observe/client/datasource/editor/ll/data/logbook/ActivityUISavePredicate.java - client/datasource/editor/ll/src/main/java/fr/ird/observe/client/datasource/editor/ll/data/observation/ActivityUISavePredicate.java - client/datasource/editor/ll/src/main/java/fr/ird/observe/client/datasource/editor/ll/data/observation/SetDetailCompositionUISavePredicate.java - client/datasource/editor/ps/src/main/java/fr/ird/observe/client/datasource/editor/ps/data/logbook/ActivityUISavePredicate.java - client/datasource/editor/ps/src/main/java/fr/ird/observe/client/datasource/editor/ps/data/observation/ActivityUISavePredicate.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/GenerateContentUIMojo.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/UiMojoRunnable.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/edit/GenerateContentEditUIModelStates.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/openable/GenerateContentOpenableUIModelStates.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/ropen/GenerateContentRootOpenableUIModelStates.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/simple/GenerateContentSimpleUIModelStates.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/table/GenerateContentTableUIModelStates.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/table/GenerateContentTableUITableModel.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/helper/ContentUIModelStatesHelper.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/referential/GenerateContentReferentialUI.java - core/api/services/src/main/resources/META-INF/i18n/Observe-labels.properties - core/api/services/src/main/resources/validators.xml - core/api/validation/src/main/filtered-resources/META-INF/validators/validation.json - core/api/validation/src/main/i18n/getters/eugene.getter - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/GearUseFeaturesDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/GearUseFeaturesDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/GearUseFeaturesMeasurementDto-create-error-validation.xml The diff was not included because it is too large. View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/0a6062e6628b1fff74b62dce… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/0a6062e6628b1fff74b62dce… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe] Pushed new branch develop-9.0.x2
by Tony CHEMIT (＠tchemit) 02 Nov '22

02 Nov '22

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

1 0

[Git][ultreiaio/ird-observe][develop-9.0.x] 6 commits: Validation - Update Public data API doc
by Tony CHEMIT (＠tchemit) 02 Nov '22

02 Nov '22

Tony CHEMIT pushed to branch develop-9.0.x at ultreiaio / ird-observe Commits: dee21e4d by Tony Chemit at 2022-11-02T12:18:16+01:00 Validation - Update Public data API doc - - - - - 42d7047a by Tony Chemit at 2022-11-02T12:18:16+01:00 GUI - Add auto-trim on content save process - - - - - a6591407 by Tony Chemit at 2022-11-02T12:18:16+01:00 GUI - Review SavePredicate API to always check if states is valid before all (this is a side effect to auto-trim before save) - - - - - cfb5a4e4 by Tony Chemit at 2022-11-02T12:18:16+01:00 Model - Introduce notBlank tagvalue (and apply them to data) - - - - - 47af6833 by Tony Chemit at 2022-11-02T12:18:16+01:00 Validation - Update data validation definitions - - - - - ce8765bb by Tony Chemit at 2022-11-02T12:18:16+01:00 Review trip i18n comment - - - - - 30 changed files: - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUIModelStates.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUITableModel.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/TableSavePredicate.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/actions/entry/SaveTableEntry.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/ropen/GenerateContentRootOpenableUIModelStates.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/table/GenerateContentTableUIModelStates.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/helper/ContentUIModelStatesHelper.java - core/api/validation/src/main/filtered-resources/META-INF/validators/validation.json - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/TripDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/TripDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ps/common/TripDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ps/common/TripDto-update-error-validation.xml - core/persistence/resources/src/main/resources/fr/ird/observe/entities/data/ll/common/Trip/validation-create.json - core/persistence/resources/src/main/resources/fr/ird/observe/entities/data/ll/common/Trip/validation-update.json - core/persistence/resources/src/main/resources/fr/ird/observe/entities/data/ps/common/Trip/validation-create.json - core/persistence/resources/src/main/resources/fr/ird/observe/entities/data/ps/common/Trip/validation-update.json - core/services/i18n/src/main/i18n/translations/services_en_GB.properties - core/services/i18n/src/main/i18n/translations/services_es_ES.properties - core/services/i18n/src/main/i18n/translations/services_fr_FR.properties - model/src/main/models/Observe/dto/attribute/notBlank.properties - server/runner/src/main/webResources/doc/api/public/data/ll/common/GearUseFeatures/validation-create-error.xml - server/runner/src/main/webResources/doc/api/public/data/ll/common/GearUseFeatures/validation-create.json - server/runner/src/main/webResources/doc/api/public/data/ll/common/GearUseFeatures/validation-update-error.xml - server/runner/src/main/webResources/doc/api/public/data/ll/common/GearUseFeatures/validation-update.json - server/runner/src/main/webResources/doc/api/public/data/ll/common/GearUseFeaturesMeasurement/validation-create-error.xml - server/runner/src/main/webResources/doc/api/public/data/ll/common/GearUseFeaturesMeasurement/validation-create.json - server/runner/src/main/webResources/doc/api/public/data/ll/common/GearUseFeaturesMeasurement/validation-update-error.xml - server/runner/src/main/webResources/doc/api/public/data/ll/common/GearUseFeaturesMeasurement/validation-update.json - server/runner/src/main/webResources/doc/api/public/data/ll/common/Trip/validation-create-error.xml - server/runner/src/main/webResources/doc/api/public/data/ll/common/Trip/validation-create.json The diff was not included because it is too large. View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/78537541c1fbc3e0b966bab0… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/78537541c1fbc3e0b966bab0… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop-9.0.x] 22 commits: GUI - Référentiel - La vérification avant sauvegarde d'un référentiel...
by Tony CHEMIT (＠tchemit) 02 Nov '22

02 Nov '22

Tony CHEMIT pushed to branch develop-9.0.x at ultreiaio / ird-observe Commits: 46069153 by Tony Chemit at 2022-10-31T14:24:25+01:00 GUI - Référentiel - La vérification avant sauvegarde d'un référentiel désactivé n'est plus en place - Closes #2507 - - - - - 83685f9e by Tony Chemit at 2022-10-31T15:44:43+01:00 GUI - Référentiel - Pouvoir de nouveau éditer le référentiel d'une base locale en utilisant l'option de configuration déjà existante - Closes #2508 - - - - - 0a6062e6 by Tony Chemit at 2022-11-01T09:27:25+01:00 update pom - - - - - 70a13cf7 by Tony Chemit at 2022-11-01T13:35:48+01:00 Model - Introduce notBlank tagvalue (and apply them to referential) - - - - - 85204998 by Tony Chemit at 2022-11-01T13:35:48+01:00 Validation - Introduce notBlankString validator - - - - - 3d3c2763 by Tony Chemit at 2022-11-01T13:36:00+01:00 Validation - Update referential validation definitions - - - - - 48bd7b1f by Tony Chemit at 2022-11-01T13:36:00+01:00 Validation - Update validation tests fixtures - - - - - dc1d1c07 by Tony Chemit at 2022-11-01T13:36:00+01:00 GUI - Better TagValues usage in UI mojo - - - - - d788a95a by Tony Chemit at 2022-11-01T13:36:00+01:00 GUI - Review SavePredicate API to always check if states is valid before all (this is a side effect to auto-trim before save) - - - - - 0b261e0c by Tony Chemit at 2022-11-01T13:36:00+01:00 GUI - Add auto-trim on content save process - - - - - 422324dd by Tony Chemit at 2022-11-01T13:36:00+01:00 Model - Introduce notBlank tagvalue (and apply them to data) - - - - - 9c2cb9cd by Tony Chemit at 2022-11-01T13:36:00+01:00 Validation - Update data validation definitions - - - - - 87bd3579 by Tony Chemit at 2022-11-01T13:36:00+01:00 Validation - Update data validation tests fixtures - - - - - b929d061 by Tony Chemit at 2022-11-01T14:03:19+01:00 Merge branch 'feature/issue_1544' into develop-9.0.x - - - - - 5f8affc8 by Tony Chemit at 2022-11-01T14:37:36+01:00 Validation - Update Public referential API doc - - - - - 74018d75 by Tony Chemit at 2022-11-01T14:37:46+01:00 Validation - Update Public data API doc - - - - - 9f412460 by Tony Chemit at 2022-11-02T09:02:27+01:00 GUI - Add auto-trim on content save process - - - - - adc268e1 by Tony Chemit at 2022-11-02T10:36:38+01:00 GUI - Review SavePredicate API to always check if states is valid before all (this is a side effect to auto-trim before save) - - - - - afd22499 by Tony Chemit at 2022-11-02T10:37:00+01:00 Model - Introduce notBlank tagvalue (and apply them to data) - - - - - 241bb2bd by Tony Chemit at 2022-11-02T11:06:57+01:00 Validation - Update data validation definitions - - - - - a8d0e9a9 by Tony Chemit at 2022-11-02T11:06:57+01:00 GUI - Review SavePredicate API to always check if states is valid before all (this is a side effect to auto-trim before save) - - - - - 78537541 by Tony Chemit at 2022-11-02T11:07:46+01:00 Review trip i18n comment - - - - - 30 changed files: - client/configuration/src/main/config/Client.ini - client/core/src/main/java/fr/ird/observe/client/datasource/api/ObserveSwingDataSource.java - client/core/src/main/java/fr/ird/observe/client/main/body/ClientConfigUI.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/ContentUIModelStates.java - + client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/actions/save/SavePredicateSupport.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/simple/ContentSimpleUIModelStatesSupport.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/ContentTableUITableModel.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/data/table/TableSavePredicate.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/referential/ContentReferentialUIModel.java - client/datasource/editor/api/src/main/java/fr/ird/observe/client/datasource/editor/api/content/referential/ReferentialSavePredicate.java - client/datasource/editor/ll/src/main/java/fr/ird/observe/client/datasource/editor/ll/data/logbook/ActivityUISavePredicate.java - client/datasource/editor/ll/src/main/java/fr/ird/observe/client/datasource/editor/ll/data/observation/ActivityUISavePredicate.java - client/datasource/editor/ll/src/main/java/fr/ird/observe/client/datasource/editor/ll/data/observation/SetDetailCompositionUISavePredicate.java - client/datasource/editor/ps/src/main/java/fr/ird/observe/client/datasource/editor/ps/data/logbook/ActivityUISavePredicate.java - client/datasource/editor/ps/src/main/java/fr/ird/observe/client/datasource/editor/ps/data/observation/ActivityUISavePredicate.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/GenerateContentUIMojo.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/UiMojoRunnable.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/edit/GenerateContentEditUIModelStates.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/openable/GenerateContentOpenableUIModelStates.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/ropen/GenerateContentRootOpenableUIModelStates.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/simple/GenerateContentSimpleUIModelStates.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/data/table/GenerateContentTableUITableModel.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/helper/ContentUIModelStatesHelper.java - client/datasource/editor/spi/src/main/java/fr/ird/observe/client/datasource/editor/spi/content/referential/GenerateContentReferentialUI.java - core/api/services/src/main/resources/META-INF/i18n/Observe-labels.properties - core/api/services/src/main/resources/validators.xml - core/api/validation/src/main/filtered-resources/META-INF/validators/validation.json - core/api/validation/src/main/i18n/getters/eugene.getter - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/GearUseFeaturesDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/common/GearUseFeaturesDto-update-error-validation.xml The diff was not included because it is too large. View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/0c6c5e20e8979820e5404f00… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/0c6c5e20e8979820e5404f00… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop-9.0.x] I can't afford to pay 10$ each week for some build that takes too much time to...
by Tony CHEMIT (＠tchemit) 30 Oct '22

30 Oct '22

Tony CHEMIT pushed to branch develop-9.0.x at ultreiaio / ird-observe Commits: 0c6c5e20 by Tony Chemit at 2022-10-30T20:09:23+01:00 I can't afford to pay 10$ each week for some build that takes too much time to save cache or other things :( Gitlab -1 - - - - - 1 changed file: - .gitlab-ci.yml Changes: ===================================== .gitlab-ci.yml ===================================== @@ -97,25 +97,25 @@ trigger-make-release: ### AUTOMATIC JOBS ### ################################################################################################### -.automatic: &automatic - only: - - /^develop.*$/i - - /^feature/.+$/i - except: - - triggers - -build: - <<: *stage_build - <<: *automatic - script: - - ultreiaio-maven-execute 'clean install -U -DskipITs' - -build-for-release: - <<: *stage_test - <<: *automatic - <<: *build-for-release - before_script: - - export BUILD_FOR_RELEASE=true +#.automatic: &automatic +# only: +# - /^develop.*$/i +# - /^feature/.+$/i +# except: +# - triggers +# +#build: +# <<: *stage_build +# <<: *automatic +# script: +# - ultreiaio-maven-execute 'clean install -U -DskipITs' +# +#build-for-release: +# <<: *stage_test +# <<: *automatic +# <<: *build-for-release +# before_script: +# - export BUILD_FOR_RELEASE=true ################################################################################################### ### AUTOMATIC RELEASE JOBS ### View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/0c6c5e20e8979820e5404f007… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/0c6c5e20e8979820e5404f007… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop] I can't afford to pay 10$ each week for some build that takes too much time to...
by Tony CHEMIT (＠tchemit) 30 Oct '22

30 Oct '22

Tony CHEMIT pushed to branch develop at ultreiaio / ird-observe Commits: d1fafc3e by Tony Chemit at 2022-10-30T20:09:37+01:00 I can't afford to pay 10$ each week for some build that takes too much time to save cache or other things :( Gitlab -1 - - - - - 1 changed file: - .gitlab-ci.yml Changes: ===================================== .gitlab-ci.yml ===================================== @@ -97,25 +97,25 @@ trigger-make-release: ### AUTOMATIC JOBS ### ################################################################################################### -.automatic: &automatic - only: - - /^develop.*$/i - - /^feature/.+$/i - except: - - triggers - -build: - <<: *stage_build - <<: *automatic - script: - - ultreiaio-maven-execute 'clean install -U -DskipITs' - -build-for-release: - <<: *stage_test - <<: *automatic - <<: *build-for-release - before_script: - - export BUILD_FOR_RELEASE=true +#.automatic: &automatic +# only: +# - /^develop.*$/i +# - /^feature/.+$/i +# except: +# - triggers +# +#build: +# <<: *stage_build +# <<: *automatic +# script: +# - ultreiaio-maven-execute 'clean install -U -DskipITs' +# +#build-for-release: +# <<: *stage_test +# <<: *automatic +# <<: *build-for-release +# before_script: +# - export BUILD_FOR_RELEASE=true ################################################################################################### ### AUTOMATIC RELEASE JOBS ### View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/d1fafc3e25bb3d291f7b0b2a5… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/d1fafc3e25bb3d291f7b0b2a5… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop] 3 commits: Générer les validateurs sur l'unicité des clef métier de collections - Closes #2505
by Tony CHEMIT (＠tchemit) 30 Oct '22

30 Oct '22

Tony CHEMIT pushed to branch develop at ultreiaio / ird-observe Commits: 2ad3e543 by Tony Chemit at 2022-10-30T17:04:30+01:00 Générer les validateurs sur l'unicité des clef métier de collections - Closes #2505 - - - - - c1b1f96d by Tony Chemit at 2022-10-30T17:16:38+01:00 Générer les validateurs sur l'unicité des clef métier de collections - Closes #2505 (update for well plan) - - - - - 84acf625 by Tony Chemit at 2022-10-30T17:16:38+01:00 Souci d'unicité sur le formPS observations / captures - Closes #2504 - - - - - 30 changed files: - core/api/services/.mvn/i18n → core/api/services/.mvn/i18n/parse-java - + core/api/services/.mvn/models-dto - core/api/services/pom.xml - + core/api/services/src/main/i18n/getters/java.getter - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDtoBaitsCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDtoBranchlinesCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDtoFloatlinesCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDtoHooksCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDtoBaitsCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDtoBranchlinesCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDtoFloatlinesCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDtoHooksCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/localmarket/SampleSampleSpeciesDtoSampleSpeciesCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/localmarket/SampleSpeciesDtoSampleSpeciesMeasureCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/localmarket/SurveyDtoSurveyPartCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/logbook/SampleSampleSpeciesDtoSampleSpeciesCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/logbook/SampleSpeciesDtoSampleSpeciesMeasureCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/logbook/WellActivityDtoWellActivitySpeciesCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/observation/SetCatchDtoCatchesCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/observation/SetSchoolEstimateDtoSchoolEstimateCollectionUniqueKeyValidator.java - core/api/validation/src/main/filtered-resources/validators.xml → core/api/services/src/main/resources/validators.xml - core/api/validation/src/main/filtered-resources/META-INF/validators/validation.json - core/api/validation/src/main/i18n/getters/validation-messages.getter - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ps/localmarket/SampleSampleSpeciesDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ps/localmarket/SampleSampleSpeciesDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ps/localmarket/SampleSpeciesDto-create-error-validation.xml The diff was not included because it is too large. View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/9f27b93b8b44a72a1405273c… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/9f27b93b8b44a72a1405273c… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop-9.0.x] 2 commits: Générer les validateurs sur l'unicité des clef métier de collections - Closes #2505
by Tony CHEMIT (＠tchemit) 30 Oct '22

30 Oct '22

Tony CHEMIT pushed to branch develop-9.0.x at ultreiaio / ird-observe Commits: f4127bc3 by Tony Chemit at 2022-10-30T17:02:23+01:00 Générer les validateurs sur l'unicité des clef métier de collections - Closes #2505 - - - - - dc9bac3b by Tony Chemit at 2022-10-30T17:02:23+01:00 Souci d'unicité sur le formPS observations / captures - Closes #2504 - - - - - 30 changed files: - core/api/services/.mvn/i18n → core/api/services/.mvn/i18n/parse-java - + core/api/services/.mvn/models-dto - core/api/services/pom.xml - + core/api/services/src/main/i18n/getters/java.getter - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDtoBaitsCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDtoBranchlinesCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDtoFloatlinesCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDtoHooksCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDtoBaitsCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDtoBranchlinesCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDtoFloatlinesCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDtoHooksCompositionCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/localmarket/SampleSampleSpeciesDtoSampleSpeciesCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/localmarket/SampleSpeciesDtoSampleSpeciesMeasureCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/localmarket/SurveyDtoSurveyPartCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/logbook/SampleSampleSpeciesDtoSampleSpeciesCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/logbook/SampleSpeciesDtoSampleSpeciesMeasureCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/observation/SetCatchDtoCatchesCollectionUniqueKeyValidator.java - + core/api/services/src/main/java/fr/ird/observe/dto/data/ps/observation/SetSchoolEstimateDtoSchoolEstimateCollectionUniqueKeyValidator.java - core/api/validation/src/main/filtered-resources/validators.xml → core/api/services/src/main/resources/validators.xml - core/api/validation/src/main/filtered-resources/META-INF/validators/validation.json - core/api/validation/src/main/i18n/getters/validation-messages.getter - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/logbook/SetGlobalCompositionDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ll/observation/SetGlobalCompositionDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ps/localmarket/SampleSampleSpeciesDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ps/localmarket/SampleSampleSpeciesDto-update-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ps/localmarket/SampleSpeciesDto-create-error-validation.xml - core/api/validation/src/main/resources/fr/ird/observe/dto/data/ps/localmarket/SampleSpeciesDto-update-error-validation.xml The diff was not included because it is too large. View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/d7c8956fe444adbe5b8c9335… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/d7c8956fe444adbe5b8c9335… You're receiving this email because of your account on gitlab.com.

1 0

[Git][ultreiaio/ird-observe][develop] tms_version va-t-elle rester dans le schéma public ? - Closes #2218
by Tony CHEMIT (＠tchemit) 27 Oct '22

27 Oct '22

Tony CHEMIT pushed to branch develop at ultreiaio / ird-observe Commits: 9f27b93b by Tony Chemit at 2022-10-27T19:35:37+02:00 tms_version va-t-elle rester dans le schéma public ? - Closes #2218 - - - - - 12 changed files: - core/persistence/test/src/test/java/org/nuiton/topia/persistence/jdbc/JdbcHelperH2Test.java - core/persistence/test/src/test/java/org/nuiton/topia/persistence/jdbc/JdbcSecurityHelperH2Test.java - core/persistence/test/src/test/java/org/nuiton/topia/persistence/security/SecurityScriptHelperTest.java - toolkit/persistence/src/main/java/org/nuiton/topia/persistence/jdbc/JdbcHelper.java - toolkit/persistence/src/main/java/org/nuiton/topia/persistence/jdbc/JdbcSecurityHelper.java - toolkit/persistence/src/main/java/org/nuiton/topia/persistence/jdbc/JdbcSecurityHelperH2.java - toolkit/persistence/src/main/java/org/nuiton/topia/persistence/jdbc/JdbcSecurityHelperPostgres.java - toolkit/persistence/src/main/java/org/nuiton/topia/service/migration/TopiaMigrationServiceContext.java - toolkit/persistence/src/main/java/org/nuiton/topia/service/migration/TopiaMigrationServiceImpl.java - toolkit/persistence/src/main/java/org/nuiton/topia/service/migration/version/MigrationServiceSqlHelper.java - toolkit/persistence/src/main/java/org/nuiton/topia/service/migration/version/TMSVersion.java - toolkit/persistence/src/main/java/org/nuiton/topia/service/sql/internal/consumer/AddVersionTableConsumer.java Changes: ===================================== core/persistence/test/src/test/java/org/nuiton/topia/persistence/jdbc/JdbcHelperH2Test.java ===================================== @@ -30,6 +30,7 @@ import org.junit.Assert; import org.junit.Before; import org.junit.ClassRule; import org.junit.Test; +import org.nuiton.topia.service.migration.version.MigrationServiceSqlHelper; import java.sql.SQLException; @@ -54,20 +55,17 @@ public class JdbcHelperH2Test extends PersistenceTestSupportWrite { @Test public void isTableExist() throws SQLException { - boolean actual = jdbcHelper.isTableExist("Public", "Tms_version"); - Assert.assertTrue(actual); + Assert.assertTrue(jdbcHelper.isTableExist(MigrationServiceSqlHelper.CURRENT.schemaName(), MigrationServiceSqlHelper.CURRENT.tableName())); + Assert.assertTrue(jdbcHelper.isTableExist(MigrationServiceSqlHelper.CURRENT.schemaName().toUpperCase(), MigrationServiceSqlHelper.CURRENT.tableName())); + Assert.assertTrue(jdbcHelper.isTableExist(MigrationServiceSqlHelper.CURRENT.schemaName().toUpperCase(), MigrationServiceSqlHelper.CURRENT.tableName().toUpperCase())); - actual = jdbcHelper.isTableExist("PUBLIC", "tms_version-fake"); - Assert.assertFalse(actual); + Assert.assertFalse(jdbcHelper.isTableExist(MigrationServiceSqlHelper.CURRENT.schemaName(), MigrationServiceSqlHelper.CURRENT.tableName()+"-Fake")); } @Test public void isSchemaExist() throws SQLException { - boolean actual = jdbcHelper.isSchemaExist("Public"); - Assert.assertTrue(actual); - - actual = jdbcHelper.isSchemaExist("PUBLIC-fake"); - Assert.assertFalse(actual); + Assert.assertTrue(jdbcHelper.isSchemaExist("Public")); + Assert.assertFalse(jdbcHelper.isSchemaExist("PUBLIC-fake")); } } ===================================== core/persistence/test/src/test/java/org/nuiton/topia/persistence/jdbc/JdbcSecurityHelperH2Test.java ===================================== @@ -31,6 +31,7 @@ import org.junit.Assert; import org.junit.Before; import org.junit.ClassRule; import org.junit.Test; +import org.nuiton.topia.service.migration.version.MigrationServiceSqlHelper; import java.util.List; import java.util.Set; @@ -62,10 +63,9 @@ public class JdbcSecurityHelperH2Test extends PersistenceTestSupportWrite { Assert.assertNotNull(actual); Assert.assertEquals(0, actual.size()); - actual = securityHelper.getTables(Set.of("PUbLIC"), Set.of()); + actual = securityHelper.getTables(Set.of(MigrationServiceSqlHelper.CURRENT.schemaName()), Set.of()); Assert.assertNotNull(actual); - Assert.assertEquals(1, actual.size()); - Assert.assertEquals(List.of(Pair.of("PUBLIC", "TMS_VERSION")), actual); + Assert.assertTrue(actual.contains(Pair.of(MigrationServiceSqlHelper.CURRENT.schemaName().toUpperCase(), MigrationServiceSqlHelper.CURRENT.tableName().toUpperCase()))); } @Test @@ -90,7 +90,7 @@ public class JdbcSecurityHelperH2Test extends PersistenceTestSupportWrite { @Test public void getTablePrivileges() { - Set<String> actual = securityHelper.getTablePrivileges("PUbLIC", "Tms_VERSION"); + Set<String> actual = securityHelper.getTablePrivileges(MigrationServiceSqlHelper.CURRENT.schemaName(), MigrationServiceSqlHelper.CURRENT.tableName()); Assert.assertNotNull(actual); Assert.assertEquals(0, actual.size()); } ===================================== core/persistence/test/src/test/java/org/nuiton/topia/persistence/security/SecurityScriptHelperTest.java ===================================== @@ -37,6 +37,7 @@ import org.junit.ClassRule; import org.junit.Test; import org.nuiton.topia.persistence.jdbc.JdbcHelper; import org.nuiton.topia.persistence.jdbc.JdbcSecurityHelper; +import org.nuiton.topia.service.migration.version.MigrationServiceSqlHelper; import java.sql.SQLException; import java.util.Map; @@ -92,7 +93,7 @@ public class SecurityScriptHelperTest extends PersistenceTestSupportWrite { SecurityScriptHelper securityScriptHelper = localTestMethodResource.getTopiaApplicationContext().newSecurityScriptHelper(); securityScriptHelper.applySecurity(users); - Set<String> actual = securityHelper.getTablePrivileges("PUBLIC", "TMS_VERSION"); + Set<String> actual = securityHelper.getTablePrivileges(MigrationServiceSqlHelper.CURRENT.schemaName().toUpperCase(), MigrationServiceSqlHelper.CURRENT.tableName().toUpperCase()); Assert.assertNotNull(actual); Assert.assertEquals(4, actual.size()); Assert.assertEquals(Set.of("DELETE", "INSERT", "UPDATE", "SELECT"), actual); ===================================== toolkit/persistence/src/main/java/org/nuiton/topia/persistence/jdbc/JdbcHelper.java ===================================== @@ -34,6 +34,8 @@ import java.sql.DriverManager; import java.sql.PreparedStatement; import java.sql.ResultSet; import java.sql.SQLException; +import java.util.LinkedList; +import java.util.List; import java.util.Objects; /** @@ -116,17 +118,38 @@ public abstract class JdbcHelper { } public String runSelectOnString(String sql) throws SQLException { - return runSelect(sql, resultSet -> resultSet.next() ? resultSet.getString(1) : null); + return runSelect(sql, resultSet -> resultSet.getString(1)); } public <T> T runSelect(String sql, SqlFunction<ResultSet, T> function) throws SQLException { try (Connection connection = openConnection()) { try (PreparedStatement preparedStatement = connection.prepareStatement(sql)) { try (ResultSet resultSet = preparedStatement.executeQuery()) { - return function.apply(resultSet); + if (resultSet.next()) { + return function.apply(resultSet); + } + return null; + } + } + } + } + + public <T> List<T> runMultipleSelect(String sql, SqlFunction<ResultSet, T> function) throws SQLException { + List<T> result = new LinkedList<>(); + try (Connection connection = openConnection()) { + try (PreparedStatement preparedStatement = connection.prepareStatement(sql)) { + try (ResultSet resultSet = preparedStatement.executeQuery()) { + while (resultSet.next()) { + + T row = function.apply(resultSet); + if (row != null) { + result.add(row); + } + } } } } + return result; } public <T> T runOnMetadata(SqlFunction<DatabaseMetaData, T> function) throws SQLException { ===================================== toolkit/persistence/src/main/java/org/nuiton/topia/persistence/jdbc/JdbcSecurityHelper.java ===================================== @@ -161,14 +161,14 @@ public interface JdbcSecurityHelper { return getLegacyVersion(); } try { - return MigrationServiceSqlHelper.DEFAULT.getVersion(jdbcHelper()).map(TMSVersion::toVersion).orElse(Version.VZERO); + return MigrationServiceSqlHelper.DEFAULT.getVersion(jdbcHelper()).map(TMSVersion::getVersion).orElse(Version.VZERO); } catch (Exception e) { - // try on legacy + // try on legacy (migration may not been applied) return getLegacyVersion(); } } private Version getLegacyVersion() { - return MigrationServiceSqlHelper.LEGACY.getVersion(jdbcHelper()).map(TMSVersion::toVersion).orElse(Version.VZERO); + return MigrationServiceSqlHelper.LEGACY.getVersion(jdbcHelper()).map(TMSVersion::getVersion).orElse(Version.VZERO); } } ===================================== toolkit/persistence/src/main/java/org/nuiton/topia/persistence/jdbc/JdbcSecurityHelperH2.java ===================================== @@ -31,7 +31,6 @@ import org.nuiton.topia.persistence.security.SecurityScriptHelper; import java.sql.ResultSet; import java.sql.SQLException; -import java.util.ArrayList; import java.util.Collection; import java.util.Collections; import java.util.HashSet; @@ -162,24 +161,21 @@ public class JdbcSecurityHelperH2 implements JdbcSecurityHelper { Set<String> safeExtraTables = extraTables.stream().map(String::toLowerCase).collect(Collectors.toSet()); try { - return jdbcHelper.runSelect("SELECT TABLE_SCHEMA, TABLE_NAME FROM INFORMATION_SCHEMA.TABLES WHERE TABLE_TYPE = 'TABLE'", resultSet -> { - List<Pair<String, String>> result = new ArrayList<>(); - while (resultSet.next()) { - String schemaName = resultSet.getString(1); - String tableName = resultSet.getString(2); - if (!safeExtraTables.contains(tableName.toLowerCase())) { - // if (POSTGIS_TABLES.contains(tableName)) { - // continue; - // } - if (schemaName == null || !safeSchemas.contains(schemaName.toLowerCase())) { - continue; - } + List<Pair<String, String>> pairs = jdbcHelper.runMultipleSelect("SELECT TABLE_SCHEMA, TABLE_NAME FROM INFORMATION_SCHEMA.TABLES WHERE TABLE_TYPE = 'TABLE'", resultSet -> { + String schemaName = resultSet.getString(1); + String tableName = resultSet.getString(2); + if (!safeExtraTables.contains(tableName.toLowerCase())) { + // if (POSTGIS_TABLES.contains(tableName)) { + // continue; + // } + if (schemaName == null || !safeSchemas.contains(schemaName.toLowerCase())) { + return null; } - result.add(Pair.of(schemaName, tableName)); } - Collections.sort(result); - return result; + return Pair.of(schemaName, tableName); }); + Collections.sort(pairs); + return pairs; } catch (Exception e) { throw new TopiaException(e); } @@ -188,14 +184,8 @@ public class JdbcSecurityHelperH2 implements JdbcSecurityHelper { @Override public Set<String> getRoles() { try { - return jdbcHelper.runSelect("SELECT name FROM INFORMATION_SCHEMA.USERS;", resultSet -> { - Set<String> users = new LinkedHashSet<>(); - while (resultSet.next()) { - String name = resultSet.getString(1); - users.add(name); - } - return users; - }); + List<String> roles = jdbcHelper.runMultipleSelect("SELECT name FROM INFORMATION_SCHEMA.USERS;", resultSet -> resultSet.getString(1)); + return Set.copyOf(roles); } catch (SQLException e) { throw new TopiaException(e); } @@ -204,7 +194,7 @@ public class JdbcSecurityHelperH2 implements JdbcSecurityHelper { @Override public boolean isOwner() { try { - return jdbcHelper.runSelect("SELECT t.* FROM INFORMATION_SCHEMA.SCHEMATA t WHERE SCHEMA_NAME = 'PUBLIC' AND SCHEMA_OWNER = CURRENT_USER", ResultSet::next); + return jdbcHelper.runSelect("SELECT t.* FROM INFORMATION_SCHEMA.SCHEMATA t WHERE SCHEMA_NAME = 'PUBLIC' AND SCHEMA_OWNER = CURRENT_USER", r-> true); } catch (SQLException e) { throw new TopiaException(e); } @@ -213,7 +203,7 @@ public class JdbcSecurityHelperH2 implements JdbcSecurityHelper { @Override public boolean isSuperUser() { try { - return jdbcHelper.runSelect("SELECT admin FROM INFORMATION_SCHEMA.USERS WHERE name = CURRENT_USER", resultSet -> resultSet.next() && resultSet.getBoolean(1)); + return jdbcHelper.runSelect("SELECT admin FROM INFORMATION_SCHEMA.USERS WHERE name = CURRENT_USER", resultSet -> resultSet.getBoolean(1)); } catch (SQLException e) { throw new TopiaException(e); } ===================================== toolkit/persistence/src/main/java/org/nuiton/topia/persistence/jdbc/JdbcSecurityHelperPostgres.java ===================================== @@ -253,7 +253,7 @@ public class JdbcSecurityHelperPostgres implements JdbcSecurityHelper { @Override public boolean isOwner() { try { - return jdbcHelper.runSelect("SELECT pg_catalog.pg_get_userbyid(d.datdba) FROM pg_catalog.pg_database d WHERE d.datname = (SELECT current_database())", resultSet -> resultSet.next() && Objects.equals(jdbcHelper.getJdbcConnectionUser(), resultSet.getString(1))); + return jdbcHelper.runSelect("SELECT pg_catalog.pg_get_userbyid(d.datdba) FROM pg_catalog.pg_database d WHERE d.datname = (SELECT current_database())", resultSet -> Objects.equals(jdbcHelper.getJdbcConnectionUser(), resultSet.getString(1))); } catch (SQLException e) { throw new TopiaException(e); } @@ -262,7 +262,7 @@ public class JdbcSecurityHelperPostgres implements JdbcSecurityHelper { @Override public boolean isSuperUser() { try { - return jdbcHelper.runSelect("SELECT usesuper FROM pg_user WHERE usename = CURRENT_USER", resultSet -> resultSet.next() && resultSet.getBoolean(1)); + return jdbcHelper.runSelect("SELECT usesuper FROM pg_user WHERE usename = CURRENT_USER", resultSet -> resultSet.getBoolean(1)); } catch (SQLException e) { throw new TopiaException(e); } @@ -275,14 +275,8 @@ public class JdbcSecurityHelperPostgres implements JdbcSecurityHelper { " FROM pg_proc p INNER JOIN pg_namespace ns ON (p.pronamespace = ns.oid)" + " WHERE ns.nspname = 'public' AND p.proname ILIKE '%s%%';", functionPattern); try { - return jdbcHelper.runSelect(sql, resultSet -> { - Set<String> result = new LinkedHashSet<>(); - while (resultSet.next()) { - String functionPrototype = resultSet.getString(1); - result.add(functionPrototype); - } - return result; - }); + List<String> result = jdbcHelper.runMultipleSelect(sql, resultSet -> resultSet.getString(1)); + return Set.copyOf(result); } catch (SQLException e) { throw new TopiaException(e); } ===================================== toolkit/persistence/src/main/java/org/nuiton/topia/service/migration/TopiaMigrationServiceContext.java ===================================== @@ -38,6 +38,9 @@ import org.nuiton.topia.service.migration.version.TMSVersion; import java.io.IOException; import java.nio.file.Files; import java.nio.file.Path; +import java.util.Date; +import java.util.List; +import java.util.Objects; import java.util.Optional; /** @@ -56,11 +59,10 @@ public class TopiaMigrationServiceContext { * Service configuration. */ protected final TopiaMigrationServiceConfiguration configuration; - protected final MigrationServiceSqlHelper sqlHelper; /** - * Jdbc helper used to perform some sql code. + * Migration sql helper (could be legacy or default). */ - private final JdbcHelper jdbcHelper; + protected final MigrationServiceSqlHelper sqlHelper; /** * Available migration resources found in class-path. */ @@ -74,17 +76,25 @@ public class TopiaMigrationServiceContext { */ protected final boolean dbNotVersioned; /** - * Is legacy TMSVersion table exists? + * Is legacy migration table exists? (only used with default mode) */ protected final boolean legacyVersionTableExist; /** - * Is TMSVersion schema exists? + * Is default migration schema exists? + */ + protected final boolean versionSchemaExist; + /** + * Is default migration table exists? */ - protected boolean versionSchemaExist; + protected final boolean versionTableExist; /** - * Is TMSVersion table exists? + * Can we apply migration versions? */ - protected boolean versionTableExist; + protected final boolean canApplyMigrationVersions; + /** + * Migration versions to apply. + */ + protected final List<Version> versionsToApply; /** * Current database version. */ @@ -114,10 +124,11 @@ public class TopiaMigrationServiceContext { try { versionTableExist = sqlHelper.isTableExist(jdbcHelper); if (versionTableExist) { + log.info("Detected standard migration table {}.", sqlHelper.gav()); versionSchemaExist = true; Optional<TMSVersion> tmsVersion = sqlHelper.getVersion(jdbcHelper); if (tmsVersion.isPresent()) { - v = tmsVersion.get().toVersion(); + v = tmsVersion.get().getVersion(); } if (v == null) { log.warn(String.format("Version not found on table %s", sqlHelper.gav())); @@ -126,9 +137,10 @@ public class TopiaMigrationServiceContext { versionSchemaExist = sqlHelper.isSchemaExist(jdbcHelper); legacyVersionTableExist = legacySqlHelper.isTableExist(jdbcHelper); if (legacyVersionTableExist) { + log.info("Detected legacy migration table {}.", sqlHelper.gav()); Optional<TMSVersion> tmsVersion = legacySqlHelper.getVersion(jdbcHelper); if (tmsVersion.isPresent()) { - v = tmsVersion.get().toVersion(); + v = tmsVersion.get().getVersion(); } if (v == null) { log.warn(String.format("Version not found on legacy table %s", legacySqlHelper.gav())); @@ -137,6 +149,7 @@ public class TopiaMigrationServiceContext { } } finally { if (v == null) { + //FIXME Is this can really happen? // la base dans ce cas n'est pas versionee. // On dit que la version de la base est 0 // et les schema de cette version 0 doivent @@ -151,7 +164,6 @@ public class TopiaMigrationServiceContext { } return new TopiaMigrationServiceContext(configuration, sqlHelper, - jdbcHelper, legacyVersionTableExist, versionSchemaExist, versionTableExist, @@ -175,9 +187,10 @@ public class TopiaMigrationServiceContext { try { versionTableExist = sqlHelper.isTableExist(jdbcHelper); if (versionTableExist) { + log.info("Detected legacy migration table {}.", sqlHelper.gav()); Optional<TMSVersion> tmsVersion = sqlHelper.getVersion(jdbcHelper); if (tmsVersion.isPresent()) { - v = tmsVersion.get().toVersion(); + v = tmsVersion.get().getVersion(); } if (v == null) { log.warn(String.format("Version not found on table %s", sqlHelper.gav())); @@ -199,7 +212,6 @@ public class TopiaMigrationServiceContext { } return new TopiaMigrationServiceContext(configuration, sqlHelper, - jdbcHelper, false, true, versionTableExist, @@ -211,15 +223,14 @@ public class TopiaMigrationServiceContext { protected TopiaMigrationServiceContext(TopiaMigrationServiceConfiguration configuration, MigrationServiceSqlHelper sqlHelper, - JdbcHelper jdbcHelper, - boolean legacyVersionTableExist, boolean versionSchemaExist, + boolean legacyVersionTableExist, + boolean versionSchemaExist, boolean versionTableExist, boolean dbNotVersioned, Version dbVersion, MigrationVersionResourceProvider resources) { this.configuration = configuration; this.sqlHelper = sqlHelper; - this.jdbcHelper = jdbcHelper; this.versionSchemaExist = versionSchemaExist; this.versionTableExist = versionTableExist; this.dbNotVersioned = dbNotVersioned; @@ -231,26 +242,24 @@ public class TopiaMigrationServiceContext { } catch (IOException e) { throw new IllegalStateException("Can't create scripts path", e); } - } - public Path getScriptPath() { - return scriptPath; - } + List<Version> allVersions = getResources().getAvailableVersions(); + log.info(String.format("Detected available versions: %1$s", allVersions)); - public boolean isVersionSchemaExist() { - return versionSchemaExist; - } + versionsToApply = getResources().getVersionsAfter(dbVersion); + log.info(String.format("Detected versions to apply: %1$s", versionsToApply)); - public boolean isVersionTableExist() { - return versionTableExist; - } + Optional<TopiaMigrationServiceAskUserToMigrate> askUserToMigrate = Optional.ofNullable(configuration.getCallback()); - public boolean isLegacyVersionTableExist() { - return legacyVersionTableExist; + // ask to perform the migration + canApplyMigrationVersions = askUserToMigrate.map(c -> c.canIMigrate(getDbVersion(), versionsToApply)).orElse(true); + if (!canApplyMigrationVersions) { + log.warn("Detected that we can not migrate versions (maybe version of database is too old?)."); + } } - public boolean isDbNotVersioned() { - return dbNotVersioned; + public Path getScriptPath() { + return scriptPath; } public Version getModelVersion() { @@ -265,35 +274,65 @@ public class TopiaMigrationServiceContext { return resources; } - public void createSchemaIfNotExist() { + public boolean adaptVersionTable(TopiaSqlSupport sqlSupport) { + + Version modelVersion = getModelVersion(); + boolean dbVersionEqualsModelVersion = Objects.equals(dbVersion, modelVersion); + if (!versionSchemaExist) { - sqlHelper.createSchema(jdbcHelper); - versionSchemaExist = true; + log.info("Create default migration schema."); + sqlHelper.createSchema(sqlSupport); } - } - - public void createTableIfNotExist() { if (!versionTableExist) { - sqlHelper.createTable(jdbcHelper); - versionTableExist = true; + log.info("Create default migration table."); + sqlHelper.createTable(sqlSupport); + } else { + if (dbVersionEqualsModelVersion) { + log.info("Database is up to date, no migration needed."); + return false; + } + } + if (legacyVersionTableExist) { + log.info("Drop legacy migration table."); + MigrationServiceSqlHelper.LEGACY.dropTable(sqlSupport); + if (dbVersionEqualsModelVersion) { + log.info("Database is up to date (version: {}), but was coming from legacy migration table, fill new migration table.", modelVersion); + saveVersion(sqlSupport, modelVersion, null); + return false; + } + log.info("Fill version table {} from legacy table.", dbVersion); + saveVersion(sqlSupport, dbVersion, null); + } else if (dbNotVersioned) { + log.info("Database is empty, no migration needed, set "); + saveVersion(sqlSupport, modelVersion, new Date()); + return false; } + // In all other cases, we can try to perform migration + return true; } - public void saveModelVersion() { - saveVersion(null, getModelVersion()); - } + public List<Version> getVersionsToApply() { - public Optional<TopiaMigrationServiceAskUserToMigrate> getAskUserToMigrate() { - return Optional.ofNullable(configuration.getCallback()); - } + if (versionsToApply.isEmpty()) { + log.info("No version to apply, no migration needed."); + // No migration, so nothing to save +// context.saveModelVersion(); + return null; + } - protected void saveVersion(TopiaSqlSupport sqlSupport, Version version) { - log.info(String.format("[ Version %s ] Saving new database version.", version)); - if (sqlSupport != null) { - sqlHelper.save(sqlSupport, version.getVersion()); - } else { - sqlHelper.save(jdbcHelper, version.getVersion()); + // ask to perform the migration + + if (!canApplyMigrationVersions) { + // user cancel migration + log.warn("Migration was cancelled by migration callback (maybe version of database is too old?)."); + return null; } + return versionsToApply; + } + + protected void saveVersion(TopiaSqlSupport sqlSupport, Version version, Date date) { + log.info(String.format("[ Version %s ] Saving new database version (date: %s).", version, date)); + sqlHelper.save(sqlSupport, version.getVersion(), date); dbVersion = version; } @@ -309,8 +348,4 @@ public class TopiaMigrationServiceContext { return new MigrationVersionResourceExecutor(migrationVersionResource, sqlSupport, configuration.getClassifier(), getScriptPath(), migrationVersionResource.getScriptVariables()); } - public void dropLegacyTable() { - MigrationServiceSqlHelper.LEGACY.dropTable(jdbcHelper); - } - } ===================================== toolkit/persistence/src/main/java/org/nuiton/topia/service/migration/TopiaMigrationServiceImpl.java ===================================== @@ -34,10 +34,10 @@ import org.nuiton.topia.service.migration.resources.MigrationVersionResource; import org.nuiton.topia.service.migration.resources.MigrationVersionResourceExecutor; import java.io.IOException; +import java.util.Date; import java.util.List; import java.util.Map; import java.util.Objects; -import java.util.Optional; /** * Topia migration service default implementation. @@ -68,81 +68,38 @@ public class TopiaMigrationServiceImpl implements TopiaMigrationService { @Override public void runSchemaMigration() throws TopiaMigrationServiceException { + log.info("Migration - Prepare internal states."); TopiaMigrationServiceContext context = TopiaMigrationServiceContext.create(configuration); - Version modelVersion = context.getModelVersion(); - Version dbVersion = context.getDbVersion(); - boolean dbNotVersioned = context.isDbNotVersioned(); - boolean legacyVersionTableExist = context.isLegacyVersionTableExist(); - boolean versionSchemaExist = context.isVersionSchemaExist(); - boolean versionTableExist = context.isVersionTableExist(); - - log.info(String.format("Starting Topia Migration Service - Model version : %s, Database version : %s", modelVersion, dbVersion)); - - if (!versionSchemaExist) { - log.info("Create migration schema."); - context.createSchemaIfNotExist(); - } - if (!versionTableExist) { - log.info("Create migration table."); - context.createTableIfNotExist(); - } - if (legacyVersionTableExist) { - log.info("Drop legacy migration table."); - context.dropLegacyTable(); - if (dbVersion.equals(modelVersion)) { - log.info("Database is up to date, but was coming from legacy migration table, fill new migration table."); - context.saveModelVersion(); + log.info("Migration - Prepare internal states done."); + + try (TopiaPersistenceContext persistenceContext = context.newPersistenceContext()) { + log.info("Migration - Adapting migration table."); + boolean canContinue = adaptVersionTable(persistenceContext, context); + log.info("Migration - Adapting migration table done."); + if (!canContinue) { + log.info(String.format("Migration - End - db version: %s (nothing more to do).", context.getDbVersion())); + return; + } + log.info("Migration - Getting migration versions to apply."); + List<Version> versionsToApply = context.getVersionsToApply(); + log.info("Migration - Getting migration versions to apply done."); + if (versionsToApply == null) { + log.info(String.format("Migration - End - db version: %s (nothing more to do).", context.getDbVersion())); return; } - } - - if (versionTableExist && dbVersion.equals(modelVersion)) { - log.info("Database is up to date, no migration needed."); - return; - } - - if (versionTableExist && dbNotVersioned) { - log.info("Database is empty, no migration needed."); - context.saveModelVersion(); - return; - } - - List<Version> allVersions = context.getResources().getAvailableVersions(); - log.info(String.format("Available versions: %1$s", allVersions)); - - List<Version> versionsToApply = context.getResources().getVersionsAfter(dbVersion); - - if (versionsToApply.isEmpty()) { - log.info("No version to apply, no migration needed."); - context.saveModelVersion(); - return; - } - - log.info(String.format("Versions to apply: %1$s", versionsToApply)); - - Optional<TopiaMigrationServiceAskUserToMigrate> askUserToMigrate = context.getAskUserToMigrate(); - - // ask to perform the migration - boolean performMigration = askUserToMigrate.map(c -> c.canIMigrate(dbVersion, versionsToApply)).orElse(true); - - log.debug("Handler choose : " + performMigration); - if (!performMigration) { - // user cancel migration - return; - } + log.info(String.format("Migration - Will apply versions: %1$s.", versionsToApply)); - long statementCount = 0; - try (TopiaPersistenceContext persistenceContext = context.newPersistenceContext()) { + long statementCount = 0; for (Version version : versionsToApply) { long t0 = TimeLog.getTime(); long versionCount = migrateVersion(persistenceContext, version, context); statementCount += versionCount; TIME_LOG.log(t0, "migrationVersion", version.toString() + " - " + versionCount + " sql statements"); } + log.info(String.format("Migration - End - db version: %s - consume %d sql statement(s)", context.getDbVersion(), statementCount)); } - log.info(String.format("Ends migration - db version: %s - consume %d sql statement(s)", context.getDbVersion(), statementCount)); } @Override @@ -150,6 +107,17 @@ public class TopiaMigrationServiceImpl implements TopiaMigrationService { configuration = null; } + protected boolean adaptVersionTable(TopiaPersistenceContext persistenceContext, TopiaMigrationServiceContext context) { + try { + boolean canContinue = context.adaptVersionTable(persistenceContext.getSqlSupport()); + persistenceContext.commit(); + return canContinue; + } catch (Exception e) { + persistenceContext.rollback(); + throw new TopiaMigrationServiceException("Exception during adapting migration table", e); + } + } + protected long migrateVersion(TopiaPersistenceContext persistenceContext, Version version, TopiaMigrationServiceContext context) { MigrationVersionResource resource = context.getResource(version); @@ -174,14 +142,15 @@ public class TopiaMigrationServiceImpl implements TopiaMigrationService { } } // here we use the sqlSupport to save version, to stay on the very same transaction - context.saveVersion(sqlSupport, version); + context.saveVersion(sqlSupport, version, new Date()); + // do commit, this migration version is now completed + persistenceContext.commit(); } catch (Exception e) { // Exception, rollback transaction persistenceContext.rollback(); throw new TopiaMigrationServiceException("Exception during schema migration on version: " + version, e); } - persistenceContext.commit(); return outStatementCount; } ===================================== toolkit/persistence/src/main/java/org/nuiton/topia/service/migration/version/MigrationServiceSqlHelper.java ===================================== @@ -27,6 +27,10 @@ import org.nuiton.topia.persistence.jdbc.JdbcHelper; import org.nuiton.topia.persistence.support.TopiaSqlSupport; import java.sql.SQLException; +import java.sql.Timestamp; +import java.util.Comparator; +import java.util.Date; +import java.util.List; import java.util.Optional; /** @@ -38,12 +42,8 @@ import java.util.Optional; * @since 9.0.7 */ public interface MigrationServiceSqlHelper { - String GET_VERSION_TABLE_STATEMENT = "SELECT version FROM %s;"; - String CREATE_VERSION_SCHEMA_STATEMENT = "CREATE SCHEMA %s;"; - String CREATE_VERSION_TABLE_STATEMENT = "CREATE TABLE %s(version VARCHAR(255) NOT NULL, PRIMARY KEY (version));"; - String DELETE_VERSION_TABLE_STATEMENT = "DELETE FROM %s;"; + String CREATE_VERSION_SCHEMA_STATEMENT = "CREATE SCHEMA IF NOT EXISTS %s;"; String DROP_VERSION_TABLE_STATEMENT = "DROP TABLE %s;"; - String FILL_VERSION_TABLE_STATEMENT = "INSERT INTO %s(version) VALUES('%s');"; /** * Legacy layout. @@ -51,6 +51,10 @@ public interface MigrationServiceSqlHelper { MigrationServiceSqlHelper LEGACY = new MigrationServiceSqlHelper() { public static final String SCHEMA_NAME = "public"; public static final String TABLE_NAME = "tms_version"; + public static final String CREATE_VERSION_TABLE_STATEMENT = "CREATE TABLE %s(version VARCHAR(255) NOT NULL, PRIMARY KEY(version));"; + public static final String FILL_VERSION_TABLE_STATEMENT = "INSERT INTO %s(version) VALUES('%s');"; + public static final String GET_VERSION_TABLE_STATEMENT = "SELECT version FROM %s;"; + public static final String DELETE_VERSION_TABLE_STATEMENT = "DELETE FROM %s;"; @Override public String schemaName() { @@ -61,15 +65,46 @@ public interface MigrationServiceSqlHelper { public String tableName() { return TABLE_NAME; } + + @Override + public String createTableSql() { + return String.format(CREATE_VERSION_TABLE_STATEMENT, gav()); + } + + @Override + public String getVersionSql() { + return String.format(GET_VERSION_TABLE_STATEMENT, gav()); + } + + @Override + public Optional<TMSVersion> getVersion(JdbcHelper jdbcHelper) { + try { + try { + TMSVersion result = jdbcHelper.runSelect(getVersionSql(), r -> new TMSVersion(r.getString(1), null)); + return Optional.ofNullable(result); + } catch (Exception e) { + throw new TopiaException("Could not obtain version", e); + } + } catch (Exception e) { + throw new TopiaException("Could not obtain version", e); + } + } + + @Override + public String saveTableSql(String version, Date date) { + return String.format(DELETE_VERSION_TABLE_STATEMENT, gav()) + String.format(FILL_VERSION_TABLE_STATEMENT, gav(), version); + } }; /** * Default layout. */ MigrationServiceSqlHelper DEFAULT = new MigrationServiceSqlHelper() { - //FIXME Find a better name public static final String SCHEMA_NAME = "common"; //FIXME Find a better name - public static final String TABLE_NAME = "tms_version"; + public static final String TABLE_NAME = "database_version"; + public static final String CREATE_VERSION_TABLE_STATEMENT = "CREATE TABLE %s(version VARCHAR(255) NOT NULL, date TIMESTAMP, PRIMARY KEY(version));"; + public static final String FILL_VERSION_TABLE_STATEMENT = "INSERT INTO %s(version, date) VALUES('%s', %s);"; + public static final String GET_VERSION_TABLE_STATEMENT = "SELECT version, date FROM %s;"; @Override public String schemaName() { @@ -80,6 +115,35 @@ public interface MigrationServiceSqlHelper { public String tableName() { return TABLE_NAME; } + + @Override + public String createTableSql() { + return String.format(CREATE_VERSION_TABLE_STATEMENT, gav()); + } + + @Override + public String getVersionSql() { + return String.format(GET_VERSION_TABLE_STATEMENT, gav()); + } + + @Override + public Optional<TMSVersion> getVersion(JdbcHelper jdbcHelper) { + try { + List<TMSVersion> result = jdbcHelper.runMultipleSelect(getVersionSql(), r -> new TMSVersion(r.getString(1), r.getTimestamp(2))); + if (result.isEmpty()) { + return Optional.empty(); + } + result.sort(Comparator.comparing(TMSVersion::getVersion).reversed()); + return Optional.of(result.get(0)); + } catch (Exception e) { + throw new TopiaException("Could not obtain version", e); + } + } + + @Override + public String saveTableSql(String version, Date date) { + return String.format(FILL_VERSION_TABLE_STATEMENT, gav(), version, date == null ? "NULL" : ("'" + new Timestamp(date.getTime()) + "'::timestamp")); + } }; /** @@ -89,7 +153,7 @@ public interface MigrationServiceSqlHelper { * <p> * We will need even after that to keep the legacy definition, to be able to migration older database using it. */ - MigrationServiceSqlHelper CURRENT = LEGACY; + MigrationServiceSqlHelper CURRENT = DEFAULT; static boolean isLegacy() { return CURRENT.equals(LEGACY); @@ -112,59 +176,22 @@ public interface MigrationServiceSqlHelper { return schemaName() + "." + tableName(); } - default Optional<TMSVersion> getVersion(JdbcHelper jdbcHelper) { - try { - String version = jdbcHelper.runSelectOnString(getVersionSql()); - return Optional.ofNullable(version == null ? null : new TMSVersion(version)); - } catch (Exception e) { - throw new TopiaException("Could not obtain version", e); - } - } - - default String getVersionSql() { - return String.format(GET_VERSION_TABLE_STATEMENT, gav()); - } + Optional<TMSVersion> getVersion(JdbcHelper jdbcHelper); - default String createSchemaSql() { - return String.format(CREATE_VERSION_SCHEMA_STATEMENT, gav()); - } + String getVersionSql(); - default String createTableSql() { - return String.format(CREATE_VERSION_TABLE_STATEMENT, gav()); - } + String createTableSql(); - default String deleteTableSql() { - return String.format(DELETE_VERSION_TABLE_STATEMENT, gav()); - } + String saveTableSql(String version, Date date); - default String fillVersionSql(String version) { - return String.format(FILL_VERSION_TABLE_STATEMENT, gav(), version); + default String createSchemaSql() { + return String.format(CREATE_VERSION_SCHEMA_STATEMENT, schemaName()); } default String dropTableSql() { return String.format(DROP_VERSION_TABLE_STATEMENT, gav()); } - default String saveTableSql(String version) { - return deleteTableSql() + fillVersionSql(version); - } - - default void save(JdbcHelper jdbcHelper, String version) { - try { - jdbcHelper.runUpdate(saveTableSql(version)); - } catch (Exception e) { - throw new TopiaException(String.format("Could not save version %s", version), e); - } - } - - default void save(TopiaSqlSupport sqlSupport, String version) { - try { - sqlSupport.executeSql(saveTableSql(version)); - } catch (Exception e) { - throw new TopiaException(String.format("Could not save version %s", version), e); - } - } - default boolean isTableExist(JdbcHelper jdbcHelper) { try { return jdbcHelper.isTableExist(schemaName(), tableName()); @@ -181,27 +208,35 @@ public interface MigrationServiceSqlHelper { } } - default void createSchema(JdbcHelper jdbcHelper) { + default void createSchema(TopiaSqlSupport sqlSupport) { try { - jdbcHelper.runUpdate(createSchemaSql()); + sqlSupport.executeSql(createSchemaSql()); } catch (Exception e) { - throw new TopiaException(String.format("Could not create version table %s", gav()), e); + throw new TopiaException(String.format("Could not create version schema %s", gav()), e); } } - default void createTable(JdbcHelper jdbcHelper) { + default void createTable(TopiaSqlSupport sqlSupport) { try { - jdbcHelper.runUpdate(createTableSql()); + sqlSupport.executeSql(createTableSql()); } catch (Exception e) { throw new TopiaException(String.format("Could not create version table %s", gav()), e); } } - default void dropTable(JdbcHelper jdbcHelper) { + default void dropTable(TopiaSqlSupport sqlSupport) { try { - jdbcHelper.runUpdate(dropTableSql()); + sqlSupport.executeSql(dropTableSql()); } catch (Exception e) { throw new TopiaException(String.format("Could not drop version table %s", gav()), e); } } + + default void save(TopiaSqlSupport sqlSupport, String version, Date date) { + try { + sqlSupport.executeSql(saveTableSql(version, date)); + } catch (Exception e) { + throw new TopiaException(String.format("Could not save version %s", version), e); + } + } } ===================================== toolkit/persistence/src/main/java/org/nuiton/topia/service/migration/version/TMSVersion.java ===================================== @@ -25,6 +25,7 @@ package org.nuiton.topia.service.migration.version; import io.ultreia.java4all.util.Version; import java.io.Serializable; +import java.util.Date; /** * @author Tony Chemit - dev(a)tchemit.fr @@ -33,27 +34,23 @@ public class TMSVersion implements Serializable { private static final long serialVersionUID = 1L; - private String version; + private final Version version; + private final Date date; - public TMSVersion() { - } - - public TMSVersion(String version) { + public TMSVersion(String version, Date date) { if (version == null || version.isEmpty()) { throw new IllegalArgumentException("version parameter can not be null nor empty."); } - this.version = version; + this.version = Version.valueOf(version); + this.date = date; } - public String getVersion() { + public Version getVersion() { return version; } - public void setVersion(String version) { - this.version = version; + public Date getDate() { + return date; } - public Version toVersion() { - return version == null || version.isEmpty() ? null : Version.valueOf(version); - } } ===================================== toolkit/persistence/src/main/java/org/nuiton/topia/service/sql/internal/consumer/AddVersionTableConsumer.java ===================================== @@ -29,6 +29,8 @@ import org.nuiton.topia.service.sql.internal.SqlRequestConsumer; import org.nuiton.topia.service.sql.internal.SqlRequestSetConsumerContext; import org.nuiton.topia.service.sql.internal.request.AddVersionTableRequest; +import java.util.Date; + /** * Created on 21/02/2021. * @@ -41,8 +43,9 @@ public class AddVersionTableConsumer implements SqlRequestConsumer<AddVersionTab public void consume(AddVersionTableRequest request, SqlRequestSetConsumerContext context) { try { SqlScriptWriter writer = context.getWriter(); + writer.writeSql(MigrationServiceSqlHelper.CURRENT.createSchemaSql()); writer.writeSql(MigrationServiceSqlHelper.CURRENT.createTableSql()); - writer.writeSql(MigrationServiceSqlHelper.CURRENT.fillVersionSql(request.getDbVersion().getVersion())); + writer.writeSql(MigrationServiceSqlHelper.CURRENT.saveTableSql(request.getDbVersion().getVersion(), new Date())); } catch (Exception e) { throw new TopiaException(String.format("Could not add version table for reason: %s", e.getMessage()), e); } View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/9f27b93b8b44a72a1405273cc… -- View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/commit/9f27b93b8b44a72a1405273cc… You're receiving this email because of your account on gitlab.com.

1 0