Merge pull request #1004 from OP-TED/prepare/1.7.5

Prepare release of SDK 1.7.5
OP-TED · Sep 2, 2024 · 02baaaf · 02baaaf
2 parents 6717840 + b0d46ed
commit 02baaaf
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 43 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,13 @@
+# SDK 1.7.5 Release Notes
+
+This release brings the following corrections in the validation rules, making them more permissive:
+
+* Remove rules that try to fetch another notice. They never had any effect, as the service they call does not exist, and they can cause errors with some schematron implementations.
+
+A comprehensive list of changes between SDK 1.7.4 and SDK 1.7.5 can be seen at <https://github.com/OP-TED/eForms-SDK/compare/1.7.4...1.7.5>
+
+You can explore the changes between those versions at <https://docs.ted.europa.eu/eforms-sdk-explorer?base=1.7.4&version=1.7.5>
+
 # SDK 1.7.4 Release Notes
 
 This release brings the following corrections in the validation rules, making them more permissive:

diff --git a/fields/fields.json b/fields/fields.json
@@ -1,6 +1,6 @@
 {
   "ublVersion" : "2.3",
-  "sdkVersion" : "eforms-sdk-1.7.4",
+  "sdkVersion" : "eforms-sdk-1.7.5",
   "metadataDatabase" : {
     "version" : "1.7.1",
     "createdOn" : "2023-04-24T12:34:54"

diff --git a/pom.xml b/pom.xml
@@ -3,7 +3,7 @@
 
   <groupId>eu.europa.ted.eforms</groupId>
   <artifactId>eforms-sdk</artifactId>
-  <version>1.7.4</version>
+  <version>1.7.5</version>
   <packaging>jar</packaging>
 
   <name>eForms SDK</name>
@@ -48,7 +48,7 @@
 
   <properties>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-    <project.build.outputTimestamp>2024-04-29T11:47:40Z</project.build.outputTimestamp>
+    <project.build.outputTimestamp>2024-08-13T08:32:14Z</project.build.outputTimestamp>
 
     <version.eforms-sdk-analyzer>1.6.0-SNAPSHOT</version.eforms-sdk-analyzer>
 

diff --git a/schematrons/dynamic/config.sch b/schematrons/dynamic/config.sch
@@ -1,5 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!-- URL prefix of the service to fetch a notice.
 	 The value must be between single quotes.
-	 The notice identifier will be appended, to build the URL used to fetch the corresponding notice XML. -->
+	 The notice identifier will be appended, to build the URL used to fetch the corresponding notice XML.
+	 Currently not used. -->
 <let xmlns="http://purl.oclc.org/dsdl/schematron" name="urlPrefix" value="'http://localhost:8080/notices/'"/>
diff --git a/schematrons/dynamic/validation-stage-6a.sch b/schematrons/dynamic/validation-stage-6a.sch
@@ -1,7 +1,4 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!--File generated from metadata database version 1.7.0 created on the 2023-03-31T15:30.-->
 <pattern id="EFORMS-validation-stage-6a" xmlns="http://purl.oclc.org/dsdl/schematron">
-	<rule context="/*/cac:ProcurementProject/cac:MainCommodityClassification[/*/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/efext:EformsExtension/efac:Changes/efbc:ChangedNoticeIdentifier]">
-		<assert id="BR-BT-00262-0300" role="ERROR" diagnostics="BT-262-Procedure" test="(cbc:ItemClassificationCode/normalize-space(text()) = fn:doc(concat($urlPrefix, /*/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/efext:EformsExtension/efac:Changes/efbc:ChangedNoticeIdentifier/normalize-space(text())))/*/cac:ProcurementProject/cac:MainCommodityClassification/cbc:ItemClassificationCode/normalize-space(text())) or not(fn:doc(concat($urlPrefix, /*/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/efext:EformsExtension/efac:Changes/efbc:ChangedNoticeIdentifier/normalize-space(text())))/*/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/efext:EformsExtension/efac:NoticeSubType/cbc:SubTypeCode/normalize-space(text()) = ('10','11','CEI','12','13','14','15','16','17','18','19','E3','20','21','22','23','24','25','26','27','28','29','30','31','32','E4','T02','33','34','35','36','37','38','39','40','E5') and (fn:doc(concat($urlPrefix, /*/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/efext:EformsExtension/efac:Changes/efbc:ChangedNoticeIdentifier/normalize-space(text())))/*/cac:ProcurementProject/cac:MainCommodityClassification/cbc:ItemClassificationCode))">rule|text|BR-BT-00262-0300</assert>
-	</rule>
 </pattern>
diff --git a/schematrons/dynamic/validation-stage-6b.sch b/schematrons/dynamic/validation-stage-6b.sch
@@ -11,40 +11,4 @@
 	</assert>
 </rule>
 
-<!-- Schematron rules to enforce that certain values in a Change Notice are the 
-	 same as those in the parent notice (the notice that is subject to the change). -->
-
-<!-- Reference to the parent notice. -->
-<let name="parentNoticeId" value="/*/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/efext:EformsExtension/efac:Changes/efbc:ChangedNoticeIdentifier/fn:normalize-space(text())"/>
-
-<!-- Function that fetches a notice based on its identifier.
-	 The value of $urlPrefix is defined in config.sch. -->
-<let name="getParentNotice" value="function($id) { fn:doc(concat($urlPrefix, $id)) }"/>
-
-<!-- XML document of the parent notice. -->
-<let name="parentNotice" value="$getParentNotice($parentNoticeId)"/>
-
-<!-- Rules applying at root or Procedure level -->
-<rule context="/*[$isChangeNotice = true()]">
-	<!-- Notice subtype -->
-	<assert role="ERROR" test="$noticeSubType eq $parentNotice/*/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/efext:EformsExtension/efac:NoticeSubType/cbc:SubTypeCode/fn:normalize-space(text())">
-		The notice subtype must be the same as for the parent notice.
-	</assert>
-
-	<!-- BT-105-Procedure Procedure Type -->
-	<assert role="ERROR" test="deep-equal( cac:TenderingProcess/cbc:ProcedureCode/fn:normalize-space(text()), $parentNotice/*/cac:TenderingProcess/cbc:ProcedureCode/fn:normalize-space(text()) )">
-		The value of "Procedure Type" (BT-105-Procedure) value must be the same as for the parent notice.
-	</assert>
-
-	<!-- BT-04-notice Procedure Identifier -->
-	<assert role="ERROR" test="cbc:ContractFolderID/fn:normalize-space(text()) = $parentNotice/*/cbc:ContractFolderID/fn:normalize-space(text())">
-		The value of "Procedure Identifier" (BT-04-notice) must be the same as for the parent notice.
-	</assert>
-
-	<!-- BT-23-Procedure Main Nature  -->
-	<assert role="ERROR" test="( cac:ProcurementProject/cbc:ProcurementTypeCode[@listName='contract-nature']/fn:normalize-space(text()) = $parentNotice/*/cac:ProcurementProject/cbc:ProcurementTypeCode[@listName='contract-nature']/fn:normalize-space(text()) )">
-		The value of "Main Nature" (BT-23-Procedure) must be the same as for the parent notice.
-	</assert>
-</rule>
-
 </pattern>