adopted-ember-addons / adopted-ember-addons/ember-validators
date validator: inconsistent meaning of allowBlank
- Lingua principale
- JavaScript
- Stelle
- 23
- Fork
- 38
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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.
Guida per i contributori
Apri la guida per i contributori
Valutazione
Questa issue non è ancora stata valutata.