#1 2022-03-23 10:42:22

danielkuettner
Member
From: Germany
Registered: 2014-08-06
Posts: 330

regression for Variants in commits after 96c8a5d3

Hello Arnaud,

this code doesn't work with a commit later than 96c8a5d3 (2022-02-12):

var
  aCCGID: Integer;

V:= _OBJ([]);
aCCGID:= 1;
V.ID:= aCCGID;

Debugger shows a EJsonException instead of a parsed json.

(Windows11, Delphi 10.4, 32/64bit)

Daniel

Offline

#2 2022-03-23 12:21:44

ab
Administrator
From: France
Registered: 2010-06-21
Posts: 14,232
Website

Re: regression for Variants in commits after 96c8a5d3

I merged your commit.

Should be fixed now.

Thanks a lot for sharing!

Offline

#3 2022-03-24 09:47:59

ab
Administrator
From: France
Registered: 2010-06-21
Posts: 14,232
Website

Re: regression for Variants in commits after 96c8a5d3

Your pull request did break the existing code.

I have fixed it properly with https://github.com/synopse/mORMot2/commit/953fd223

Offline

#4 2022-03-24 18:45:34

danielkuettner
Member
From: Germany
Registered: 2014-08-06
Posts: 330

Re: regression for Variants in commits after 96c8a5d3

Thanks!

I've tested your changes and it works in my project without issues.

Offline

Board footer

Powered by FluxBB