@abaplint/core 2.101.34 → 2.101.36
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
|
@@ -19,7 +19,7 @@ class CloudTypes {
|
|
|
19
19
|
key: "cloud_types",
|
|
20
20
|
title: "Check cloud types",
|
|
21
21
|
shortDescription: `Checks that the package does not contain any object types unsupported in cloud ABAP.`,
|
|
22
|
-
tags: [_irule_1.RuleTag.SingleFile],
|
|
22
|
+
tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Syntax],
|
|
23
23
|
};
|
|
24
24
|
}
|
|
25
25
|
getDescription(objectType) {
|
|
@@ -2070,6 +2070,9 @@ ${indentation} output = ${uniqueName}.\n`;
|
|
|
2070
2070
|
continue;
|
|
2071
2071
|
}
|
|
2072
2072
|
const correspondingBody = s.findDirectExpression(Expressions.CorrespondingBody);
|
|
2073
|
+
if ((correspondingBody === null || correspondingBody === void 0 ? void 0 : correspondingBody.getFirstToken().getStr().toUpperCase()) === "BASE") {
|
|
2074
|
+
continue;
|
|
2075
|
+
}
|
|
2073
2076
|
const uniqueName = this.uniqueName(firstToken.getStart(), lowFile.getFilename(), highSyntax);
|
|
2074
2077
|
const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
|
|
2075
2078
|
let type = this.findType(s, lowFile, highSyntax);
|
|
@@ -33,7 +33,9 @@ class UnnecessaryPragma extends _abap_rule_1.ABAPRule {
|
|
|
33
33
|
|
|
34
34
|
* NO_TEXT without texts
|
|
35
35
|
|
|
36
|
-
* SUBRC_OK where sy-subrc is checked
|
|
36
|
+
* SUBRC_OK where sy-subrc is checked
|
|
37
|
+
|
|
38
|
+
NO_HANDLER inside macros are not checked`,
|
|
37
39
|
tags: [_irule_1.RuleTag.SingleFile],
|
|
38
40
|
badExample: `TRY.
|
|
39
41
|
...
|
|
@@ -164,6 +166,7 @@ ENDIF.`,
|
|
|
164
166
|
}
|
|
165
167
|
if (next
|
|
166
168
|
&& next.get() instanceof _statement_1.Comment
|
|
169
|
+
&& !(statement.get() instanceof _statement_1.MacroContent)
|
|
167
170
|
&& next.concatTokens().toUpperCase().includes("#EC NO_HANDLER")) {
|
|
168
171
|
return true;
|
|
169
172
|
}
|
|
@@ -10,6 +10,7 @@ const edit_helper_1 = require("../edit_helper");
|
|
|
10
10
|
class UnnecessaryReturnConf extends _basic_rule_config_1.BasicRuleConfig {
|
|
11
11
|
}
|
|
12
12
|
exports.UnnecessaryReturnConf = UnnecessaryReturnConf;
|
|
13
|
+
// todo: make this rule more intelligent, eg RETURN. ENDTRY. ENDMETHOD.
|
|
13
14
|
class UnnecessaryReturn extends _abap_rule_1.ABAPRule {
|
|
14
15
|
constructor() {
|
|
15
16
|
super(...arguments);
|