@abaplint/core 2.102.60 → 2.102.62

package/build/src/abap/1_lexer/tokens/_token.js

@@ -3,6 +3,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.Token = void 0;
 const position_1 = require("../../../position");
 class Token {
+    [Symbol.for("debug.description")]() {
+        return `${this.constructor.name} ${this.str}`;
+    }
     constructor(start, str) {
         this.start = start;
         this.str = str;

package/build/src/abap/5_syntax/expressions/sql_compare.js

@@ -4,12 +4,16 @@ exports.SQLCompare = void 0;
 const Expressions = require("../../2_statements/expressions");
 const nodes_1 = require("../../nodes");
 const basic_1 = require("../../types/basic");
+const source_1 = require("./source");
 const sql_source_1 = require("./sql_source");
 class SQLCompare {
     runSyntax(node, scope, filename, tables) {
         var _a;
         let sourceType;
         let token;
+        for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
+            new source_1.Source().runSyntax(s, scope, filename);
+        }
         for (const s of node.findAllExpressions(Expressions.SQLSource)) {
             for (const child of s.getChildren()) {
                 if (child instanceof nodes_1.ExpressionNode) {

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

@@ -23,6 +23,9 @@ class ScopeData {
     }
 }
 class SpaghettiScopeNode extends ScopeData {
+    [Symbol.for("debug.description")]() {
+        return `SpaghettiSN ${this.identifier.sname} ${this.identifier.stype}`;
+    }
     constructor(identifier, parent) {
         super();
         this.identifier = identifier;

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

@@ -3,6 +3,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.TypedIdentifier = void 0;
 const _identifier_1 = require("../4_file_information/_identifier");
 class TypedIdentifier extends _identifier_1.Identifier {
+    [Symbol.for("debug.description")]() {
+        return `${this.constructor.name} ${this.getName()}:${this.getType().constructor.name}`;
+    }
     static from(id, type, meta) {
         return new TypedIdentifier(id.getToken(), id.getFilename(), type, meta);
     }

package/build/src/objects/_abap_object.js

@@ -5,6 +5,9 @@ const _abstract_object_1 = require("./_abstract_object");
 const xml_utils_1 = require("../xml_utils");
 const abap_parser_1 = require("../abap/abap_parser");
 class ABAPObject extends _abstract_object_1.AbstractObject {
+    [Symbol.for("debug.description")]() {
+        return `${this.constructor.name} ${this.getName()}`;
+    }
     constructor(name) {
         super(name);
         this.parsed = [];

package/build/src/registry.js

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

package/build/src/rules/check_subrc.js

@@ -38,7 +38,7 @@ class CheckSubrc extends _abap_rule_1.ABAPRule {
 
 If sy-dbcnt is checked after database statements, it is considered okay.
 
-"SELECT SINGLE @abap_true FROM " is considered as an existence check
+"SELECT SINGLE @abap_true FROM " is considered as an existence check, also "SELECT COUNT( * )" is considered okay
 
 If IS ASSIGNED is checked after assigning, it is considered okay.
 
@@ -89,6 +89,8 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
             else if (config.selectTable === true
                 && statement.get() instanceof Statements.Select
                 && statement.concatTokens().toUpperCase().startsWith("SELECT SINGLE ") === false
+                && statement.concatTokens().toUpperCase().startsWith("SELECT COUNT( * ) ") === false
+                && statement.concatTokens().toUpperCase().startsWith("SELECT COUNT(*) ") === false
                 && this.isChecked(i, statements) === false
                 && this.checksDbcnt(i, statements) === false) {
                 issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.102.60",
+    "version": "2.102.62",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -53,9 +53,9 @@
         "@microsoft/api-extractor": "^7.38.0",
         "@types/chai": "^4.3.9",
         "@types/mocha": "^10.0.3",
-        "@types/node": "^20.8.7",
+        "@types/node": "^20.8.8",
         "chai": "^4.3.10",
-        "eslint": "^8.51.0",
+        "eslint": "^8.52.0",
         "mocha": "^10.2.0",
         "c8": "^8.0.1",
         "source-map-support": "^0.5.21",