From b6f4c5f3153523b2096947088757f621d3126d03 Mon Sep 17 00:00:00 2001
From: Fred Atherden <43879983+fred-atherden@users.noreply.github.com>
Date: Mon, 27 Nov 2023 13:17:16 +0000
Subject: [PATCH] Downgrade prc-pub-review-test-1 to warning

---
 src/final-JATS-schematron.sch                               | 2 +-
 src/final-JATS-schematron.xsl                               | 6 +++---
 src/final-package-JATS-schematron.sch                       | 2 +-
 src/pre-JATS-schematron.sch                                 | 2 +-
 src/pre-JATS-schematron.xsl                                 | 6 +++---
 src/schematron.sch                                          | 4 ++--
 .../gen/prc-pub-review-tests/prc-pub-review-test-1/fail.xml | 2 +-
 .../gen/prc-pub-review-tests/prc-pub-review-test-1/pass.xml | 2 +-
 .../prc-pub-review-test-1/prc-pub-review-test-1.sch         | 2 +-
 test/xspec/schematron.sch                                   | 2 +-
 test/xspec/schematron.xspec                                 | 4 ++--
 11 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/src/final-JATS-schematron.sch b/src/final-JATS-schematron.sch
index 12cd9b1e63..7eec150f56 100644
--- a/src/final-JATS-schematron.sch
+++ b/src/final-JATS-schematron.sch
@@ -4184,7 +4184,7 @@
       <assert test="anonymous" role="error" id="prc-reviewer-test-2">[prc-reviewer-test-2] A reviewer contrib in a PRC article must have a child anonymous element. This one does not - <value-of select="."/>.</assert>
     </rule></pattern><pattern id="prc-pub-review-tests-pattern"><rule context="article[e:is-prc(.)]" id="prc-pub-review-tests">
       
-      <report test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )" role="error" id="prc-pub-review-test-1">[prc-pub-review-test-1] This article has individual public reviews, and also either a consensus or a joint public review. This must be incorrect.</report>
+      <report test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )" role="warning" id="prc-pub-review-test-1">[prc-pub-review-test-1] This article has individual public reviews, and also either a consensus or a joint public review, which is highgly unusual. Is this correct?</report>
     </rule></pattern>
   
   <pattern id="sub-article-doi-checks-pattern"><rule context="sub-article/front-stub/article-id[@pub-id-type='doi']" id="sub-article-doi-checks">
diff --git a/src/final-JATS-schematron.xsl b/src/final-JATS-schematron.xsl
index a9c8e30524..624576fc50 100644
--- a/src/final-JATS-schematron.xsl
+++ b/src/final-JATS-schematron.xsl
@@ -24565,15 +24565,15 @@
    <xsl:template match="article[e:is-prc(.)]" priority="1000" mode="M379">
       <svrl:fired-rule xmlns:svrl="http://purl.oclc.org/dsdl/svrl" context="article[e:is-prc(.)]" id="prc-pub-review-tests"/>
 
-		    <!--REPORT error-->
+		    <!--REPORT warning-->
       <xsl:if test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )">
          <svrl:successful-report xmlns:svrl="http://purl.oclc.org/dsdl/svrl" test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and ( sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')] or sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')] )">
             <xsl:attribute name="id">prc-pub-review-test-1</xsl:attribute>
-            <xsl:attribute name="role">error</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>[prc-pub-review-test-1] This article has individual public reviews, and also either a consensus or a joint public review. This must be incorrect.</svrl:text>
+            <svrl:text>[prc-pub-review-test-1] This article has individual public reviews, and also either a consensus or a joint public review, which is highgly unusual. Is this correct?</svrl:text>
          </svrl:successful-report>
       </xsl:if>
       <xsl:apply-templates select="*" mode="M379"/>
diff --git a/src/final-package-JATS-schematron.sch b/src/final-package-JATS-schematron.sch
index 94844acbfa..c491fdd177 100644
--- a/src/final-package-JATS-schematron.sch
+++ b/src/final-package-JATS-schematron.sch
@@ -4190,7 +4190,7 @@
       <assert test="anonymous" role="error" id="prc-reviewer-test-2">A reviewer contrib in a PRC article must have a child anonymous element. This one does not - <value-of select="."/>.</assert>
     </rule></pattern><pattern id="prc-pub-review-tests-pattern"><rule context="article[e:is-prc(.)]" id="prc-pub-review-tests">
       
