Ticket-Ansicht zeigt fremde/alte Belege an ("Zu diesem Ticket gehören Belege")

[Avatarsia]

Beschreibung

In der Ticket-Ansicht erscheint oben teilweise die Meldung „Zu diesem Ticket gehören Belege: …" mit Belegen, die mit dem Ticket nichts zu tun haben — bei uns z. B. ein brandneues Ticket, dem Belege aus dem Jahr 2022 zugeordnet wurden.

Wann tritt es auf

Betroffen sind vor allem Instanzen mit älterem Datenbestand (viele Belege mit Dateianhängen) und vergleichsweise neuen Tickets. Die falschen Zuordnungen wirken zufällig und ändern sich je nach Ticket.

Ursache

Die Zuordnung Ticket ↔ Beleg läuft über gemeinsame Dateianhänge (GetTicketBelege() in www/lib/class.erpapi.php). Beim Abgleich wird aber nur die numerische ID verglichen, nicht der Objekttyp des Stichwort-Eintrags. Dadurch matcht eine Ticket-Nachricht mit z. B. ID 7 auch Datei-Verknüpfungen von Angebot 7, Auftrag 7 usw. — und deren Belege werden dem Ticket angezeigt.

Vorschlag

Beim Abgleich zusätzlich auf den Objekttyp „Ticket" filtern, damit nur echte Ticket-Anhänge als Verbindung zählen. Wir haben das bei uns lokal so gelöst und können bei Interesse einen Pull Request stellen.

(Beobachtet auf einem aktuellen master-Stand, MariaDB, migrierter Xentral-Altbestand.)

[OpenXE-ERP]

Das Problem ist bei uns auch aufgetreten, ich habe es mir angesehen und würde folgende Änderung vornehmen:

WHERE dsb.objekt IN( 'angebot', 'auftrag', 'verbindlichkeit', 'lieferantengutschrift' ) AND t.id = '".$ticketid."'

=>

WHERE dsb.objekt IN( 'angebot', 'auftrag', 'verbindlichkeit', 'lieferantengutschrift' ) AND dst.objekt LIKE 'ticket%' AND t.id = '".$ticketid."'

Damit sind auch die Anhänge am Ticket direkt und an der Ticketnachricht abgedeckt. Passt das zu Euren Beobachtungen?

[Avatarsia]

Kurzes Update: Wir haben den Fix bei uns umgesetzt und er läuft seit gestern produktiv — die falschen Beleg-Zuordnungen sind damit verschwunden. Die korrekten Zuordnungen (Datei hängt sowohl am Ticket als auch am Beleg) funktionieren unverändert weiter.

Die Änderung ist bewusst klein gehalten: eine zusätzliche Bedingung in der Abfrage, damit beim Abgleich nur echte Ticket-Anhänge zählen. Die Gegenrichtung (GetBelegTickets) macht das übrigens schon genauso — insofern stellt der Fix nur die Symmetrie her.

Commit in unserem Fork: Avatarsia/OpenXE@fb35ea0 (Branch fix/ticket-belege-objekt-filter)

Wenn Interesse besteht, stellen wir gerne einen Pull Request.

[OpenXE-ERP]

Danke für die Info, da haben wir fast gleichzeitig geschrieben. Fix ist raus, habe gleich noch die like-Bedingung bei GetBelegTickets ergänzt.

8328f76

Bitte testet das Upgrade...

[Avatarsia]

Danke für die schnelle Umsetzung!

Eine Beobachtung zur LIKE 'ticket%'-Variante: Die Anhänge im Dateien-Tab des Tickets werden als ticket_header gespeichert — dort ist parameter aber die Ticket-ID (vgl. DateiUpload('ticket_header', $id) in ticket.php), während der Abgleich in beiden Funktionen dst.parameter mit der Nachrichten-ID (ticket_nachricht.id) vergleicht. Durch das LIKE laufen diese Zeilen jetzt mit durch, werden dabei aber mit der falschen ID-Art verglichen. Folge:

1. Die ursprüngliche Fehlzuordnung kann in kleinerem Maßstab wieder auftreten — ein Kopf-Anhang von Ticket A taucht an dem Ticket auf, dessen Nachrichten-ID zufällig der Ticket-ID von A entspricht.
2. Die Kopf-Anhänge selbst werden nur zufällig richtig zugeordnet (nur wenn Nachrichten-ID == Ticket-ID).

Damit die Kopf-Anhänge sauber mitzählen, bräuchten sie einen eigenen Abgleich-Zweig statt des LIKE, sinngemäß:

( dst.objekt = 'Ticket'        AND dst.parameter = tn.id )
OR
( dst.objekt = 'ticket_header' AND dst.parameter = t.id )

(analog in GetBelegTickets).

Zum Testen: Wir fahren produktiv aktuell noch unsere strikte Variante (= 'Ticket', ohne Kopf-Anhänge); euren Stand nehmen wir mit dem nächsten Upgrade mit und geben Rückmeldung.

Was hältst du davon?

[OpenXE-ERP]

Ja, Du hast recht. Die Verknüpfung auf den Ticketkopf ist eigentlich sowieso sinnlos, man kann momentan sowieso nur Dateien aus den Nachrichten zu Belegen zuordnen.

Ich habe es jetzt so wie in Deinem Vorschlag umgesetzt.