From 9d6a0e2237965ae6f452d09f76ca52480d6b21f5 Mon Sep 17 00:00:00 2001
From: Fred Atherden <43879983+fred-atherden@users.noreply.github.com>
Date: Tue, 10 Dec 2024 09:07:37 +0000
Subject: [PATCH] VOR - account for sub-articles in dupe orcid tests

---
 src/final-JATS-schematron.sch                 |  2 +-
 src/final-JATS-schematron.xsl                 | 33 ++++++++-------
 src/final-package-JATS-schematron.sch         |  2 +-
 src/pre-JATS-schematron.sch                   |  2 +-
 src/pre-JATS-schematron.xsl                   | 41 ++++++++++---------
 src/schematron.sch                            |  8 ++--
 .../orcid-tests/final-orcid-test-3/fail.xml   |  2 +-
 .../final-orcid-test-3/final-orcid-test-3.sch |  4 +-
 .../orcid-tests/final-orcid-test-3/pass.xml   |  2 +-
 .../gen/orcid-tests/pre-orcid-test-3/fail.xml |  2 +-
 .../gen/orcid-tests/pre-orcid-test-3/pass.xml |  2 +-
 .../pre-orcid-test-3/pre-orcid-test-3.sch     |  4 +-
 test/xspec/schematron.sch                     |  8 ++--
 test/xspec/schematron.xspec                   |  8 ++--
 14 files changed, 63 insertions(+), 57 deletions(-)

diff --git a/src/final-JATS-schematron.sch b/src/final-JATS-schematron.sch
index 28a2366e87..76cf028450 100644
--- a/src/final-JATS-schematron.sch
+++ b/src/final-JATS-schematron.sch
@@ -1962,7 +1962,7 @@
 	  
 	  
 	  
-	  <report test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)" role="error" id="final-orcid-test-3">[final-orcid-test-3] <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</report>
+	  <assert test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1" role="error" id="final-orcid-test-3">[final-orcid-test-3] <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</assert>
 		
 		</rule></pattern><pattern id="email-tests-pattern"><rule context="article-meta//email" id="email-tests">
 		
diff --git a/src/final-JATS-schematron.xsl b/src/final-JATS-schematron.xsl
index bf3ffd4a57..651166b1d5 100644
--- a/src/final-JATS-schematron.xsl
+++ b/src/final-JATS-schematron.xsl
@@ -9143,21 +9143,24 @@
             </svrl:failed-assert>
          </xsl:otherwise>
       </xsl:choose>
-      <!--REPORT error-->
-      <xsl:if test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)">
-         <svrl:successful-report xmlns:svrl="http://purl.oclc.org/dsdl/svrl" test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)">
-            <xsl:attribute name="id">final-orcid-test-3</xsl:attribute>
-            <xsl:attribute name="role">error</xsl:attribute>
-            <xsl:attribute name="location">
-               <xsl:apply-templates select="." mode="schematron-select-full-path"/>
-            </xsl:attribute>
-            <svrl:text>[final-orcid-test-3] <xsl:text/>
-               <xsl:value-of select="e:get-name(parent::*/name[1])"/>
-               <xsl:text/>'s ORCiD is the same as another author's - <xsl:text/>
-               <xsl:value-of select="."/>
-               <xsl:text/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</svrl:text>
-         </svrl:successful-report>
-      </xsl:if>
+      <!--ASSERT error-->
+      <xsl:choose>
+         <xsl:when test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1"/>
+         <xsl:otherwise>
+            <svrl:failed-assert xmlns:svrl="http://purl.oclc.org/dsdl/svrl" test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1">
+               <xsl:attribute name="id">final-orcid-test-3</xsl:attribute>
+               <xsl:attribute name="role">error</xsl:attribute>
+               <xsl:attribute name="location">
+                  <xsl:apply-templates select="." mode="schematron-select-full-path"/>
+               </xsl:attribute>
+               <svrl:text>[final-orcid-test-3] <xsl:text/>
+                  <xsl:value-of select="e:get-name(parent::*/name[1])"/>
+                  <xsl:text/>'s ORCiD is the same as another author's - <xsl:text/>
+                  <xsl:value-of select="."/>
+                  <xsl:text/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</svrl:text>
+            </svrl:failed-assert>
+         </xsl:otherwise>
+      </xsl:choose>
       <xsl:apply-templates select="*" mode="M107"/>
    </xsl:template>
    <xsl:template match="text()" priority="-1" mode="M107"/>
