Fusionner le compteur d'innovation dans celui d'xp #1
Labels
No Label
Core
bug
duplicate
easy
enhancement
help wanted
invalid
performance
proposal
question
security
warning
wontfix
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: devs/PCv5#1
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
Actuellement, on a deux compteurs de points par membre : xp (équivalent aux points de participations dans la version actuelle) et innovation (attribués pour la création de contenus dans la v5).
Le truc, c'est qu'ultimement le niveau qu'on veut afficher sur le profil (avec la courbe d'xp qu'on avait précisément affinée) ne prend qu'un argument, et du coup on se retrouve à faire
level(xp + innovation)
ou quelque chose du genre.Comme le compteur d'xp reçoit de toute façon des points de plusieurs sources, je propose de tout fusionner en un seul compteur quitte à avoir sur la page de profil un détail du nombre de programmes/images/etc uploadés pour préserver leur mise en valeur.
(Proposé par @Lephenixnoir)
Ok, pour ça, mais régler le bug de la bdd et du dépassement les 1 Milliard d'xp Actuellement quand on dépasse les 32 bits on fait une 500 car c'est enregistré en bdd
Heu, on ira pas dépasser le milliard xD
Totoyo, qui a actuellement le plus de points n'en est qu'à 16000. Même en considérant qu'on donne plus de points, disons 1000 fois plus pour avoir énormément de marge, on arrive à 16 millions. Y'a de quoi faire avant d'aller taper le milliard.
En gros l'idée était juste de capper dans le code parce que sinon un admin peut mettre un nombre énorme dans le champ d'xp sur la page de modification d'un profil et causer une erreur à la mise à jour de la base de données car le type est trop petit.
Ah oui. Bah c'est juste une condition à ne pas oublier x)
Du coup, pour l'histoire de fusionner les compteurs ?
Va pour la fusion.
Du coup je m'en occupe, ce soir ou demain.
Fait par
21ebfb7d2a
(j'ai oublié de mentionner l'issue dans le commit, my bad). Ce commit supprime le compteur d'innovation, tout ira donc dans le compteur d'XP.