Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AA0248 Add 'this' qualification. - Missing Runtime version check #7742

Open
pri-kise opened this issue May 6, 2024 · 0 comments
Open

AA0248 Add 'this' qualification. - Missing Runtime version check #7742

pri-kise opened this issue May 6, 2024 · 0 comments

Comments

@pri-kise
Copy link

pri-kise commented May 6, 2024

Please include the following with each issue:

1. Describe the bug
The CodeCop should check for the runtime specified in the app.json and only trigger the warning AA0248 then.

2. To Reproduce
Steps to reproduce the behavior:

    procedure SaveContactSync(CurrUserSecurityID: Guid; var Contact: Record Contact; NewID: Text)
    var
        RecordRef: RecordRef;
    begin
        RecordRef.GetTable(Contact);
        SaveSync(CurrUserSecurityID, RecordRef, Enum::"PTE Entity"::Contact, NewID); // Wrong warning for this
    end;

    procedure SaveSync(CurrUserSecurityID: Guid; var RecordRef: RecordRef; MgEntity: Enum "PTE Entity"; NewID: Text)
    var

    begin
        // Magic
    end;

.

3. Expected behavior
Similiar to other properties the CodeCop should respect the runtime version.

4. Actual behavior
The runtime is ignored.

5. Versions:

  • AL Language: v14.0.1002061 (pre-release)

Internal work item: AB#534070

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants