@abaplint/transpiler-cli 2.11.95 → 2.12.0

package/abap_transpile

@@ -1,2 +1,2 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 require("./build/bundle");

package/build/bundle.js

@@ -47,7 +47,6 @@ class TranspilerConfig {
                 ignoreSyntaxCheck: false,
                 addFilenames: true,
                 addCommonJS: true,
-                skipConstants: false,
                 unknownTypes: transpiler_1.UnknownTypesEnum.compileError,
             },
         };
@@ -6509,7 +6508,7 @@ const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/co
 const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/index.js");
 class ReportName extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.seq)((0, combi_1.regex)(/^[\w/$%]+$/), (0, combi_1.star)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.Dash), (0, combi_1.regex)(/^\w+$/))));
+        return (0, combi_1.seq)((0, combi_1.regex)(/^[\w/$%&]+$/), (0, combi_1.star)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.Dash), (0, combi_1.regex)(/^\w+$/))));
     }
 }
 exports.ReportName = ReportName;
@@ -7067,13 +7066,14 @@ const sql_field_name_1 = __webpack_require__(/*! ./sql_field_name */ "./node_mod
 const sql_source_1 = __webpack_require__(/*! ./sql_source */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_source.js");
 class SQLCase extends combi_1.Expression {
     getRunnable() {
+        const abap = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), _1.SimpleFieldChain2);
         const field = (0, combi_1.altPrio)(_1.SQLAggregation, SQLCase, _1.SQLFunction, _1.SQLPath, sql_field_name_1.SQLFieldName, constant_1.Constant);
         const sub = (0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/", "&&"), (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.WParenLeftW)), field, (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.WParenRightW)));
         const sourc = (0, combi_1.altPrio)(SQLCase, _1.SQLAggregation, _1.SQLFunction, sql_source_1.SQLSource, constant_1.Constant);
-        const val = (0, combi_1.altPrio)(sql_cond_1.SQLCond, constant_1.Constant, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), _1.SimpleSource3));
+        const val = (0, combi_1.altPrio)(sql_cond_1.SQLCond, constant_1.Constant, abap);
         const when = (0, combi_1.seq)("WHEN", val, "THEN", sourc, (0, combi_1.starPrio)(sub));
         const els = (0, combi_1.seq)("ELSE", sourc);
-        return (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)("CASE", (0, combi_1.opt)(sql_field_name_1.SQLFieldName), (0, combi_1.plusPrio)(when), (0, combi_1.optPrio)(els), "END"));
+        return (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)("CASE", (0, combi_1.opt)((0, combi_1.altPrio)(sql_field_name_1.SQLFieldName, abap)), (0, combi_1.plusPrio)(when), (0, combi_1.optPrio)(els), "END"));
     }
 }
 exports.SQLCase = SQLCase;
@@ -14110,7 +14110,8 @@ class ModifyEntities {
         const updateFrom = (0, combi_1.seq)("UPDATE FROM", expressions_1.Source, (0, combi_1.opt)(relating));
         const deleteFrom = (0, combi_1.seq)("DELETE FROM", expressions_1.Source);
         const updateFields = (0, combi_1.seq)("UPDATE", fieldsWith);
-        const operation = (0, combi_1.alt)((0, combi_1.seq)("UPDATE SET FIELDS WITH", expressions_1.Source), (0, combi_1.seq)("CREATE SET FIELDS WITH", expressions_1.Source), updateFields, deleteFrom, updateFrom, create, execute, (0, combi_1.seq)("CREATE", (0, combi_1.opt)(by), (0, combi_1.optPrio)("AUTO FILL CID"), (0, combi_1.altPrio)(withh, fieldsWith)));
+        const updateSetFields = (0, combi_1.seq)("UPDATE SET FIELDS WITH", expressions_1.Source);
+        const operation = (0, combi_1.alt)(updateSetFields, (0, combi_1.seq)("CREATE SET FIELDS WITH", expressions_1.Source), updateFields, deleteFrom, updateFrom, create, execute, (0, combi_1.seq)("CREATE", (0, combi_1.opt)(by), (0, combi_1.optPrio)("AUTO FILL CID"), (0, combi_1.altPrio)(withh, fieldsWith)));
         const failed = (0, combi_1.seq)("FAILED", expressions_1.Target);
         const result = (0, combi_1.seq)("RESULT", expressions_1.Target);
         const mapped = (0, combi_1.seq)("MAPPED", expressions_1.Target);
@@ -14120,7 +14121,7 @@ class ModifyEntities {
         const create2 = (0, combi_1.seq)("CREATE", fieldsWith, (0, combi_1.opt)((0, combi_1.seq)("CREATE BY", expressions_1.AssociationName, fieldsWith)));
         const create3 = (0, combi_1.seq)("CREATE BY", expressions_1.AssociationName, fieldsWith);
         const create4 = (0, combi_1.seq)("CREATE FROM", expressions_1.Source, (0, combi_1.plus)((0, combi_1.seq)("CREATE BY", expressions_1.AssociationName, "FROM", expressions_1.Source)));
-        const entity = (0, combi_1.seq)("ENTITY", (0, combi_1.opt)("IN LOCAL MODE"), (0, combi_1.alt)(expressions_1.NamespaceSimpleName, expressions_1.EntityAssociation), (0, combi_1.alt)(execute, create, updateFields, deleteFrom, updateFrom, create2, create3, create4));
+        const entity = (0, combi_1.seq)("ENTITY", (0, combi_1.opt)("IN LOCAL MODE"), (0, combi_1.alt)(expressions_1.NamespaceSimpleName, expressions_1.EntityAssociation), (0, combi_1.alt)(execute, create, updateFields, deleteFrom, updateSetFields, updateFrom, create2, create3, create4));
         return (0, combi_1.ver)(version_1.Version.v754, (0, combi_1.seq)("MODIFY", (0, combi_1.alt)(entities, entity), end));
     }
 }
