No último resumo que uma das minhas contas recebeu, nenhum dos tópicos foi duplicado, mas os comentários abaixo deles foram (“Posts Populares”). Havia cinco espaços e duas duplicatas (três posts únicos), ambas do mesmo tópico.
Editar: Estou analisando alguns resumos históricos que foram enviados para diferentes contas de teste/administrador:
O anterior também tem duas duplicatas nos Posts Populares, também desse mesmo tópico.
O anterior a esse tinha um tópico duplicado, mas nenhum Post Popular duplicado. Há uma duplicata em “Novidades para você”.
@j127, seria interessante saber quantas tags seus tópicos de resumo tinham. Em minhas amostras recentes, a duplicação corresponde ao número de tags em um tópico. Pode ser coincidência — ou não.
Consegui reproduzir o problema em meu site Discourse local adicionando uma tag à configuração do site digest suppress tags e, em seguida, criando um tópico com várias tags.
if SiteSetting.digest_suppress_tags.present?
tag_ids = Tag.where_name(SiteSetting.digest_suppress_tags.split("|")).pluck(:id)
if tag_ids.present?
topics =
topics.joins("LEFT JOIN topic_tags tg ON topics.id = tg.topic_id").where(
"tg.tag_id NOT IN (?) OR tg.tag_id IS NULL",
tag_ids,
)
end
end
Editar: Não acho que um LEFT JOIN seja necessário aqui. As colunas da tabela topic_tags unida não parecem ser usadas posteriormente no método. A correção pode ser tão simples quanto:
if SiteSetting.digest_suppress_tags.present?
tag_ids = Tag.where_name(SiteSetting.digest_suppress_tags.split("|")).pluck(:id)
if tag_ids.present?
topics =
topics.where.not(id: TopicTag.where(tag_id: tag_ids).select(:topic_id))
end
end
Deixarei para a equipe do Discourse descobrir a melhor abordagem. O método for_digest é executado muitas vezes e precisa ser eficiente.
@simon – muito obrigado por investigar isso! Tenho certeza de que deu muito trabalho analisar. Aguardarei o retorno da equipe do Discourse e a ação sobre sua análise.
Não está totalmente claro quando os commits mesclados estarão disponíveis no canal beta. Se eu estiver na versão 3.4.0.beta3-dev (53f9c81790) e ela não estiver solicitando uma atualização da GUI, uma atualização pela linha de comando captará essa alteração?
Tenho certeza de que deu muito trabalho analisar. Aguardarei o retorno da equipe do Discourse e a ação sobre sua análise.
Esse era de fato o bug