From 210e4a1b417e1cf5cc619205db8260b3f3ac2e21 Mon Sep 17 00:00:00 2001
From: Eduard Wisch <data@data-it-solution.de>
Date: Sat, 15 Aug 2026 00:40:08 +0200
Subject: [PATCH] Dauertest strukturiert darstellen (Web-Karte + PDF) statt
 Fliesstext
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Der Dauertest liefert seit App-Phase 2 deutlich mehr Ergebnisfelder
(Zeitreihe in Minuten-Buckets, Ausfallsegmente, p95, laengster Ausfall).
netdiagFormatResult() (Web-Karte) und netdiagPdfFlattenResult() (PDF) wuerden
das zu einem einzigen Fliesstext zusammenkleben — die Web-Karte kuerzt jeden
Wert zusaetzlich per dol_trunc(...,200), eine Ausfallliste mit mehreren
Eintraegen waere also mitten im Satz abgeschnitten.

- netdiagFormatResult($json, $tool='') bekommt einen optionalen zweiten
  Parameter; fuer $tool==='stresstest' greift netdiagFormatStressTest() statt
  der generischen Kuerzung: Kennzahlenzeile (Ziel, Dauer, Takt, Proben,
  Verlust, Oe/Min/Max/p95, laengster Ausfall) + eine echte <ul>-Ausfallliste,
  ungekuerzt. netdiagprotocol_card.php uebergibt jetzt $m->tool.
- netdiagPdfStressTest() im PDF-Generator macht dasselbe mit TCPDF-Zellen:
  Kennzahlenzeile + Ausfalltabelle als eigene Zeilen. Die Zeitreihe
  (`verlauf`) wird bewusst NICHT gedruckt — als Diagramm in der App nuetzlich,
  als Fliesstext auf Papier nicht.
- Beide Formatierer nutzen dieselben vorformatierten Ausfall-Zeilen
  ("HH:MM:SS – HH:MM:SS (Xs)"), die auch die App in ihrer Messungen-Liste
  zeigt — eine Quelle der Wahrheit statt mehrfacher Formatierung.

Lokal verifiziert (Test-Dolibarr, Modul manuell deployt): Web-Karte zeigt
Kennzahlenzeile + Ausfaelle-Liste mit echten Uhrzeiten statt Rohschluessel,
PDF ebenso (2 Seiten, TCPDF-Ausfalltabelle je Lauf lesbar).

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 lib/netdiag.lib.php      | 73 +++++++++++++++++++++++++++++++++++-
 lib/netdiag_pdf.lib.php  | 80 +++++++++++++++++++++++++++++++++++++++-
 netdiagprotocol_card.php |  2 +-
 3 files changed, 151 insertions(+), 4 deletions(-)

diff --git a/lib/netdiag.lib.php b/lib/netdiag.lib.php
index 2272722..df0e80d 100644
--- a/lib/netdiag.lib.php
+++ b/lib/netdiag.lib.php
@@ -95,12 +95,18 @@ function netdiagGetOutputDir()
  * Ein Mess-Ergebnis (JSON) lesbar als HTML aufbereiten.
  *
  * Generisch: rendert flache Schlüssel/Wert-Paare und einfache Listen,
- * damit auch künftige Tools ohne Code-Änderung dargestellt werden.
+ * damit auch künftige Tools ohne Code-Änderung dargestellt werden. Jeder Wert
+ * wird auf 200 Zeichen gekürzt (`dol_trunc`) — für Skalare unschädlich, aber
+ * ein Dauertest mit vielen Ausfällen/Minuten-Buckets würde dabei mitten im
+ * Satz abgeschnitten. Für `$tool === 'stresstest'` deshalb eigener, nicht
+ * gekürzter Zweig (siehe netdiagFormatStressTest) — dieselbe Notlösung wie
+ * netdiagPdfStressTest() im PDF-Generator.
  *
  * @param  string $json JSON-String des Ergebnisses
+ * @param  string $tool Werkzeug-ID der Messung (optional, für Sonderfälle)
  * @return string       HTML-Schnipsel
  */
