Gli argomenti uniti non vengono più visualizzati in cima

Sembra che un bug sia entrato nella versione 3.6.0.beta2-latest (e2eb9fe93f) che non era presente negli aggiornamenti precedenti che ho effettuato la scorsa settimana.

Diff, se può essere utile:

C’è un problema durante l’unione degli argomenti.

In precedenza, l’argomento in cui mi univo veniva aggiornato.

Questo non sta più accadendo e entrambi gli argomenti scompaiono dalla vista.

Ecco l’argomento originale che ho unito, 21 minuti fa:

Vai su /latest e puoi vedere l’argomento chiuso, ma nessun segno dell’argomento in cui l’ho unito :person_shrugging:

Allo stesso modo, andando nella categoria Travel, nessun segno dell’argomento che viene aggiornato qui :person_shrugging:

Se vado all’argomento in cui mi sono unito, quindi Reset Bump Date, esso appare magicamente in /latest all’ora corretta dell’ultimo post che è stato unito ad esso.


Questo problema non sembra verificarsi se unisco e seleziono la casella per preservare l’ordine cronologico.

Ho provato sia con account staff che con account moderatore, posso replicare ogni volta con entrambi i tipi di account.

1 Mi Piace

Sospetto sia correlato a questo :thinking:

2 Mi Piace

Giusto per chiarire il problema qui con uno scenario.

  • L’utente crea un nuovo argomento chiamato “Argomento XYZ”
  • Lo staff unisce quel nuovo argomento in un argomento esistente chiamato “Argomento ABC”
    – Pulizia standard per prevenire argomenti duplicati
  • “Argomento XYZ” è ora scomparso dalla vista pubblica perché è stato non elencato e unito
  • “Argomento ABC” non è visibile in /latest (o altrove di recente) perché non è stato aggiornato
  • Nessun utente è nemmeno a conoscenza che un nuovo argomento / post sia mai stato creato (o cosa sia ora tecnicamente una risposta a un argomento più vecchio perché è stato unito) perché è stato spostato in un argomento che potrebbe avere un anno e non è stato aggiornato

Non sono sicuro di quanto bene lo stia spiegando :blush:

2 Mi Piace

Sì, penso che tu stia descrivendo bene il problema.

@christo Penso che dobbiamo considerare alcune combinazioni diverse qui:

Dato che stiamo unendo l’argomento A (un singolo post) nell’argomento B, ci sono i seguenti casi da considerare:

    1. il post dell’argomento A è più recente dell’ultimo post dell’argomento B
    • a. viene selezionata la conservazione della cronologia
    • b. la conservazione della cronologia non è selezionata
    1. il post dell’argomento A è più vecchio dell’ultimo post dell’argomento B
    • c. viene selezionata la conservazione della cronologia
    • d. la conservazione della cronologia non è selezionata

Penso che il comportamento previsto sia che l’argomento venga aggiornato in tutti i casi, tranne nel caso 2(c).

In ogni caso tranne 2(c), il post dall’argomento A verrebbe aggiunto alla fine dell’argomento B, quindi penso che l’unione dovrebbe impostare la data di aggiornamento in base a quando è avvenuta l’unione.

Non sono esattamente sicuro di cosa mi aspetterei che accadesse nel caso 2(d), se l’amministratore in seguito eseguisse “reimposta data di aggiornamento”. Penso che probabilmente dovrebbe aggiornare la data di aggiornamento alla data del post creato più di recente nell’argomento (ad esempio, il penultimo post).

@martin tu ed io abbiamo pensato alla logica della data di aggiornamento in altri scenari: quanto sopra ha senso per te?

2 Mi Piace

Sì, basandomi esclusivamente sul fatto che è sempre stato così fino agli aggiornamenti che ho installato questa settimana.

Ancora una volta, il problema che abbiamo ora è che i nuovi argomenti/post semplicemente svaniscono nel nulla poiché non incrementano più l’argomento in cui sono stati uniti, cosa che è sempre accaduta in passato.

Sono d’accordo.

1 Mi Piace

Sì, penso che abbia senso :+1:

2 Mi Piace

Sì, sembra che questo fosse un bug che ho introdotto nel commit collegato sopra. Avevo intenzione di correggere un problema per cui la data bumped_at a volte veniva retrocessa nel tempo quando un primo post più vecchio dell’ultimo post nell’argomento di destinazione veniva unito. Ma sembra che abbia anche rotto il caso in cui il primo post è più recente.

Ho una correzione in corso e aggiornerò qui quando sarà risolta.

1 Mi Piace

Ecco la PR per correggere questo problema. Aggiornerò di nuovo una volta approvata e unita.

3 Mi Piace

Grazie per la rapida correzione @christo :person_bowing:

3 Mi Piace

Va bene, questa correzione è stata unita. Dovrebbe tornare a funzionare come previsto qui.

2 Mi Piace