BC2021w1. Return record from table procedure

[DmitryKatson]

Describe the bug
When i create a procedure on the table or tableextension and return the record this doesn't work

To Reproduce

this returns empty record
tableextension 50100 "PurchaseLineExt" extends "Purchase Line" { procedure FindFirstRec(): Record "Purchase Line" begin Rec.FindFirst(); end; }

this returns first record
tableextension 50100 "PurchaseLineExt" extends "Purchase Line" { procedure FindFirstRec() PurchLine: Record "Purchase Line" begin PurchLine.FindFirst(); end; }

Expected behavior
Should return Rec

Screenshots

5. Versions:

• AL Language:
  7.0.426055
• Business Central:

[IamLerxx]

Please clarify.
tableextension 50100 "PurchaseLineExt" extends "Purchase Line" { procedure FindFirstRec(): Record "Purchase Line" begin Rec.FindFirst(); end; }

": Record "Purchase Line"" just says, that the return type of the function is a Record of Type Purchase Line.
This return type has no connection to the Rec.
To get the desired functionality add a "exit(Rec);" after the FindFirst.
tableextension 50100 "PurchaseLineExt" extends "Purchase Line" { procedure FindFirstRec(): Record "Purchase Line" begin Rec.FindFirst(); exit(Rec); end; }

[dzzzb]

Do you mean the first one? Why should it return Rec?? Rec means the global current record, not the result of the function. You did not name the result so as to assign to it, nor did you assign to it with exit(), so of course it's still default initialised. It's not related to Rec.

[dzzzb]

To get the desired functionality add a "exit(Rec);" after the FindFirst.

Sure, but then doing that is pointless, since the Rec.FindFirst() already changed the state of the instance on which the method was called, so they can just use that, instead of also copying it to the result.

[IamLerxx]

To get the desired functionality add a "exit(Rec);" after the FindFirst.

Sure, but then doing that is pointless, since the Rec.FindFirst() already changed the state of the instance on which the method was called, so they can just use that, instead of also copying it to the result.

In case of TableExtensions you are right.

In case of PageExtensions returning the Rec would be good as it would make the difference between:
myPage.FindFirstRec();
myPage.GetRecord(myRec);
and
MyRec := MyPage.FindFirstRec();

Also in the example provided by dkatson the return type of the function is set, so it should return something.

[qutreson]

I am closing this issue as this is the expected behavior. The other comments on the issue provide more information on how to handle this scenario.