Closed mgkprod closed 5 years ago
innocenzi
commented
5 years ago 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 :
mgkprod
commented
5 years ago 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.
innocenzi
commented
5 years ago 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. :/
mgkprod
commented
5 years ago https://github.com/4sucres/board/issues/44#issuecomment-494426705
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
innocenzi
commented
5 years ago Est-ce que ces options peuvent aider ?
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)