From 2f4b4480565dbd9374f2d24bed8885764394a6cf Mon Sep 17 00:00:00 2001
From: Brian Pepple <bdpepple@gmail.com>
Date: Tue, 14 Jul 2026 10:12:46 -0400
Subject: [PATCH] Add optional CommunityRating element to v1.1 schema

Adds AverageRating (decimal, 0-5.0) and RatingCount (optional positiveInteger) sub-elements for tracking community rating data, along with sample XML and schema validation tests.
---
 drafts/v1.1/MetronInfo.xsd | 15 ++++++++++
 drafts/v1.1/Sample.xml     |  4 +++
 tests/test.py              | 56 ++++++++++++++++++++++++++++++++++++++
 3 files changed, 75 insertions(+)

diff --git a/drafts/v1.1/MetronInfo.xsd b/drafts/v1.1/MetronInfo.xsd
index 1c7c9e5..220312b 100644
--- a/drafts/v1.1/MetronInfo.xsd
+++ b/drafts/v1.1/MetronInfo.xsd
@@ -29,6 +29,7 @@
             <xs:element name="Reprints" type="reprintsType" minOccurs="0" />
             <xs:element name="GTIN" type="gtinType" minOccurs="0" />
             <xs:element name="AgeRating" type="ageRatingType" minOccurs="0" default="Unknown" />
+            <xs:element name="CommunityRating" type="communityRatingType" minOccurs="0" />
             <xs:element name="URLs" type="urlsType" minOccurs="0" />
             <xs:element name="Credits" type="creditsType" minOccurs="0" />
             <xs:element name="LastModified" type="xs:dateTime" minOccurs="0" />
@@ -232,6 +233,13 @@
         </xs:all>
     </xs:complexType>
 
+    <xs:complexType name="communityRatingType">
+        <xs:all>
+            <xs:element name="AverageRating" type="averageRatingType" />
+            <xs:element name="RatingCount" type="xs:positiveInteger" minOccurs="0" />
+        </xs:all>
+    </xs:complexType>
+
     <!-- Simple Types -->
     <xs:simpleType name="formatType">
         <xs:restriction base="xs:string">
@@ -326,6 +334,13 @@
         </xs:restriction>
     </xs:simpleType>
 
+    <xs:simpleType name="averageRatingType">
+        <xs:restriction base="xs:decimal">
+            <xs:minInclusive value="0" />
+            <xs:maxInclusive value="5.0" />
+        </xs:restriction>
+    </xs:simpleType>
+
     <xs:simpleType name="countryCode">
         <xs:restriction base="xs:string">
             <xs:pattern value="[A-Z][A-Z]" />
diff --git a/drafts/v1.1/Sample.xml b/drafts/v1.1/Sample.xml
index 8a1ceb1..2bfda50 100644
--- a/drafts/v1.1/Sample.xml
+++ b/drafts/v1.1/Sample.xml
@@ -91,6 +91,10 @@
         <UPC>76194130593600111</UPC>
     </GTIN>
     <AgeRating>Everyone</AgeRating>
+    <CommunityRating>
+        <AverageRating>4.5</AverageRating>
+        <RatingCount>1250</RatingCount>
+    </CommunityRating>
     <Reprints>
         <Reprint id="65498">Foo Bar #001 (2002)</Reprint>
         <Reprint>Foo Bar #002 (2022)</Reprint>
diff --git a/tests/test.py b/tests/test.py
index e77b3b8..22ca775 100644
--- a/tests/test.py
+++ b/tests/test.py
@@ -44,6 +44,27 @@
                 '<?xml version="1.0" encoding="UTF-8"?><MetronInfo>'
                 "<Series><Name>Foo</Name></Series><Number>1</Number><PageCount>0</PageCount></MetronInfo>",
         ),
+        (
+                TEST_V11_XSD,
+                '<?xml version="1.0" encoding="UTF-8"?><MetronInfo>'
+                "<Series><Name>Foo</Name></Series><Number />"
+                "<CommunityRating><AverageRating>4.5</AverageRating><RatingCount>1250</RatingCount></CommunityRating>"
+                "</MetronInfo>",
+        ),
+        (
+                TEST_V11_XSD,
+                '<?xml version="1.0" encoding="UTF-8"?><MetronInfo>'
+                "<Series><Name>Foo</Name></Series><Number />"
+                "<CommunityRating><AverageRating>0</AverageRating></CommunityRating>"
+                "</MetronInfo>",
+        ),
+        (
+                TEST_V11_XSD,
+                '<?xml version="1.0" encoding="UTF-8"?><MetronInfo>'
+                "<Series><Name>Foo</Name></Series><Number />"
+                "<CommunityRating><AverageRating>5.0</AverageRating></CommunityRating>"
+                "</MetronInfo>",
+        ),
     ],
     ids=[
         "valid_xml",
@@ -54,6 +75,9 @@
         "v11_volume_zero",
         "v11_alternative_number",
         "v11_alternative_number_absent",
+        "v11_community_rating",
+        "v11_community_rating_min",
+        "v11_community_rating_max",
     ],
 )
 def test_valid(xsd: Path, xml: Path | str) -> None:
@@ -92,6 +116,34 @@ def test_valid(xsd: Path, xml: Path | str) -> None:
                 "<Series><Name>Foo</Name></Series><Number>1</Number>"
                 "<AlternativeNumber>1A</AlternativeNumber><AlternativeNumber>1B</AlternativeNumber></MetronInfo>",
         ),
+        (
+                TEST_V11_XSD,
+                '<?xml version="1.0" encoding="UTF-8"?><MetronInfo>'
+                "<Series><Name>Foo</Name></Series><Number />"
+                "<CommunityRating><AverageRating>5.1</AverageRating></CommunityRating>"
+                "</MetronInfo>",
+        ),
+        (
+                TEST_V11_XSD,
+                '<?xml version="1.0" encoding="UTF-8"?><MetronInfo>'
+                "<Series><Name>Foo</Name></Series><Number />"
+                "<CommunityRating><AverageRating>-0.1</AverageRating></CommunityRating>"
+                "</MetronInfo>",
+        ),
+        (
+                TEST_V11_XSD,
+                '<?xml version="1.0" encoding="UTF-8"?><MetronInfo>'
+                "<Series><Name>Foo</Name></Series><Number />"
+                "<CommunityRating><RatingCount>100</RatingCount></CommunityRating>"
+                "</MetronInfo>",
+        ),
+        (
+                TEST_V11_XSD,
+                '<?xml version="1.0" encoding="UTF-8"?><MetronInfo>'
+                "<Series><Name>Foo</Name></Series><Number />"
+                "<CommunityRating><AverageRating>4.5</AverageRating><RatingCount>0</RatingCount></CommunityRating>"
+                "</MetronInfo>",
+        ),
     ],
     ids=[
         "dup_primary_attr_xml",
@@ -101,6 +153,10 @@ def test_valid(xsd: Path, xml: Path | str) -> None:
         "v11_negative_page_count",
         "v11_negative_volume",
         "v11_duplicate_alternative_number",
+        "v11_community_rating_too_high",
+        "v11_community_rating_negative",
+        "v11_community_rating_missing_average",
+        "v11_community_rating_zero_count",
     ],
 )
 def test_invalid(xsd: Path, xml: Path | str) -> None: