Project

General

Profile

Actions

Feature #13572

closed

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

Added by Carsten Rose about 2 years ago. Updated over 1 year ago.

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

100%

Estimated time:
Discuss:
Prio Planung:
No
Vote:

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 View clipboard-202112221546-pmwz8.png 82 KB Carsten Rose, 22.12.2021 15:46
newError.png View newError.png 43.3 KB Enis Nuredini, 24.12.2021 15:44
code1.png View code1.png 73.9 KB Enis Nuredini, 24.12.2021 15:50
code2.png View code2.png 39.4 KB Enis Nuredini, 24.12.2021 15:50
Actions #1

Updated by Enis Nuredini about 2 years 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 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

Actions #3

Updated by Carsten Rose about 2 years ago

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

Updated by Carsten Rose almost 2 years ago

  • Target version changed from 355 to 22.5.0
Actions #5

Updated by Carsten Rose almost 2 years ago

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

Updated by Carsten Rose almost 2 years ago

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

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.

Actions #8

Updated by Enis Nuredini over 1 year ago

  • Status changed from Feedback to Closed
Actions #9

Updated by Carsten Rose over 1 year ago

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

Also available in: Atom PDF