Feature #13572
closed
Form Load: misleading error message on trying to load non existent primary record
Added by Carsten Rose over 2 years ago.
Updated over 1 year ago.
Description
Wird beim laden eines Forms eine Record-ID (r=...) angegeben die in der Primary Table nicht existiert, gibt es eine Fehlemeldung 'Error build Form Edit Link':
Die Meldung ist schlecht (auch wenn sie technisch korrekt ist).
Bitte eine Meldung der Art 'Error: record id "..." not found in table "<table>"'.
Files
Ich habe ein Code implementiert der zu folgendem Ergebnis führt:
Code ist in der Database.php drin wo auch die DbException raus geht:
Hier wird die Exception ausgegeben:
- Tracker changed from Support to Feature
Hallo Enis
Vielen Dank fuer's fixen und kommentieren. Folgende Anmerkungen:
- Wenn moeglich wuerde ich in den Commits ein Text der Art Refs #13572 ... oder Fixes #13572... verwenden (Git / Project Keywords).
- Damit ist es dann ueberfluessig Screenshots vom Code im Ticket anzufuegen und wir haben direkt die Aenderung im Ticket verlinkt.
BTW: ich habe keinen Branch mit '13572' im Namen gefunden - hast du ihn schon committet? Ist allerdings nicht mehr noetig - ich habe einen fix commitet.
- Klar, wenn du zusaetzlich dokumentieren/fragen/hinweisen moechtest auf einen Fix, sind Screenshots nicht schlecht.
- Wenn du den Text in der Database Klasse aenderst ist es fuer alle SQL Abfragen - dann stimmt die Bezeichnung 'Tabelle' nicht, es koennen auch mehrere sein. Das war der Grund warum ich von 'Rows' spreche, das ist allgemeiner (zwar korrekt aber leider schwieriger zu verstehen) und trifft auch fuer Queries mit mehr als einer Tabelle zu.
- Das Ticket moechte eine Verbesserung fuer einen ganz spezifischen Falls: Beim laden des Forms. Ich habe es nun gefixt. Schau Dir bitte mal meine Implementierung an.
CU
Carsten
- Status changed from Feedback to Closed
- % Done changed from 0 to 100
- Target version changed from 355 to 22.5.0
- Status changed from Closed to Feedback
- Assignee changed from Carsten Rose to Enis Nuredini
- Target version changed from 22.5.0 to 22.5.feedback-enis
- Das Ticket moechte eine Verbesserung fuer einen ganz spezifischen Falls: Beim laden des Forms. Ich habe es nun gefixt. Schau Dir bitte mal meine Implementierung an.
Hab ich mir genauer angeschaut. Deine Implementierung ist direkter/einfacher und auf den Fall ausgerichtet.
- Status changed from Feedback to Closed
- Target version changed from 22.5.feedback-enis to 22.10.0
Also available in: Atom
PDF
Updated by Enis Nuredini 
Updated by Carsten Rose