zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Le karma n'est plus "voted" sur les membres récents #1577

Closed Alex-D closed 9 years ago

Alex-D commented 10 years ago

Sujet du forum

Un nouveau membre signale que lorsqu'il vote, après rechargement de la page son vote est bien compté mais il n'a pas de retour visuel (vert/rouge) sur celui qu'il a cliqué.

Axylium commented 10 years ago

Peut-être faudrait-il réouvrir ici. J'ai actuellement le même problème que le membre du post, depuis peu.

SpaceFox commented 9 years ago

Corrigé et mergé --> fermé.

Situphen commented 9 years ago

Il faut réouvrir car le problème est réapparu pour certains membres ! https://zestedesavoir.com/forums/sujet/1376/les-pouces-vertrouge/?page=2#p50815 Je pense que c'est le template tag profile qui bug chez certains membres et pas d'autres !

Eskimon commented 9 years ago

@Situphen on peut refermer ici ?

Situphen commented 9 years ago

Non, #2560 ne corrige pas le problème !

Eskimon commented 9 years ago

Alors là c'est mystique !!

Test sur la beta :

W.T.F !

SpaceFox commented 9 years ago

Je soupçonne un problème d'identifiant dans les tables user et profile.

Ton compte a les mêmes ID dans les 2 tables mais ce n'est pas le cas des nouveaux comptes… il suffit qu'on enregistre l'un et recherche l'autre pour que ça foire. Le 2 mai 2015 11:18, "Eskimon" notifications@github.com a écrit :

Alors là c'est mystique !!

Test sur la beta :

  • Avec mon vieux compte Eskimon : Je plussoie -> rafraichit -> tout roule
  • Avec un nouveau compte bidon : Je plussoie -> rafraichit -> le simbole du pouce n'est pas vert !

W.T.F !

— Reply to this email directly or view it on GitHub https://github.com/zestedesavoir/zds-site/issues/1577#issuecomment-98338170 .

Eskimon commented 9 years ago

Oh comme c'est bien vu !!

Tout se joue là donc : https://github.com/zestedesavoir/zds-site/blob/dev/templates/misc/message.part.html#L283

Eskimon commented 9 years ago

@SpaceFox tu pourrais trouver de combien d'indices est le décalage des id entre profile et user . Parce que la c'est inquiétant comme problème je trouve. Et surtout "a part la fusion de profile et user comment resoudre les problemes ici" ?

SpaceFox commented 9 years ago

Y'a 1 de décalage : les derniers IDs sont de 2717 / 2718.

Le dernier membre avec les IDs communs est le n°756 "Le Berger". Le premier avec des IDs différents est le 757 (member_profile) / 758 (user_id) "captain28".

On parle de gens qui se sont enregistrées le 24 juillet 2014.

Eskimon commented 9 years ago

Quels sont les impacts si on décide de faire un reset des id a partir de l'id 750 par exemple (histoire de réaligner les deux tables) ?

SpaceFox commented 9 years ago

On ne fait pas.

Ce sont des IDs techniques, ça doit fonctionner même si les IDs sont complètement différents d'une table à l'autre.

Et si on est incapables de gérer ça proprement, on fusionne ces deux tables.

Eskimon commented 9 years ago

Ce sont des IDs techniques, ça doit fonctionner même si les IDs sont complètement différents d'une table à l'autre.

De toute évidence pour Django c'est pas le cas... Problème du framework ou on a fait une merde quelque part ?

SpaceFox commented 9 years ago

On a forcément fait de la merde quelque part, en se mélangeant les pinceaux entre member_profile.id et auth_user.id : comme à la base c'était les mêmes (alors qu'absolument rien ne le garantit), on est parti du principe que c'était les mêmes et qu'on pouvait utiliser indifféremment l'un ou l'autre.

GerardPaligot commented 9 years ago

On me rappelle comment on peut merge ces 2 tables ?

Eskimon commented 9 years ago

On me rappelle comment on peut merge ces 2 tables ?

Avec plaisir ! https://github.com/zestedesavoir/zds-site/issues/2295

Eskimon commented 9 years ago

@firm1 avait corrigé ce truc, je ferme :)