Skip to content

Commit 82aa444

Browse files
alicelaalicela
authored
Merge pull request #61 from InseeFr/acceptance
Acceptance
2 parents 956be73 + c5675ac commit 82aa444

11 files changed

+96
-133
lines changed

‎metadataapi-changeLog.txt‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
3.3.0 : Passage à Java 11 + Correctif sur les chef lieu
2+
3.2.1 : Passage à Java 11 pour les tests de charge (non mis en production
13
3.2.0 : Ajout des services pour les circonscriptions territoriales (Geo)
24
3.1.0 : Ajout des services des collectivités d'Outre-mer (Geo)
35
3.0.5 : Montée de version log4J

‎pom.xml‎

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,15 @@
77
<packaging>war</packaging>
88
<name>Implementation of the RMéS metadata API</name>
99

10-
<version>3.2.0</version>
10+
<version>3.3.0</version>
1111

1212
<properties>
1313
<title>API RM\u00e9S</title>
1414
<description>API sur les m\u00e9tadonn\u00e9es de l'Insee</description>
1515

1616
<!-- Dependencies' version -->
17-
<maven.compiler.source>1.8</maven.compiler.source>
18-
<maven.compiler.target>1.8</maven.compiler.target>
17+
<maven.compiler.source>11</maven.compiler.source>
18+
<maven.compiler.target>11</maven.compiler.target>
1919
<jersey.version>2.29.1</jersey.version>
2020
<jackson.version>2.10.1</jackson.version>
2121
<jakarta.version>4.0.3</jakarta.version>
@@ -30,7 +30,9 @@
3030

3131
<!-- Configuration Sonar -->
3232
<sonar.exclusions>src/main/java/fr/insee/rmes/queries/**/*,
33-
src/main/java/fr/insee/rmes/modeles/**/*</sonar.exclusions>
33+
src/main/java/fr/insee/rmes/modeles/**/*,
34+
src/main/java/fr/insee/rmes/utils/XmlUtils.java
35+
</sonar.exclusions>
3436
<sonar.projectKey>${project.groupId}:${project.artifactId}</sonar.projectKey>
3537
<sonar.projectName>Metadata-API</sonar.projectName>
3638
<sonar.core.codeCoveragePlugin>jacoco</sonar.core.codeCoveragePlugin>
@@ -39,7 +41,7 @@
3941
<sonar.language>java</sonar.language>
4042
<version.maven-jacoco>0.8.5</version.maven-jacoco>
4143
<version.maven-sonar>3.7.0.1746</version.maven-sonar>
42-
<argLine>-Xms256m -Xmx512m -XX:MaxPermSize=128m -ea -Dfile.encoding=UTF-8</argLine>
44+
<argLine>-Xms256m -Xmx512m -ea -Dfile.encoding=UTF-8</argLine>
4345

4446
</properties>
4547

@@ -80,7 +82,7 @@
8082
</dependency>
8183
<dependency>
8284
<groupId>org.apache.logging.log4j</groupId>
83-
<artifactId>log4j-slf4j-impl</artifactId>
85+
<artifactId>log4j-slf4j18-impl</artifactId>
8486
<version>${log.version}</version>
8587
</dependency>
8688

@@ -208,6 +210,11 @@
208210
</resource>
209211
</resources>
210212
<plugins>
213+
<plugin>
214+
<groupId>org.apache.maven.plugins</groupId>
215+
<artifactId>maven-war-plugin</artifactId>
216+
<version>3.3.2</version>
217+
</plugin>
211218
<plugin>
212219
<groupId>org.sonarsource.scanner.maven</groupId>
213220
<artifactId>sonar-maven-plugin</artifactId>

‎src/main/java/fr/insee/rmes/modeles/geo/territoire/CirconscriptionTerritoriale.java‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,6 @@
44
import javax.xml.bind.annotation.XmlAccessorType;
55
import javax.xml.bind.annotation.XmlRootElement;
66

7-
import com.fasterxml.jackson.annotation.JsonProperty;
8-
import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlProperty;
97
import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlRootElement;
108

119
import fr.insee.rmes.modeles.geo.EnumTypeGeographie;

‎src/main/java/fr/insee/rmes/queries/geo/GeoQueries.java‎

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -315,13 +315,8 @@ private static String getAscendantOrDescendantsQuery(
315315
params.put(FILTRE, filtreNom);
316316
params.put(COM,com);
317317
params.put(ASCENDANT, String.valueOf(ascendant));
318-
//
319-
// if (EnumTypeGeographie.DISTRICT.equals(typeOrigine)) {
320-
// return buildRequest(QUERIES_FOLDER, "getAscendantsOrDescendantsByCodeTypeDateDistrict.ftlh", params);
321-
// }
322-
// else {
323318
return buildRequest(QUERIES_FOLDER, "getAscendantsOrDescendantsByCodeTypeDate.ftlh", params);
324-
// }
319+
325320
}
326321

327322
private static String getPreviousOrNextQuery(
@@ -351,11 +346,7 @@ private static String getTerritoireFiltre(String code, String date, String filtr
351346
Map<String, Object> params = buildCodeAndDateAndFilterParams(code, date, filtreNom,com);
352347
params.put("territoire", typeGeo.getTypeObjetGeo());
353348
params.put("chefLieu", typeGeo.getChefLieuPredicate());
354-
// if (EnumTypeGeographie.DISTRICT.equals(typeGeo)) {
355-
// return buildRequest(QUERIES_FOLDER, "getTerritoireByCodeAndDateDistrict.ftlh", params);
356-
// } else {
357349
return buildRequest(QUERIES_FOLDER, "getTerritoireByCodeDateNomcommune.ftlh", params);
358-
// }
359350
}
360351

361352
private static Map<String, Object> buildCodeAndDateAndFilterParams(String code, String date, String filtreNom, boolean com) {

‎src/main/resources/request/geographie/getAllProjectionByTypeDate.ftlh‎

Lines changed: 32 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,22 @@ WHERE {
1212

1313

1414
OPTIONAL {
15-
?origine ((igeo:sousPrefecture|igeo:prefecture|igeo:prefectureDeRegion)/igeo:codeINSEE) ?chefLieuOrigine .
15+
?uri (igeo:sousPrefecture|igeo:prefecture|igeo:prefectureDeRegion) ?chefLieuRDF .
16+
?chefLieuRDF igeo:codeINSEE ?chefLieuOrigine .
17+
18+
OPTIONAL {
19+
?evenementCreationChefLieu igeo:creation ?chefLieuRDF ;
20+
igeo:date ?dateCreationChefLieu .
21+
}
22+
OPTIONAL {
23+
?evenementSuppressionChefLieu igeo:suppression ?chefLieuRDF ;
24+
igeo:date ?dateSuppressionChefLieu.
25+
}
26+
27+
<#if date != "*">
28+
FILTER(!BOUND(?dateCreationChefLieu) || ?dateCreationChefLieu <= '${date}'^^xsd:date)
29+
FILTER(!BOUND(?dateSuppressionChefLieu) || ?dateSuppressionChefLieu > '${date}'^^xsd:date)
30+
</#if>
1631
}
1732

1833

@@ -56,7 +71,22 @@ WHERE {
5671
}
5772

5873
OPTIONAL {
59-
?uri ((igeo:sousPrefecture|igeo:prefecture|igeo:prefectureDeRegion)/igeo:codeINSEE) ?chefLieu.
74+
?uri (igeo:sousPrefecture|igeo:prefecture|igeo:prefectureDeRegion) ?chefLieuRDF .
75+
?chefLieuRDF igeo:codeINSEE ?chefLieu .
76+
77+
OPTIONAL {
78+
?evenementCreationChefLieu igeo:creation ?chefLieuRDF ;
79+
igeo:date ?dateCreationChefLieu .
80+
}
81+
OPTIONAL {
82+
?evenementSuppressionChefLieu igeo:suppression ?chefLieuRDF ;
83+
igeo:date ?dateSuppressionChefLieu.
84+
}
85+
86+
<#if date != "*">
87+
FILTER(!BOUND(?dateCreationChefLieu) || ?dateCreationChefLieu <= '${date}'^^xsd:date)
88+
FILTER(!BOUND(?dateSuppressionChefLieu) || ?dateSuppressionChefLieu > '${date}'^^xsd:date)
89+
</#if>
6090
}
6191

6292
FILTER(!BOUND(?dateCreation) || ?dateCreation <= '${dateProjection}'^^xsd:date)

‎src/main/resources/request/geographie/getAscendantsOrDescendantsByCodeTypeDate.ftlh‎

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,22 @@ FROM <http://rdf.insee.fr/graphes/geo/cog>
2121
</#if>
2222

2323
OPTIONAL {
24-
?uri ((igeo:sousPrefecture|igeo:prefecture|igeo:prefectureDeRegion)/igeo:codeINSEE) ?chefLieu.
24+
?uri (igeo:sousPrefecture|igeo:prefecture|igeo:prefectureDeRegion) ?chefLieuRDF .
25+
?chefLieuRDF igeo:codeINSEE ?chefLieu .
26+
27+
OPTIONAL {
28+
?evenementCreationChefLieu igeo:creation ?chefLieuRDF ;
29+
igeo:date ?dateCreationChefLieu .
30+
}
31+
OPTIONAL {
32+
?evenementSuppressionChefLieu igeo:suppression ?chefLieuRDF ;
33+
igeo:date ?dateSuppressionChefLieu.
34+
}
35+
36+
<#if date != "*">
37+
FILTER(!BOUND(?dateCreationChefLieu) || ?dateCreationChefLieu <= '${date}'^^xsd:date)
38+
FILTER(!BOUND(?dateSuppressionChefLieu) || ?dateSuppressionChefLieu > '${date}'^^xsd:date)
39+
</#if>
2540
}
2641

2742
OPTIONAL {

‎src/main/resources/request/geographie/getAscendantsOrDescendantsByCodeTypeDateCOM.ftlh‎

Lines changed: 0 additions & 46 deletions
This file was deleted.

‎src/main/resources/request/geographie/getAscendantsOrDescendantsByCodeTypeDateDistrict.ftlh‎

Lines changed: 0 additions & 30 deletions
This file was deleted.

‎src/main/resources/request/geographie/getPreviousOrNextByCodeTypeDate.ftlh‎

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,22 @@ WHERE {
5151
}
5252

5353
OPTIONAL {
54-
?uri ((igeo:sousPrefecture|igeo:prefecture|igeo:prefectureDeRegion)/igeo:codeINSEE) ?chefLieu.
54+
?uri (igeo:sousPrefecture|igeo:prefecture|igeo:prefectureDeRegion) ?chefLieuRDF .
55+
?chefLieuRDF igeo:codeINSEE ?chefLieu .
56+
57+
OPTIONAL {
58+
?evenementCreationChefLieu igeo:creation ?chefLieuRDF ;
59+
igeo:date ?dateCreationChefLieu .
60+
}
61+
OPTIONAL {
62+
?evenementSuppressionChefLieu igeo:suppression ?chefLieuRDF ;
63+
igeo:date ?dateSuppressionChefLieu.
64+
}
65+
66+
<#if date != "*">
67+
FILTER(!BOUND(?dateCreationChefLieu) || ?dateCreationChefLieu <= '${date}'^^xsd:date)
68+
FILTER(!BOUND(?dateSuppressionChefLieu) || ?dateSuppressionChefLieu > '${date}'^^xsd:date)
69+
</#if>
5570
}
5671

5772

‎src/main/resources/request/geographie/getProjectionByCodeTypeDate.ftlh‎

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,22 @@ WHERE {
5858
}
5959

6060
OPTIONAL {
61-
?uri ((igeo:sousPrefecture|igeo:prefecture|igeo:prefectureDeRegion)/igeo:codeINSEE) ?chefLieu.
61+
?uri (igeo:sousPrefecture|igeo:prefecture|igeo:prefectureDeRegion) ?chefLieuRDF .
62+
?chefLieuRDF igeo:codeINSEE ?chefLieu .
63+
64+
OPTIONAL {
65+
?evenementCreationChefLieu igeo:creation ?chefLieuRDF ;
66+
igeo:date ?dateCreationChefLieu .
67+
}
68+
OPTIONAL {
69+
?evenementSuppressionChefLieu igeo:suppression ?chefLieuRDF ;
70+
igeo:date ?dateSuppressionChefLieu.
71+
}
72+
73+
<#if date != "*">
74+
FILTER(!BOUND(?dateCreationChefLieu) || ?dateCreationChefLieu <= '${date}'^^xsd:date)
75+
FILTER(!BOUND(?dateSuppressionChefLieu) || ?dateSuppressionChefLieu > '${date}'^^xsd:date)
76+
</#if>
6277
}
6378

6479
FILTER(!BOUND(?dateCreation) || ?dateCreation <= '${dateProjection}'^^xsd:date)

0 commit comments

Comments
 (0)