When moving a solved topic to a category that disallows solutions, remove solved

AI-generated summary

The discussion addresses a bug where moving a solved topic to a category that disallows solutions failed to remove the “solved” status. codinghorror initially reported the issue, noting that topics moved out of categories with solutions retained their solved markers. sam agreed to the cleanup, and bekicot submitted a PR adding a plugin hook to handle category changes. tgxworld suggested using a model callback instead of a new event trigger. fhe and nildarar confirmed the issue persisted, explaining that admins could not uncheck the solution after moving the topic to an incompatible category. lindsey initially claimed a fix was in place, but Moin clarified that the previous fix only addressed moving individual posts, not entire topics. lindsey acknowledged the misunderstanding and lindsey committed to fixing the topic-level issue. Finally, lindsey confirmed the behavior was updated: when a solved topic is moved to a category without solution support, the accepted solution is automatically removed, and admins receive a notification before the move is completed.

I’ve seen this too when moving topics out of a category where solved was set, any way to fix this @sam?

  • When moving a topic out of a category with solutions, remove solved stuff

  • When moving a topic into a category with solution, make solved possible

2 curtidas

Agree, we should get this cleaned up, for now pr-welcome

4 curtidas

Added Trigger DiscourseEvent when topic category is changed by bekicot · Pull Request #4785 · discourse/discourse · GitHub, so that plugin can hook into :topic_category_changed. I think it is required in order to fix this issue.

5 curtidas

Thanks for the PR @bekicot :thumbsup:

I was just curious but is it possible to avoid adding a DiscourseEvent trigger and just add a model callback which checks when the category_id has been changed?

2 curtidas

Seems like this is still happening.

A topic marked as solution which is moved out of a category with solutions remains marked as solution (and cannot be unchecked as such)

1 curtida

Quando uma das respostas é marcada incorretamente como a resposta correta, o administrador percebe que a categoria está errada e move o tópico para outra categoria. A resposta marcada continua sendo exibida como a resposta correta, mesmo que esse recurso não esteja habilitado na nova categoria.

Nesses casos, o administrador não pode cancelar a resposta marcada. Deve-se alterar a categoria para aquela em que esse recurso está habilitado, cancelar a resposta correta e, em seguida, alterar a categoria novamente.

Seria um pouco mais fácil se os gestores pudessem cancelar a resposta correta em todas as categorias (mesmo naquelas com esse recurso desativado).

Abordamos isso garantindo que, se uma postagem de solução for movida para fora do tópico, o tópico retorne ao estado não resolvido.

O usuário que está movendo a postagem de solução é notificado sobre o impacto antes de concluir a ação de mover:

4 curtidas

Para mim, isso parece ser uma correção para Carelessly splitting a solved post to a new topic breaks things

Parece que não corrige o problema descrito na publicação original, que fala sobre um tópico inteiro sendo movido para outra categoria onde as soluções não estão habilitadas, e não sobre mover algumas postagens para outro tópico. Quando movo `personal_messages:username` search filter doesn't work, que é um tópico resolvido, para Contribute > Feature, onde as soluções não são permitidas, a solução não é removida e não há nenhum aviso.

Isso aborda este problema:

Mas eu posso ver que há mais no problema, então podemos mantê-lo aberto.

Hmm, ainda não entendi. Não foi isso que acabei de tentar?

Um tópico marcado como solução

Escolhi este tópico que estava marcado como resolvido `personal_messages:username` search filter doesn't work

que foi movido para fora de uma categoria com soluções

Eu o movi para Contribute > Feature, onde as soluções não estão habilitadas

permanece marcado como solução (e não pode ser desmarcado como tal)

E ainda estava marcado como resolvido. O que a correção mudou nesse processo?

2 curtidas

Desculpe, agora eu entendi o que você está dizendo. Eu estava lendo isso errado, como uma postagem marcada como solução que é movida para fora de uma categoria.

Vou colocar esta na fila para consertar, pois acho que deve ser uma pequena melhoria no trabalho que fizemos com a postagem resolvida.

3 curtidas

Atualizamos esse comportamento para que, quando um tópico resolvido for movido para uma categoria que não suporte soluções, a solução aceita será removida do tópico. Notificaremos os administradores sobre isso antes que eles concluam a movimentação:

5 curtidas

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