From 652da4cc803d41d9b1386ccc68d71af1beada76c Mon Sep 17 00:00:00 2001
From: Syd Bauman <sydb@users.noreply.github.com>
Date: Fri, 1 Nov 2024 13:02:20 -0400
Subject: [PATCH] Pay better attention to the (#2611)

role= attribute of Schematron assertions & reports.
---
 P5/Utilities/validator.xsl | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/P5/Utilities/validator.xsl b/P5/Utilities/validator.xsl
index ada5e88de3..7141662eac 100644
--- a/P5/Utilities/validator.xsl
+++ b/P5/Utilities/validator.xsl
@@ -46,7 +46,13 @@
   <xsl:template match="svrl:failed-assert|svrl:successful-report">
     <xsl:variable name="outGI">
       <xsl:choose>
-        <xsl:when test="@role='nonfatal'">WARNING</xsl:when>
+	<!-- We explicitly test for the 6 values oXygen provides in its
+	     “sample values include” list, plus the 1 other value that
+	     actually occurs in P5. -->
+	<xsl:when test="@role = ('info','information')">INFORMATION</xsl:when>
+	<xsl:when test="@role = ('warn','warning')"    >WARNING</xsl:when>
+	<xsl:when test="@role eq 'nonfatal'"           >WARNING</xsl:when>
+	<xsl:when test="@role = ('error','fatal')"     >ERROR</xsl:when>
         <xsl:otherwise>ERROR</xsl:otherwise>
       </xsl:choose>
     </xsl:variable>