Zusammengeführte Themen werden nicht mehr nach oben verschoben

Ein Fehler scheint sich in 3.6.0.beta2-latest (e2eb9fe93f) eingeschlichen zu haben, der in den vorherigen Updates, die ich letzte Woche vorgenommen habe, nicht vorhanden war.

Diff, falls es hilft:

Es gibt ein Problem beim Zusammenführen von Themen.

Zuvor wurde das Thema, in das ich zusammengeführt habe, nach oben verschoben.

Dies geschieht nun nicht mehr und beide Themen verschwinden aus dem Blickfeld.

Hier ist das ursprüngliche Thema, das ich vor 21 Minuten zusammengeführt habe:

Gehen Sie zu /latest und Sie können das geschlossene Thema sehen, aber keine Spur von dem Thema, in das ich es zusammengeführt habe :person_shrugging:

Ebenso gibt es in der Kategorie Travel keine Spur davon, dass das Thema hier nach oben verschoben wird :person_shrugging:

Wenn ich zu dem Thema gehe, in das ich zusammengeführt habe, und dann Reset Bump Date auswähle, erscheint es magischerweise in /latest mit dem korrekten Zeitstempel des letzten Beitrags, der in dieses Thema zusammengeführt wurde.


Dieses Problem scheint nicht aufzutreten, wenn ich zusammenführe und das Kontrollkästchen zum Erhalten der chronologischen Reihenfolge aktiviere.

Ich habe sowohl Staff- als auch Moderatorenkonten ausprobiert, kann es mit beiden Kontotypen jedes Mal reproduzieren.

1 „Gefällt mir“

Ich vermute, es hat damit zu tun :thinking:

2 „Gefällt mir“

Nur um das Problem hier mit einem Szenario zu verdeutlichen.

  • Ein Benutzer erstellt ein neues Thema namens „Thema XYZ“
  • Ein Mitarbeiter führt dieses neue Thema in ein bestehendes Thema namens „ABC Thema“ zusammen.
    – Standard-Housekeeping zur Vermeidung doppelter Themen
  • „Thema XYZ“ ist nun aus der öffentlichen Ansicht verschwunden, da es nicht mehr aufgeführt und zusammengeführt wurde.
  • „ABC Thema“ ist nicht in /latest (oder anderswo kürzlich) sichtbar, da es nicht nach oben verschoben wurde.
  • Kein Benutzer weiß, dass jemals ein neues Thema/Beitrag erstellt wurde (oder was jetzt technisch eine Antwort auf ein älteres Thema ist, da es zusammengeführt wurde), da es in ein Thema verschoben wurde, das möglicherweise ein Jahr alt ist und nicht nach oben verschoben wurde.

Ich bin mir nicht sicher, wie gut ich das erkläre :blush:

2 „Gefällt mir“

Ja, ich denke, Sie beschreiben das Problem gut.

@christo Ich denke, wir müssen hier ein paar verschiedene Kombinationen berücksichtigen:

Da wir Thema A (einen einzelnen Beitrag) in Thema B zusammenführen, sind die folgenden Fälle zu berücksichtigen:

    1. Der Beitrag von Thema A ist neuer als der letzte Beitrag von Thema B
    • a. Chronologie beibehalten ist ausgewählt
    • b. Chronologie beibehalten ist nicht ausgewählt
    1. Der Beitrag von Thema A ist älter als der letzte Beitrag von Thema B
    • c. Chronologie beibehalten ist ausgewählt
    • d. Chronologie beibehalten ist nicht ausgewählt

Ich denke, das erwartete Verhalten ist, dass das Thema in allen Fällen hochgestuft wird, außer in 2(c).

In jedem Fall außer 2(c) wird der Beitrag von Thema A am Ende von Thema B angehängt, daher denke ich, dass die Zusammenführung das Hochstufungsdatum basierend auf dem Zeitpunkt des Zusammenführens festlegen sollte.

Ich bin mir jedoch nicht ganz sicher, was in Fall 2(d) passieren würde, wenn der Administrator später „Hochstufungsdatum zurücksetzen“ ausführt. Ich denke, es sollte wahrscheinlich das Hochstufungsdatum auf das Datum des zuletzt erstellten Beitrags im Thema aktualisieren (z. B. den vorletzten Beitrag).

@martin Sie und ich haben über die Logik des Hochstufungsdatums in anderen Szenarien nachgedacht – ergibt das oben Sinn für Sie?

2 „Gefällt mir“

Ja, allein schon deshalb, weil es bis zu den Updates, die ich diese Woche installiert habe, immer so war.

Wieder einmal ist das Problem, das wir jetzt haben, dass neue Themen/Beiträge einfach in dünner Luft verschwinden, da sie das Thema, in das sie zusammengeführt wurden, nicht mehr hochstufen – was sie in der Vergangenheit immer getan haben.

Dem stimme ich zu.

1 „Gefällt mir“

Ja, das ergibt Sinn :+1:

2 „Gefällt mir“

Ja, das sieht so aus, als wäre das ein Fehler, den ich in dem oben verlinkten Commit eingeführt habe. Ich wollte ein Problem beheben, bei dem das bumped_at-Datum manchmal in der Zeit zurückgeschoben wurde, wenn ein erster Beitrag, der älter als der neueste Beitrag im Zielthema war, zusammengeführt wurde. Aber es sieht so aus, als hätte ich auch den Fall kaputt gemacht, bei dem der erste Beitrag neuer ist.

Ich arbeite gerade an einer Lösung dafür und werde hier Bescheid geben, wenn das Problem behoben ist.

1 „Gefällt mir“

Hier ist der PR zur Behebung dieses Problems. Ich werde erneut aktualisieren, sobald er genehmigt und zusammengeführt wurde.

3 „Gefällt mir“

Danke für die schnelle Korrektur @christo :person_bowing:

3 „Gefällt mir“

In Ordnung, dieser Fix wurde zusammengeführt. Sollte hier wieder wie vorgesehen funktionieren.

2 „Gefällt mir“