microsoft / AL

Home of the Dynamics 365 Business Central AL Language extension for Visual Studio Code. Used to track issues regarding the latest version of the AL compiler and developer tools available in the Visual Studio Code Marketplace or as part of the AL Developer Preview builds for Dynamics 365 Business Central.
MIT License
719 stars 242 forks source link

/ in name of record breaks symbol building #7720

Closed Itexu closed 2 months ago

Itexu commented 2 months ago

1. Describe the bug If downloading symbols and a there is a record (might be other objects) that contains a / in its name, the compiler cant read it. Happens with C/AL Table 823 "Name/Value Buffer". Considering this is a default table and you can also name objects containing a /, this seems to be a bug.

It seems to happen only as parameter.

You get the message: Argument 5: cannot convert from 'Record "Name/Value Buffer" temporary' to 'MissingTypeSymbol'

2. To Reproduce 1: Have version of "AL Language extension for Microsoft Dynamics 365 Business Central" greater than version 11.7 (latest correct working version) 2: Have a Table containing a / (like C/AL Table 823 "Name/Value Buffer") 3: Have procedure with that object as a parameter in an extension. 4: Call that procedure from a different extension 5: Execute "AL: Download Symbols" (in pop-up menu) 6: Try to compile

codeunit x "CallingCodeunit"
{
local procedure CallingFunction()
var
        ltrecNameValueBuffer: Record "Name/Value Buffer" temporary;
        lcunCodeunitInOtherExtension: Codeunit CodeunitInOtherExtension;
begin
        lcunCodeunitInOtherExtension.FunctionInOtherExtenstion(ltrecNameValueBuffer);
end;
}

codeunit y "CodeunitInOtherExtension"
{
procedure FunctionInOtherExtenstion(var ptrecNameValueBuffer: Record "Name/Value Buffer" temporary)
begin

end;
}

3. Expected behavior Correct reference of symbols so it can be compiled

4. Actual behavior Generates error: Argument 5: cannot convert from 'Record "Name/Value Buffer" temporary' to 'MissingTypeSymbol'

5. Versions:

Final Checklist

Please remember to do the following:

kalberes commented 2 months ago

Version 14 aka C/SIDE is not supported anymore. Official answer: image