sucresware/4sucres

Ancienneté dans le profil

mgkprod opened this issue · 5 comments

image

L'affichage de l'ancienneté dans le profil n'est pas bon pour les tous membres inscrits dans les dernières 24h. C'est une erreur d'affichage.

En effet, le calcul actuel est : Si le membre est inscrit depuis moins d'une journée, on affiche "aujourd'hui"
Or, il faudrait : Si le membre est inscrit depuis le début de la journée, alors on affiche aujourd'hui
(il faudra faire attention aussi à "Membre depuis 0 jours" qui pourra apparaître)

Pour l'instant la modification s'effectue ici en attendant un refacto pour tout bouger dans le controller :

https://github.com/4sucres/board/blob/6b911edf1f7a1f4f2a296c879b075ac8e2555a6a/resources/views/user/show.blade.php#L33

(l'image c'est uniquement pour le contexte, le "bug" n'est pas présent dessus)

Je pense qu'il faut éviter de faire ça à la main, parce que des librairies existent précisément pour ce genre de problème récurrent.

Ma référence dans Symfony est intégrée à Twig, mais en PHP, Carbon est connue.

La fonctionnalité qui nous intéresse est là : https://carbon.nesbot.com/docs/#api-humandiff.

Autres ressources :

Il me semblait important de l'afficher en "jours", le choix est volontaire.
Évidemment, diffForHumans() est déjà utilisé un peu partout et Carbon est configuré a la bon heure et en français.

Il me semblait important de l'afficher en "jours", le choix est volontaire.

Comme ça ?

echo Carbon::now()->subDays(5)->diffForHumans(); // 5 days ago

Évidemment, diffForHumans() est déjà utilisé un peu partout et Carbon est configuré a la bon heure et en français.

D'accord, autant pour moi, j'ai pas vérifié. Du coup je crois que mes suggestions sont hors-sujet. :/

#44 (comment)
Comment il se comporte au bout de quelques mois, quelques années ? Il me semble qu'on perds la précision et qu'on arrive dans des x months ago

Est-ce que ces options peuvent aider ?