TASVideos / tasvideos

The code for the live TASVideos website
https://tasvideos.org/
GNU General Public License v3.0
62 stars 29 forks source link

Posts moved to restricted subforum are still shown as most recent for their original subforum #1696

Closed YoshiRulz closed 11 months ago

YoshiRulz commented 12 months ago

edit: Turns out this was a bug in the spam button for forum mods.

Both "Last Post" and the unread badge are wrong.

Found during circumstances of #1694.

"Mark all Posts as read" on the General subforum didn't work, but the global one did work.

adelikat commented 12 months ago

This is just the fallout from a failed attempt to remove the topic from a spam operation

SamsaraTAS commented 11 months ago

This is still happening with the spam button.

Masterjun3 commented 11 months ago

The issue is our denormalization with our post<->subforum relation. When pressing the spam button, we only set the new topic of the post as seen below. But we also need to set the subforum of the post. https://github.com/TASVideos/tasvideos/blob/6c29e17e2ca5c97680053abb749596846e4ffb86/TASVideos/Pages/Forum/Posts/Edit.cshtml.cs#L279