keystonejs/keystone-classic

TextType - Over-length value validates when both max and min specified

Ouverte

#4 536 ouverte le 28 déc. 2017

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)JavaScript (2 288 forks)batch import
4.x candidatebughelp wanted

Métriques du dépôt

Stars
 (14 656 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Expected behavior

When both max and min are specified in a field definition, over length strings should be refused.

Actual/Current behavior

Over length string accepted

Steps to reproduce the actual/current behavior

Define a field as per below: name: { type: Types.Text, required: true, min: 3, max: 5 }, Enter a string longer than 5 characters Submit

Environment

Keystone master (0ca9eb18c00fb051c579105ae7d283f98b182d8d) Node v9.2.0

Notes

Issue seems to be caused by a922b38a12bad632ef6f5d992ea91b363bb78d4f. If min length validation passes, all other validation results are discarded.

Guide contributeur

Direction de recherche
Inspectez le commit a922b38a12bad632ef6f5d992ea91b363bb78d4f pour comprendre comment la validation de longueur minimale ignore incorrectement les autres résultats de validation. Ensuite, tracez la logique de validation dans le gestionnaire de champ TextType pour corriger le problème.
Stack technique
javascript
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75