@abaplint/transpiler-cli 2.13.23 → 2.13.25

package/build/bundle.js

@@ -6084,10 +6084,11 @@ exports.Or = Or;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.ParameterException = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/index.js");
 const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 class ParameterException extends combi_1.Expression {
     getRunnable() {
-        const name = (0, combi_1.altPrio)("OTHERS", _1.ParameterName);
+        const name = (0, combi_1.altPrio)("OTHERS", (0, combi_1.seq)(_1.ParameterName, (0, combi_1.tok)(tokens_1.Dash), _1.ComponentName), _1.ParameterName);
         return (0, combi_1.seq)(name, (0, combi_1.optPrio)((0, combi_1.seq)("=", (0, combi_1.altPrio)(_1.Integer, _1.SimpleFieldChain), (0, combi_1.optPrio)((0, combi_1.seq)("MESSAGE", _1.Target)))));
     }
 }
@@ -22505,6 +22506,7 @@ const table_type_1 = __webpack_require__(/*! ../types/basic/table_type */ "./nod
 const enum_type_1 = __webpack_require__(/*! ../types/basic/enum_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/enum_type.js");
 const cgeneric_type_1 = __webpack_require__(/*! ../types/basic/cgeneric_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/cgeneric_type.js");
 const Expressions = __webpack_require__(/*! ../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const nodes_1 = __webpack_require__(/*! ../nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
 // todo: refactor to static? for performance
 class TypeUtils {
     constructor(scope) {
@@ -22755,6 +22757,18 @@ class TypeUtils {
           return false;
         }
         */
+        var _a;
+        if (node.getChildren().length === 4
+            && node.getFirstChild() instanceof nodes_1.TokenNodeRegex
+            && ((_a = node.getFirstChild()) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase()) === "BOOLC") {
+            return true;
+        }
+        if (node.getChildren().length >= 3) {
+            const second = node.getChildren()[1];
+            if (second.getFirstToken().getStr() === "&&") {
+                return true;
+            }
+        }
         const calculated = node.findFirstExpression(Expressions.MethodCallChain) !== undefined
             || node.findFirstExpression(Expressions.StringTemplate) !== undefined
             || node.findFirstExpression(Expressions.ArithOperator) !== undefined;
@@ -22768,7 +22782,7 @@ class TypeUtils {
         return this.isAssignable(source, target);
     }
     isAssignableStrict(source, target, node) {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k;
         const calculated = node ? this.isCalculated(node) : false;
         /*
             console.dir(source);
@@ -22827,16 +22841,15 @@ class TypeUtils {
         else if (source instanceof basic_1.StringType) {
             if (target instanceof basic_1.StructureType && this.structureContainsString(target)) {
                 return false;
-                /*
-              } else if (target instanceof CharacterType) {
-                if (source.getAbstractTypeData()?.derivedFromConstant === true) {
-                  return true;
+            }
+            else if (target instanceof basic_1.CharacterType) {
+                if (((_g = source.getAbstractTypeData()) === null || _g === void 0 ? void 0 : _g.derivedFromConstant) === true) {
+                    return true;
                 }
                 return false;
-                */
             }
             else if (target instanceof basic_1.IntegerType) {
-                if (((_g = source.getAbstractTypeData()) === null || _g === void 0 ? void 0 : _g.derivedFromConstant) === true) {
+                if (((_h = source.getAbstractTypeData()) === null || _h === void 0 ? void 0 : _h.derivedFromConstant) === true) {
                     return true;
                 }
                 return false;
@@ -22846,7 +22859,7 @@ class TypeUtils {
                 return false;
             }
             else if (target instanceof basic_1.XSequenceType || target instanceof basic_1.XStringType) {
-                if (((_h = source.getAbstractTypeData()) === null || _h === void 0 ? void 0 : _h.derivedFromConstant) === true) {
+                if (((_j = source.getAbstractTypeData()) === null || _j === void 0 ? void 0 : _j.derivedFromConstant) === true) {
                     return true;
                 }
                 return false;
@@ -22890,7 +22903,7 @@ class TypeUtils {
                 return false;
             }
             else if (target instanceof basic_1.Integer8Type || target instanceof basic_1.PackedType) {
-                if (((_j = source.getAbstractTypeData()) === null || _j === void 0 ? void 0 : _j.derivedFromConstant) === true) {
+                if (((_k = source.getAbstractTypeData()) === null || _k === void 0 ? void 0 : _k.derivedFromConstant) === true) {
                     return true;
                 }
                 return false;
@@ -25161,8 +25174,8 @@ class FieldChain {
             && node.findDirectExpression(Expressions.ComponentName)) {
             // workaround for names with dashes, eg. "sy-repid"
             const concat = node.concatTokens().replace(/ /g, "");
-            const offset = ((_b = node.findDirectExpression(Expressions.FieldOffset)) === null || _b === void 0 ? void 0 : _b.concatTokens()) || "";
-            const length = ((_c = node.findDirectExpression(Expressions.FieldLength)) === null || _c === void 0 ? void 0 : _c.concatTokens()) || "";
+            const offset = ((_b = node.findDirectExpression(Expressions.FieldOffset)) === null || _b === void 0 ? void 0 : _b.concatTokens().replace(/ /g, "")) || "";
+            const length = ((_c = node.findDirectExpression(Expressions.FieldLength)) === null || _c === void 0 ? void 0 : _c.concatTokens().replace(/ /g, "")) || "";
             const found = input.scope.findVariable(concat.replace(offset, "").replace(length, ""));
             if (found) {
                 if (refType) {
@@ -29754,6 +29767,12 @@ class CallFunction {
         for (const s of node.findDirectExpressions(Expressions.Source)) {
             source_1.Source.runSyntax(s, input);
         }
+        for (const d of node.findDirectExpressions(Expressions.Destination)) {
+            const source = d.findFirstExpression(Expressions.Source);
+            if (source) {
+                source_1.Source.runSyntax(source, input);
+            }
+        }
         const fp = node.findDirectExpression(Expressions.FunctionParameters);
         if (fp) {
             function_parameters_1.FunctionParameters.runSyntax(fp, input);
@@ -31308,11 +31327,15 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.FetchNextCursor = void 0;
 const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js");
+const sql_source_1 = __webpack_require__(/*! ../expressions/sql_source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js");
 class FetchNextCursor {
     runSyntax(node, input) {
         for (const t of node.findAllExpressions(Expressions.Target)) {
             target_1.Target.runSyntax(t, input);
         }
+        for (const s of node.findAllExpressions(Expressions.SQLSourceSimple)) {
+            sql_source_1.SQLSource.runSyntax(s, input);
+        }
     }
 }
 exports.FetchNextCursor = FetchNextCursor;
@@ -42215,6 +42238,7 @@ class Config {
                 skipGeneratedProxyInterfaces: false,
                 useApackDependencies: false,
                 skipIncludesWithoutMain: false,
+                errorOnDuplicateFilenames: false,
             },
             dependencies: [{
                     url: "https://github.com/abaplint/deps",
@@ -42278,6 +42302,9 @@ class Config {
         if (this.config.global.skipIncludesWithoutMain === undefined) {
             this.config.global.skipIncludesWithoutMain = false;
         }
+        if (this.config.global.errorOnDuplicateFilenames === undefined) {
+            this.config.global.errorOnDuplicateFilenames = false;
+        }
         this.checkVersion();
     }
     get() {
@@ -55195,7 +55222,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.119.16";
+        return "2.119.19";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -55311,6 +55338,9 @@ class Registry {
                 continue;
             }
             let found = this.findOrCreate(f.getObjectName(), f.getObjectType());
+            if (this.conf.getGlobal().errorOnDuplicateFilenames === true) {
+                this.checkDuplicateFilename(filename, found.getFiles());
+            }
             if (dependency === false && found && this.isDependency(found)) {
                 this.removeDependency(found);
                 found = this.findOrCreate(f.getObjectName(), f.getObjectType());
@@ -55319,6 +55349,22 @@ class Registry {
         }
         return this;
     }
+    checkDuplicateFilename(filename, files) {
+        const basename = Registry.filenameBasename(filename);
+        if (basename === "PACKAGE.DEVC.XML") {
+            return;
+        }
+        for (const existingFile of files) {
+            if (Registry.filenameBasename(existingFile.getFilename()) === basename) {
+                throw new Error("Duplicate filename: " + filename + " already exists as " + existingFile.getFilename());
+            }
+        }
+    }
+    static filenameBasename(filename) {
+        const normalized = filename.replace(/\\/g, "/");
+        const last = normalized.split("/").pop();
+        return (last !== null && last !== void 0 ? last : normalized).toUpperCase();
+    }
     addFiles(files) {
         this._addFiles(files, false);
         return this;
@@ -72533,10 +72579,6 @@ class ParserBadExceptions extends _abap_rule_1.ABAPRule {
             title: "Parser Error, bad EXCEPTIONS in CALL FUNCTION",
             shortDescription: `Checks for syntax not recognized by abaplint, related to EXCEPTIONS in CALL FUNCTION.`,
             tags: [_irule_1.RuleTag.Syntax, _irule_1.RuleTag.SingleFile],
-            /*
-            badExample: `IF ( foo = 'bar').`,
-            goodExample: `IF ( foo = 'bar' ).`,
-            */
         };
     }
     getConfig() {
@@ -72551,6 +72593,15 @@ class ParserBadExceptions extends _abap_rule_1.ABAPRule {
             if (!(statement.get() instanceof Statements.CallFunction)) {
                 continue;
             }
+            const exceptionTokens = this.findExceptionTokens(statement);
+            if (exceptionTokens !== undefined
+                && this.containsMixedExceptionSyntax(statement.findAllExpressions(Expressions.ParameterException))) {
+                const message = "Bad EXCEPTIONS syntax in CALL FUNCTION";
+                for (const token of exceptionTokens) {
+                    issues.push(issue_1.Issue.atToken(file, token, message, this.getMetadata().key, this.conf.severity));
+                }
+                continue;
+            }
             for (const e of statement.findAllExpressions(Expressions.ParameterException)) {
                 if (e.findDirectTokenByText("=") === undefined) {
                     const message = "Bad EXCEPTIONS syntax in CALL FUNCTION";
@@ -72560,6 +72611,18 @@ class ParserBadExceptions extends _abap_rule_1.ABAPRule {
         }
         return issues;
     }
+    containsMixedExceptionSyntax(exceptions) {
+        return exceptions.some(e => e.findDirectTokenByText("=") === undefined)
+            && exceptions.some(e => e.findDirectTokenByText("=") !== undefined);
+    }
+    findExceptionTokens(statement) {
+        const tokens = statement.getTokens();
+        const index = tokens.findIndex(t => t.getStr().toUpperCase() === "EXCEPTIONS");
+        if (index === -1) {
+            return undefined;
+        }
+        return tokens.slice(index).filter(t => t.getStr() !== ".");
+    }
 }
 exports.ParserBadExceptions = ParserBadExceptions;
 //# sourceMappingURL=parser_bad_exceptions.js.map
@@ -76256,6 +76319,7 @@ const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@ab
 const Objects = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
 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 include_graph_1 = __webpack_require__(/*! ../utils/include_graph */ "./node_modules/@abaplint/core/build/src/utils/include_graph.js");
 class SuperclassFinalConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.SuperclassFinalConf = SuperclassFinalConf;
@@ -76263,6 +76327,7 @@ class SuperclassFinal extends _abap_rule_1.ABAPRule {
     constructor() {
         super(...arguments);
         this.conf = new SuperclassFinalConf();
+        this.graph = undefined;
     }
     getMetadata() {
         return {
@@ -76301,6 +76366,9 @@ class SuperclassFinal extends _abap_rule_1.ABAPRule {
                     }
                 }
             }
+            if (found === undefined && obj instanceof Objects.Program) {
+                found = this.findInRelatedProgramIncludes(file, sup);
+            }
             if (found === undefined) {
                 const clas = this.reg.getObject("CLAS", sup);
                 if (clas) {
@@ -76320,6 +76388,33 @@ class SuperclassFinal extends _abap_rule_1.ABAPRule {
         }
         return output;
     }
+    findInRelatedProgramIncludes(file, superClassName) {
+        if (this.graph === undefined) {
+            this.graph = new include_graph_1.IncludeGraph(this.reg);
+        }
+        const mainFilenames = this.graph.listMainForInclude(file.getFilename());
+        if (mainFilenames.length === 0) {
+            return undefined;
+        }
+        for (const object of this.reg.getObjectsByType("PROG")) {
+            if (!(object instanceof Objects.Program)) {
+                continue;
+            }
+            const programFile = object.getMainABAPFile();
+            if (programFile === undefined) {
+                continue;
+            }
+            const programMainFilenames = this.graph.listMainForInclude(programFile.getFilename());
+            if (mainFilenames.some(filename => programMainFilenames.includes(filename)) === false) {
+                continue;
+            }
+            const found = programFile.getInfo().getClassDefinitionByName(superClassName);
+            if (found !== undefined) {
+                return found;
+            }
+        }
+        return undefined;
+    }
 }
 exports.SuperclassFinal = SuperclassFinal;
 //# sourceMappingURL=superclass_final.js.map
@@ -80618,6 +80713,9 @@ class DatabaseSetup {
         const populateTables = new populate_tables_1.PopulateTables(this.reg);
         // INSERT data
         for (const obj of this.reg.getObjects()) {
+            if (options?.populateTables?.tadir !== false) {
+                insert.push(populateTables.insertTADIR(obj));
+            }
             if (obj instanceof abaplint.Objects.MessageClass) {
                 insert.push(...populateTables.insertT100(obj));
             }
@@ -80667,6 +80765,7 @@ class PopulateTables {
     hasSEOSUBCOTX;
     hasT000;
     hasT100;
+    hasTADIR;
     hasWWWPARAMS;
     constructor(reg) {
         this.hasREPOSRC = reg.getObject("TABL", "REPOSRC") !== undefined;
@@ -80675,6 +80774,7 @@ class PopulateTables {
         this.hasSEOSUBCOTX = reg.getObject("TABL", "SEOSUBCOTX") !== undefined;
         this.hasT000 = reg.getObject("TABL", "T000") !== undefined;
         this.hasT100 = reg.getObject("TABL", "T100") !== undefined;
+        this.hasTADIR = reg.getObject("TABL", "TADIR") !== undefined;
         this.hasWWWPARAMS = reg.getObject("TABL", "WWWPARAMS") !== undefined;
     }
     insertREPOSRC(obj) {
@@ -80788,6 +80888,15 @@ class PopulateTables {
         // todo, this should take the client number from the json abap_transpile.json settings
         return `INSERT INTO t000 ("mandt", "cccategory", "ccnocliind") VALUES ('123', '', '');`;
     }
+    insertTADIR(obj) {
+        if (!this.hasTADIR) {
+            return "";
+        }
+        const type = obj.getType().toUpperCase();
+        const name = obj.getName().toUpperCase();
+        return `INSERT INTO "tadir" ("pgmid", "object", "obj_name", "devclass", "korrnum", "srcsystem", "delflag", "genflag", "edtflag", "masterlang")
+      VALUES ('R3TR', '${type}', '${this.escape(name)}', '$TMP', '', 'ABC', '', '', '', 'E');`;
+    }
     insertWWWPARAMS(obj) {
         if (!this.hasWWWPARAMS) {
             return [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.13.23",
+  "version": "2.13.25",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,8 +27,8 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.119.16",
-    "@abaplint/transpiler": "^2.13.23",
+    "@abaplint/core": "^2.119.19",
+    "@abaplint/transpiler": "^2.13.25",
     "@types/glob": "^8.1.0",
     "@types/node": "^24.12.2",
     "@types/progress": "^2.0.7",