@abaplint/core 2.113.139 → 2.113.141

package/build/src/abap/2_statements/expressions/select_loop.js

@@ -20,7 +20,7 @@ class SelectLoop extends combi_1.Expression {
         const tab = (0, combi_1.seq)(_1.SQLIntoTable, (0, combi_1.alt)(pack, (0, combi_1.seq)(_1.SQLFrom, pack), (0, combi_1.seq)(pack, _1.SQLFrom)));
         const packTab = (0, combi_1.seq)(pack, _1.SQLIntoTable);
         const into = (0, combi_1.altPrio)(sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList);
-        const perm = (0, combi_1.per)(_1.SQLFrom, where, sql_up_to_1.SQLUpTo, sql_order_by_1.SQLOrderBy, sql_having_1.SQLHaving, _1.SQLClient, bypass, _1.SQLGroupBy, _1.SQLForAllEntries, (0, combi_1.alt)(tab, into, packTab));
+        const perm = (0, combi_1.per)(_1.SQLFrom, where, sql_up_to_1.SQLUpTo, sql_order_by_1.SQLOrderBy, sql_having_1.SQLHaving, _1.SQLClient, bypass, _1.SQLGroupBy, _1.SQLForAllEntries, (0, combi_1.alt)(tab, sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList, packTab));
         const strict = (0, combi_1.seq)(_1.SQLFrom, (0, combi_1.ver)(version_1.Version.v750, sql_fields_loop_1.SQLFieldsLoop), (0, combi_1.optPrio)(_1.SQLForAllEntries), (0, combi_1.optPrio)((0, combi_1.seq)(where, (0, combi_1.optPrio)(sql_order_by_1.SQLOrderBy), into, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo))));
         const aggr = (0, combi_1.seq)(_1.SQLAggregation, into, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), _1.SQLFrom, where, _1.SQLGroupBy);
         const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)((0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), sql_field_list_loop_1.SQLFieldListLoop, perm), strict, aggr), (0, combi_1.optPrio)(sql_hints_1.SQLHints));

package/build/src/abap/2_statements/expressions/sql_field.js

@@ -6,11 +6,10 @@ const _1 = require(".");
 const version_1 = require("../../../version");
 const tokens_1 = require("../../1_lexer/tokens");
 const sql_function_1 = require("./sql_function");
-const simple_field_chain_1 = require("./simple_field_chain");
 const sql_path_1 = require("./sql_path");
 class SQLField extends combi_1.Expression {
     getRunnable() {
-        const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), simple_field_chain_1.SimpleFieldChain));
+        const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), _1.SimpleFieldChain2));
         const as = (0, combi_1.seq)("AS", _1.SQLAsName);
         const field = (0, combi_1.altPrio)(_1.SQLAggregation, _1.SQLCase, sql_function_1.SQLFunction, sql_path_1.SQLPath, _1.SQLFieldName, abap, _1.Constant);
         const sub = (0, combi_1.plusPrio)((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))));

package/build/src/abap/2_statements/expressions/sql_field_list_loop.js

@@ -12,7 +12,7 @@ class SQLFieldListLoop extends combi_1.Expression {
         const comma = (0, combi_1.opt)((0, combi_1.ver)(version_1.Version.v740sp05, ","));
         const as = (0, combi_1.seq)("AS", _1.SQLAsName);
         const someField = (0, combi_1.seq)(_1.SQLField, comma);
-        const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), _1.SimpleFieldChain));
+        const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), _1.SimpleFieldChain2));
         const fieldList = (0, combi_1.seq)((0, combi_1.star)(someField), (0, combi_1.alt)(_1.SQLFieldName, abap, sql_path_1.SQLPath, _1.Constant), (0, combi_1.optPrio)(as), comma, (0, combi_1.star)(someField));
         const fields = (0, combi_1.alt)("*", _1.Dynamic, fieldList);
         return fields;

package/build/src/abap/2_statements/statements/ranges.js

@@ -7,7 +7,7 @@ const version_1 = require("../../../version");
 class Ranges {
     getMatcher() {
         const occurs = (0, combi_1.seq)("OCCURS", expressions_1.Source);
-        const ret = (0, combi_1.seq)("RANGES", expressions_1.SimpleName, "FOR", expressions_1.SimpleFieldChain2, (0, combi_1.optPrio)(occurs), (0, combi_1.optPrio)(expressions_1.FieldLength));
+        const ret = (0, combi_1.seq)("RANGES", expressions_1.DefinitionName, "FOR", expressions_1.SimpleFieldChain2, (0, combi_1.optPrio)(occurs), (0, combi_1.optPrio)(expressions_1.FieldLength));
         return (0, combi_1.verNot)(version_1.Version.Cloud, ret);
     }
 }

