Project

General

Profile

Actions

Feature #13572

open

Form Load: misleading error message on trying to load non existent primary record

Added by Carsten Rose 6 months ago. Updated about 1 month ago.

Status:
Feedback
Priority:
Normal
Assignee:
Enis Nuredini
Target version:
Start date:
22.12.2021
Due date:
% Done:

100%

Estimated time:
Discuss:

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

clipboard-202112221546-pmwz8.png (82 KB) clipboard-202112221546-pmwz8.png Carsten Rose, 22.12.2021 15:46
newError.png (43.3 KB) newError.png Enis Nuredini, 24.12.2021 15:44
code1.png (73.9 KB) code1.png Enis Nuredini, 24.12.2021 15:50
code2.png (39.4 KB) code2.png Enis Nuredini, 24.12.2021 15:50
Actions #1

Updated by Enis Nuredini 6 months ago

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:

Actions #2

Updated by Carsten Rose 6 months 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

Actions #3

Updated by Carsten Rose 6 months ago

  • Status changed from Feedback to Closed
  • % Done changed from 0 to 100
Actions #4

Updated by Carsten Rose about 1 month ago

  • Target version changed from 22.6.0 to 22.5.0
Actions #5

Updated by Carsten Rose about 1 month ago

  • Status changed from Closed to Feedback
  • Assignee changed from Carsten Rose to Enis Nuredini
Actions #6

Updated by Carsten Rose about 1 month ago

  • Target version changed from 22.5.0 to 22.5.feedback-enis
Actions

Also available in: Atom PDF