movedoa
movedoa
Only the variable definition gets flagged here. The rule should also flag temp-table fields and parameters with the default init value. ``` DEFINE VARIABLE cFoo AS CHARACTER NO-UNDO INIT "":U....
See: https://knowledgebase.progress.com/articles/Article/P133222 Using the default buffer for temp tables can lead to unexpected behaviour.
The compiler only forces you to define INPUT, OUTPUT etc. in method calls when there are overloads that make the call ambiguous. This makes code very hard to read since...
Just tripped over this. Trimming a path variable with \ will make the path invalid in Windows if it is a UNC path. Since TRIM with \ is most likely...
There is the startup parameter -noroutineinwhere and there is a rule idea for a sonar check that does the same thing https://github.com/Riverside-Software/sonar-openedge/issues/554. But a strict version of this would be...
A rule that checks if there is a missed opportunity for a join would be great. Joins are faster then seperate queries (and will be even faster in the future)...
It would be nice to have a rule that checks if a call of an internal method is made with "THIS-OBJECT" or the class name for static methods Wrong: `FooMethod()`...
This is a 13k lines class. When im at the beginning of the file, sonarlint reports 7 issues.  Then i scroll to the end of the file and save...
Invalid number of parameters in SUBSTITUTE statement doesnt report missing substitution parameters
V 2.24 The rule doesnt seem to check if all variables are actually used in the base-string. ``` DEFINE VARIABLE lcFoo AS CHARACTER NO-UNDO. /* 2 parameter(s) expected by SUBSTITUTE...
Rule request - Blocks that are not affected by BLOCK-LEVEL ON ERROR UNDO, THROW need error handling
There are blocks that are not effected by BLOCK-LEVEL ON ERROR UNDO, THROW. https://docs.progress.com/bundle/openedge-abl-error-handling-122/page/BLOCK-LEVEL-ON-ERROR-UNDO-THROW-statement.html A rule that checks if every DO and DO WHILE block has an error handling directive...