Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐞 Ajout du resetValidation dans les champs custom #3914

Merged
merged 2 commits into from
May 13, 2024
Merged

Conversation

alemangui
Copy link
Collaborator

@alemangui alemangui commented May 13, 2024

Cette erreur a été vu dans la page pour ajouter des achats lors qu'on cliquait sur "Valider et ajouter un nouveau" :

image

La raison derrière est que nos composants custom n'ont pas surchargé la méthode resetValidation.

Problème

1- Lors qu'on clique sur "Valider et ajouter un nouveau" on sauvegarde l'achat, et puis on reste dans la même page, mais avec le formulaire en blanc. Ça vient de ce bout de code :

if (!stayOnPage) this.$router.push({ name: "PurchasesHome" }).then(notify)
else {
      ...
      this.$refs.form.resetValidation()
      ...
}

2- La ligne this.$refs.form.resetValidation() itère sur les champs du formulaire et appelle resetValidation sur chaqu'un. Voici le code vuetify :

resetValidation () {
      this.inputs.forEach(input => input.resetValidation())
      this.resetErrorBag()
}

3- Lors que ça arrive à nos champs custom, resetValidation n'existe pas. Donc l'erreur est levée.

Solution

Simplement ajouter la méthode pour chaque champ custom et faire en sorte qu'elle fasse l'appel au composant pertinent.

@alemangui alemangui marked this pull request as draft May 13, 2024 12:41
@alemangui alemangui marked this pull request as ready for review May 13, 2024 12:48
@alemangui alemangui requested a review from hfroot May 13, 2024 12:48
@alemangui alemangui changed the base branch from staging to main May 13, 2024 12:53
@alemangui alemangui merged commit 032e143 into main May 13, 2024
5 checks passed
@alemangui alemangui deleted the purchase-bug-fix branch May 13, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants