project-schema: Add Organization.beneficialOwners

open-contracting · Nov 12, 2023 · 889eb3a · 889eb3a
1 parent 97ec8a4
commit 889eb3a
Show file tree

Hide file tree

Showing 8 changed files with 701 additions and 21 deletions.
diff --git a/docs/_static/i18n.csv b/docs/_static/i18n.csv
@@ -140,10 +140,49 @@ parties/contactPoint/telephone,Telephone,True,
 parties/contactPoint/faxNumber,Fax number,True,
 parties/contactPoint/url,URL,False,
 parties/roles,Party roles,False,
+parties/beneficialOwners,Beneficial owners,False,
+parties/beneficialOwners,Person,False,
+parties/beneficialOwners/id,ID,True,
+parties/beneficialOwners/name,Name,True,
+parties/beneficialOwners/identifier,Identifier,False,
+parties/beneficialOwners/identifier,Identifier,False,
+parties/beneficialOwners/identifier/scheme,Scheme,True,
+parties/beneficialOwners/identifier/id,ID,True,
+parties/beneficialOwners/identifier/legalName,Legal Name,True,
+parties/beneficialOwners/identifier/uri,URI,False,
+parties/beneficialOwners/nationalities,Nationalities,False,
+parties/beneficialOwners/address,Address,False,
+parties/beneficialOwners/address,Address,False,
+parties/beneficialOwners/address/streetAddress,Street address,True,
+parties/beneficialOwners/address/locality,Locality,True,
+parties/beneficialOwners/address/region,Region,True,
+parties/beneficialOwners/address/postalCode,Postal code,True,
+parties/beneficialOwners/address/countryName,Country name,True,
+parties/beneficialOwners/email,Email,False,
+parties/beneficialOwners/faxNumber,Fax number,True,
+parties/beneficialOwners/telephone,Telephone,True,
+parties/beneficialOwners/jobTitle,Job title,True,
 parties/people,People,False,
 parties/people,Person,False,
-parties/people/id,Identifier,True,
+parties/people/id,ID,True,
 parties/people/name,Name,True,
+parties/people/identifier,Identifier,False,
+parties/people/identifier,Identifier,False,
+parties/people/identifier/scheme,Scheme,True,
+parties/people/identifier/id,ID,True,
+parties/people/identifier/legalName,Legal Name,True,
+parties/people/identifier/uri,URI,False,
+parties/people/nationalities,Nationalities,False,
+parties/people/address,Address,False,
+parties/people/address,Address,False,
+parties/people/address/streetAddress,Street address,True,
+parties/people/address/locality,Locality,True,
+parties/people/address/region,Region,True,
+parties/people/address/postalCode,Postal code,True,
+parties/people/address/countryName,Country name,True,
+parties/people/email,Email,False,
+parties/people/faxNumber,Fax number,True,
+parties/people/telephone,Telephone,True,
 parties/people/jobTitle,Job title,True,
 publicAuthority,Public authority,False,
 publicAuthority,Organization reference,False,

diff --git a/docs/examples/blank.json b/docs/examples/blank.json
@@ -165,6 +165,29 @@
           "name": "",
           "jobTitle": ""
         }
+      ],
+      "beneficialOwners": [
+        {
+          "id": "string",
+          "name": "string",
+          "identifier": {
+            "scheme": "string",
+            "id": "string"
+          },
+          "nationalities": [
+            "string"
+          ],
+          "address": {
+            "streetAddress": "string",
+            "locality": "string",
+            "region": "string",
+            "postalCode": "string",
+            "countryName": "string"
+          },
+          "email": "string",
+          "faxNumber": "string",
+          "telephone": "string"
+        }
       ]
     }
   ],

diff --git a/docs/examples/example.json b/docs/examples/example.json
@@ -312,6 +312,29 @@
           "roles": [
             "supplier",
             "tenderer"
+          ],
+          "beneficialOwners": [
+            {
+              "id": "1",
+              "name": "Juan Perez",
+              "identifier": {
+                "scheme": "PRY-IDCARD",
+                "id": "12345"
+              },
+              "nationalities": [
+                "PY"
+              ],
+              "address": {
+                "streetAddress": "Avenida Eusebio Ayala 1347",
+                "locality": "Asunción",
+                "region": "Gran Asunción",
+                "postalCode": "1001",
+                "countryName": "Paraguay"
+              },
+              "email": "[email protected]",
+              "faxNumber": "+595210000001",
+              "telephone": "+595210000000"
+            }
           ]
         },
         {

diff --git a/docs/reference/codelists.md b/docs/reference/codelists.md
@@ -67,6 +67,13 @@ Projects with a `status` of 'completed' may be displayed in a list of archived p
 :file: ../../build/current_lang/codelists/milestoneType.csv
 ```
 
+### country
+
+```{csv-table-no-translate}
+:header-rows: 1
+:file: ../../build/current_lang/codelists/country.csv
+```
+
 ## Open codelists
 
 ### DocumentType

diff --git a/docs/reference/schema.md b/docs/reference/schema.md
@@ -456,7 +456,7 @@ Each `Organization` has the following fields:
 
 ```{jsonschema} ../../build/current_lang/project-schema.json
 :pointer: /definitions/Organization
-:collapse: name,id,identifier,additionalIdentifiers,address,contactPoint,roles,people
+:collapse: name,id,identifier,additionalIdentifiers,address,contactPoint,roles,beneficialOwners,people
 :addtargets:
 ```
 
@@ -574,6 +574,7 @@ When working with data, users ought to be aware that addresses might not always
 This sub-schema is referenced by the following properties:
 * [`Location/address`](project-schema.json,/definitions/Location,address)
 * [`Organization/address`](project-schema.json,/definitions/Organization,address)
+* [`Person/address`](project-schema.json,/definitions/Person,address)
 
 Each `Address` has the following fields:
 
@@ -734,6 +735,7 @@ For example, if identifying a company in Colombia, look up its identifier in the
 This sub-schema is referenced by the following properties:
 * [`Organization/identifier`](project-schema.json,/definitions/Organization,identifier)
 * [`Organization/additionalIdentifiers`](project-schema.json,/definitions/Organization,additionalIdentifiers)
+* [`Person/identifier`](project-schema.json,/definitions/Person,identifier)
 
 Each `Identifier` has the following fields:
 
@@ -891,6 +893,7 @@ Use this object when you need to disclose the details of people associated with,
 ```
 
 This sub-schema is referenced by the following properties:
+* [`Organization/beneficialOwners`](project-schema.json,/definitions/Organization,beneficialOwners)
 * [`Organization/people`](project-schema.json,/definitions/Organization,people)
 
 Each `Person` has the following fields:
@@ -901,7 +904,7 @@ Each `Person` has the following fields:
 
 ```{jsonschema} ../../build/current_lang/project-schema.json
 :pointer: /definitions/Person
-:collapse: id,name,jobTitle
+:collapse: id,name,identifier,nationalities,address,email,faxNumber,telephone,jobTitle
 :addtargets:
 ```
 

diff --git a/manage.py b/manage.py
@@ -334,21 +334,32 @@ def update_sub_schema_reference(schema):
         ])
 
         # Paths that don't appear in the example data at all
