Don't validate against empty fields

Author: phpfui

Tests/Unit/ValidationTest.php

@@ -132,6 +132,18 @@ public function testComparison() : void
 		$gtDate = '2023-01-02';
 		$ltDate = '2022-12-31';
 
+		$crud->equal = $date;
+		$crud->not_equal = $gtDate;
+		$crud->gt_field = $gtDate;
+		$crud->gte_field = $date;
+		$crud->lt_field = $ltDate;
+		$crud->lte_field = $date;
+		$crud->eq_field = $date;
+		$crud->neq_field = $ltDate;
+		$validator->validate();
+		$errors = $validator->getErrors();
+		$this->assertEmpty($errors);
+
 		$crud->equal = $date;
 		$crud->not_equal = $gtDate;
 		$crud->gt_field = $gtDate;
@@ -978,7 +990,6 @@ public function testUnique() : void
 		$crud->product_code = 'NWTCFV-91';
 		$validator->validate();
 		$errors = $validator->getErrors();
-//		fwrite(STDERR, print_r($errors, true));
 		$this->assertCount(1, $errors);
 		$this->assertContains('NWTCFV-91 is not unique', $errors['product_code']);
 		}

src/PHPFUI/ORM/Validator.php

@@ -489,7 +489,7 @@ private function validate_eq_field(mixed $value, array $parameters, array $field
 		$field = $parameters[0] ?? '';
 		$compare = $this->record[$field];
 
-		return $value == $compare ? '' : \PHPFUI\ORM::trans('.validator.eq_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
+		return empty($compare) || $value == $compare ? '' : \PHPFUI\ORM::trans('.validator.eq_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
 		}
 
 	/**
@@ -512,7 +512,7 @@ private function validate_gt_field(mixed $value, array $parameters, array $field
 		$field = $parameters[0] ?? '';
 		$compare = $this->record[$field];
 
-		return $value > $compare ? '' : \PHPFUI\ORM::trans('.validator.gt_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
+		return empty($compare) || $value > $compare ? '' : \PHPFUI\ORM::trans('.validator.gt_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
 		}
 
 	/**
@@ -524,7 +524,7 @@ private function validate_gte_field(mixed $value, array $parameters, array $fiel
 		$field = $parameters[0] ?? '';
 		$compare = $this->record[$field];
 
-		return $value >= $compare ? '' : \PHPFUI\ORM::trans('.validator.gte_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
+		return empty($compare) || $value >= $compare ? '' : \PHPFUI\ORM::trans('.validator.gte_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
 		}
 
 	/**
@@ -596,7 +596,7 @@ private function validate_lt_field(mixed $value, array $parameters, array $field
 		$field = $parameters[0] ?? '';
 		$compare = $this->record[$field];
 
-		return $value < $compare ? '' : \PHPFUI\ORM::trans('.validator.lt_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
+		return empty($compare) || $value < $compare ? '' : \PHPFUI\ORM::trans('.validator.lt_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
 		}
 
 	/**
@@ -608,7 +608,7 @@ private function validate_lte_field(mixed $value, array $parameters, array $fiel
 		$field = $parameters[0] ?? '';
 		$compare = $this->record[$field];
 
-		return $value <= $compare ? '' : \PHPFUI\ORM::trans('.validator.lte_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
+		return empty($compare) || $value <= $compare ? '' : \PHPFUI\ORM::trans('.validator.lte_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
 		}
 
 	/**
@@ -705,7 +705,7 @@ private function validate_neq_field(mixed $value, array $parameters, array $fiel
 		$field = $parameters[0] ?? '';
 		$compare = $this->record[$field];
 
-		return $value != $compare ? '' : \PHPFUI\ORM::trans('.validator.neq_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
+		return empty($compare) || $value != $compare ? '' : \PHPFUI\ORM::trans('.validator.neq_field', ['value' => $value, 'field' => $field, 'compare' => $compare]);
 		}
 
 	/**