package/build/src/abap/5_syntax/statements/ranges.js

@@ -9,7 +9,7 @@ const assert_error_1 = require("../assert_error");
 class Ranges {
     runSyntax(node, input) {
         var _a;
-        const nameToken = (_a = node.findFirstExpression(Expressions.SimpleName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+        const nameToken = (_a = node.findFirstExpression(Expressions.DefinitionName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
         const typeExpression = node.findFirstExpression(Expressions.SimpleFieldChain2);
         if (typeExpression === undefined) {
             throw new assert_error_1.AssertError("Ranges, unexpected node");

package/build/src/registry.js

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

package/build/src/rules_runner.js

@@ -72,8 +72,9 @@ class RulesRunner {
         }
         (_c = input === null || input === void 0 ? void 0 : input.progress) === null || _c === void 0 ? void 0 : _c.set(rules.length, "Initialize Rules");
         for (const rule of rules) {
+            const start = Date.now();
             if ((input === null || input === void 0 ? void 0 : input.outputPerformance) === true) {
-                process.stderr.write("Initializing rule " + rule.getMetadata().key + "\n");
+                process.stderr.write("Initializing rule " + rule.getMetadata().key);
             }
             else {
                 (_d = input === null || input === void 0 ? void 0 : input.progress) === null || _d === void 0 ? void 0 : _d.tick("Initialize Rules - " + rule.getMetadata().key);
@@ -82,6 +83,9 @@ class RulesRunner {
                 throw new Error(rule.getMetadata().key + " missing initialize method");
             }
             rule.initialize(this.reg);
+            if ((input === null || input === void 0 ? void 0 : input.outputPerformance) === true) {
+                process.stderr.write(", " + (Date.now() - start) + "ms\n");
+            }
             rulePerformance[rule.getMetadata().key] = 0;
         }
         (_e = input === null || input === void 0 ? void 0 : input.progress) === null || _e === void 0 ? void 0 : _e.set(check.length, "Finding Issues");

package/build/src/utils/include_graph.js

@@ -66,9 +66,6 @@ class IncludeGraph {
         this.graph = new Graph();
         this.build();
     }
-    getIssues() {
-        return this.issues;
-    }
     listMainForInclude(filename) {
         const ret = [];
         if (filename === undefined) {
@@ -88,6 +85,17 @@ class IncludeGraph {
                 ret.push(i);
             }
         }
+        const v = this.graph.findVertexByFilename(file.getFilename());
+        if (v !== undefined
+            && v.include === true
+            && this.listMainForInclude(v.filename).length === 0) {
+            const f = this.reg.getFileByName(v.filename);
+            if (f === undefined) {
+                throw new Error("findUnusedIncludes internal error");
+            }
+            const issue = issue_1.Issue.atPosition(f, new position_1.Position(1, 1), "INCLUDE not used anywhere", new check_include_1.CheckInclude().getMetadata().key, severity_1.Severity.Error);
+            ret.push(issue);
+        }
         return ret;
     }
     ///////////////////////////////
@@ -128,21 +136,6 @@ class IncludeGraph {
                 }
             }
         }
-        this.findUnusedIncludes();
-    }
-    findUnusedIncludes() {
-        for (const v of Object.values(this.graph.verticesFilenameIndex)) {
-            if (v.include === true) {
-                if (this.listMainForInclude(v.filename).length === 0) {
-                    const f = this.reg.getFileByName(v.filename);
-                    if (f === undefined) {
-                        throw new Error("findUnusedIncludes internal error");
-                    }
-                    const issue = issue_1.Issue.atPosition(f, new position_1.Position(1, 1), "INCLUDE not used anywhere", new check_include_1.CheckInclude().getMetadata().key, severity_1.Severity.Error);
-                    this.issues.push(issue);
-                }
-            }
-        }
     }
     addVertices() {
         for (const o of getABAPObjects(this.reg)) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.113.139",
+    "version": "2.113.141",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -53,7 +53,7 @@
         "@microsoft/api-extractor": "^7.52.8",
         "@types/chai": "^4.3.20",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.0.10",
+        "@types/node": "^24.0.12",
         "chai": "^4.5.0",
         "eslint": "^9.30.1",
         "mocha": "^11.7.1",