@abaplint/cli 2.113.149 → 2.113.151

package/build/cli.js

@@ -20892,6 +20892,12 @@ class BuiltInMethod extends _identifier_1.Identifier {
     getAll() {
         throw new Error("BuiltInMethod->getAll, Method not implemented");
     }
+    getEventName() {
+        return undefined;
+    }
+    getEventClass() {
+        return undefined;
+    }
     getImporting() {
         const ret = [];
         for (const i in this.method.mandatory) {
@@ -39839,10 +39845,14 @@ class EventDefinition extends _identifier_1.Identifier {
         super(found.getFirstToken(), input.filename);
         this.parameters = [];
         this.parse(node, input);
+        this.is_static = node.getFirstToken().getStr().toUpperCase() === "CLASS";
     }
     getParameters() {
         return this.parameters;
     }
+    isStatic() {
+        return this.is_static;
+    }
     ///////////////
     parse(node, input) {
         for (const e of node.findAllExpressions(expressions_1.MethodParam)) {
@@ -40262,7 +40272,7 @@ const _reference_1 = __webpack_require__(/*! ../5_syntax/_reference */ "./node_m
 class MethodDefinition extends _identifier_1.Identifier {
     // todo: final flag
     constructor(node, visibility, input) {
-        var _a, _b;
+        var _a, _b, _c, _d;
         if (!(node.get() instanceof statements_1.MethodDef)) {
             throw new Error("MethodDefinition, expected MethodDef as part of input node");
         }
@@ -40284,8 +40294,11 @@ class MethodDefinition extends _identifier_1.Identifier {
             }
         }
         this.eventHandler = false;
-        if (node.findDirectExpression(Expressions.EventHandler)) {
+        const handler = node.findDirectExpression(Expressions.EventHandler);
+        if (handler) {
             this.eventHandler = true;
+            this.eventName = (_a = handler.findDirectExpression(Expressions.EventName)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
+            this.eventClass = (_b = handler.findDirectExpression(Expressions.ClassName)) === null || _b === void 0 ? void 0 : _b.concatTokens().toUpperCase();
         }
         this.abstract = false;
         if (node.findDirectExpression(Expressions.Abstract)) {
@@ -40297,7 +40310,7 @@ class MethodDefinition extends _identifier_1.Identifier {
             this.static = true;
         }
         this.raising = [];
-        for (const r of ((_a = node.findDirectExpression(Expressions.MethodDefRaising)) === null || _a === void 0 ? void 0 : _a.findAllExpressions(Expressions.ClassName)) || []) {
+        for (const r of ((_c = node.findDirectExpression(Expressions.MethodDefRaising)) === null || _c === void 0 ? void 0 : _c.findAllExpressions(Expressions.ClassName)) || []) {
             const token = r.getFirstToken();
             const name = token.getStr();
             this.raising.push(name);
@@ -40313,7 +40326,7 @@ class MethodDefinition extends _identifier_1.Identifier {
             }
         }
         this.exceptions = [];
-        for (const r of ((_b = node.findDirectExpression(Expressions.MethodDefExceptions)) === null || _b === void 0 ? void 0 : _b.findAllExpressions(Expressions.NamespaceSimpleName)) || []) {
+        for (const r of ((_d = node.findDirectExpression(Expressions.MethodDefExceptions)) === null || _d === void 0 ? void 0 : _d.findAllExpressions(Expressions.NamespaceSimpleName)) || []) {
             const token = r.getFirstToken();
             const name = token.getStr();
             this.exceptions.push(name);
@@ -40339,6 +40352,12 @@ class MethodDefinition extends _identifier_1.Identifier {
     isEventHandler() {
         return this.eventHandler;
     }
+    getEventName() {
+        return this.eventName;
+    }
+    getEventClass() {
+        return this.eventClass;
+    }
     getParameters() {
         return this.parameters;
     }
@@ -53154,7 +53173,7 @@ class Table extends _abstract_object_1.AbstractObject {
                 }
                 if (found instanceof Types.StructureType) {
                     if (field.GROUPNAME !== undefined) {
-                        components.push({ name: field.GROUPNAME, type: found });
+                        components.push({ name: field.GROUPNAME, type: found, asInclude: true });
                     }
                     if (field.FIELDNAME.startsWith(".INCLU-") === false
                         || field.FIELDNAME === ".INCLU--AP") {
@@ -54677,7 +54696,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.149";
+        return "2.113.151";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -68453,6 +68472,7 @@ exports.MethodParameterNames = MethodParameterNames;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.MixReturning = exports.MixReturningConf = void 0;
+/* eslint-disable max-len */
 const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
 const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
@@ -68472,9 +68492,10 @@ class MixReturning extends _abap_rule_1.ABAPRule {
             key: "mix_returning",
             title: "Mix of returning and exporting",
             shortDescription: `Checks that methods don't have a mixture of returning and exporting/changing parameters`,
-            // eslint-disable-next-line max-len
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-either-returning-or-exporting-or-changing-but-not-a-combination`,
-            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-either-returning-or-exporting-or-changing-but-not-a-combination
+
+This syntax is not allowed on versions earlier than 740sp02, https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abennews-740-abap_objects.htm#!ABAP_MODIFICATION_1@1@`,
+            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Syntax],
             badExample: `CLASS lcl DEFINITION.
   PUBLIC SECTION.
     METHODS

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.149",
+    "version": "2.113.151",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.149",
+        "@abaplint/core": "^2.113.151",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",