Cristin64

cristin_ms/cristin_xml.py

@@ -167,6 +167,9 @@ def add_persons(root: ET.Element,
                                                 0].innehaver_sluttdato,
                                            'stillingsbetegnelse': stil_model.stillingsnavn
                                        })
+            if not all_ansettelser:
+                continue
+
 
         # Validate and adjust values if possible, otherwise skip person
         values = {
@@ -200,29 +203,27 @@ def add_persons(root: ET.Element,
         for k, v in values_none_okay.items():
             if v is not None:
                 set_text(person, k, v)
-
-        if len(all_ansettelser) > 0:
-            ansettelser = ET.SubElement(person, 'ansettelser')
-            for ans_num in range(len(all_ansettelser)):
-                ansettelse = ET.SubElement(ansettelser, 'ansettelse')
-                ans_dict = all_ansettelser[ans_num]
-                set_text(ansettelse, 'institusjonsnr',
-                         ans_dict['institusjonsnr'])
-                set_text(ansettelse, 'avdnr', ans_dict['avd'])
-                set_text(ansettelse, 'undavdnr', ans_dict['undavd'])
-                set_text(ansettelse, 'gruppenr', ans_dict['gruppe'])
-                # Use last 4 digits of stillingskode. API typically gives 20001234
-                # but we only want 1234
-                set_text(ansettelse, 'stillingskode',
-                         ans_dict['stillingskode'])
-                set_text(ansettelse, 'datoFra',
-                         ans_dict['datoFra'])
-                set_text(ansettelse, 'datoTil',
-                         ans_dict['datoTil'])
-                if ans_dict['stillingsbetegnelse']:
-                    set_text(ansettelse, 'stillingsbetegnelse',
-                             ans_dict['stillingsbetegnelse'])
-                # set_text(ansettelse, 'stillingsandel', ???)
+        
+        ansettelser = ET.SubElement(person, 'ansettelser')
+        for ans_dict in all_ansettelser:
+            ansettelse = ET.SubElement(ansettelser, 'ansettelse')
+            set_text(ansettelse, 'institusjonsnr',
+                     ans_dict['institusjonsnr'])
+            set_text(ansettelse, 'avdnr', ans_dict['avd'])
+            set_text(ansettelse, 'undavdnr', ans_dict['undavd'])
+            set_text(ansettelse, 'gruppenr', ans_dict['gruppe'])
+            # Use last 4 digits of stillingskode. API typically gives 20001234
+            # but we only want 1234
+            set_text(ansettelse, 'stillingskode',
+                     ans_dict['stillingskode'])
+            set_text(ansettelse, 'datoFra',
+                     ans_dict['datoFra'])
+            set_text(ansettelse, 'datoTil',
+                     ans_dict['datoTil'])
+            if ans_dict['stillingsbetegnelse']:
+                set_text(ansettelse, 'stillingsbetegnelse',
+                         ans_dict['stillingsbetegnelse'])
+            # set_text(ansettelse, 'stillingsandel', ???)
 
 
 def make_root_element() -> ET.Element: