Skip to content

feat: migrate validation from ValidatorFX to FormsFX - #11

Merged
vladcheck merged 1 commit into
mainfrom
vladcheck/forms-fx-migration
Oct 6, 2026
Merged

vladcheck merged 1 commit into
mainfrom
vladcheck/forms-fx-migration

Conversation

@vladcheck

@vladcheck vladcheck commented Oct 6, 2026 •

Copy link
Copy Markdown
Owner

Удаляет ValidationFX (избыточен при наличии FormsFX) и переводит валидацию на FormsFX.

  • Убран net.synedra:validatorfx из pom.xml
  • Валидаторы отдают FormsFX validators() (CustomValidator + StringLengthValidator); чистая функция validate(String) оставлена источником истины
  • LoginFormController строит FormsFX Form (StringField/PasswordField) и показывает его через FormRenderer; LoginForm.fxml сведён к контейнеру формы + кнопке отправки
  • Удалены ставшие ненужными FieldError, FieldErrorDecoration и стартер тулкита ValidatorFxTest
  • Все 5 тестов валидаторов обновлены под состояние FormsFX-полей (isValid/errorMessages)

Проверка: mvn verify — 27 тестов зелёные, spotless чистый, 0 нарушений checkstyle.

@vladcheck vladcheck linked an issue Oct 6, 2026 that may be closed by this pull request
@vladcheck
vladcheck requested review from Deatheh and miyagi18 October 6, 2026 08:26
@vladcheck vladcheck self-assigned this Oct 6, 2026
@vladcheck vladcheck added the enhancement New feature or request label Oct 6, 2026
@vladcheck
vladcheck merged commit 5427c67 into main Oct 6, 2026
8 checks passed
@vladcheck
vladcheck deleted the vladcheck/forms-fx-migration branch October 6, 2026 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Переехать на FormsFX

1 participant