-function netdiagFormatResult($json)
+function netdiagFormatResult($json, $tool = '')
 {
 	if (empty($json)) {
 		return '<span class="opacitymedium">-</span>';
@@ -112,6 +118,9 @@ function netdiagFormatResult($json)
 	if (!is_array($data)) {
 		return dol_escape_htmltag((string) $data);
 	}
+	if ($tool === 'stresstest') {
+		return netdiagFormatStressTest($data);
+	}
 
 	$out = '<div class="netdiag-result">';
 	foreach ($data as $key => $val) {
@@ -132,3 +141,63 @@ function netdiagFormatResult($json)
 	$out .= '</div>';
 	return $out;
 }
+
+/**
+ * Dauer-/Stresstest-Ergebnis strukturiert als HTML aufbereiten (Kennzahlen +
+ * Ausfallliste, ungekürzt) statt es wie ein generisches Werkzeug zu
+ * behandeln — sonst schneidet dol_trunc(...,200) die Ausfallliste eines
+ * längeren Laufs mitten im Satz ab.
+ *
+ * @param  array<string,mixed> $data bereits dekodiertes result-JSON
+ * @return string                    HTML-Schnipsel
+ */
+function netdiagFormatStressTest($data)
+{
+	if (!empty($data['hinweis']) || !empty($data['fehler'])) {
+		return '<span class="opacitymedium">'.dol_escape_htmltag((string) ($data['hinweis'] ?? $data['fehler'])).'</span>';
+	}
+
+	$sum = array();
+	if (isset($data['host'])) {
+		$sum[] = 'Ziel: '.$data['host'];
+	}
+	if (isset($data['dauerSekunden'])) {
+		$sum[] = 'Dauer: '.round($data['dauerSekunden'] / 60).' min';
+	}
+	if (isset($data['intervallSek'])) {
+		$sum[] = 'Takt: '.$data['intervallSek'].' s';
+	}
+	if (isset($data['gesendet']) && isset($data['empfangen'])) {
+		$sum[] = 'Proben: '.$data['empfangen'].'/'.$data['gesendet'];
+	}
+	if (isset($data['verlustProzent'])) {
+		$sum[] = 'Verlust: '.$data['verlustProzent'].' %';
+	}
+	if (isset($data['avgMs']) && $data['avgMs'] !== null) {
+		$sum[] = 'Ø '.$data['avgMs'].' ms';
+	}
+	if (isset($data['minMs']) && isset($data['maxMs']) && $data['minMs'] !== null) {
+		$sum[] = 'Min/Max '.$data['minMs'].'/'.$data['maxMs'].' ms';
+	}
+	if (isset($data['p95Ms']) && $data['p95Ms'] !== null) {
+		$sum[] = 'p95 '.$data['p95Ms'].' ms';
+	}
+	if (isset($data['laengsterAusfallSek']) && $data['laengsterAusfallSek'] !== null) {
+		$sum[] = 'längster Ausfall '.$data['laengsterAusfallSek'].' s';
+	}
+
+	$out = '<div class="netdiag-result"><div class="netdiag-kv">'.dol_escape_htmltag(implode('  |  ', $sum)).'</div>';
+
+	$ausfaelle = (isset($data['ausfaelle']) && is_array($data['ausfaelle'])) ? $data['ausfaelle'] : array();
+	if (empty($ausfaelle)) {
+		$out .= '<div class="opacitymedium">Kein Ausfall während des Laufs.</div>';
+	} else {
+		$out .= '<div><strong>Ausfälle ('.count($ausfaelle).'):</strong></div><ul class="netdiag-outages">';
+		foreach ($ausfaelle as $line) {
+			$out .= '<li>'.dol_escape_htmltag((string) $line).'</li>';
+		}
+		$out .= '</ul>';
+	}
+	$out .= '</div>';
+	return $out;
+}
diff --git a/lib/netdiag_pdf.lib.php b/lib/netdiag_pdf.lib.php
index 279e7dd..c7b2a59 100644
--- a/lib/netdiag_pdf.lib.php
+++ b/lib/netdiag_pdf.lib.php
@@ -162,7 +162,11 @@ function netdiagGeneratePdf($db, $protocol, $outputlangs)
 		$pdf->Cell(150, 6, dol_trunc($title, 80), 1, 0, 'L');
 		$pdf->Cell(30, 6, $statusmarks[$st].' '.$statuslabels[$st], 1, 1, 'C', true);
 		$pdf->SetFont('', '', 8);
-		$pdf->MultiCell(180, 5, netdiagPdfFlattenResult($m->result), 1, 'L');
+		if ($m->tool === 'stresstest') {
+			netdiagPdfStressTest($pdf, $m->result);
+		} else {
+			$pdf->MultiCell(180, 5, netdiagPdfFlattenResult($m->result), 1, 'L');
+		}
 	}
 	if (empty($measurements)) {
 		$pdf->SetFont('', '', 9);
@@ -212,3 +216,77 @@ function netdiagPdfFlattenResult($json)
 	}
 	return implode('  |  ', $parts);
 }