diff --git a/src/final-package-JATS-schematron.sch b/src/final-package-JATS-schematron.sch
index 7955264ea7..c1825c41fa 100644
--- a/src/final-package-JATS-schematron.sch
+++ b/src/final-package-JATS-schematron.sch
@@ -1968,7 +1968,7 @@
 	  
 	  
 	  
-	  <report test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)" role="error" id="final-orcid-test-3"><value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</report>
+	  <assert test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1" role="error" id="final-orcid-test-3"><value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</assert>
 		
 		</rule></pattern><pattern id="email-tests-pattern"><rule context="article-meta//email" id="email-tests">
 		
diff --git a/src/pre-JATS-schematron.sch b/src/pre-JATS-schematron.sch
index efa84a5017..55e86d4e2d 100644
--- a/src/pre-JATS-schematron.sch
+++ b/src/pre-JATS-schematron.sch
@@ -1905,7 +1905,7 @@
 		<!-- Needs updating to only allow https when this is implemented -->
 	  <assert test="matches(.,'^http[s]?://orcid.org/[\d]{4}-[\d]{4}-[\d]{4}-[\d]{3}[0-9X]$')" role="error" id="orcid-test-2">[orcid-test-2] contrib-id[@contrib-id-type="orcid"] must contain a valid ORCID URL in the format 'https://orcid.org/0000-0000-0000-0000'</assert>
 	  
-	  <report test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)" role="warning" id="pre-orcid-test-3">[pre-orcid-test-3] <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD - <value-of select="."/> - is associated with <value-of select="count(preceding::contrib-id[@contrib-id-type='orcid' and text()=$text]) + count(following::contrib-id[@contrib-id-type='orcid' and text()=$text]) + 1"/> authors. Please confirm which author this ORCiD belongs to.'.</report>
+	  <assert test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1" role="warning" id="pre-orcid-test-3">[pre-orcid-test-3] <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD - <value-of select="."/> - is associated with <value-of select="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text])"/> authors. Please confirm which author this ORCiD belongs to.'.</assert>
 	  
 	  
 		
diff --git a/src/pre-JATS-schematron.xsl b/src/pre-JATS-schematron.xsl
index e3aa4cf665..53d1729f6d 100644
--- a/src/pre-JATS-schematron.xsl
+++ b/src/pre-JATS-schematron.xsl
@@ -9034,25 +9034,28 @@
             </svrl:failed-assert>
          </xsl:otherwise>
       </xsl:choose>
-      <!--REPORT warning-->
-      <xsl:if test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)">
-         <svrl:successful-report xmlns:svrl="http://purl.oclc.org/dsdl/svrl" test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)">
-            <xsl:attribute name="id">pre-orcid-test-3</xsl:attribute>
-            <xsl:attribute name="role">warning</xsl:attribute>
-            <xsl:attribute name="location">
-               <xsl:apply-templates select="." mode="schematron-select-full-path"/>
-            </xsl:attribute>
-            <svrl:text>[pre-orcid-test-3] <xsl:text/>
-               <xsl:value-of select="e:get-name(parent::*/name[1])"/>
-               <xsl:text/>'s ORCiD is the same as another author's - <xsl:text/>
-               <xsl:value-of select="."/>
-               <xsl:text/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD - <xsl:text/>
-               <xsl:value-of select="."/>
-               <xsl:text/> - is associated with <xsl:text/>
-               <xsl:value-of select="count(preceding::contrib-id[@contrib-id-type='orcid' and text()=$text]) + count(following::contrib-id[@contrib-id-type='orcid' and text()=$text]) + 1"/>
-               <xsl:text/> authors. Please confirm which author this ORCiD belongs to.'.</svrl:text>
-         </svrl:successful-report>
-      </xsl:if>
+      <!--ASSERT warning-->
+      <xsl:choose>
+         <xsl:when test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1"/>
+         <xsl:otherwise>
+            <svrl:failed-assert xmlns:svrl="http://purl.oclc.org/dsdl/svrl" test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1">
+               <xsl:attribute name="id">pre-orcid-test-3</xsl:attribute>
+               <xsl:attribute name="role">warning</xsl:attribute>
+               <xsl:attribute name="location">
+                  <xsl:apply-templates select="." mode="schematron-select-full-path"/>
+               </xsl:attribute>
+               <svrl:text>[pre-orcid-test-3] <xsl:text/>
+                  <xsl:value-of select="e:get-name(parent::*/name[1])"/>
+                  <xsl:text/>'s ORCiD is the same as another author's - <xsl:text/>
+                  <xsl:value-of select="."/>
+                  <xsl:text/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD - <xsl:text/>
+                  <xsl:value-of select="."/>
+                  <xsl:text/> - is associated with <xsl:text/>
+                  <xsl:value-of select="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text])"/>
+                  <xsl:text/> authors. Please confirm which author this ORCiD belongs to.'.</svrl:text>
+            </svrl:failed-assert>
+         </xsl:otherwise>
+      </xsl:choose>
       <xsl:apply-templates select="*" mode="M105"/>
    </xsl:template>
    <xsl:template match="text()" priority="-1" mode="M105"/>
diff --git a/src/schematron.sch b/src/schematron.sch
index 72547a088d..315d9118fd 100644
--- a/src/schematron.sch
+++ b/src/schematron.sch
@@ -2551,13 +2551,13 @@
         role="error" 
         id="orcid-test-2">contrib-id[@contrib-id-type="orcid"] must contain a valid ORCID URL in the format 'https://orcid.org/0000-0000-0000-0000'</assert>
 	  
-	  <report test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)" 
+	  <assert test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1" 
         role="warning" 
-        id="pre-orcid-test-3"><value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD - <value-of select="."/> - is associated with <value-of select="count(preceding::contrib-id[@contrib-id-type='orcid' and text()=$text]) + count(following::contrib-id[@contrib-id-type='orcid' and text()=$text]) + 1"/> authors. Please confirm which author this ORCiD belongs to.'.</report>
+        id="pre-orcid-test-3"><value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD - <value-of select="."/> - is associated with <value-of select="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text])"/> authors. Please confirm which author this ORCiD belongs to.'.</assert>
 	  
-	  <report test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)" 
+	  <assert test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1" 
         role="error" 
-        id="final-orcid-test-3"><value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</report>
+        id="final-orcid-test-3"><value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</assert>
 		
 		</rule>
 	
diff --git a/test/tests/gen/orcid-tests/final-orcid-test-3/fail.xml b/test/tests/gen/orcid-tests/final-orcid-test-3/fail.xml
index 7003e767b5..db51220582 100644
--- a/test/tests/gen/orcid-tests/final-orcid-test-3/fail.xml
+++ b/test/tests/gen/orcid-tests/final-orcid-test-3/fail.xml
@@ -1,6 +1,6 @@
 <?oxygen SCHSchema="final-orcid-test-3.sch"?>
 <!--Context: contrib-id[@contrib-id-type='orcid']
-Test: report    (preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)
+Test: assert    count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1
 Message: 's ORCiD is the same as another author's - . Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors. -->
 <root xmlns:ali="http://www.niso.org/schemas/ali/1.0/" xmlns:mml="http://www.w3.org/1998/Math/MathML" xmlns:xlink="http://www.w3.org/1999/xlink">
   <article>
diff --git a/test/tests/gen/orcid-tests/final-orcid-test-3/final-orcid-test-3.sch b/test/tests/gen/orcid-tests/final-orcid-test-3/final-orcid-test-3.sch
index b3ab1c68c4..0672483b9d 100644
--- a/test/tests/gen/orcid-tests/final-orcid-test-3/final-orcid-test-3.sch
+++ b/test/tests/gen/orcid-tests/final-orcid-test-3/final-orcid-test-3.sch
@@ -1363,8 +1363,8 @@
   <pattern id="article-metadata">
     <rule context="contrib-id[@contrib-id-type='orcid']" id="orcid-tests">
       <let name="text" value="."/>
-      <report test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)" role="error" id="final-orcid-test-3">
-        <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</report>
+      <assert test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1" role="error" id="final-orcid-test-3">
+        <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</assert>
     </rule>
   </pattern>
   <pattern id="root-pattern">
