FindSet deprecated?!?

JlT

Fired up VS Code with runtime 11.0 and got this little varning. What should we use instead?

JlT

So it is the use of two parameters that will be deprecated. FindSet(false) doesn't give a warning.

bbrown

JlT wrote: »

So it is the use of two parameters that will be deprecated. FindSet(false) doesn't give a warning.

JlT wrote: »

So it is the use of two parameters that will be deprecated. FindSet(false) doesn't give a warning.

Do you know which parameter was deprecated?

DenSter

I think UpdateKey is removed because it was not really working anymore anyway.

https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/methods-auto/record/record-findset-method

RockWithNAV

@DenSter - Is it so? It use to work in the NAV Version. Not Sure of BC?

primeap

DenSter wrote: »

I think UpdateKey is removed because it was not really working anymore anyway.

Meaning we cannot do a rename within a loop with FindSet ?

jordi79

If I can recall, UpdateKey in FINDSET() is used when you are modifying a FILTER key in the loop.
e.g.

Rec.SETRANGE(Fld1, Value1);
IF Rec.FINDSET(TRUE, TRUE) then REPEAT
  Rec.Fld1 := 'Some other value';
  Rec.MODIFY;
UNTIL Rec.NEXT = 0;

Does this mean with runtime 11.0, this code will have to be refactored?

txeriff

Don't worry it's just they will remove the two optional parameters (forupdate and updatekey). it will be just one parameter (forupdate), so it will be findset(true or false).

if you want to remove those errors you can set something in the app.json file or force install of older AL extension