Eduard Wisch
0b3db47ec1
Vorab: drei Punkte der Roadmap-Liste waren Fehlannahmen. Eine Analyse mit anschließender Gegenprüfung (jeder Befund musste einen Widerlegungsversuch überstehen) hat sie ausgeräumt, bevor Code geändert wurde: - "ab Seite 2 alles nach rechts verschoben" existiert nicht. Nachgemessen am Prod-PDF ND2026-0015 mit pdftotext -bbox: Seite 1 und Seite 2 beginnen beide bei 16,0 mm. Die echten Umbruchfehler waren andere. - measure_status validieren war seit Phase 1 erledigt. - Werkzeug-IDs / Teilnetz-Gruppierung / TCPDF-Fußzeile: verworfen, die vorgeschlagenen Änderungen hätten das PDF verschlechtert. PDF (alle Punkte am mehrseitigen Dokument nachgeprüft): - Tabellenkopf der Geräteliste wird auf Folgeseiten wiederholt. Vorher standen ab Seite 2 unbeschriftete Spalten — bei leeren MAC/Hostname-Feldern vier namenlose Spalten. - Messungs-Titelzeile und Ergebnis werden zusammengehalten. Vorher blieb die Überschrift samt Ampel am Seitenende allein zurück, darunter ein leerer, unten offener Rahmen; in einem Testlauf über 61 Umbruchlagen 5-mal (~8 %). - Spalte "Gerätetyp" hatte 15 mm, ließ aber 10 Zeichen zu — "Chromecast/TV" lief bis 199,0 mm bei 195 mm Tabellenkante über den Rahmen in den Druckrand. - Deutsche Bezeichnungen mit Einheiten statt roher JSON-Schlüssel: aus "VerlustProzent: 0 | MinMs: 4.4 | UptimeSek: 8123456" wird "Paketverlust: 0 % | Kürzeste Antwortzeit: 4.4 ms | Betriebszeit: 94 Tage 1 Std". Als Whitelist (netdiagKundenfelder(), gemeinsam für Karte und PDF) — interne Felder wie arpAvailable, mdnsOk, probed, answered fallen damit automatisch heraus. - "ARP-Tabelle nicht lesbar (/proc/net/arp) — braucht Root" wird beim Drucken zu einem kundentauglichen Satz. Altdaten stehen so in der DB, deshalb Ersetzung beim Drucken statt nur in der App. Gerätemerkmale (der eigentliche Roadmap-Punkt): Der Techniker sah in der App "Drucker HP, Port 9100", im Kundenprotokoll stand nur die IP. Die Felder fehlten dabei nicht in der Übertragung, sondern durchgängig — ein Fix allein in der API wäre folgenlos geblieben, weil Dolibarrs setSaveQuery() nur deklarierte $fields schreibt. Ergänzt über die ganze Kette: sql/llx_netdiag_device.sql + neue Migration llx_netdiag_device_v2.sql (ADD COLUMN IF NOT EXISTS, wiederholbar, läuft bei jedem Modul-Update), NetDiagDevice::$fields + Properties, api/protocols.php POST und GET, Kartenansicht und PDF. Neu: netbios_name, mdns_name, mdns_services, custom_name, open_ports, found_via, last_seen. Im PDF steht jetzt statt "192.168.178.20" die Zeile "Brother HL-L2350DW · Brother · Drucker · 80,443,9100" — der Name kommt aus mDNS, obwohl der Hostname leer ist. Sprachschlüssel Vendor -> NetDiagVendor: Die Gegenprüfung hielt den Punkt für falsch (Translate::load() ist first-wins, im CLI-Test kam "Hersteller"), im Browser stand in der Kartenansicht aber "Lieferant" — im HTTP-Kontext lädt Dolibarr vorher andere Sprachdateien als im CLI. Statt der Ursache nachzugehen jetzt ein eigener, kollisionsfreier Schlüssel; im Browser gegengeprüft. Nebenbei: doppeltes "OK OK" beim Status 0 im PDF. Gegen das Test-Dolibarr geprüft: Sync über die echte API (Login, POST, GET), Felder in der DB kontrolliert, Kartenansicht im Browser, mehrseitiges PDF gerendert und angesehen, Migration zweimal ausgeführt (idempotent).
26 lines
1.6 KiB
SQL
26 lines
1.6 KiB
SQL
-- Copyright (C) 2026 Eduard Wisch <data@data-it-solution.de>
|
|
--
|
|
-- This program is free software; you can redistribute it and/or modify
|
|
-- it under the terms of the GNU General Public License as published by
|
|
-- the Free Software Foundation; either version 3 of the License, or
|
|
-- (at your option) any later version.
|
|
--
|
|
-- Nachtrag zu llx_netdiag_device: Gerätemerkmale, die die App seit dem
|
|
-- IP-Scanner-Umbau ermittelt, die aber bisher nirgends ankamen.
|
|
--
|
|
-- Der Techniker sah in der App „Drucker HP, Port 9100" — im Kundenprotokoll
|
|
-- stand nur die IP. Die Felder fehlten dabei nicht etwa in der Übertragung,
|
|
-- sondern durchgängig: in dieser Tabelle, in NetDiagDevice::$fields, im
|
|
-- App-Payload und in der Anzeige. Diese Datei ergänzt die Tabelle.
|
|
--
|
|
-- Läuft bei jedem Modul-Update mit (DolibarrModules::_load_tables führt alle
|
|
-- llx_*.sql aus, alphabetisch — „.sql" vor „_v2.sql"). „IF NOT EXISTS" macht
|
|
-- den Lauf wiederholbar, ohne dass ein Update mit Fehler abbricht.
|
|
|
|
ALTER TABLE llx_netdiag_device ADD COLUMN IF NOT EXISTS netbios_name varchar(64) AFTER hostname;
|
|
ALTER TABLE llx_netdiag_device ADD COLUMN IF NOT EXISTS mdns_name varchar(255) AFTER netbios_name;
|
|
ALTER TABLE llx_netdiag_device ADD COLUMN IF NOT EXISTS mdns_services varchar(512) AFTER mdns_name;
|
|
ALTER TABLE llx_netdiag_device ADD COLUMN IF NOT EXISTS custom_name varchar(255) AFTER mdns_services;
|
|
ALTER TABLE llx_netdiag_device ADD COLUMN IF NOT EXISTS open_ports varchar(255) AFTER devicetype;
|
|
ALTER TABLE llx_netdiag_device ADD COLUMN IF NOT EXISTS found_via varchar(16) AFTER open_ports;
|
|
ALTER TABLE llx_netdiag_device ADD COLUMN IF NOT EXISTS last_seen datetime AFTER found_via;
|