adopted-ember-addons / adopted-ember-addons/ember-validators

date validator: inconsistent meaning of allowBlank

オープン
#65 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
23
フォーク
38
PR マージ指標
30日以内にマージされた PR はありません

説明

The [date validation](https://github.com/offirgolan/ember-validators/blob/fbca781275be19281ac8b568a1720f4f03e005eb/addon/date.js#L17-L39) has an option `isBlank` which is documented like so:

> If true, skips validation if the value is empty

This accurately describes the implementation, which skips validation if `isEmpty` returns true for the value.

[Other validators](https://github.com/offirgolan/ember-validators/blob/fbca781275be19281ac8b568a1720f4f03e005eb/addon/number.js#L14-L15) use `allowBlank` in combination with an `isBlank` check, and `allowNone` for an `isEmpty` check. This is inconsistent; blank should always mean blank, and empty/none should always mean empty.

There may be other validators with this problem, but I haven't checked.

Should this be fixed, and if so, how? One option is to deprecate the current `allowBlank` option in favor of `allowNone`, which would behave the same way. At the next major release, you could then add a real `allowBlank` option if desired.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。