-      <report test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )" role="error" id="prc-pub-review-test-1">This article has individual public reviews, and also either a consensus or a joint public review. This must be incorrect.</report>
+      <report test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )" role="warning" id="prc-pub-review-test-1">This article has individual public reviews, and also either a consensus or a joint public review, which is highgly unusual. Is this correct?</report>
     </rule></pattern>
   
   <pattern id="sub-article-doi-checks-pattern"><rule context="sub-article/front-stub/article-id[@pub-id-type='doi']" id="sub-article-doi-checks">
diff --git a/src/pre-JATS-schematron.sch b/src/pre-JATS-schematron.sch
index b20003f40b..a4bfcbf29f 100644
--- a/src/pre-JATS-schematron.sch
+++ b/src/pre-JATS-schematron.sch
@@ -4129,7 +4129,7 @@
       <assert test="anonymous" role="error" id="prc-reviewer-test-2">[prc-reviewer-test-2] A reviewer contrib in a PRC article must have a child anonymous element. This one does not - <value-of select="."/>.</assert>
     </rule></pattern><pattern id="prc-pub-review-tests-pattern"><rule context="article[e:is-prc(.)]" id="prc-pub-review-tests">
       
-      <report test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )" role="error" id="prc-pub-review-test-1">[prc-pub-review-test-1] This article has individual public reviews, and also either a consensus or a joint public review. This must be incorrect.</report>
+      <report test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )" role="warning" id="prc-pub-review-test-1">[prc-pub-review-test-1] This article has individual public reviews, and also either a consensus or a joint public review, which is highgly unusual. Is this correct?</report>
     </rule></pattern>
   
   <pattern id="sub-article-doi-checks-pattern"><rule context="sub-article/front-stub/article-id[@pub-id-type='doi']" id="sub-article-doi-checks">
diff --git a/src/pre-JATS-schematron.xsl b/src/pre-JATS-schematron.xsl
index 55e794e06a..4d6e27a499 100644
--- a/src/pre-JATS-schematron.xsl
+++ b/src/pre-JATS-schematron.xsl
@@ -24446,15 +24446,15 @@
    <xsl:template match="article[e:is-prc(.)]" priority="1000" mode="M377">
       <svrl:fired-rule xmlns:svrl="http://purl.oclc.org/dsdl/svrl" context="article[e:is-prc(.)]" id="prc-pub-review-tests"/>
 
-		    <!--REPORT error-->
+		    <!--REPORT warning-->
       <xsl:if test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )">
          <svrl:successful-report xmlns:svrl="http://purl.oclc.org/dsdl/svrl" test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and ( sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')] or sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')] )">
             <xsl:attribute name="id">prc-pub-review-test-1</xsl:attribute>
-            <xsl:attribute name="role">error</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>[prc-pub-review-test-1] This article has individual public reviews, and also either a consensus or a joint public review. This must be incorrect.</svrl:text>
+            <svrl:text>[prc-pub-review-test-1] This article has individual public reviews, and also either a consensus or a joint public review, which is highgly unusual. Is this correct?</svrl:text>
          </svrl:successful-report>
       </xsl:if>
       <xsl:apply-templates select="*" mode="M377"/>
diff --git a/src/schematron.sch b/src/schematron.sch
index 5ca8ed28c5..3b93cc855f 100644
--- a/src/schematron.sch
+++ b/src/schematron.sch
@@ -7549,8 +7549,8 @@ else self::*/local-name() = $allowed-p-blocks"
         or
         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]
         )"
-        role="error" 
-        id="prc-pub-review-test-1">This article has individual public reviews, and also either a consensus or a joint public review. This must be incorrect.</report>
+        role="warning" 
+        id="prc-pub-review-test-1">This article has individual public reviews, and also either a consensus or a joint public review, which is highgly unusual. Is this correct?</report>
     </rule>
     
   </pattern>
diff --git a/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/fail.xml b/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/fail.xml
index cba3d9474a..7b2acf224e 100644
--- a/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/fail.xml
+++ b/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/fail.xml
@@ -1,7 +1,7 @@
 <?oxygen SCHSchema="prc-pub-review-test-1.sch"?>
 <!--Context: article[e:is-prc(.)]
 Test: report    sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and ( sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')] or sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')] )
