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

En mode connecté, on fait une requête pour CHAQUE MP RECU ! #2090

Closed
SpaceFox opened this issue Jan 19, 2015 · 3 comments

Comments

@SpaceFox
Copy link
Member

commented Jan 19, 2015

Quand on est connecté, on passe par ce bout de code sur toutes les pages :

zds-site/templates/base.html ligne 258 : {% with topics=user|interventions_privatetopics %}

Il fait cette requête pour chaque MP que vous avez reçu !

SELECT `mp_privatetopic`.`id`, `mp_privatetopic`.`title`, `mp_privatetopic`.`subtitle`, `mp_privatetopic`.`author_id`, `mp_privatetopic`.`last_message_id`, `mp_privatetopic`.`pubdate` FROM `mp_privatetopic` WHERE `mp_privatetopic`.`id` = <id membre>

C'est à corriger d'urgence. Ça passe dans la v1.5.

Et après ça on s'étonne que certaines personnes trouvaient le site lent...

@SpaceFox SpaceFox added this to the "Futur proche" (v1.x) milestone Jan 19, 2015

@SpaceFox SpaceFox added the Bloquant label Jan 19, 2015

@SpaceFox

This comment has been minimized.

Copy link
Member Author

commented Jan 19, 2015

J'ai MAJ la première version (mauvaise interprétation) avec la triste réalité...

@SpaceFox SpaceFox self-assigned this Jan 19, 2015

@SpaceFox SpaceFox changed the title La page des membres va chercher des infos sur les MP des membres affichés ! En mode connecté, on fait une requête pour CHAQUE MP RECU ! Jan 19, 2015

@SpaceFox

This comment has been minimized.

Copy link
Member Author

commented Jan 19, 2015

... et c'est là que j'ai une bonne raison de râler pour les commentaires : le problème vient de templatetags/interventions.py, méthode def interventions_privatetopics(user):.

Qui fait des trucs dégueulasses sans le moindre commentaire. Là où ça aurait couté 2 minutes de mettre 2 lignes de commentaires pour expliquer comment ce truc fonctionne, je passe 2 heures à essayer de le comprendre.

@SpaceFox

This comment has been minimized.

Copy link
Member Author

commented Jan 21, 2015

Corrigé donc fermé.

@SpaceFox SpaceFox closed this Jan 21, 2015

@SpaceFox SpaceFox removed the Bloquant label Jan 21, 2015

@SpaceFox SpaceFox modified the milestones: Version 1.6, "Futur proche" (v1.x) Feb 10, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.