Additional commit for CFM-371

arlen · Jul 26, 2024 · 5716c19 · 5716c19
1 parent 22e9679
commit 5716c19
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/app/availableplugins/PipelineToolkit/src/Model/Table/PersonRoleMappingsTable.php b/app/availableplugins/PipelineToolkit/src/Model/Table/PersonRoleMappingsTable.php
@@ -159,12 +159,12 @@ public function validationDefault(Validator $validator): Validator {
     ]);
     $validator->notEmptyString('attribute');
 
+    // The required fields here aren't ideal, since they vary by attribute
     $validator->add('affiliation_type_id', [
       'content' => ['rule' => 'isInteger']
     ]);
-    $validator->notEmptyString('affiliation_type_id');
+    $validator->allowEmptyString('affiliation_type_id');
 
-    // The required fields here aren't ideal, since they vary by attribute
     $this->registerStringValidation($validator, $schema, 'ad_hoc_tag', false);
 
     $validator->add('comparison', [
@@ -182,12 +182,12 @@ public function validationDefault(Validator $validator): Validator {
     $validator->add('target_cou_id', [
       'content' => ['rule' => 'isInteger']
     ]);
-    $validator->notEmptyString('target_cou_id');
+    $validator->allowEmptyString('target_cou_id');
 
     $validator->add('target_affiliation_type_id', [
       'content' => ['rule' => 'isInteger']
     ]);
-    $validator->notEmptyString('target_affiliation_type_id');
+    $validator->allowEmptyString('target_affiliation_type_id');
 
     return $validator; 
   }