@abaplint/core 2.113.149 → 2.113.151

package/build/abaplint.d.ts

@@ -609,6 +609,8 @@ declare class BuiltInMethod extends Identifier implements IMethodDefinition, IMe
     getRequiredParameters(): readonly TypedIdentifier[];
     getOptional(): readonly string[];
     getAll(): readonly TypedIdentifier[];
+    getEventName(): undefined;
+    getEventClass(): undefined;
     getImporting(): readonly TypedIdentifier[];
     getDefaultImporting(): string | undefined;
     getExporting(): readonly TypedIdentifier[];
@@ -3313,6 +3315,7 @@ export declare interface IEdit {
 declare interface IEventDefinition {
     getName(): string;
     getParameters(): readonly TypedIdentifier[];
+    isStatic(): boolean;
 }
 
 declare class If implements IStructure {
@@ -3351,7 +3354,9 @@ declare interface IFunctionModuleParameter {
 }
 
 declare interface IGlobalConfig {
-    /** input files, glob format */
+    /** input files, glob format
+     * @uniqueItems true
+     */
     files: string | string[];
     skipGeneratedBOPFInterfaces?: boolean;
     /** Skips generated table maintenances, determined via TOBJ object */
@@ -3445,6 +3450,8 @@ export declare interface IMethodDefinition extends Identifier {
     isAbstract(): boolean;
     isStatic(): boolean;
     isEventHandler(): boolean;
+    getEventName(): string | undefined;
+    getEventClass(): string | undefined;
     getParameters(): IMethodParameters;
     getRaising(): readonly string[];
     getExceptions(): readonly string[];
@@ -4445,6 +4452,8 @@ declare class MethodDefinition extends Identifier implements IMethodDefinition {
     private readonly parameters;
     private readonly redefinition;
     private readonly eventHandler;
+    private readonly eventName;
+    private readonly eventClass;
     private readonly abstract;
     private readonly static;
     private readonly raising;
@@ -4457,6 +4466,8 @@ declare class MethodDefinition extends Identifier implements IMethodDefinition {
     isAbstract(): boolean;
     isStatic(): boolean;
     isEventHandler(): boolean;
+    getEventName(): string | undefined;
+    getEventClass(): string | undefined;
     getParameters(): MethodParameters_2;
     getRaising(): readonly string[];
     getExceptions(): readonly string[];

package/build/src/abap/5_syntax/_builtin.js

@@ -28,6 +28,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) {

package/build/src/abap/types/event_definition.js

@@ -18,10 +18,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)) {

package/build/src/abap/types/method_definition.js

@@ -9,7 +9,7 @@ const _reference_1 = require("../5_syntax/_reference");
 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");
         }
@@ -31,8 +31,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)) {
@@ -44,7 +47,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);
@@ -60,7 +63,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);
@@ -86,6 +89,12 @@ class MethodDefinition extends _identifier_1.Identifier {
     isEventHandler() {
         return this.eventHandler;
     }
+    getEventName() {
+        return this.eventName;
+    }
+    getEventClass() {
+        return this.eventClass;
+    }
     getParameters() {
         return this.parameters;
     }

package/build/src/objects/table.js

@@ -122,7 +122,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") {

package/build/src/registry.js

@@ -67,7 +67,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.149";
+        return "2.113.151";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/mix_returning.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.MixReturning = exports.MixReturningConf = void 0;
+/* eslint-disable max-len */
 const issue_1 = require("../issue");
 const _abap_rule_1 = require("./_abap_rule");
 const Statements = require("../abap/2_statements/statements");
@@ -20,9 +21,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/core",
-    "version": "2.113.149",
+    "version": "2.113.151",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",