Infinitiweb / code-conventions

MIT License
8 stars 3 forks source link

Убрать пункт про выбрасывание исключения для методов названия которых начинаются c check и validate #8

Closed tolvinskii closed 6 years ago

tolvinskii commented 6 years ago

Предлагаю выпилить этот пункт https://github.com/Infinitiweb/code-conventions/blob/master/PHP.md#-Методы-названия-которых-начинаются-c-check-и-validate-должны-выбрасывать-исключения-и-не-возвращать-значения

в yii2 свои функции валидации должны возвращать bool

пример кода в модели в rules [['inn'], 'validateInn']

public function validateInn() { if (...) return true; return false; }

hello-omny commented 6 years ago

Создание Валидаторов в Yii2

Просто в данном случае addError === исключение, и можно жить дальше. Либо вносим поправку что для Yii2 разрешено и допустимо.

tolvinskii commented 6 years ago

Оставляем как есть, валидация в yii не возвращает ничего, а addError === исключение