is it possible to delete a zero ID through ORM?

dcoun · 2024-02-15 08:17:20

I have noticed that the following works:
if MemberExists(TOrmsometing,0) then ....
but the following does not:
Delete(TOrmsometing,0)

Is it possible in mormot.orm.sql line 1317 to allow zero IDs to be deleted through orm?

function TRestStorageExternal.EngineDelete(TableModelIndex: integer;
  ID: TID): boolean;
begin
  if (ID <= 0) or
     (TableModelIndex < 0) or
     (Model.Tables[TableModelIndex] <> fStoredClass) then
    result := false

Thank you in advance

pvn0 · 2024-02-15 11:27:23

zero is invalid, a valid ID value is >=1.

Last edited by pvn0 (2024-02-15 11:27:41)

dcoun · 2024-02-15 12:00:08

I just post it because MemberExists(TOrmsometing,0), a ORM method, accept it and it works
I tried also to run the tests without errors after changing TRestStorageExternal.EngineDelete to be like the following :

function TRestStorageExternal.EngineDelete(TableModelIndex: integer;
  ID: TID): boolean;
begin
  if (ID < 0) or
     (TableModelIndex < 0) or
     (Model.Tables[TableModelIndex] <> fStoredClass) then
    result := false

I know that ID:0 has a meaning for Add/AddOrUpdate in Delete as in MemberExists I am not sure.
I am not sure also about using Delete in batches

dcoun · 2024-02-20 08:07:06

@ab do you consider the above change as acceptable for the Delete as it works for MembereExists?

Thank you in advance

ab · 2024-02-20 11:26:44

The value 0 has a meaning of invalid ID everywhere in the framework by convention.

Perhaps it is not properly checked everywhere but it is what is expected.

So I don't consider allowing to delete an ID = 0 when we can't Add or Update such ID.

dcoun · 2024-02-20 12:27:06

No problem. I did in an other way. Thank you

mORMot Open Source

#1 2024-02-15 08:17:20

is it possible to delete a zero ID through ORM?

#2 2024-02-15 11:27:23

Re: is it possible to delete a zero ID through ORM?

#3 2024-02-15 12:00:08

Re: is it possible to delete a zero ID through ORM?

#4 2024-02-20 08:07:06

Re: is it possible to delete a zero ID through ORM?

#5 2024-02-20 11:26:44

Re: is it possible to delete a zero ID through ORM?

#6 2024-02-20 12:27:06

Re: is it possible to delete a zero ID through ORM?

Board footer