Corrige le bug de recherche qui fait se tromper de produit

Jojo144 / compteur_du_gase

Logiciel de gestion de comptes et de stock pour un GASE (Groupement d'Achat en Service Épicerie)

GNU General Public License v3.0

6 stars 3 forks source link

Corrige le bug de recherche qui fait se tromper de produit #68

Closed JocelynDelalande closed 2 years ago

JocelynDelalande commented 2 years ago

(désolé pour le diff touffu, mais voilà ce qui arrive avec du code pas très bien indenté :p)

Fix #67

Jojo144 commented 2 years ago

Tu peux vérifier quand est-ce qu'il faut mettre des "this" et quand est-ce que qu'il ne faut pas en mettre ? J'ai pas très bien compris cette histoire.

Sinon ça me parait très bien.

Donc le comportement quand on clique sur "Dans Panier" alors qu'il y a une quantité masquée c'est de ne pas la mettre dans le panier, mais de ne pas l'effacer par contre. C'est bien ça hein ?

JocelynDelalande commented 2 years ago

Donc le comportement quand on clique sur "Dans Panier" alors qu'il y a une quantité masquée c'est de ne pas la mettre dans le panier, mais de ne pas l'effacer par contre. C'est bien ça hein ?

C'est ça… Mais surtout ça résoud le bug causé par le fait de détruire/reconstruire les input à la volée, là on les masque/affiche et tout se passe bien :-).

(en plus c'est plus performant).

JocelynDelalande commented 2 years ago

C'est corrigé.

JocelynDelalande commented 2 years ago

Tu peux vérifier quand est-ce qu'il faut mettre des "this" et quand est-ce que qu'il ne faut pas en mettre ? J'ai pas très bien compris cette histoire.

Si j'ai bien compris :

dans le code JS « pur » : il faut mettre this quand on référence un attribut de l'objet instance de Vue depuis une méthode de cet dernier, que ça soit pour faire référence à quelque-chose de methods, de data ou quelque-chose d'initialisé manuellement genre this.bidule = 42.
dans le code de la template, les attributs/méthodes listées dans methods et data sont accessibles directement, sans utiliser "this". Par contre on a pas accès aux trucs initialisés manuellement.

Jojo144 commented 2 years ago

Super !