diff --git a/test/tests/gen/orcid-tests/final-orcid-test-3/pass.xml b/test/tests/gen/orcid-tests/final-orcid-test-3/pass.xml
index 985254eb40..fb460e3474 100644
--- a/test/tests/gen/orcid-tests/final-orcid-test-3/pass.xml
+++ b/test/tests/gen/orcid-tests/final-orcid-test-3/pass.xml
@@ -1,6 +1,6 @@
 <?oxygen SCHSchema="final-orcid-test-3.sch"?>
 <!--Context: contrib-id[@contrib-id-type='orcid']
-Test: report    (preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)
+Test: assert    count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1
 Message: 's ORCiD is the same as another author's - . Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors. -->
 <root xmlns:ali="http://www.niso.org/schemas/ali/1.0/" xmlns:mml="http://www.w3.org/1998/Math/MathML" xmlns:xlink="http://www.w3.org/1999/xlink">
   <article>
diff --git a/test/tests/gen/orcid-tests/pre-orcid-test-3/fail.xml b/test/tests/gen/orcid-tests/pre-orcid-test-3/fail.xml
index d2ca79a4bc..0466e299cf 100644
--- a/test/tests/gen/orcid-tests/pre-orcid-test-3/fail.xml
+++ b/test/tests/gen/orcid-tests/pre-orcid-test-3/fail.xml
@@ -1,6 +1,6 @@
 <?oxygen SCHSchema="pre-orcid-test-3.sch"?>
 <!--Context: contrib-id[@contrib-id-type='orcid']
-Test: report    (preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)
+Test: assert    count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1
 Message: 's ORCiD is the same as another author's - . Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD -  - is associated with  authors. Please confirm which author this ORCiD belongs to.'. -->
 <root xmlns:ali="http://www.niso.org/schemas/ali/1.0/" xmlns:mml="http://www.w3.org/1998/Math/MathML" xmlns:xlink="http://www.w3.org/1999/xlink">
   <article>
diff --git a/test/tests/gen/orcid-tests/pre-orcid-test-3/pass.xml b/test/tests/gen/orcid-tests/pre-orcid-test-3/pass.xml
index 5d38b4c72e..a7977aef50 100644
--- a/test/tests/gen/orcid-tests/pre-orcid-test-3/pass.xml
+++ b/test/tests/gen/orcid-tests/pre-orcid-test-3/pass.xml
@@ -1,6 +1,6 @@
 <?oxygen SCHSchema="pre-orcid-test-3.sch"?>
 <!--Context: contrib-id[@contrib-id-type='orcid']
-Test: report    (preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)
+Test: assert    count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1
 Message: 's ORCiD is the same as another author's - . Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD -  - is associated with  authors. Please confirm which author this ORCiD belongs to.'. -->
 <root xmlns:ali="http://www.niso.org/schemas/ali/1.0/" xmlns:mml="http://www.w3.org/1998/Math/MathML" xmlns:xlink="http://www.w3.org/1999/xlink">
   <article>
diff --git a/test/tests/gen/orcid-tests/pre-orcid-test-3/pre-orcid-test-3.sch b/test/tests/gen/orcid-tests/pre-orcid-test-3/pre-orcid-test-3.sch
index 30e3768e19..e731081219 100644
--- a/test/tests/gen/orcid-tests/pre-orcid-test-3/pre-orcid-test-3.sch
+++ b/test/tests/gen/orcid-tests/pre-orcid-test-3/pre-orcid-test-3.sch
@@ -1363,8 +1363,8 @@
   <pattern id="article-metadata">
     <rule context="contrib-id[@contrib-id-type='orcid']" id="orcid-tests">
       <let name="text" value="."/>
-      <report test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)" role="warning" id="pre-orcid-test-3">
-        <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD - <value-of select="."/> - is associated with <value-of select="count(preceding::contrib-id[@contrib-id-type='orcid' and text()=$text]) + count(following::contrib-id[@contrib-id-type='orcid' and text()=$text]) + 1"/> authors. Please confirm which author this ORCiD belongs to.'.</report>
+      <assert test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1" role="warning" id="pre-orcid-test-3">
+        <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD - <value-of select="."/> - is associated with <value-of select="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text])"/> authors. Please confirm which author this ORCiD belongs to.'.</assert>
     </rule>
   </pattern>
   <pattern id="root-pattern">
