Old user avatars in quoted text?

I’m not sure if this is quite a bug, or just some behaviour that I don’t quite get, but it appears that quoting someone in a post stores their avatar at that point in time with the quote. Like here:

It seems like this might be a regression of this thing:
https://meta.discourse.org/t/after-changing-avatar-quotes-still-show-the-old-avatar/21122

Not a bug; it’s always been that way.

2 « J'aime »

I suppose its because the posts are baked with the URL at the time. Do you want a fix for this, or is it too potentially resource intensive to trigger a rebake each time someone changes their avatar?

Changing the avatar already triggers a rebake of posts containing quotes, but it’s currently quite resource intensive to find and rebake all posts. So, it’s possible that some of them won’t be updated. Post revisions aren’t updated at all.

I have plans to fix all of this when I’m working on improvements for user renaming/anonymizing in one of the next releases. It will require a new table that keeps track of all the places where a user is quoted, mentioned, oneboxed, and what not.

If you want to work on this, please keep in mind that rebaking just to update the avatar isn’t needed. I’d suggest reusing code from UpdatUsername. It’s a lot faster.

8 « J'aime »

Cela se produit after_save mais utilise will_save_change, donc je suis à peu près sûr que cela ne se déclenche jamais. Il semble que cela fonctionnait avant, mais ce commit de 2017 (par erreur, je suppose) est passé de _changed à will_save_change.

3 « J'aime »

Je pense que nous avons également quelques sujets de Bug à ce sujet :

4 « J'aime »

Merci @dsims d’avoir trouvé la cause première de ce problème ! J’ai créé une PR avec un correctif.

4 « J'aime »

Ce sujet a été automatiquement fermé après 4 jours. Les nouvelles réponses ne sont plus autorisées.