@abaplint/cli 2.119.16 → 2.119.18

package/build/cli.js

@@ -7174,10 +7174,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)))));
     }
 }
@@ -23595,6 +23596,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) {
@@ -23845,6 +23847,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;
@@ -23858,7 +23872,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);
@@ -23917,16 +23931,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;
@@ -23936,7 +23949,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;
@@ -23980,7 +23993,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;
@@ -26251,8 +26264,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) {
@@ -30844,6 +30857,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);
@@ -32398,11 +32417,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;
@@ -43305,6 +43328,7 @@ class Config {
                 skipGeneratedProxyInterfaces: false,
                 useApackDependencies: false,
                 skipIncludesWithoutMain: false,
+                errorOnDuplicateFilenames: false,
             },
             dependencies: [{
                     url: "https://github.com/abaplint/deps",
@@ -43368,6 +43392,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() {
@@ -56285,7 +56312,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.119.16";
+        return "2.119.18";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -56405,10 +56432,29 @@ class Registry {
                 this.removeDependency(found);
                 found = this.findOrCreate(f.getObjectName(), f.getObjectType());
             }
+            if (this.conf.getGlobal().errorOnDuplicateFilenames === true) {
+                this.checkDuplicateFilename(filename, found.getFiles());
+            }
             found.addFile(f);
         }
         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;
@@ -73623,10 +73669,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() {
@@ -73641,6 +73683,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";
@@ -73650,6 +73701,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
@@ -77346,6 +77409,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;
@@ -77353,6 +77417,7 @@ class SuperclassFinal extends _abap_rule_1.ABAPRule {
     constructor() {
         super(...arguments);
         this.conf = new SuperclassFinalConf();
+        this.graph = undefined;
     }
     getMetadata() {
         return {
@@ -77391,6 +77456,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) {
@@ -77410,6 +77478,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

package/package.json

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