You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Предлагаю разбить статью "Валидаторы" на отдельные части и превратить в целый раздел.
Сейчас статья хоть и решает поставленную задачу (знакомит разработчиков с валидаторами), но выглядит местами сильно бедной:
не показывает примеров почему так нужно делать,
неявное отличие терминов правило и валидатор сбивает с толку при разработке собственных правил,
перечень документированных валидаторов стремиться к нулю.
Сама по себе статья содержит ошибки:
Класса Bitrix\Main\Validation\Rule\Composite\Validatable нет, его заменил Bitrix\Main\Validation\Rule\Recursive\Validatable
Неявное описание про отсутствие валидации для nullable поля - при заполнении его значением null валидация осуществляется.
Возможно разделение и нейминг не сильно удачный, но на мой взгляд читается проще.
Добрый день!
По issue #75 решили, что вложенность в документации не будем увеличивать. Поэтому разбивать статью по валидации и делать отдельный раздел не будем.
Нужно объединить ваши материалы в одну статью перед ревью.
Другой вариант, чтобы вы не тратили много времени — можем просто забрать материалы себе на обработку, объединить и поправить оформление самостоятельно 🙂
Как вам удобнее?
Другой вариант, чтобы вы не тратили много времени — можем просто забрать материалы себе на обработку, объединить и поправить оформление самостоятельно 🙂
Как вам удобнее?
Вот даже не знаю.
Я не против, если вы возьмете хорошие части валидаторов и включите их в статью, но с учетом #75 я не считаю что то что написано должно быть в этой огромной портянке. Возможно имеет смысл часть данных и переработать, но я пока не вижу как лучше это сделать.
Возможно у вас есть идеи?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Предлагаю разбить статью "Валидаторы" на отдельные части и превратить в целый раздел.
Сейчас статья хоть и решает поставленную задачу (знакомит разработчиков с валидаторами), но выглядит местами сильно бедной:
Сама по себе статья содержит ошибки:
Bitrix\Main\Validation\Rule\Composite\Validatableнет, его заменилBitrix\Main\Validation\Rule\Recursive\Validatablenullableполя - при заполнении его значением null валидация осуществляется.Возможно разделение и нейминг не сильно удачный, но на мой взгляд читается проще.