-        paths_to_skip = ['forecasts/0/observations/0/value', 'metrics/0/observations/0/value']
+        paths_to_skip = ['forecasts/0/observations/0/value',
+                         'metrics/0/observations/0/value',
+                         'parties/0/beneficialOwners/0',
+                         'parties/0/people/0/address',
+                         'parties/0/people/0/identifier']
 
         # Add examples
         definition["references"] = get_definition_references(schema, defn)
         for ref in definition["references"]:
-            if ref[0] not in schema['definitions'] and '/'.join(ref) not in paths_to_skip:
-                if ref[-1] == '0':
-                    ref.pop(-1)
+            if ref[0] not in schema['definitions']:
+                skip = False
 
-                definition["content"].extend([
-                  "```{jsoninclude} ../../docs/examples/example.json\n",
-                  f":jsonpointer: /projects/0/{'/'.join(ref)}\n",
-                  f":title: {'/'.join(ref)}\n",
-                  "```\n\n"
-                ])
+                for path_to_skip in paths_to_skip:
+                    if path_to_skip == '/'.join(ref)[:len(path_to_skip)]:
+                        skip = True
+
+                if not skip:
+                    if ref[-1] == '0':
+                        ref.pop(-1)
+
+                    definition["content"].extend([
+                      "```{jsoninclude} ../../docs/examples/example.json\n",
+                      f":jsonpointer: /projects/0/{'/'.join(ref)}\n",
+                      f":title: {'/'.join(ref)}\n",
+                      "```\n\n"
+                    ])
 
         definition["content"].extend([
             "````\n\n",
@@ -453,7 +464,8 @@ def copy_element(name, replacements=None, root='definitions'):
 
     ocds_base_url = 'https://standard.open-contracting.org/1.1/en/'
 
-    builder = ProfileBuilder('1__1__5', {'budget': 'master', 'transaction_milestones': 'master'})
+    builder = ProfileBuilder('1__1__5',
+                             {'budget': 'master', 'transaction_milestones': 'master', 'beneficialOwners': 'master'})
     ppp_schema = get(f'{ppp_base_url}release-schema.json').json()
     ppp_schema = builder.patched_release_schema(schema=ppp_schema)
 
@@ -474,6 +486,7 @@ def copy_element(name, replacements=None, root='definitions'):
         'relatedProjectScheme.csv',
         'relatedProject.csv',
         'classificationScheme.csv',
+        'country.csv',
     }
     ocds_codelists = {
         'currency.csv',
@@ -499,7 +512,6 @@ def copy_element(name, replacements=None, root='definitions'):
         'LinkedRelease',  # Similar to linked release in OCDS
         'Modification',
         'RelatedProject',  # Similar to relatedProcess in OCDS
-        'Person',
         'SimpleIdentifier',
     }
     ocds_definitions = {
@@ -519,6 +531,7 @@ def copy_element(name, replacements=None, root='definitions'):
         'Transaction',
         'Milestone',
         'MilestoneReference',
+        'Person',
     }
     compare(schema['definitions'], infra_definitions, ocds_definitions,
             'schema/project-level/project-schema.json#/definitions', 'definitions')
@@ -750,6 +763,14 @@ def copy_element(name, replacements=None, root='definitions'):
     # Original from standard: "The title of the milestone being referenced, this must match the title of a milestone described elsewhere in a release about this contracting process." # noqa: E501
     schema['definitions']['MilestoneReference']['properties']['title']['description'] = "The title of the milestone being referenced, this must match the title of a milestone in this project or contracting process's `.milestones`."  # noqa: E501
 
+    copy_element('Person')
+    schema['definitions']['Person']['properties']['jobTitle'] = {
+          "title": "Job title",
+          "description": "The job title of the person (for example, Financial Manager).",
+          "type": "string",
+          "minLength": 1
+        }
+
     remove_null_and_pattern_properties(schema)
     remove_integer_identifier_types(schema)
     remove_deprecated_properties(schema)