chore: 4.20.0#1091
Open
EmmanuelDemey wants to merge 786 commits into
Open
Conversation
* fix: disable test compilation for docker build * feat: improve CI * fix: solve unit test * fix: unit tests * fix: unit tests * feat: use HIE * feat: remove default stamp * feat: add ddi3 to ddi4 converters (#1025) * fix: disable test compilation for docker build * feat: improve CI * fix: solve unit test * fix: unit tests * fix: unit tests * feat: add ddi3 to ddi4 converters * fix: solve unit test * fix: solve sonar issues * refactor: rename queries (#1029) * fix: disable test compilation for docker build * feat: improve CI * fix: solve unit test * fix: unit tests * fix: unit tests * refactor: rename all queries class * feat: be able to validate json schema (#1030) * refactor: delete log * Update version to 4.15.0-rc1 in pom.xml * feat: add unit tests for Collections * feat: migrate rdfconnection interface * feat: migrate rasmp service * Update version to 4.15.0-rc2 in pom.xml * testing with more maven logs * test to force end of lines of mvnw * fix typo * test avec nouveau dockerfile * adding missing module * fix: add missing port * fix: remove wrong properties * feat: add organizations endpoint with stamp and label retrieval from GraphDB - Add OrganisationOption model with stamp and label fields - Create OrganisationRepository server-side port - Implement OrganisationGraphDBRepository using SPARQL queries via FreeMarker - Add /v2/stamps endpoint to StampResources returning full organization data - Add comprehensive unit and integration tests using Testcontainers * feat: fix issue with organisations * feat: add FROM property when fetching series by stamp * feat: migrate some sparql queries * Update version from 4.15.0-rc3 to 4.15.0-rc4 * feat: add other organisation graph * Update version from 4.15.0-rc4 to 4.15.0-rc5 * Update version from 4.15.0-rc5 to 4.15.0 --------- Co-authored-by: Nicolas Echallier <rkouere@users.noreply.github.com>
Comment on lines
+503
to
+505
| ColecticaSetItem[] setItems = restClient | ||
| .get() | ||
| .uri(setUrl) |
Comment on lines
+1436
to
+1438
| ColecticaSetItem[] setItems = restClient | ||
| .get() | ||
| .uri(setUrl) |
Comment on lines
+580
to
+584
| logger.info( | ||
| "Fetching DDI4 Group from Colectica API for agencyId: {}, id: {}", | ||
| agencyId, | ||
| id | ||
| ); |
Comment on lines
+596
to
+599
| logger.info( | ||
| "Fetching full DDI set for Group from: {}", | ||
| ddisetUrl | ||
| ); |
Comment on lines
+610
to
+613
| logger.error( | ||
| "Received empty response from Colectica API for ddiset URL: {}", | ||
| ddisetUrl | ||
| ); |
Comment on lines
+1099
to
+1103
| logger.info( | ||
| "Updating full physical instance {}/{} with all DDI objects in Colectica", | ||
| agencyId, | ||
| id | ||
| ); |
Comment on lines
+1162
to
+1166
| logger.info( | ||
| "Successfully updated full physical instance with id: {} ({} items saved)", | ||
| id, | ||
| colecticaItems.size() | ||
| ); |
Comment on lines
+1776
to
+1779
| logger.info( | ||
| "Searching StudyUnit XML by operationIri: {}", | ||
| operationIri | ||
| ); |
Comment on lines
+667
to
+669
| byte[] ddisetBytes = restClient | ||
| .get() | ||
| .uri(ddisetUrl) |
|
Adopt the new ddi-schema.json structure for localized strings: Label,
Title, Description and VariableName are now flat arrays of langString
objects ({ "@language", "@value" }) instead of nested
{ Content/String: [ { MultilingualStringValue: { LanguageTag, Value } } ] }.
- Replace MultilingualString* / Label / Title / Description / VariableName
records with a single LangString record + LangStrings helper.
- Parent records (Citation, Ddi4Category/CodeList/Variable/DataRelationship,
LogicalRecord) now hold List<LangString>.
- Update Ddi3XmlReader/Writer, AbstractDDIItemConverter, DDIRepositoryImpl,
LocalColecticaGroupInitConfiguration and CodeListSummaryResponse.
- Update all related tests.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.




No description provided.