[v18] Date incorrecte dans les notifications

[firm1]

J'ai reçu une notification pour un topic que je suivais avant la v18.

Selon la notification elle datait d'il y'a 7h, sauf qu'en cliquant dessus, je suis tombé sur un message qui date d'il y'a 7 minutes. On peut le voir sur les screens ci-dessous.

Notification avant le clic :

Notification après le clic :

Il faudrait retomber sur ce cas pour identifier facilement le bug. Si jamais j'ai une meilleur idée du truc j'édite le titre qui n'est pas très parlant.

[gustavi]

ping @GerardPaligot

[firm1]

Après visite du sujet, et après le post de quelqu'un d'autre, j'ai a nouveau la notification avec la mauvaise heure.

La preuve en image :

Pour information les 7h correspondent à la date de la notification que j'avais AVANT la MEP de la v18. On dirait que les dates des notifications existantes n'ont pas été migrés par la MEP.

[GerardPaligot]

Ca risque d'être compliqué dans la mesure où il n'existait pas de date à une notification puisque pas d'objet. C'était une info calculé à chaque fois.

J'essayerais de voir ce que je peux faire mais je n'ai pas beaucoup de temps en ce moment et cela, jusqu'au 9 juin. Autrement dit, si quelqu'un d'autres veut être sur cette regression, qu'il ne s'en prive pas.

[GerardPaligot]

Juste une précision : Ca concerne que les notifications actives avant la MEP de la v18 + les nouvelles notifications sur ces sujets qui avaient des notifications actives ? Ou la portée est plus large ?

[firm1]

Si j'en crois ce que je vois. ça concerne tous les topics que je suivais avant la MEP.

Le problème est le même pour les notifications des contenus.

[GerardPaligot]

Et tu sais à quoi correspond ce 7h ? Et si c'est toujours 7h ?

[GerardPaligot]

Je ne comprends pas parce que à vue de nez, le code semble cohérent :

• Récupère le dernier message non lu du sujet suivi : https://github.com/zestedesavoir/zds-site/blob/dev/zds/notification/management/commands/migrate_subscriptions.py#L28
• Ou le dernier message : https://github.com/zestedesavoir/zds-site/blob/dev/zds/notification/management/commands/migrate_subscriptions.py#L30
• Affectation de la date du message à la date de la notification : https://github.com/zestedesavoir/zds-site/blob/dev/zds/notification/management/commands/migrate_subscriptions.py#L41

Le problème est le même pour les notifications des contenus.

• Même chose pour les contenus : https://github.com/zestedesavoir/zds-site/blob/dev/zds/notification/management/commands/migrate_subscriptions.py#L113

[firm1]

Ah bah je viens d'inspecter et le public est plus restreint. le 07h correspond aux notifications que je n'ai pas lus JUSTE AVANT qui étaient activent avant la MEP.

Les anciens topics que je suivaient sont OK.

[ShiiFu]

J'ai eu le même problème aujourd'hui. J'ai lu toutes mes notifications ce matin et en début d'après midi je vois ceci :

Et en cliquant dessus je vois bien le soucis de concordance des dates. (Si je marque non lu, la notification revenu a toujours la mauvaise date)

et

En cliquant sur toutes les notifications j'ai bien les bonnes dates.

Pour précision ce sont des sujets suivis depuis un moment (donc avant la MEP de ce week-end). Si vous avez besoin de plus d'informations n'hésitez pas.

[DevHugo]

/blague Je suis célèbre, je tiens à remercier tout d'abord ma mére sans qui … o/

Même souci qu’au dessus un sujet souscrit avant la MEP. Joseph poste dedans y'a 45 minutes, la date de la notification. J'ai l’impression que c'est la date de l'avant dernière notification pour le sujet.

[firm1]

J'ai l’impression que c'est la date de l'avant dernière notification pour le sujet.

Dans mon cas ce n'est pas ça. Moi il me renvoit toujours a une notification trop vieille (difficile de l'identifier)

[cgabard]

Ce matin j'ai eu la meme chose qu'Hugo : Une notif mais la date était celle de l'avant dernier message (juste avant celui concerné par la notif)

[SpaceFox]

Je confirme que dans certains cas, la date de la notif est foireus.

2016-06-02 9:48 GMT+02:00 Christophe Gabard notifications@github.com:

Ce matin j'ai eu la meme chose qu'Hugo : Une notif mais la date était
celle de l'avant dernier message (juste avant celui concerné par la notif)

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#3628 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AFhKnJhHOy1JYGk8jU2b4JF1RTP4wpNyks5qHorZgaJpZM4IpA9Z
.

[GerardPaligot]

Ok, je pense avoir compris le bug de cette issue :

La valeur pubdate d'une notification est enregistrée automatiquement à la sauvegarde d'une nouvelle instance du modèle (comme en témoigne la déclaration du champ dans le modèle) mais comme nous réutilisons la notification existante si elle existe, nous ne mettons pas à jour sa valeur à l'envoi de la notification.

Ceci est théorique parce que non testé (toujours pas le temps en ce moment) mais fait sens.

Pour corriger le bug, il faut simplement mettre à jour la valeur de pubdate avec la valeur de content.pubdate et pour tester, il suffit de générer 2 notifications pour un même contenu notifiable (pensé à renseigner une valeur manuellement à la première notification pour éviter d'avoir la même date si Django est un rapide dans l'exécution du test).

[gustavi]

v18.2 inc