diff --git a/test/xspec/schematron.sch b/test/xspec/schematron.sch
index f2c5a422a6..88d81e09c6 100644
--- a/test/xspec/schematron.sch
+++ b/test/xspec/schematron.sch
@@ -2146,11 +2146,11 @@
 		<!-- Needs updating to only allow https when this is implemented -->
 	  <assert test="matches(.,'^http[s]?://orcid.org/[\d]{4}-[\d]{4}-[\d]{4}-[\d]{3}[0-9X]$')" role="error" id="orcid-test-2">contrib-id[@contrib-id-type="orcid"] must contain a valid ORCID URL in the format 'https://orcid.org/0000-0000-0000-0000'</assert>
 	  
-	  <report test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)" role="warning" id="pre-orcid-test-3">
-        <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD - <value-of select="."/> - is associated with <value-of select="count(preceding::contrib-id[@contrib-id-type='orcid' and text()=$text]) + count(following::contrib-id[@contrib-id-type='orcid' and text()=$text]) + 1"/> authors. Please confirm which author this ORCiD belongs to.'.</report>
+	  <assert test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1" role="warning" id="pre-orcid-test-3">
+        <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please add an author query - 'This ORCiD - <value-of select="."/> - is associated with <value-of select="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text])"/> authors. Please confirm which author this ORCiD belongs to.'.</assert>
 	  
-	  <report test="(preceding::contrib-id[@contrib-id-type='orcid']/text() = $text) or (following::contrib-id[@contrib-id-type='orcid']/text() = $text)" role="error" id="final-orcid-test-3">
-        <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</report>
+	  <assert test="count(ancestor::contrib-group//contrib-id[@contrib-id-type='orcid' and .=$text]) = 1" role="error" id="final-orcid-test-3">
+        <value-of select="e:get-name(parent::*/name[1])"/>'s ORCiD is the same as another author's - <value-of select="."/>. Duplicated ORCiDs are not allowed. If it is clear who the ORCiD belongs to, remove the duplicate. If it is not clear please raise a query with production so that they can raise it with the authors.</assert>
 		
 		</rule>
   </pattern>
diff --git a/test/xspec/schematron.xspec b/test/xspec/schematron.xspec
index 679b3152c3..db6ccc007b 100644
--- a/test/xspec/schematron.xspec
+++ b/test/xspec/schematron.xspec
@@ -1811,22 +1811,22 @@
       </x:scenario>
       <x:scenario label="pre-orcid-test-3-pass">
         <x:context href="../tests/gen/orcid-tests/pre-orcid-test-3/pass.xml"/>
-        <x:expect-not-report id="pre-orcid-test-3" role="warning"/>
+        <x:expect-not-assert id="pre-orcid-test-3" role="warning"/>
         <x:expect-not-assert id="orcid-tests-xspec-assert" role="error"/>
       </x:scenario>
       <x:scenario label="pre-orcid-test-3-fail">
         <x:context href="../tests/gen/orcid-tests/pre-orcid-test-3/fail.xml"/>
-        <x:expect-report id="pre-orcid-test-3" role="warning"/>
+        <x:expect-assert id="pre-orcid-test-3" role="warning"/>
         <x:expect-not-assert id="orcid-tests-xspec-assert" role="error"/>
       </x:scenario>
       <x:scenario label="final-orcid-test-3-pass">
         <x:context href="../tests/gen/orcid-tests/final-orcid-test-3/pass.xml"/>
-        <x:expect-not-report id="final-orcid-test-3" role="error"/>
+        <x:expect-not-assert id="final-orcid-test-3" role="error"/>
         <x:expect-not-assert id="orcid-tests-xspec-assert" role="error"/>
       </x:scenario>
       <x:scenario label="final-orcid-test-3-fail">
         <x:context href="../tests/gen/orcid-tests/final-orcid-test-3/fail.xml"/>
-        <x:expect-report id="final-orcid-test-3" role="error"/>
+        <x:expect-assert id="final-orcid-test-3" role="error"/>
         <x:expect-not-assert id="orcid-tests-xspec-assert" role="error"/>
       </x:scenario>
     </x:scenario>