WTEL-10373 [phone validation]

[Kontentski]

feat: add phone validation for members

[webitel-review]

🤖 Webitel Code Review

Цей пул-реквест додає валідацію формату призначення (destination) для комунікацій учасників (Member) у методі IsValid. Проте безумовне застосування валідації телефону до всіх типів комунікацій несе високий ризик порушення роботи не-телефонних каналів зв'язку.

📋 Walkthrough (2 файл(и/ів))

Файл	Зміни
model/cc_member.go	Додано виклик функції validatePhoneNumber для перевірки формату поля Destination у комунікаціях учасника.
model/cc_member_test.go	Додано юніт-тести для перевірки валідації призначення комунікацій з різними сценаріями (валідні, пусті, невалідні формати).

Знахідки

• [high] model/cc_member.go:483 — Метод IsValid тепер безумовно валідує Destination як номер телефону за допомогою validatePhoneNumber. Проте MemberCommunication може використовуватися для інших типів комунікацій (наприклад, email, чати, SIP-адреси тощо), де формат відрізняється від телефонного номера. Це призведе до помилок валідації для будь-яких не-телефонних каналів зв'язку. Необхідно або обмежити цю валідацію лише для телефонних типів комунікацій, або переконатися, що інші типи не використовують це поле.
• [low] model/cc_member_test.go:36 — Тест очікує, що номер телефону з пробілами (380 99 111 22 33) є невалідним. Якщо регулярний вираз дійсно забороняє пробіли, це може створити проблеми для користувачів, які копіюють номери у такому форматі. Варто перевірити, чи не доцільніше очищати пробіли (або дозволити їх у регулярному виразі) перед валідацією.

---

_{Index-grounded review across the Webitel codebase. Знахідки можуть бути неточними — перевіряйте перед застосуванням.}