jsonschema: fix partOf subfields display

* Closes #626. * Makes the `status` deposit field required to prevent submitting a deposit with no status from the API. Co-Authored-by: Pascal Repond <pascal.repond@rero.ch>
rero · Aug 25, 2022 · 09050b7 · 09050b7
1 parent 190c4f6
commit 09050b7
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/sonar/modules/deposits/jsonschemas/deposits/deposit-v1.0.0_src.json b/sonar/modules/deposits/jsonschemas/deposits/deposit-v1.0.0_src.json
@@ -4,7 +4,8 @@
   "title": "SONAR deposit v1.0.0",
   "type": "object",
   "required": [
-    "pid"
+    "pid",
+    "status"
   ],
   "additionalProperties": false,
   "properties": {
@@ -573,7 +574,7 @@
               "type": "string",
               "minLength": 1,
               "form": {
-                "hideExpression": "field.parent.parent.model && !['coar:c_beb9', 'coar:c_6501', 'coar:c_998f', 'coar:c_dcae04bc'].includes(field.parent.parent.model.documentType)",
+                "hideExpression": "field.parent.parent.model && !['coar:c_3248', 'coar:c_3e5a', 'coar:c_ba08', 'coar:c_beb9', 'coar:c_6501', 'coar:c_998f', 'coar:c_dcae04bc', 'coar:c_816b', 'coar:c_5794', 'coar:c_6670', 'coar:c_c94f'].includes(field.parent.parent.model.documentType)",
                 "templateOptions": {
                   "placeholder": "Example: 1, 4-5, …"
                 }
@@ -584,7 +585,7 @@
               "type": "string",
               "minLength": 1,
               "form": {
-                "hideExpression": "field.parent.parent.model && !['coar:c_beb9', 'coar:c_6501', 'coar:c_998f', 'coar:c_dcae04bc'].includes(field.parent.parent.model.documentType)",
+                "hideExpression": "field.parent.parent.model && !['coar:c_3e5a', 'coar:c_ba08', 'coar:c_beb9', 'coar:c_6501', 'coar:c_998f', 'coar:c_dcae04bc', 'coar:c_816b', 'coar:c_5794', 'coar:c_6670', 'coar:c_c94f'].includes(field.parent.parent.model.documentType)",
                 "templateOptions": {
                   "placeholder": "Example: 10, 22-24, …"
                 }

diff --git a/sonar/modules/documents/jsonschemas/documents/document-v1.0.0_src.json b/sonar/modules/documents/jsonschemas/documents/document-v1.0.0_src.json
@@ -1581,15 +1581,15 @@
             "type": "string",
             "minLength": 1,
             "form": {
-              "hideExpression": "!['coar:c_3e5a', 'coar:c_ba08', coar:c_beb9', 'coar:c_6501', 'coar:c_998f', 'coar:c_dcae04bc'].includes(field.parent.parent.parent.model.documentType)"
+              "hideExpression": "!['coar:c_3248', 'coar:c_3e5a', 'coar:c_ba08', 'coar:c_beb9', 'coar:c_6501', 'coar:c_998f', 'coar:c_dcae04bc', 'coar:c_816b', 'coar:c_5794', 'coar:c_6670', 'coar:c_c94f'].includes(field.parent.parent.parent.model.documentType)"
             }
           },
           "numberingIssue": {
             "title": "Issue",
             "type": "string",
             "minLength": 1,
             "form": {
-              "hideExpression": "!['coar:c_3e5a', 'coar:c_ba08', 'coar:c_beb9', 'coar:c_6501', 'coar:c_998f', 'coar:c_dcae04bc'].includes(field.parent.parent.parent.model.documentType)"
+              "hideExpression": "!['coar:c_3e5a', 'coar:c_ba08', 'coar:c_beb9', 'coar:c_6501', 'coar:c_998f', 'coar:c_dcae04bc', 'coar:c_816b', 'coar:c_5794', 'coar:c_6670', 'coar:c_c94f'].includes(field.parent.parent.parent.model.documentType)"
             }
           },
           "numberingPages": {

diff --git a/tests/ui/users/test_user_jsonresolvers.py b/tests/ui/users/test_user_jsonresolvers.py
@@ -37,6 +37,7 @@ def test_user_resolver(app, organisation, roles):
     record = DepositRecord.create(
         {'user': {
             '$ref': 'https://sonar.ch/api/users/1'
-        }}, with_bucket=False)
+        },
+        'status': 'in_progress'}, with_bucket=False)
 
     assert record.replace_refs().get('user')['email'] == 'admin@test.com'