[Validator] fixed raising violations to a maximum of one

This commit is contained in:
Michael Hirschler 2015-12-02 14:59:04 +01:00
parent bc642fb633
commit 7860bb463d

View File

@ -38,6 +38,8 @@ class BicValidator extends ConstraintValidator
->setParameter('{{ value }}', $this->formatValue($value)) ->setParameter('{{ value }}', $this->formatValue($value))
->setCode(Bic::INVALID_LENGTH_ERROR) ->setCode(Bic::INVALID_LENGTH_ERROR)
->addViolation(); ->addViolation();
return;
} }
// must contain alphanumeric values only // must contain alphanumeric values only
@ -46,6 +48,8 @@ class BicValidator extends ConstraintValidator
->setParameter('{{ value }}', $this->formatValue($value)) ->setParameter('{{ value }}', $this->formatValue($value))
->setCode(Bic::INVALID_CHARACTERS_ERROR) ->setCode(Bic::INVALID_CHARACTERS_ERROR)
->addViolation(); ->addViolation();
return;
} }
// first 4 letters must be alphabetic (bank code) // first 4 letters must be alphabetic (bank code)
@ -54,6 +58,8 @@ class BicValidator extends ConstraintValidator
->setParameter('{{ value }}', $this->formatValue($value)) ->setParameter('{{ value }}', $this->formatValue($value))
->setCode(Bic::INVALID_BANK_CODE_ERROR) ->setCode(Bic::INVALID_BANK_CODE_ERROR)
->addViolation(); ->addViolation();
return;
} }
// next 2 letters must be alphabetic (country code) // next 2 letters must be alphabetic (country code)
@ -62,6 +68,8 @@ class BicValidator extends ConstraintValidator
->setParameter('{{ value }}', $this->formatValue($value)) ->setParameter('{{ value }}', $this->formatValue($value))
->setCode(Bic::INVALID_COUNTRY_CODE_ERROR) ->setCode(Bic::INVALID_COUNTRY_CODE_ERROR)
->addViolation(); ->addViolation();
return;
} }
// should contain uppercase characters only // should contain uppercase characters only
@ -70,6 +78,8 @@ class BicValidator extends ConstraintValidator
->setParameter('{{ value }}', $this->formatValue($value)) ->setParameter('{{ value }}', $this->formatValue($value))
->setCode(Bic::INVALID_CASE_ERROR) ->setCode(Bic::INVALID_CASE_ERROR)
->addViolation(); ->addViolation();
return;
} }
} }
} }