Ersteller statt letztem Antwortendem lesen

Ein Antrag hing bei einem Spieler, der ihn weder geschrieben noch
kommentiert hatte. Die Ursache steht im HTML der Boarduebersicht: Der
Ersteller "-Daniel-" hat sein Konto geloescht und steht dort ohne
Profil-Link. Der einzige Profil-Link der Zeile gehoerte damit dem letzten
Antwortenden - und genau den nahm der Parser, weil er nach dem ersten Treffer
im ganzen Zeilenblock suchte.

Jetzt wird die Zeile vor der Spalte "letzter Beitrag" abgeschnitten und
zuerst in der Zelle gesucht, die den Ersteller fuehrt. Erst wenn dort kein
Link steht, gilt der blosse Text - so wie es bei einem geloeschten Konto der
Fall ist. Auf vier Stichprobenseiten des Archivs trifft das fuenf von achtzig
Threads; hochgerechnet liegen im Bestand also rund drei Dutzend falsch.

Die falschen Zuordnungen bleiben nicht stehen: Aendert sich der Ersteller
eines bekannten Threads, wird seine Zuordnung neu bestimmt - aber nur, wenn
die bisherige von selbst zustande kam. Wer von Hand zugeordnet hat, wusste
mehr als der Abgleich, das bleibt unangetastet. Die Abschlussmeldung nennt,
wie viele Threads dabei gewandert sind.

server/src/lib/forum.ts

@@ -45,11 +45,23 @@ const TOPIC_LINK =
 const USER_LINK =
   /<a\s+href="([^"]*user\/(\d+)-[^"]*)"[^>]*class="userLink"[^>]*>(?:<i[^>]*><\/i>)?\s*(?:<span[^>]*>)?\s*([^<]{1,80})/;
 
-// Wer sein Konto gelöscht hat, steht ohne Profillink in der Zelle. Der Name
-// ist trotzdem da und bleibt die einzige Spur zum Spieler - ohne diesen
-// Rückfall fallen solche Threads bei der Zuordnung durchs Raster.
+/**
+ * Die Zelle mit dem Ersteller. Dort steht er verlässlich - mit Profillink,
+ * wenn es das Konto noch gibt, sonst als blosser Text.
+ */
 const AUTHOR_CELL = /<li class="messageGroupAuthor">([\s\S]{0,400}?)<\/li>/;
 
+/**
+ * Ab hier beginnt die Spalte mit dem **letzten Beitrag**.
+ *
+ * Dort steht ein anderer Name, und wer ihn erwischt, ordnet den Antrag dem
+ * falschen Spieler zu. Das passiert genau dann, wenn der Ersteller sein Konto
+ * geloescht hat: er hat dann keinen Profillink, der letzte Antwortende schon.
+ * Deshalb wird die Zeile vorher abgeschnitten, statt sich auf die Reihenfolge
+ * der Treffer zu verlassen.
+ */
+const LAST_POST_COLUMN = /<li class="columnLastPost"/;
+
 // Ein zurückgelassener Verweis ist kein eigener Thread.
 const MOVED_MARKER = /wbbTopicLinkMoved|class="[^"]*movedTopic|>\s*Verschoben\s*</i;
 
