JsonToObject überreden ein leeres Objekt zu verarbeiten

[fisipjm]

Also du kommst nicht in die Exception rein?
Ich habe es mit der 10.2 und der 12 getestet, geht bei beiden nicht.

[Uwe Raabe]

In der Tat, mit Delphi 12 bekomme ich den Fehler auch. Der relevante Code ist dort neu hinzugekommen, versäumt aber die Prüfung auf LField <> nil .

Eigentlich sollte die Prüfung schon viel früher erfolgen:

Delphi-Quellcode:

    LField := GetFieldType(Data, FieldName);
    if (LField <> nil) and not (LField.DataType.TypeKind in [tkClass, tkInterface]) and not LField.IsWritable then
    begin
      Inc(I);
      Continue;
    end;

Das Continue sollte auch bei LField = nil passieren.

Das QP ja aktuell nichts mehr annimmt werde ich mal andere Kanäle versuchen.

[Uwe Raabe]

Das hatte schon jemand anderes gemeldet: https://quality.embarcadero.com/browse/RSP-43488

Ein Fix ist offenbar für Update 1 geplant.

[fisipjm]

Ging also in der 10.2 nicht.
Ging dann mal in der 11.2
Und geht dann in der 12.0 wieder nicht

Ist doch zum weine. Danke für deine Recherche.

[Uwe Raabe]

Sowas passiert halt, wenn man nicht genügend Unit-Tests hat...

[peterbelow]

Zitat von Uwe Raabe:

Sowas passiert halt, wenn man nicht genügend Unit-Tests hat...

Murphy's First Law für Programmierer: "Man hat nie genug Unit-Tests!"

[Uwe Raabe]

Solange wir das noch nicht haben

Zitat:

Hey, JIRA! Schreibe einen Unit-Test für RSP-43488.

wäre es vielleicht hilfreich, wenn der Bug-Reporter das übernimmt. Das bewirkt vielleicht auch noch einen Push bezüglich Unit-Tests für den eigenen Sourcecode, wenn man sich dann sowas angewöhnt hat.

Falls sich jemand bewogen fühlt selbst zu den Delphi-Unit-Tests etwas beizutragen: Die Tests in diesem Repo werden aktuell zu den Regression-Tests für Delphi mit herangezogen.

Ein entsprechender Unit-Test damals für Delphi 10.2 hätte den neuen Fehler in Delphi 12 vermutlich verhindert.