add: odk:client-editable submission config

- similar to auto_send / auto_delete, but requested to allow more
  true/false-y values than just 'true' accepted for those.

Author: lindsay-stevens

pyxform/constants.py

@@ -34,6 +34,7 @@
 SUBMISSION_URL = "submission_url"
 AUTO_SEND = "auto_send"
 AUTO_DELETE = "auto_delete"
+CLIENT_EDITABLE = "client_editable"
 DEFAULT_FORM_NAME = "data"
 DEFAULT_LANGUAGE_KEY = "default_language"
 DEFAULT_LANGUAGE_VALUE = "default"

pyxform/survey.py

@@ -215,6 +215,7 @@ def recursive_dict():
     "title",
     "version",
     constants.ALLOW_CHOICE_DUPLICATES,
+    constants.CLIENT_EDITABLE,
     constants.COMPACT_DELIMITER,
     constants.COMPACT_PREFIX,
     constants.ENTITY_FEATURES,
@@ -262,6 +263,7 @@ def __init__(self, **kwargs):
         self.auto_delete: str | None = None
         self.auto_send: str | None = None
         self.clean_text_values: bool = False
+        self.client_editable: bool = False
         self.instance_xmlns: str | None = None
         self.namespaces: str | None = None
         self.omit_instanceID: bool = False
@@ -727,7 +729,15 @@ def xml_model(self):
             node("instance", self.xml_instance()),
         )
 
-        if self.submission_url or self.public_key or self.auto_send or self.auto_delete:
+        if any(
+            (
+                self.submission_url,
+                self.public_key,
+                self.auto_send,
+                self.auto_delete,
+                self.client_editable,
+            )
+        ):
             submission_attrs = {}
             if self.submission_url:
                 submission_attrs["action"] = self.submission_url
@@ -738,6 +748,8 @@ def xml_model(self):
                 submission_attrs["orx:auto-send"] = self.auto_send
             if self.auto_delete:
                 submission_attrs["orx:auto-delete"] = self.auto_delete
+            if self.client_editable:
+                submission_attrs["odk:client-editable"] = "true"
             submission_node = node("submission", **submission_attrs)
             model_children.insert(0, submission_node)
 

pyxform/xls2json.py

@@ -481,6 +481,10 @@ def workbook_to_json(
         settings["add_none_option"] = aliases.yes_no.get(
             settings["add_none_option"], False
         )
+    if constants.CLIENT_EDITABLE in settings:
+        settings[constants.CLIENT_EDITABLE] = aliases.yes_no.get(
+            settings.get(constants.CLIENT_EDITABLE, "no"), False
+        )
 
     # Here we create our json dict root with default settings:
     id_string = settings.get(

tests/test_settings.py

@@ -292,3 +292,71 @@ def test_primary_instance_attribute__multiple(self):
                 """,
             ],
         )
+
+    def test_client_editable_setting__active(self):
+        """Should find the odk:client-editable attribute in the submission config."""
+        # Set to an alias of True.
+        md1 = """
+        | settings |
+        |          | client_editable |
+        |          | yes             |
+        | survey |       |      |       |
+        |        | type  | name | label |
+        |        | text  | q1   | hello |
+        """
+        # Set the same way as another submission setting (e.g. auto_send).
+        md2 = """
+        | settings |
+        |          | client_editable |
+        |          | true            |
+        | survey |       |      |       |
+        |        | type  | name | label |
+        |        | text  | q1   | hello |
+        """
+        for md in (md1, md2):
+            self.assertPyxformXform(
+                md=md,
+                xml__xpath_match=[
+                    """/h:html/h:head/x:model/x:submission[@odk:client-editable = 'true']""",
+                ],
+            )
+
+    def test_client_editable_setting__inactive(self):
+        """Should not find the odk:client-editable attribute in the submission config."""
+        # Set to an alias of False.
+        md1 = """
+        | settings |
+        |          | client_editable |
+        |          | no              |
+        | survey |       |      |       |
+        |        | type  | name | label |
+        |        | text  | q1   | hello |
+        """
+        # Not set.
+        md2 = """
+        | survey |       |      |       |
+        |        | type  | name | label |
+        |        | text  | q1   | hello |
+        """
+        for md in (md1, md2):
+            self.assertPyxformXform(
+                md=md,
+                xml__xpath_match=[
+                    """/h:html/h:head/x:model[not(./x:submission/@odk:client-editable)]""",
+                ],
+            )
+        # Set to false, with other setting that triggers `submission` element.
+        md3 = """
+        | settings |
+        |          | client_editable | auto_send |
+        |          | false           | true      |
+        | survey |       |      |       |
+        |        | type  | name | label |
+        |        | text  | q1   | hello |
+        """
+        self.assertPyxformXform(
+            md=md3,
+            xml__xpath_match=[
+                """/h:html/h:head/x:model/x:submission[not(@odk:client-editable)]""",
+            ],
+        )