Problème déconsigne d'écocup

[Juknum]

Il est possible de déconsigner plus d'écocup que ce que l'utilisateur à déjà consigné

[klmp200]

C'est une feature 😉
https://github.com/ae-utbm/sith3/blob/master/sith/settings.py#L473

[Juknum]

C'est clairement pas une feature quand cette limite n'est pas respectée. On a réussi à déconsigner 50 ecocups d'un coup.

[klmp200]

Par hasard, ils n'auraient psa crée plusieurs produits d'ecocup qui correspondent plus à ceux suivis par la feature ?
il faut plus de détails, quels produits sont affectés comment c'est utilisé ? Sinon on ne peut pas investiger.

[imperosol]

C'est clairement pas une feature quand cette limite n'est pas respectée. On a réussi à déconsigner 50 ecocups d'un coup.

Pour le coup, je suis vraiment curieux sur ça.
Soit c'est une incompréhension et il n'y a pas de problème, soit c'est très grave, mais dans les deux cas, il faut vraiment beaucoup plus de détails que ça.

Qui a remarqué ce détail ? Quel jour ? Quelle heure ? Où ? Quel code le produit avait ?

[klmp200]

Je veux bien m'intéresser au problème mais comme expliqué, il me faut plus de détails.
Sachant que je n'ai accès à aucune donnée de production, je ne peux pas y vérifier mes hypothèses et j'ai aucune raison de penser que la fonctionnalitée originale ne fonctionne plus sachant qu'ellle fonctionnait très bien avant et rien n'a été modifié de ce côté.

[Juknum]

[klmp200]

La feature fonctionne par rapport à l'ID du produit, pas par rapport à ses charactéristiques.
Il existe 1 produit de consigne et 1 produit de déconsigne qui sont définis dans la configuration du logiciel.
Je ne peux pas déterminer avec les informations que tu me fournis si le site est bien configuré par rapport aux produits que tu me présente.
Au vu de tous les changements qui ont eu lieu au niveau des consignes/déconsignes, il n'est pas à exclure que les gestionnaires de comptoires aient recrée les consignes/déconsignes à partir de nouveaux produits qui ne seraient donc pas traqués par la feature. Il se peut aussi qu'ils aient crées plusieurs produits de consignes qui ne sont également pas suivis.

Je n'ai pas ces informations dans ces screenshots.

[Juknum]

Quand tu dis ID, c'est l'ID de l'article dans la base de donnée ? ce serait hardcodé du coup ? Car je n'ai aucun moyen de te dire si l'ID de la déconsigne actuelle correspond à celle qui serait enregistrée.

Si c'est hardcodé, dans ce cas il faudrait ajouter une catégorie "Déconsigne" pour ça fonctionne avec n'importe quel article, nouveau ou ancien.

[klmp200]

Tu retrouve l'id du produit dans l'url
C'est hardcodé dans la config
C'est bel et bien ce que je soupçonait, la fonctionnalité va très bien, c'est l'entrée utilisateur qui ne va pas.

Le problème d'une catégorie « Déconsigne » c'est qu'il faut lier la déconsigne à un produit. Si on veut faire ça de manière dynamique ça nécessite une interface utilisateur assez compliquée.

Le plus simple c'est de faire des paires de produit « consigne/déconsigne » dans la config.

Conclusion, comme je l'ai indiqué au début, ce n'est pas un bug et ça fonctionne parfaitement, c'est un problème humain ;)

[Juknum]

@TheoDurr tu pourrais checker la config du site steuplé avec les ids :

• 1151 - déconsigne écocup
• 2730 - déconsigne pichet

[TheoDurr]

On a bien une config pour la consigne/déconsigne écocup.

https://github.com/ae-utbm/sith3/blob/b7f20fed6c89eb69f5ae8e163115535e35690ead/sith/settings.py#L468-L470

Il y a même un seuil maximal configuré entre la différence consigne/déconsigne :

https://github.com/ae-utbm/sith3/blob/b7f20fed6c89eb69f5ae8e163115535e35690ead/sith/settings.py#L472-L473

Donc cela me semble être particulier comme comportement :/

En ce qui concerne le pichet par contre ouais c'est pas du tout dans la config.

Edit : Là j'ai mentionné le fichier de config de base, mais la config ne diffère pas en production, après vérification.

[imperosol]

Ok, j'ai un peu enquêté, et en effet, il s'agit d'un exploit assez énervé dû à un enchainement de fonctions assez mal codées et très peu maintenables.

Plus précisément, le bug date du 15 août 2017 et a été introduit... par Sli.
Ca tient au fait très simple que si on déconsigne plusieurs gobelets en même temps (13XDCONS), il y a un problème de calcul dans la fonction qui regarde si on a le droit de déconsigner, puisque celle-ci considèrera toujours qu'on en déconsigne un seul à la fois.

C'est un bug assez énervé, mais ça reste "pas trop grave" dans la mesure où il est resté là 6 ans sans qu'on le voie et que seuls les barmen y ont accès (et que ceux-ci sont censés savoir qu'on ne peut pas déconsigner plus de trois ecocups).
C'est à résoudre, bien sûr, mais on n'est pas sur une urgence absolue.

Pour ce qui est du temps que ça peut prendre à réparer, c'est dur à dire. Ca peut prendre une heure comme ça peut prendre des semaines. Tout ce qui touche aux clics tient dans une seule vue monstrueuse avec 1000 fois trop de responsabilités ; donc il ne faut pas négliger la possibilité que la résolution du bug oblige à modifier plus de choses dans le plat de spaghettis.

[klmp200]

J'aurais du mettre des tests 😩

[TheoDurr]

On sait quoi faire maintenant du coup ! Ecrire le petit test en priorité pour s'assurer que ça fonctionne et corriger tout ça ;).

On fera sûrement un déploiement dans le mois pour fixer ça. On attendra pas avril

[imperosol]

Je serais pas si optimiste. Même pour des bugs mineurs, c'est dur de savoir combien de temps ça prendra, parce qu'on peut avoir d'autres obligations en dehors du site ou que quelque chose d'autre arrive peu après qui prend la priorité. Pour ce bug là, comme je l'ai dit, c'est encore plus incertain, parce que ça se trouve dans une partie spaghetti.

[imperosol]

On fera sûrement un déploiement dans le mois pour fixer ça. On attendra pas avril

Finalement, on a attendu août :)

[Juknum]

C'est sûr que dire ça sa fait avancer le schmilblick :)

[Juknum]

On a trouvé ça : si la personne a déjà déconsignée >= 3 ecocup elle peut plus en deconsigner davantage mais rien n'empeche quelqu'un de deconsigner + de 3 ecocup d'un coup.

Du coup pour contourner la limite on peut reconsigner des ecocups (jusqu'à ce que l'utilisateur soit au dessus de -3) et déconsigner derrière (autant qu'on veut : le bug)

[imperosol]

Potentiellement réparé avec #959
A suivre.