diff --git a/app/availableplugins/PipelineToolkit/src/Model/Table/PersonRoleMappingsTable.php b/app/availableplugins/PipelineToolkit/src/Model/Table/PersonRoleMappingsTable.php index f2f98d16f..9a7526f78 100644 --- 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; }