Feature #13572
closedForm Load: misleading error message on trying to load non existent primary record
100%
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
Updated by Enis Nuredini about 2 years ago
- File newError.png newError.png added
- File code1.png code1.png added
- File code2.png code2.png added
- Status changed from New to Feedback
- Assignee changed from Enis Nuredini to Carsten Rose
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:
Updated by Carsten Rose about 2 years ago
- 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
Updated by Carsten Rose about 2 years ago
- Status changed from Feedback to Closed
- % Done changed from 0 to 100
Applied in changeset typo3-qfq|d9e1d48bc5dc03647c560f35b64e0f4e7341b6d9.
Updated by Carsten Rose almost 2 years ago
- Target version changed from 355 to 22.5.0
Updated by Carsten Rose almost 2 years ago
- Status changed from Closed to Feedback
- Assignee changed from Carsten Rose to Enis Nuredini
Updated by Carsten Rose almost 2 years ago
- Target version changed from 22.5.0 to 22.5.feedback-enis
Updated by Enis Nuredini over 1 year ago
- Prio Planung set to No
- 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.
Updated by Carsten Rose over 1 year ago
- Target version changed from 22.5.feedback-enis to 22.10.0