@@ -86,11 +98,17 @@ export function parseBoard(html: string): ForumThread[] {
     if (!Number.isInteger(threadId) || seen.has(threadId)) continue;
     seen.add(threadId);
 
-    const user = USER_LINK.exec(block);
+    // Alles ab der Spalte "letzter Beitrag" gehört nicht mehr zum Ersteller.
+    const kopf = block.split(LAST_POST_COLUMN)[0] ?? block;
+
+    // Zuerst die Zelle, die den Ersteller führt. Erst wenn es sie nicht gibt,
+    // wird weiter oben gesucht - etwa im Avatar, der ebenfalls auf ihn zeigt.
+    const zelle = AUTHOR_CELL.exec(kopf);
+    const user = USER_LINK.exec(zelle?.[1] ?? kopf);
+
     let authorName = user ? decode(user[3] ?? '') || null : null;
-    if (authorName === null) {
-      const cell = AUTHOR_CELL.exec(block);
-      if (cell) authorName = decode((cell[1] ?? '').replace(/<[^>]*>/g, ' ')) || null;
+    if (authorName === null && zelle) {
+      authorName = decode((zelle[1] ?? '').replace(/<[^>]*>/g, ' ')) || null;
     }
 
     LABEL.lastIndex = 0;

server/src/services/forum.ts

@@ -110,6 +110,9 @@ export interface ForumScanSummary {
   /// in der Arbeitsliste, gehen aber auch nicht verloren.
   unassignedThreads: number;
   matchedPlayers: number;
+  /// Threads, deren Ersteller sich geändert hat und die deshalb einem anderen
+  /// Spieler zugeschlagen wurden.
+  reassigned: number;
   /// Wie oft das Forum den Abruf gebremst hat. Über null heißt: der Lauf hat
   /// länger gedauert, gelesen wurde trotzdem alles.
   rateLimited: number;
@@ -230,6 +233,7 @@ async function scanIntern(ctx: ActionContext): Promise<ForumScanSummary> {
     skippedLabels: 0,
     unassignedThreads: 0,
     matchedPlayers: 0,
+    reassigned: 0,
     rateLimited: 0,
     failedBoards: [],
   };
@@ -364,6 +368,31 @@ async function scanIntern(ctx: ActionContext): Promise<ForumScanSummary> {
     // Eine später nachgetragene Zuordnung nicht wieder überschreiben.
     const nachgetragen = existing.playerId === null && playerId !== null;
 
+    /**
+     * Ein anderer Ersteller als beim letzten Mal.
+     *
+     * Das kommt vor, wenn jemand sich im Forum umbenennt - und es kam vor,
+     * weil der Ersteller einmal falsch gelesen wurde: bei einem geloeschten
+     * Konto stand in der Zeile nur noch der letzte Antwortende mit Profil,
+     * und der landete als Ersteller in den Daten. Solche Threads hingen
+     * anschliessend beim falschen Spieler.
+     *
+     * Korrigiert wird nur, was von selbst zustande kam: wenn die bisherige
+     * Zuordnung genau die ist, die der alte Name ergeben haette. Eine von
+     * Hand gesetzte bleibt stehen - wer sie gesetzt hat, wusste mehr als der
+     * Abgleich.
+     */
+    const autorGeaendert = existing.authorName !== thread.authorName;
+    const zuordnungWarAutomatisch =
+      existing.playerId !== null &&
+      existing.authorName !== null &&
+      byForumName.get(existing.authorName.toLowerCase()) === existing.playerId;
+    const zuordnungKorrigieren =
+      autorGeaendert &&
+      zuordnungWarAutomatisch &&
+      existing.needsClassification &&
+      existing.playerId !== playerId;
+
     // Das Antragsdatum kam erst später dazu; bestehende Threads holen es hier
     // nach, ohne dass ein vorhandener Wert überschrieben wird.
     const datumFehlt = existing.postedAt === null && thread.postedAt !== null;
@@ -373,6 +402,7 @@ async function scanIntern(ctx: ActionContext): Promise<ForumScanSummary> {
     const geaendert =
       verschoben ||
       nachgetragen ||
+      zuordnungKorrigieren ||
       datumFehlt ||
       profilFehlt ||
       existing.title !== thread.title ||
@@ -394,6 +424,11 @@ async function scanIntern(ctx: ActionContext): Promise<ForumScanSummary> {
         ...(datumFehlt ? { postedAt: thread.postedAt } : {}),
         ...(profilFehlt ? { authorProfileUrl: thread.authorProfileUrl } : {}),
         ...(nachgetragen ? { player: { connect: { id: playerId as number } } } : {}),
+        ...(zuordnungKorrigieren
+          ? playerId === null
+            ? { player: { disconnect: true } }
+            : { player: { connect: { id: playerId } } }
+          : {}),
         ...(verschoben
           ? {
               currentBoard: board,
@@ -406,6 +441,8 @@ async function scanIntern(ctx: ActionContext): Promise<ForumScanSummary> {
       },
     });
 
+    if (zuordnungKorrigieren) summary.reassigned += 1;
+
     if (verschoben) {
       historie.push({ threadId: thread.threadId, board });
       summary.movedThreads += 1;