@@ -15062,7 +15063,8 @@ class ReadEntities {
         const result = (0, combi_1.seq)("RESULT", expressions_1.Target);
         const failed = (0, combi_1.seq)("FAILED", expressions_1.Target);
         const reported = (0, combi_1.seq)("REPORTED", expressions_1.Target);
-        const entity = (0, combi_1.seq)("ENTITY", expressions_1.NamespaceSimpleName, (0, combi_1.opt)((0, combi_1.seq)("BY", expressions_1.AssociationName)), (0, combi_1.alt)(fields, from, all), (0, combi_1.optPrio)(result));
+        const foo = (0, combi_1.seq)((0, combi_1.opt)((0, combi_1.seq)("BY", expressions_1.AssociationName)), (0, combi_1.alt)(fields, from, all), (0, combi_1.optPrio)(result));
+        const entity = (0, combi_1.seq)("ENTITY", expressions_1.NamespaceSimpleName, (0, combi_1.plus)(foo));
         const s = (0, combi_1.seq)("ENTITIES OF", expressions_1.NamespaceSimpleName, (0, combi_1.opt)("IN LOCAL MODE"), (0, combi_1.plus)(entity), (0, combi_1.optPrio)((0, combi_1.seq)("LINK", expressions_1.Target)), (0, combi_1.optPrio)((0, combi_1.per)(failed, reported)));
         const byall = (0, combi_1.seq)("BY", expressions_1.AssociationName, all);
         const by = (0, combi_1.seq)("BY", expressions_1.AssociationName, fields);
@@ -16505,11 +16507,13 @@ class Submit {
         const cover = (0, combi_1.seq)("SAP COVER PAGE", expressions_1.Source);
         const copies = (0, combi_1.seq)("COPIES", expressions_1.Source);
         const datasetExpiration = (0, combi_1.seq)("DATASET EXPIRATION", expressions_1.Source);
+        const coverText = (0, combi_1.seq)("COVER TEXT", expressions_1.Source);
+        const listName = (0, combi_1.seq)("LIST NAME", expressions_1.Source);
         const keep = (0, combi_1.seq)("KEEP IN SPOOL", expressions_1.Source);
         const imm = (0, combi_1.seq)("IMMEDIATELY", expressions_1.Source);
         const dest = (0, combi_1.seq)("DESTINATION", expressions_1.Source);
         const language = (0, combi_1.seq)("LANGUAGE", expressions_1.Source);
-        const perm = (0, combi_1.per)((0, combi_1.plus)(awith), selectionTable, (0, combi_1.plus)(awith), spool, lineSize, lineCount, archive, user, sset, ssetp, keep, cover, imm, layout, dest, language, free, newList, uss, copies, datasetExpiration, "TO SAP-SPOOL", "WITHOUT SPOOL DYNPRO", "VIA SELECTION-SCREEN", exporting, expressions_1.AndReturn, job);
+        const perm = (0, combi_1.per)((0, combi_1.plus)(awith), selectionTable, (0, combi_1.plus)(awith), spool, lineSize, lineCount, archive, user, sset, ssetp, keep, cover, imm, layout, dest, coverText, listName, language, free, newList, uss, copies, datasetExpiration, "TO SAP-SPOOL", "WITHOUT SPOOL DYNPRO", "VIA SELECTION-SCREEN", exporting, expressions_1.AndReturn, job);
         const ret = (0, combi_1.seq)("SUBMIT", prog, (0, combi_1.opt)(perm));
         return (0, combi_1.verNot)(version_1.Version.Cloud, ret);
     }
@@ -20276,7 +20280,9 @@ class BuiltIn {
         // https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abennews-610-system.htm
         const id3 = new tokens_1.Identifier(new position_1.Position(this.row++, 1), "sy-repid");
         const syrepid = new _typed_identifier_1.TypedIdentifier(id3, BuiltIn.filename, new basic_1.CharacterType(40, { qualifiedName: "sy-repid" }), ["read_only" /* IdentifierMeta.ReadOnly */, "built-in" /* IdentifierMeta.BuiltIn */]);
-        return [sy, syst, syrepid];
+        const id4 = new tokens_1.Identifier(new position_1.Position(this.row++, 1), "syst-repid");
+        const systrepid = new _typed_identifier_1.TypedIdentifier(id4, BuiltIn.filename, new basic_1.CharacterType(40, { qualifiedName: "syst-repid" }), ["read_only" /* IdentifierMeta.ReadOnly */, "built-in" /* IdentifierMeta.BuiltIn */]);
+        return [sy, syst, syrepid, systrepid];
     }
     buildConstant(name, type, value) {
         const id = new tokens_1.Identifier(new position_1.Position(this.row++, 1), name);
@@ -22736,7 +22742,7 @@ class TypeUtils {
                     if (!(sourceRowType instanceof basic_1.StructureType)) {
                         return false;
                     }
-                    else if (!(this.structureContainsString(sourceRowType))
+                    else if (!this.structureContainsString(sourceRowType)
                         && this.structureContainsVoid(sourceRowType) === false) {
                         return false;
                     }
@@ -22746,7 +22752,7 @@ class TypeUtils {
                     if (!(targetRowType instanceof basic_1.StructureType)) {
                         return false;
                     }
-                    else if (!(this.structureContainsString(targetRowType))
+                    else if (!this.structureContainsString(targetRowType)
                         && this.structureContainsVoid(targetRowType) === false) {
                         return false;
                     }
@@ -23296,7 +23302,7 @@ class BasicTypes {
             const name = typename.concatTokens();
             const type = (_d = this.input.scope.getDDIC().lookupDDLS(name)) === null || _d === void 0 ? void 0 : _d.type;
             if (type) {
-                return new Types.TableType(basic_1.VoidType.get("RapTodo"), options);
+                return new Types.TableType(basic_1.VoidType.get("RAP-TODO"), options);
             }
             else if (this.input.scope.getDDIC().inErrorNamespace(name)) {
                 return new Types.UnknownType(`DDLS ${name} not found`);
@@ -24544,7 +24550,7 @@ class CorrespondingBody {
         }
         const base = (_a = node.findDirectExpression(Expressions.CorrespondingBodyBase)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(Expressions.Source);
         if (base) {
-            source_1.Source.runSyntax(base, input);
+            source_1.Source.runSyntax(base, input, targetType);
         }
         let type = undefined;
         for (const s of node.findDirectExpressions(Expressions.Source)) {
@@ -29583,7 +29589,7 @@ class ClassImplementation {
         }
         else {
             // todo: instead of the void type, do proper typing, ie. only empty constructor method
-            input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(new tokens_1.Identifier(new position_1.Position(1, 1), "super"), _builtin_1.BuiltIn.filename, basic_1.VoidType.get("noSuper")));
+            input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(new tokens_1.Identifier(new position_1.Position(1, 1), "super"), _builtin_1.BuiltIn.filename, basic_1.VoidType.get("noSuperClass")));
         }
         input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(new tokens_1.Identifier(new position_1.Position(1, 1), "me"), _builtin_1.BuiltIn.filename, new basic_1.ObjectReferenceType(classDefinition)));
         helper.addAliasedAttributes(classDefinition); // todo, this is not correct, take care of instance vs static
@@ -40046,7 +40052,7 @@ const cds_integer_1 = __webpack_require__(/*! ./cds_integer */ "./node_modules/@
 class CDSArithmetics extends combi_1.Expression {
     getRunnable() {
         const name = (0, combi_1.seq)(_1.CDSName, (0, combi_1.optPrio)((0, combi_1.seq)(".", _1.CDSName)));
-        const val = (0, combi_1.altPrio)(cds_integer_1.CDSInteger, _1.CDSFunction, _1.CDSCase, _1.CDSCast, _1.CDSString, name);
+        const val = (0, combi_1.altPrio)(cds_integer_1.CDSInteger, _1.CDSFunction, _1.CDSCase, _1.CDSCast, _1.CDSString, _1.CDSAggregate, name);
         const operator = (0, combi_1.altPrio)("+", "-", "*", "/");
         const operatorValue = (0, combi_1.seq)(operator, val);
         const paren = (0, combi_1.seq)("(", val, (0, combi_1.plusPrio)(operatorValue), ")");
@@ -40171,7 +40177,8 @@ const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node
 class CDSCast extends combi_1.Expression {
     getRunnable() {
         const name = (0, combi_1.seq)(_1.CDSName, (0, combi_1.starPrio)((0, combi_1.seq)(".", _1.CDSName)));
-        return (0, combi_1.seq)("CAST", "(", (0, combi_1.altPrio)(_1.CDSFunction, _1.CDSCase, _1.CDSAggregate, _1.CDSArithmetics, CDSCast, _1.CDSString, name), "AS", _1.CDSType, (0, combi_1.optPrio)((0, combi_1.seq)("PRESERVING", "TYPE")), ")");
+        const first = (0, combi_1.altPrio)(_1.CDSFunction, _1.CDSCase, _1.CDSAggregate, _1.CDSArithmetics, CDSCast, _1.CDSString, _1.CDSInteger, name);
+        return (0, combi_1.seq)("CAST", "(", first, "AS", _1.CDSType, (0, combi_1.optPrio)((0, combi_1.seq)("PRESERVING", "TYPE")), ")");
     }
 }
 exports.CDSCast = CDSCast;
@@ -40487,7 +40494,7 @@ const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node
 class CDSFunctionInput extends combi_1.Expression {
     getRunnable() {
         const qualified = (0, combi_1.seq)(_1.CDSName, (0, combi_1.opt)(_1.CDSParameters), (0, combi_1.starPrio)((0, combi_1.seq)(".", _1.CDSName, (0, combi_1.opt)(_1.CDSParameters))));
-        const input = (0, combi_1.altPrio)(_1.CDSCast, _1.CDSFunction, _1.CDSArithmetics, _1.CDSCase, _1.CDSString, qualified, _1.CDSInteger);
+        const input = (0, combi_1.altPrio)(_1.CDSArithmetics, _1.CDSCast, _1.CDSFunction, _1.CDSCase, _1.CDSString, qualified, _1.CDSInteger);
         return input;
     }
 }
@@ -40579,7 +40586,7 @@ class CDSJoin extends combi_1.Expression {
     getRunnable() {
         const cond = (0, combi_1.seq)(_1.CDSSource, "ON", cds_condition_1.CDSCondition);
         const foo = (0, combi_1.altPrio)((0, combi_1.seq)("(", cond, ")"), cond);
-        return (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.altPrio)("LEFT OUTER TO ONE", "LEFT OUTER", "INNER", "CROSS")), "JOIN", foo);
+        return (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.altPrio)("LEFT OUTER TO ONE", "LEFT OUTER", "INNER", "CROSS", "RIGHT OUTER")), "JOIN", foo);
     }
 }
 exports.CDSJoin = CDSJoin;
@@ -53450,7 +53457,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.217";
+        return "2.113.222";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -54757,6 +54764,7 @@ ENDINTERFACE.`,
         }
         issues.push(...this.checkTypes(stru, file));
         issues.push(...this.checkMethods(stru, file));
+        issues.push(...this.checkEvents(stru, file));
         return issues;
     }
     check(fields, column, file) {
@@ -54816,6 +54824,26 @@ ENDINTERFACE.`,
         }
         return issues;
     }
+    checkEvents(stru, file) {
+        const issues = [];
+        const events = stru.findAllStatements(Statements.Events);
+        for (const e of events) {
+            const fields = [];
+            const params = e.findAllExpressions(Expressions.MethodParam);
+            let column = 0;
+            for (const p of params) {
+                const children = p.getChildren();
+                const name = children[children.length - 2];
+                fields.push({
+                    nameEnd: name.getLastToken().getEnd(),
+                    after: p.findFirstExpression(Expressions.TypeParam).getFirstToken().getStart()
+                });
+                column = Math.max(column, name.getFirstToken().getEnd().getCol() + 1);
+            }
+            issues.push(...this.check(fields, column, file));
+        }
+        return issues;
+    }
     checkTypes(stru, file) {
         const issues = [];
         const types = stru.findAllStructuresRecursive(Structures.Types);
@@ -57117,6 +57145,7 @@ class CloudTypes {
             || obj instanceof Objects.ApplicationLogObject
             || obj instanceof Objects.CommunicationScenario
             || obj instanceof Objects.DataControl
+            || obj instanceof Objects.Namespace
             || obj instanceof Objects.KnowledgeTransferDocument
             || obj instanceof Objects.DataDefinition
             || obj instanceof Objects.DataElement
@@ -59488,7 +59517,6 @@ ${indentation}${uniqueName} = ${source.concatTokens()}.\n${indentation}`);
         const source = high.findExpressionAfterToken("MESSAGE");
         if ((source === null || source === void 0 ? void 0 : source.get()) instanceof Expressions.MessageSourceSource
             && ((_a = source.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.Source) {
-            ;
             const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
             const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
             const firstToken = high.getFirstToken();
@@ -62920,7 +62948,7 @@ class FullyTypeConstants extends _abap_rule_1.ABAPRule {
         for (const stat of file.getStatements()) {
             if ((stat.get() instanceof Statements.Constant
                 || (this.conf.checkData === true && stat.get() instanceof Statements.Data))
-                && (!this.isTyped(stat))) {
+                && !this.isTyped(stat)) {
                 const type = stat.get() instanceof Statements.Constant ? "constant definition" : "data definition";
                 let token = (_a = stat.findFirstExpression(expressions_1.NamespaceSimpleName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
                 if (token === undefined) {
@@ -62935,7 +62963,7 @@ class FullyTypeConstants extends _abap_rule_1.ABAPRule {
         return issues;
     }
     isTyped(stat) {
-        return (stat.findFirstExpression(expressions_1.Type) || stat.findFirstExpression(expressions_1.TypeTable));
+        return stat.findFirstExpression(expressions_1.Type) || stat.findFirstExpression(expressions_1.TypeTable);
     }
 }
 exports.FullyTypeConstants = FullyTypeConstants;
@@ -63591,20 +63619,51 @@ WRITE 'world'.`,
     ////////////////
     analyzeIf(file, node) {
         var _a;
-        if (node.getChildren().length !== 4) {
+        const ifBody = node.findDirectStructure(Structures.Body);
+        const elseIfBodies = node.findDirectStructures(Structures.ElseIf) || [];
+        const elseBody = (_a = node.findDirectStructure(Structures.Else)) === null || _a === void 0 ? void 0 : _a.findDirectStructure(Structures.Body);
+        if (elseBody === undefined) {
             return [];
         }
-        const ifBody = node.findDirectStructure(Structures.Body);
-        if (node.findDirectStructure(Structures.ElseIf)) {
+        const ifFirst = ifBody === null || ifBody === void 0 ? void 0 : ifBody.getFirstChild();
+        const elseFirst = elseBody === null || elseBody === void 0 ? void 0 : elseBody.getFirstChild();
+        const ifLast = ifBody === null || ifBody === void 0 ? void 0 : ifBody.getLastChild();
+        const elseLast = elseBody === null || elseBody === void 0 ? void 0 : elseBody.getLastChild();
+        if (elseIfBodies.length > 0) {
+            let firstMatch = true;
+            let lastMatch = true;
+            for (const elseif of elseIfBodies) {
+                const elseifBody = elseif.findDirectStructure(Structures.Body);
+                const elseifFirst = elseifBody === null || elseifBody === void 0 ? void 0 : elseifBody.getFirstChild();
+                const elseifLast = elseifBody === null || elseifBody === void 0 ? void 0 : elseifBody.getLastChild();
+                if (ifFirst === undefined
+                    || ifLast === undefined
+                    || elseLast === undefined
+                    || elseFirst === undefined) {
+                    return [];
+                }
+                if (elseifFirst === undefined
+                    || ifFirst.concatTokens() !== elseifFirst.concatTokens()
+                    || elseFirst.concatTokens() !== elseifFirst.concatTokens()) {
+                    firstMatch = false;
+                }
+                if (elseifLast === undefined
+                    || ifLast.concatTokens() !== elseifLast.concatTokens()
+                    || elseLast.concatTokens() !== elseifLast.concatTokens()) {
+                    lastMatch = false;
+                }
+            }
+            if (firstMatch === true || lastMatch === true) {
+                const message = "Identical contents";
+                const issue = issue_1.Issue.atToken(file, node.getFirstToken(), message, this.getMetadata().key, this.conf.severity);
+                return [issue];
+            }
             return [];
         }
-        const elseBody = (_a = node.findDirectStructure(Structures.Else)) === null || _a === void 0 ? void 0 : _a.findDirectStructure(Structures.Body);
-        if (elseBody === undefined || ifBody === undefined) {
+        else if (elseBody === undefined || ifBody === undefined) {
             return [];
         }
         {
-            const ifFirst = ifBody.getFirstChild();
-            const elseFirst = elseBody.getFirstChild();
             if (ifFirst === undefined || elseFirst === undefined || this.isChained(ifFirst)) {
                 return [];
             }
@@ -63615,8 +63674,6 @@ WRITE 'world'.`,
             }
         }
         {
-            const ifLast = ifBody.getLastChild();
-            const elseLast = elseBody.getLastChild();
             if (ifLast === undefined || elseLast === undefined || this.isChained(ifLast)) {
                 return [];
             }
@@ -64045,7 +64102,15 @@ class ImplementMethods extends _abap_rule_1.ABAPRule {
                     const issue = issue_1.Issue.atIdentifier(found, "Do not implement abstract method \"" + md.name + "\"", this.getMetadata().key, this.conf.severity);
                     ret.push(issue);
                 }
-                continue;
+                if (def.isAbstract) {
+                    continue;
+                }
+                else {
+                    const message = "Abstract methods can only be defined in abstract classes, " + md.name;
+                    const issue = issue_1.Issue.atIdentifier(md.identifier, message, this.getMetadata().key, this.conf.severity);
+                    ret.push(issue);
+                    break;
+                }
             }
             if (impl === undefined) {
                 const message = "Class implementation for \"" + def.name + "\" not found";
@@ -64148,8 +64213,19 @@ class ImplementMethods extends _abap_rule_1.ABAPRule {
                 return [idef];
             }
             for (const m of this.findInterfaceMethods(idef)) {
-                if (interfaceInfo.abstractMethods.includes(m.method.name.toUpperCase())) {
-                    continue;
+                if (this.isAbstract(m, interfaceInfo, def)) {
+                    if (def.isAbstract) {
+                        continue;
+                    }
+                    else {
+                        // there is some bug here, which I cannot reproduce right now
+                        /*
+                        const message = "Abstract methods can only be defined in abstract classes, " + m.method.name;
+                        const issue = Issue.atIdentifier(m.method.identifier, message, this.getMetadata().key, this.conf.severity);
+                        ret.push(issue);
+                        break;
+                        */
+                    }
                 }
                 if (this.isImplemented(m, def, impl) === false) {
                     const message = "Implement method \"" + m.method.name + "\" from interface \"" + m.objectName + "\"";
@@ -64167,6 +64243,23 @@ class ImplementMethods extends _abap_rule_1.ABAPRule {
         }
         return ret;
     }
+    isAbstract(m, interfaceInfo, def) {
+        if (interfaceInfo.abstractMethods.includes(m.method.name.toUpperCase())) {
+            return true;
+        }
+        if (!def.superClassName) {
+            return false;
+        }
+        // look up in superclass if method is abstract there
+        const superClass = this.findClass(def.superClassName);
+        const superInterface = superClass === null || superClass === void 0 ? void 0 : superClass.def.interfaces.find(iface => iface.name.toUpperCase() === m.objectName.toUpperCase());
+        if (superClass && superInterface) {
+            return this.isAbstract(m, superInterface, superClass.def);
+        }
+        else {
+            return false;
+        }
+    }
     isImplemented(m, def, impl) {
         if (impl === undefined) {
             return false;
@@ -69547,6 +69640,7 @@ const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@ab
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 const version_1 = __webpack_require__(/*! ../version */ "./node_modules/@abaplint/core/build/src/version.js");
+const __1 = __webpack_require__(/*! .. */ "./node_modules/@abaplint/core/build/src/index.js");
 class Parser702ChainingConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.Parser702ChainingConf = Parser702ChainingConf;
@@ -69558,7 +69652,7 @@ class Parser702Chaining extends _abap_rule_1.ABAPRule {
     getMetadata() {
         return {
             key: "parser_702_chaining",
-            title: "Parser Error, bad chanining on 702",
+            title: "Parser Error, bad chaining on 702",
             shortDescription: `ABAP on 702 does not allow for method chaining with IMPORTING/EXPORTING/CHANGING keywords,
 this rule finds these and reports errors.
 Only active on target version 702 and below.`,
@@ -69599,13 +69693,47 @@ Only active on target version 702 and below.`,
                     || param.findDirectTokenByText("CHANGING")
                     || param.findDirectTokenByText("EXCEPTIONS")) {
                     const message = "This kind of method chaining not possible in 702";
-                    const issue = issue_1.Issue.atPosition(file, param.getFirstToken().getStart(), message, this.getMetadata().key, this.conf.severity);
-                    issues.push(issue);
+                    this.pushIssue(message, file, param, issues);
+                }
+            }
+        }
+        // after a value assignment (move statement whose source is a method call, or method parameter assignment),
+        // there can't be any EXPORTING/IMPORTING/CHANGING/EXCEPTIONS
+        for (const statement of file.getStatements()) {
+            if (!(statement.get() instanceof __1.Statements.Move)) {
+                continue;
+            }
+            const source = statement.findDirectExpression(Expressions.Source);
+            if (source === undefined) {
+                continue;
+            }
+            this.ensureSourceHasNoProceduralKeywords(source, file, issues);
+        }
+        for (const methodParameters of stru.findAllExpressions(Expressions.MethodParameters)) {
+            for (const params of methodParameters.findAllExpressions(Expressions.ParameterS)) {
+                const source = params.findDirectExpression(Expressions.Source);
+                if (source === undefined) {
+                    continue;
                 }
+                this.ensureSourceHasNoProceduralKeywords(source, file, issues);
             }
         }
         return issues;
     }
+    ensureSourceHasNoProceduralKeywords(source, file, issues) {
+        const forbiddenTokens = ["EXPORTING", "IMPORTING", "CHANGING", "EXCEPTIONS"];
+        for (const param of source.findAllExpressions(Expressions.MethodParameters)) {
+            const usedForbiddenToken = forbiddenTokens.find(text => param.findDirectTokenByText(text));
+            if (usedForbiddenToken) {
+                const message = `Unexpected word ${usedForbiddenToken} in functional method call`;
+                this.pushIssue(message, file, param, issues);
+            }
+        }
+    }
+    pushIssue(message, file, node, issues) {
+        const issue = issue_1.Issue.atPosition(file, node.getFirstToken().getStart(), message, this.getMetadata().key, this.conf.severity);
+        issues.push(issue);
+    }
 }
 exports.Parser702Chaining = Parser702Chaining;
 //# sourceMappingURL=parser_702_chaining.js.map
@@ -74529,9 +74657,9 @@ ENDFORM.`,
         for (let i = 0; i < statements.length; i++) {
             const node = statements[i];
             const nodeType = node.get();
-            if ((nodeType instanceof Statements.MethodImplementation
+            if (nodeType instanceof Statements.MethodImplementation
                 || nodeType instanceof Statements.Form
-                || nodeType instanceof Statements.FunctionModule)) {
+                || nodeType instanceof Statements.FunctionModule) {
                 statementCounter = 0;
                 continue;
             }
@@ -77653,8 +77781,8 @@ class PopulateTables {
         if (!this.hasT000) {
             return "";
         }
-        // todo, this should take the client number from the settings
-        return `INSERT INTO t000 ('mandt', 'cccategory', 'ccnocliind') VALUES ('123', '', '');`;
+        // todo, this should take the client number from the json abap_transpile.json settings
+        return `INSERT INTO t000 ("mandt", "cccategory", "ccnocliind") VALUES ('123', '', '');`;
     }
     escape(value) {
         let ret = value.replace(/\'/g, "''");
@@ -82985,9 +83113,13 @@ import "./_top.mjs";\n`;
 import runtime from "@abaplint/runtime";
 globalThis.abap = new runtime.ABAP();\n`;
         }
-        ret += `${this.buildImports(reg, useImport, options)}
-
-export async function initializeABAP() {\n`;
+        if (options?.setup?.filename === undefined || options?.setup?.filename === "") {
+            ret += `// no setup logic specified in config\n`;
+        }
+        else {
+            ret += `const setup = await import("${options?.setup?.filename}");\n`;
+        }
+        ret += `export async function initializeABAP() {\n`;
         ret += `  const sqlite = [];\n`;
         for (const i of dbSetup.schemas.sqlite) {
             ret += `  sqlite.push(\`${i}\`);\n`;
@@ -83008,14 +83140,15 @@ export async function initializeABAP() {\n`;
             ret += `  insert.push(\`${i}\`);\n`;
         }
         ret += `\n`;
-        if (options?.extraSetup === undefined || options?.extraSetup === "") {
-            ret += `// no setup logic specified in config\n`;
+        if (options?.setup?.preFunction !== undefined) {
+            ret += `  await setup.${options?.setup?.preFunction}(globalThis.abap, schemas, insert);\n`;
         }
-        else {
-            ret += `  const {setup} = await import("${options?.extraSetup}");\n` +
-                `  await setup(globalThis.abap, schemas, insert);\n`;
+        ret += `}\n\n`;
+        ret += `await initializeABAP();\n\n`;
+        ret += `${this.buildImports(reg, useImport, options)}`;
+        if (options?.setup?.postFunction !== undefined) {
+            ret += `\n\nawait setup.${options?.setup?.postFunction}(globalThis.abap, schemas, insert);\n`;
         }
-        ret += `}`;
         return ret;
     }
     buildImports(reg, useImport, options) {
@@ -92010,12 +92143,11 @@ class UnitTest {
 import fs from "fs";
 import path from "path";
 import {fileURLToPath} from "url";
-import {initializeABAP} from "./init.mjs";
+import "./init.mjs";
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
 async function run() {
-  await initializeABAP();
   let lt_input = new abap.types.Table(new abap.types.Structure({class_name: new abap.types.Character(30), testclass_name: new abap.types.Character(30), method_name: new abap.types.Character(30)}), {"withHeader":false,"type":"STANDARD","isUnique":false,"keyFields":[]});
   let ls_input = new abap.types.Structure({class_name: new abap.types.Character(30), testclass_name: new abap.types.Character(30), method_name: new abap.types.Character(30)});
   let ls_result = new abap.types.Structure({list: new abap.types.Table(new abap.types.Structure({class_name: new abap.types.Character(30), testclass_name: new abap.types.Character(30), method_name: new abap.types.Character(30), expected: new abap.types.String(), actual: new abap.types.String(), status: new abap.types.String(), runtime: new abap.types.Integer(), message: new abap.types.String(), js_location: new abap.types.String(), console: new abap.types.String()}), {"withHeader":false,"type":"STANDARD","isUnique":false,"keyFields":[]}), json: new abap.types.String()});
@@ -92154,7 +92286,7 @@ run().then(() => {
         };
         let ret = `/* eslint-disable curly */
 /* eslint-disable max-len */
-import {initializeABAP} from "./init.mjs";
+import "./init.mjs";
 
 function getData() {
   const ret = [];\n`;
@@ -92181,7 +92313,6 @@ function getData() {
 async function run() {
   const skipCritical = process.argv[2] === "--skip-critical";
   const onlyCritical = process.argv[2] === "--only-critical";
-  await initializeABAP();
   for (const st of getData()) {
     const imported = await import(st.filename);
     const localClass = imported[st.localClass];
@@ -103311,9 +103442,6 @@ class Progress {
 }
 async function loadLib(config) {
     const files = [];
-    if (config.lib && config.lib !== "" && config.libs === undefined) {
-        config.libs = [{ url: config.lib }];
-    }
     for (const l of config.libs || []) {
         let dir = "";
         let cleanupFolder = false;

package/build/types.d.ts

@@ -6,10 +6,6 @@ export interface ITranspilerConfig {
     /** list of regex, case insensitive */
     exclude_filter?: string[];
     output_folder: string;
-    /** to be deprecated, "lib", use "libs" instead
-     * @deprecated
-     */
-    lib?: string;
     libs?: {
         url?: string;
         folder?: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.11.95",
+  "version": "2.12.0",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,17 +27,17 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.113.217",
-    "@abaplint/transpiler": "^2.11.95",
+    "@abaplint/core": "^2.113.222",
+    "@abaplint/transpiler": "^2.12.0",
     "@types/glob": "^8.1.0",
-    "@types/node": "^24.5.2",
+    "@types/node": "^24.6.2",
     "@types/progress": "^2.0.7",
     "glob": "=7.2.0",
     "progress": "^2.0.3",
     "ts-json-schema-generator": "^2.4.0",
-    "typescript": "^5.9.2",
+    "typescript": "^5.9.3",
     "p-limit": "^3.1.0",
     "webpack-cli": "^6.0.1",
-    "webpack": "^5.101.3"
+    "webpack": "^5.102.0"
   }
 }