+
+/**
+ * Dauer-/Stresstest strukturiert ins PDF schreiben statt als Fließtext.
+ *
+ * netdiagPdfFlattenResult() würde Kennzahlen, Ausfallliste und Zeitreihe zu
+ * einem einzigen langen Absatz zusammenkleben — bei einer Stunde Laufzeit mit
+ * 60 Minuten-Buckets kaum lesbar. Hier stattdessen eine kompakte
+ * Kennzahlenzeile plus eine echte Ausfalltabelle. Die Zeitreihe (`verlauf`)
+ * wird bewusst NICHT gedruckt — sie ist als Diagramm in der App nützlich,
+ * als reiner Text auf Papier nicht.
+ *
+ * @param  TCPDF  $pdf  PDF-Objekt (Cursor steht direkt hinter der Titelzeile)
+ * @param  string $json result-JSON der Messung
+ * @return void
+ */
+function netdiagPdfStressTest($pdf, $json)
+{
+	$data = json_decode((string) $json, true);
+	if (!is_array($data)) {
+		$pdf->MultiCell(180, 5, (string) $json, 1, 'L');
+		return;
+	}
+	if (!empty($data['hinweis']) || !empty($data['fehler'])) {
+		$pdf->MultiCell(180, 5, (string) ($data['hinweis'] ?? $data['fehler']), 1, 'L');
+		return;
+	}
+
+	$sum = array();
+	if (isset($data['host'])) {
+		$sum[] = 'Ziel: '.$data['host'];
+	}
+	if (isset($data['dauerSekunden'])) {
+		$sum[] = 'Dauer: '.round($data['dauerSekunden'] / 60).' min';
+	}
+	if (isset($data['intervallSek'])) {
+		$sum[] = 'Takt: '.$data['intervallSek'].' s';
+	}
+	if (isset($data['gesendet']) && isset($data['empfangen'])) {
+		$sum[] = 'Proben: '.$data['empfangen'].'/'.$data['gesendet'];
+	}
+	if (isset($data['verlustProzent'])) {
+		$sum[] = 'Verlust: '.$data['verlustProzent'].' %';
+	}
+	if (isset($data['avgMs']) && $data['avgMs'] !== null) {
+		$sum[] = 'Ø '.$data['avgMs'].' ms';
+	}
+	if (isset($data['minMs']) && isset($data['maxMs']) && $data['minMs'] !== null) {
+		$sum[] = 'Min/Max '.$data['minMs'].'/'.$data['maxMs'].' ms';
+	}
+	if (isset($data['p95Ms']) && $data['p95Ms'] !== null) {
+		$sum[] = 'p95 '.$data['p95Ms'].' ms';
+	}
+	if (isset($data['laengsterAusfallSek']) && $data['laengsterAusfallSek'] !== null) {
+		$sum[] = 'längster Ausfall '.$data['laengsterAusfallSek'].' s';
+	}
+	$pdf->MultiCell(180, 5, implode('  |  ', $sum), 1, 'L');
+
+	// Ausfalltabelle — je Zeile "HH:MM:SS – HH:MM:SS (Xs)", vorformatiert von
+	// der App (dieselben Zeilen, die auch in der Messungen-Liste in der App
+	// stehen — eine Quelle der Wahrheit statt zwei Formatierungen).
+	$ausfaelle = (isset($data['ausfaelle']) && is_array($data['ausfaelle'])) ? $data['ausfaelle'] : array();
+	if (empty($ausfaelle)) {
+		$pdf->SetFont('', 'I', 8);
+		$pdf->Cell(180, 5, 'Kein Ausfall während des Laufs.', 1, 1, 'L');
+		$pdf->SetFont('', '', 8);
+		return;
+	}
+	$pdf->SetFont('', 'B', 8);
+	$pdf->Cell(180, 5, 'Ausfälle ('.count($ausfaelle).')', 1, 1, 'L');
+	$pdf->SetFont('', '', 8);
+	foreach ($ausfaelle as $line) {
+		$pdf->Cell(180, 5, dol_trunc((string) $line, 120), 1, 1, 'L');
+	}
+}
diff --git a/netdiagprotocol_card.php b/netdiagprotocol_card.php
index 4f84b9c..89d4743 100644
--- a/netdiagprotocol_card.php
+++ b/netdiagprotocol_card.php
@@ -232,7 +232,7 @@ foreach ($measurements as $m) {
 	print '<td>'.dol_escape_htmltag($m->category).'</td>';
 	print '<td>'.dol_escape_htmltag($m->tool).'</td>';
 	print '<td>'.dol_escape_htmltag($m->label).'</td>';
-	print '<td>'.netdiagFormatResult($m->result).'</td>';
+	print '<td>'.netdiagFormatResult($m->result, $m->tool).'</td>';
 	print '<td class="center"><span class="badge '.$statuscss[$st].'">'.$langs->trans($statuslabels[$st]).'</span></td>';
 	print '</tr>';
 }