al-code-outline icon indicating copy to clipboard operation
al-code-outline copied to clipboard

RemoveProceduresSemicolon - does not remove semicolon from triggers

Open ThorstenEngelsGOB opened this issue 1 year ago • 2 comments

The new procedure RemoveProceduresSemicolon removes the redundant semicolon at the end of a function but not at the end of a trigger, such as trigger OnRename(); or trigger OnModify; The search criterion and the fix should be extended to trigger so that the LinterCOP used no longer issues the warning LC00024.

ThorstenEngelsGOB avatar Feb 28 '24 08:02 ThorstenEngelsGOB

I just wanted to report the same. It ignores also to triggers in codeunits, think of OnInstallAppPerCompany in an install codeunit.

NKarolak avatar Mar 14 '24 08:03 NKarolak

ControlAddin Triggers are ignored, too.

            usercontrol(Magic; "PTE Magic")
            {
                ApplicationArea = all;

                trigger OnControlIsReady(); //Not needed semicolon
                begin
                    CurrPage.Magic.InitializeControl();
                end;
            }

pri-kise avatar Apr 10 '24 12:04 pri-kise

Thank you for reporting your issue. The fix is available in the last release.

anzwdev avatar May 28 '24 20:05 anzwdev