@abaplint/core 2.94.17 → 2.94.18
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/build/src/registry.js
CHANGED
|
@@ -29,7 +29,7 @@ class DefinitionsTop extends _abap_rule_1.ABAPRule {
|
|
|
29
29
|
return {
|
|
30
30
|
key: "definitions_top",
|
|
31
31
|
title: "Place definitions in top of routine",
|
|
32
|
-
shortDescription: `Checks that definitions are placed at the beginning of METHODs and
|
|
32
|
+
shortDescription: `Checks that definitions are placed at the beginning of METHODs, FORMs and FUNCTIONs.`,
|
|
33
33
|
extendedInformation: `If the routine has inline definitions then no issues are reported
|
|
34
34
|
|
|
35
35
|
https://docs.abapopenchecks.org/checks/17/`,
|
|
@@ -56,7 +56,7 @@ https://docs.abapopenchecks.org/checks/17/`,
|
|
|
56
56
|
if (containsUnknown === true) {
|
|
57
57
|
return [];
|
|
58
58
|
}
|
|
59
|
-
const routines = structure.findAllStructuresMulti([Structures.Form, Structures.Method]);
|
|
59
|
+
const routines = structure.findAllStructuresMulti([Structures.Form, Structures.Method, Structures.FunctionModule]);
|
|
60
60
|
for (const r of routines) {
|
|
61
61
|
// one fix per routine
|
|
62
62
|
this.fixed = false;
|
|
@@ -83,6 +83,9 @@ https://docs.abapopenchecks.org/checks/17/`,
|
|
|
83
83
|
if (get instanceof _statement_1.Comment) {
|
|
84
84
|
continue;
|
|
85
85
|
}
|
|
86
|
+
else if (get instanceof Statements.FunctionModule) {
|
|
87
|
+
continue;
|
|
88
|
+
}
|
|
86
89
|
else if (get instanceof Statements.Form) {
|
|
87
90
|
continue;
|
|
88
91
|
}
|