-Message: This article has individual public reviews, and also either a consensus or a joint public review. This must be incorrect. -->
+Message: This article has individual public reviews, and also either a consensus or a joint public review, which is highgly unusual. Is this correct? -->
 <root xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:mml="http://www.w3.org/1998/Math/MathML" xmlns:ali="http://www.niso.org/schemas/ali/1.0/">
   <article>
     <front>
diff --git a/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/pass.xml b/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/pass.xml
index cb442fc552..238b05fc03 100644
--- a/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/pass.xml
+++ b/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/pass.xml
@@ -1,7 +1,7 @@
 <?oxygen SCHSchema="prc-pub-review-test-1.sch"?>
 <!--Context: article[e:is-prc(.)]
 Test: report    sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and ( sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')] or sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')] )
-Message: This article has individual public reviews, and also either a consensus or a joint public review. This must be incorrect. -->
+Message: This article has individual public reviews, and also either a consensus or a joint public review, which is highgly unusual. Is this correct? -->
 <root xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:mml="http://www.w3.org/1998/Math/MathML" xmlns:ali="http://www.niso.org/schemas/ali/1.0/">
   <article>
     <front>
diff --git a/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/prc-pub-review-test-1.sch b/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/prc-pub-review-test-1.sch
index a967dc0457..648cdafb60 100644
--- a/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/prc-pub-review-test-1.sch
+++ b/test/tests/gen/prc-pub-review-tests/prc-pub-review-test-1/prc-pub-review-test-1.sch
@@ -1287,7 +1287,7 @@
   </xsl:function>
   <pattern id="sub-article-version-2">
     <rule context="article[e:is-prc(.)]" id="prc-pub-review-tests">
-      <report test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )" role="error" id="prc-pub-review-test-1">This article has individual public reviews, and also either a consensus or a joint public review. This must be incorrect.</report>
+      <report test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )" role="warning" id="prc-pub-review-test-1">This article has individual public reviews, and also either a consensus or a joint public review, which is highgly unusual. Is this correct?</report>
     </rule>
   </pattern>
   <pattern id="root-pattern">
diff --git a/test/xspec/schematron.sch b/test/xspec/schematron.sch
index 3581d1fc14..070a639b27 100644
--- a/test/xspec/schematron.sch
+++ b/test/xspec/schematron.sch
@@ -5376,7 +5376,7 @@
   <pattern id="prc-pub-review-tests-pattern">
     <rule context="article[e:is-prc(.)]" id="prc-pub-review-tests">
       
-      <report test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )" role="error" id="prc-pub-review-test-1">This article has individual public reviews, and also either a consensus or a joint public review. This must be incorrect.</report>
+      <report test="sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'reviewer #')] and (         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'consensus')]          or         sub-article[@article-type='referee-report']/front-stub//article-title[starts-with(lower-case(.),'joint')]         )" role="warning" id="prc-pub-review-test-1">This article has individual public reviews, and also either a consensus or a joint public review, which is highgly unusual. Is this correct?</report>
     </rule>
   </pattern>
   
diff --git a/test/xspec/schematron.xspec b/test/xspec/schematron.xspec
index 0102b86182..3789e99ae7 100644
--- a/test/xspec/schematron.xspec
+++ b/test/xspec/schematron.xspec
@@ -9979,12 +9979,12 @@
     <x:scenario label="prc-pub-review-tests">
       <x:scenario label="prc-pub-review-test-1-pass">
         <x:context href="../tests/gen/prc-pub-review-tests/prc-pub-review-test-1/pass.xml"/>
-        <x:expect-not-report id="prc-pub-review-test-1" role="error"/>
+        <x:expect-not-report id="prc-pub-review-test-1" role="warning"/>
         <x:expect-not-assert id="prc-pub-review-tests-xspec-assert" role="error"/>
       </x:scenario>
       <x:scenario label="prc-pub-review-test-1-fail">
         <x:context href="../tests/gen/prc-pub-review-tests/prc-pub-review-test-1/fail.xml"/>
-        <x:expect-report id="prc-pub-review-test-1" role="error"/>
+        <x:expect-report id="prc-pub-review-test-1" role="warning"/>
         <x:expect-not-assert id="prc-pub-review-tests-xspec-assert" role="error"/>
       </x:scenario>
     </x:scenario>