Allow Language metadata to have multiple values

Author: rgaudin

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 - Using pylibzim `3.1.0`
+- ZIM metadata check now allows multiple values (comma-separated) for `Language`
 
 ### Removed
 

src/zimscraperlib/zim/metadata.py

@@ -59,8 +59,12 @@ def validate_date(name: str, value: Union[datetime.datetime, datetime.date, str]
 
 def validate_language(name: str, value: Union[Iterable[str], str]):
     """ensures Language metadata is a single or list of ISO-639-3 codes"""
-    if name == "Language" and not is_valid_iso_639_3(value):
-        raise ValueError(f"{value} is not ISO-639-3.")
+    if name == "Language":
+        if isinstance(value, str):
+            value = value.split(",")
+        for code in value:
+            if not is_valid_iso_639_3(code):
+                raise ValueError(f"{code} is not ISO-639-3.")
 
 
 def validate_counter(name: str, value: str):

tests/zim/test_zim_creator.py

@@ -594,6 +594,11 @@ def test_config_metadata(tmp_path, png_image):
         ("Language", "bam", True),
         ("Language", "fr", False),
         ("Language", "en", False),
+        ("Language", "fra,eng", True),
+        ("Language", "fra,eng,bam", True),
+        ("Language", "fra,en,bam", False),
+        ("Language", "eng,", False),
+        ("Language", "eng, fra", False),
         ("Counter", "1", False),
         ("Description", "X" * 80, True),
         ("Description", "X" * 81, False),