[v 15.6]fix #2873 avec mysql

[artragis]

Mysql ne permet pas d'utiliser un queryset comme si c'était une liste, j'ai donc fait des "all" juste avant.

Q	R
Correction de bugs ?	[oui]
Nouvelle Fonctionnalité ?	[non]
Tickets (issues) concernés	#2873

note pour la QA :

Il vous faudra 2 utilisateurs

avec l'utilisateur n°1

• créez un topic

avec l'utilisateur n°2

• répondez au topic
• vérifiez dans la liste "toutes les notifications" que le topic n'est pas en gras

avec l'utilisateur n°1

• vérifiez qu'il y a une notification, mais avant de la lire, allez dans "toutes les notifications" et vérifiez que le topic est en gras.
• vérifiez aussi avant de la lire qu'elle est affichée comme non lue dans son forum

[landscape-bot]

Repository health decreased by 0.01% when pulling 3820047 on artragis:patch-10 into 179ed0f on zestedesavoir:prod.

• 2 new problems were found (including 0 errors and 0 code smells).
• No problems were fixed.

[Situphen]

Erreur de PEP8 ! Je te conseille vraiment de mettre un git hook comme dit dans la doc. ;)

[SpaceFox]

Note pour QA : vérifier aussi que ça marche aussi quand on fait "Non lu" sur un message de forum.

[artragis]

y'a pas de githook sur l'éditeur de github (et le fix était trop simple
pour que je m'amuse à rebooter sur linux)

Le 05/07/2015 16:23, Situphen a écrit :

Erreur de PEP8 ! Je te conseille vraiment de mettre un /git hook/
comme dit dans la doc
http://zds-site.readthedocs.org/fr/latest/utils/git-pre-hook.html. ;)

—
Reply to this email directly or view it on GitHub
#2889 (comment).

[Situphen]

@gustavi : Ok

Pour la QA, il faut aussi vérifier que les sujets non lus et non suivis sont en gras !

[landscape-bot]

Code quality remained the same when pulling 873161b on artragis:patch-10 into 179ed0f on zestedesavoir:prod.

[pierre-24]

Ça marche aussi ^^

Par contre, faut tester avec mysql aussi (mais comme c'est artragis qui fait la PR, je peux le faire :p )

[pierre-24]

Rapport de QA: désolé, mais non.

>>>> print(type(context['topics'].all()))
<class 'django.db.models.query.QuerySet'>

Tant que c'est pas "évalué", ça reste un QuerySet et ça plante toujours. La seule solution, c'est les listes :o

EDIT: la preuve:

(avec MySQL, évidement)

[artragis]

fixed + documentation + commentaires.

[landscape-bot]

Code quality remained the same when pulling c149e9e on artragis:patch-10 into 179ed0f on zestedesavoir:prod.

[Situphen]

authenticated

[landscape-bot]

Code quality remained the same when pulling e2d1d06 on artragis:patch-10 into 179ed0f on zestedesavoir:prod.

[pierre-24]

Rapport de QA: presque, encore un petit effort:

• Les sujets non-lu sont en gras: oui
• Les sujets suivis avec un nouveau message sont en gras: oui
• Le sujet est à nouveau en gras quand on fait "non-lu": oui

... SAUF sur la page qui liste les sujets d'un utilisateur sur son profil ;)

[artragis]

j'y pensais pas à celle là :)

Le 06/07/2015 19:48, Pierre Beaujean a écrit :

Rapport de QA: presque, encore un petit effort:

• Les sujets non-lu sont en gras: oui
• Les sujets suivis avec un nouveau message sont en gras: oui
• Le sujet est à nouveau en gras quand on fait "non-lu": oui

... SAUF sur la page qui liste les sujets d'un utilisateur sur son
profil ;)

—
Reply to this email directly or view it on GitHub
#2889 (comment).

[artragis]

voilou.

[landscape-bot]

Code quality remained the same when pulling a2c77aa on artragis:patch-10 into 179ed0f on zestedesavoir:prod.

[artragis]

Tu imagines même pas le temps que j'ai mis à comprendre. Mais c'set fait.

[landscape-bot]

Code quality remained the same when pulling 3c507c0 on artragis:patch-10 into 179ed0f on zestedesavoir:prod.

[pierre-24]

Rapport de QA: Ok.

Mais à mon avis, on acceptera jamais ça sans un squash

[landscape-bot]

Code quality remained the same when pulling 3ffbe31 on artragis:patch-10 into 179ed0f on zestedesavoir:prod.

[artragis]

rebase fait.

2015-07-07 8:42 GMT+02:00 landscape-bot notifications@github.com:

[image: Code Health] https://landscape.io/diff/192266
Code quality remained the same when pulling 3ffbe31
artragis@3ffbe31
on artragis:patch-10 into 179ed0f
179ed0f
on zestedesavoir:prod.

—
Reply to this email directly or view it on GitHub
#2889 (comment)
.

[SpaceFox]

Il va falloir refaire cette PR : on ne merge jamais rien directement dans prod. Le merge doit se faire dans cette branche de hotfix que je viens de créer.

J'ai pas vérifié, mais est-ce que ta branche partait bien de prod ?

[artragis]

elle partait de prod

Le 9 juillet 2015 14:01, SpaceFox notifications@github.com a écrit :

Il va falloir refaire cette PR : on ne merge jamais rien directement dans
prod. Le merge doit se faire dans cette branche de hotfix
https://github.com/zestedesavoir/zds-site/tree/hotfix/v15.6b que je
viens de créer.

J'ai pas vérifié, mais est-ce que ta branche partait bien de prod ?

—
Reply to this email directly or view it on GitHub
#2889 (comment)
.