Los temas fusionados ya no se actualizan

Parece que un error se ha introducido en 3.6.0.beta2-latest (e2eb9fe93f) que no estaba presente en las actualizaciones anteriores que hice la semana pasada.

Diferencia, si ayuda:

Hay un problema al fusionar temas.

Anteriormente, el tema en el que me fusiono se actualizaba.

Esto ya no sucede y ambos temas desaparecen de la vista.

Aquí está el tema original que fusioné, hace 21 minutos:

Ve a /latest y podrás ver el tema cerrado, pero ni rastro del tema en el que lo fusioné :person_shrugging:

Del mismo modo, al ir a la categoría Travel, tampoco hay rastro de que el tema se haya actualizado aquí :person_shrugging:

Si voy al tema en el que me fusioné, luego a Reset Bump Date, mágicamente aparece en /latest con la marca de tiempo correcta de la última publicación que se fusionó en él.


Este problema no parece ocurrir si fusiono y marco la casilla para preservar el orden cronológico.

He intentado con cuentas de personal y moderador, puedo replicarlo cada vez con ambos tipos de cuenta.

1 me gusta

Sospecho que está relacionado con esto :thinking:

2 Me gusta

Solo para aclarar el problema aquí con un escenario.

  • El usuario crea un nuevo tema llamado “Tema XYZ”
  • El personal fusiona ese nuevo tema en un tema existente llamado “Tema ABC”
    – Limpieza estándar para evitar temas duplicados
  • “Tema XYZ” ha desaparecido de la vista pública porque no ha sido listado y se ha fusionado
  • “Tema ABC” no es visible en /latest (o en ningún otro lugar recientemente) porque no ha sido actualizado
  • Ningún usuario es siquiera consciente de que se creó un nuevo tema / publicación (o lo que ahora es técnicamente una respuesta a un tema más antiguo porque se fusionó) porque se ha movido a un tema que podría tener un año y no se actualizó

No estoy seguro de lo bien que lo explico :blush:

2 Me gusta

Sí, creo que estás describiendo bien el problema.

@christo Creo que debemos considerar algunas combinaciones diferentes aquí:

Dado que estamos fusionando el tema A (una sola publicación) en el tema B, hay los siguientes casos a considerar:

    1. La publicación del tema A es más reciente que la última publicación del tema B
    • a. se selecciona mantener la cronología
    • b. no se selecciona mantener la cronología
    1. La publicación del tema A es anterior a la última publicación del tema B
    • c. se selecciona mantener la cronología
    • d. no se selecciona mantener la cronología

Creo que el comportamiento esperado es que el tema se actualice en todos los casos, excepto en el 2(c).

En todos los casos excepto en el 2(c), la publicación del tema A se agregaría al final del tema B, por lo que creo que la fusión debería establecer la fecha de actualización en función de cuándo ocurrió la fusión.

Sin embargo, no estoy seguro de lo que esperaría que sucediera en el caso 2(d), si el administrador luego realiza “restablecer fecha de actualización”. Creo que probablemente debería actualizar la fecha de actualización a la fecha de la publicación creada más recientemente en el tema (por ejemplo, la penúltima publicación).

@martin tú y yo hemos estado pensando en la lógica de la fecha de actualización en otros escenarios: ¿tiene sentido para ti lo anterior?

2 Me gusta

Sí, basándome únicamente en que siempre ha sido así hasta las actualizaciones que instalé esta semana.

De nuevo, el problema que tenemos ahora es que los nuevos temas/publicaciones simplemente desaparecen en el aire, ya que ya no impulsan el tema en el que se fusionaron, donde siempre lo han hecho en el pasado.

Estoy de acuerdo.

1 me gusta

Sí, creo que tiene sentido :+1:

2 Me gusta

Sí, parece que este fue un error que introduje en el commit enlazado anteriormente. Tenía la intención de solucionar un problema en el que la fecha bumped_at a veces se retrasaba en el tiempo cuando se fusionaba una primera publicación más antigua que la publicación más reciente en el tema de destino. Pero parece que también rompí el caso en el que la primera publicación es más nueva.

Tengo una solución en progreso y actualizaré aquí cuando se resuelva.

1 me gusta

Aquí está el PR para solucionar este problema. Volveré a actualizar una vez que sea aprobado y fusionado.

3 Me gusta

Gracias por la rápida solución @christo :person_bowing:

3 Me gusta

Bien, esta corrección se ha fusionado. Debería volver a funcionar como se esperaba aquí.

2 Me gusta