Tópicos mesclados não estão mais sendo atualizados

Parece que um bug entrou na versão 3.6.0.beta2-latest (e2eb9fe93f) que não estava presente nas atualizações anteriores que fiz na semana passada.

Diff, se ajudar:

Há um problema ao mesclar tópicos.

Anteriormente, o tópico que eu mesclava era promovido.

Isso não está mais acontecendo e ambos os tópicos desaparecem de vista.

Aqui está o tópico original que eu mesclei, 21 minutos atrás:

Vá para /latest e você poderá ver o tópico fechado, mas nenhum sinal do tópico em que o mesclei :person_shrugging:

Da mesma forma, indo para a categoria Travel, nenhum sinal do tópico sendo promovido aqui também :person_shrugging:

Se eu for ao tópico em que mesclei e clicar em Reset Bump Date, ele magicamente aparece em /latest com o timestamp correto da última postagem que foi mesclada nele.


Este problema não parece ocorrer se eu mesclar e marcar a caixa para preservar a ordem cronológica.

Tentei com contas de staff e moderador, consigo reproduzir todas as vezes com ambos os tipos de conta.

1 curtida

Suspeito que esteja relacionado a isto :thinking:

2 curtidas

Apenas para esclarecer o problema aqui com um cenário.

  • O usuário cria um novo tópico chamado “Tópico XYZ”
  • A equipe mescla esse novo tópico em um tópico existente chamado “Tópico ABC”
    – Organização padrão para evitar tópicos duplicados
  • “Tópico XYZ” desapareceu da visualização pública porque foi deslistado e mesclado
  • “Tópico ABC” não está visível em /latest (ou em qualquer outro lugar recentemente) porque não foi atualizado
  • Nenhum usuário está ciente de que um novo tópico / post foi criado (ou o que agora é tecnicamente uma resposta a um tópico mais antigo porque foi mesclado) porque foi movido para um tópico que pode ter um ano e não foi atualizado

Não tenho certeza se estou explicando bem isso :blush:

2 curtidas

Sim, acho que você está descrevendo bem o problema.

@christo Acho que precisamos considerar algumas combinações diferentes aqui:

Dado que estamos mesclando o tópico A (uma única postagem) no tópico B, existem os seguintes casos a serem considerados:

    1. A postagem do tópico A é mais recente que a última postagem do tópico B
    • a. Manter cronologia está selecionado
    • b. Manter cronologia não está selecionado
    1. A postagem do tópico A é mais antiga que a última postagem do tópico B
    • c. Manter cronologia está selecionado
    • d. Manter cronologia não está selecionado

Acho que o comportamento esperado é que o tópico seja atualizado em todos os casos, exceto no 2(c).

Em todos os casos, exceto no 2(c), a postagem do tópico A seria anexada ao final do tópico B, então acho que a mesclagem deve fazer com que a data de atualização seja definida com base em quando a mesclagem ocorreu.

Não tenho certeza do que eu esperaria que acontecesse no caso 2(d), se o administrador mais tarde fizer “reiniciar data de atualização”. Acho que provavelmente deveria atualizar a data de atualização para ser a data da postagem criada mais recentemente no tópico (por exemplo, a penúltima postagem).

@martin você e eu temos pensado na lógica de data de atualização em outros cenários - isso faz sentido para você?

2 curtidas

Sim, com base unicamente no fato de sempre ter sido assim até as atualizações que instalei esta semana.

Novamente, o problema que temos agora é que novos tópicos/posts simplesmente desaparecem no ar, pois não estão mais impulsionando o tópico em que foram mesclados – o que sempre aconteceu no passado.

Concordo.

1 curtida

Sim, acho que faz sentido :+1:

2 curtidas

Sim, parece que este foi um bug que introduzi no commit vinculado acima. Eu pretendia corrigir um problema em que a data bumped_at às vezes era retrocedida no tempo quando uma primeira postagem mais antiga que a postagem mais recente no tópico de destino era mesclada. Mas parece que também quebrei o caso em que a primeira postagem é mais nova.

Tenho uma correção em andamento e atualizarei aqui quando for resolvida.

1 curtida

Aqui está o PR para corrigir este problema. Atualizarei novamente assim que for aprovado e mesclado.

3 curtidas

Obrigado pela correção rápida @christo :person_bowing:

3 curtidas

Ok, este conserto foi mesclado. Deve voltar a funcionar como pretendido aqui.

2 curtidas