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 curtidas

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 curtidas

Isso acontece after_save, mas usa will_save_change, então tenho certeza de que nunca é acionado. Parece que funcionava antes, mas este commit em 2017 (presumo que por engano) mudou de _changed para will_save_change.

3 curtidas

Acho que temos alguns tópicos de Bug nisso também:

4 curtidas

Obrigado @dsims por encontrar a causa raiz deste problema! Criei um PR com uma correção.

4 curtidas

Este tópico foi automaticamente fechado após 4 dias. Novas respostas não são mais permitidas.