Adapt phet schema following 3.0.0 release

Author: benoit74

dispatcher/backend/src/common/schemas/offliners/__init__.py

@@ -1,4 +1,3 @@
-from common.schemas import SerializableSchema
 from common.schemas.offliners.devdocs import DevDocsFlagsSchema
 from common.schemas.offliners.freecodecamp import FreeCodeCampFlagsSchema
 from common.schemas.offliners.gutenberg import GutenbergFlagsSchema
@@ -11,6 +10,7 @@
     NautilusFlagsSchemaRelaxed,
 )
 from common.schemas.offliners.openedx import OpenedxFlagsSchema
+from common.schemas.offliners.phet import PhetFlagsSchema
 from common.schemas.offliners.sotoki import SotokiFlagsSchema
 from common.schemas.offliners.ted import TedFlagsSchema
 from common.schemas.offliners.wikihow import WikihowFlagsSchema
@@ -28,15 +28,11 @@
     "NautilusFlagsSchema",
     "NautilusFlagsSchemaRelaxed",
     "OpenedxFlagsSchema",
+    "PhetFlagsSchema",
     "SotokiFlagsSchema",
     "TedFlagsSchema",
     "WikihowFlagsSchema",
     "YoutubeFlagsSchema",
     "ZimitFlagsSchema",
     "ZimitFlagsSchemaRelaxed",
 )
-
-
-class PhetFlagsSchema(SerializableSchema):
-    class Meta:
-        ordered = True

dispatcher/backend/src/common/schemas/offliners/phet.py

@@ -0,0 +1,51 @@
+from marshmallow import fields
+
+from common.schemas import SerializableSchema, String
+
+
+class PhetFlagsSchema(SerializableSchema):
+    class Meta:
+        ordered = True
+
+    includeLanguages = String(
+        required=False,
+        metadata={
+            "label": "Include Languages",
+            "description": "Spoken languages to ZIM, one ZIM per language and/or all in"
+            " a `mul` ZIM.",
+        },
+    )
+    excludeLanguages = String(
+        required=False,
+        metadata={
+            "label": "Exclude Languages",
+            "description": "Spoken languages to not ZIM.",
+        },
+    )
+    withoutLanguageVariants = fields.Boolean(
+        truthy=[True],
+        falsy=[False],
+        metadata={
+            "label": "Without Language Variants",
+            "description": "Exclude languages with Country variant. For example `en_CA`"
+            " will not be ZIMed with this argument.",
+        },
+    )
+    createMul = fields.Boolean(
+        truthy=[True],
+        falsy=[False],
+        metadata={
+            "label": "Create `mul` ZIM",
+            "description": "Create a ZIM file with all languages (by default, scraper "
+            "creates only one ZIM per language)",
+        },
+    )
+    mulOnly = fields.Boolean(
+        truthy=[True],
+        falsy=[False],
+        metadata={
+            "label": "Only `mul` ZIM",
+            "description": "Create only `mul` ZIM, skip ZIM files for individual "
+            "languages",
+        },
+    )

dispatcher/backend/src/utils/offliners.py

@@ -27,6 +27,7 @@
     Offliner.kolibri: od("kolibri2zim", True, False),
     Offliner.devdocs: od("devdocs2zim", True, False),
     Offliner.mindtouch: od("mindtouch2zim", True, True),
+    Offliner.phet: od("phet2zim", False, False),
 }
 
 
@@ -42,9 +43,6 @@ def command_for(offliner, flags, mount_point):
 
     for an offliner,  flags:dict and a mount_point:Path (task volume)"""
 
-    if offliner == Offliner.phet:
-        return ["/bin/bash", "-c", "'cd /phet && npm i && npm start'"]
-
     offliner_def = OFFLINER_DEFS[offliner]
     cmd = offliner_def.cmd
     if offliner_def.std_output: