From 5716c1965d19c743ba015fc594f7c8f877ce7751 Mon Sep 17 00:00:00 2001 From: Benn Oshrin Date: Fri, 26 Jul 2024 14:50:23 -0400 Subject: [PATCH] Additional commit for CFM-371 --- .../src/Model/Table/PersonRoleMappingsTable.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) 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; }