@abaplint/cli 2.102.51 → 2.102.53

package/build/cli.js

@@ -7516,7 +7516,7 @@ const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/co
 const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 class SQLFields extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.seq)("FIELDS", _1.SQLFieldList);
+        return (0, combi_1.seq)("FIELDS", (0, combi_1.opt)("DISTINCT"), _1.SQLFieldList);
     }
 }
 exports.SQLFields = SQLFields;
@@ -25674,20 +25674,23 @@ class Select {
                     if (isDynamic) {
                         throw new Error(`dynamic field list, inlining not possible`);
                     }
-                    if (isSimple.test(field.code) && dbSources.length === 1 && dbSources[0] !== undefined) {
-                        const dbType = dbSources[0].parseType(scope.getRegistry());
-                        let type = new basic_1.VoidType("SELECT_todo");
-                        if (dbType instanceof basic_1.StructureType) {
-                            type = dbType.getComponentByName(field.code);
-                            if (type === undefined) {
-                                throw new Error(`handleInto, internal error, should be checked earlier`);
+                    let type = new basic_1.VoidType("SELECT_todo");
+                    if (isSimple.test(field.code)) {
+                        for (const dbSource of dbSources) {
+                            if (dbSource === undefined) {
+                                continue;
+                            }
+                            const dbType = dbSource.parseType(scope.getRegistry());
+                            if (dbType instanceof basic_1.StructureType) {
+                                const found = dbType.getComponentByName(field.code);
+                                if (found) {
+                                    type = found;
+                                    break;
+                                }
                             }
                         }
-                        new inline_data_1.InlineData().runSyntax(inline, scope, filename, type);
-                    }
-                    else {
-                        new inline_data_1.InlineData().runSyntax(inline, scope, filename, new basic_1.VoidType("SELECT_todo"));
                     }
+                    new inline_data_1.InlineData().runSyntax(inline, scope, filename, type);
                 }
             }
         }
@@ -36940,7 +36943,8 @@ class Message {
         return this.message;
     }
     getPlaceholderCount() {
-        return (this.getMessage().match(/&/g) || []).length;
+        const escaped = (this.getMessage().match(/&&/g) || []).length;
+        return (this.getMessage().match(/&/g) || []).length - escaped * 2;
     }
 }
 exports.Message = Message;
@@ -49145,7 +49149,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.51";
+        return "2.102.53";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -61906,6 +61910,11 @@ const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./
 const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
 const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
 class MSAGConsistencyConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        /** paramters must be numbered */
+        this.numericParamters = true;
+    }
 }
 exports.MSAGConsistencyConf = MSAGConsistencyConf;
 class MSAGConsistency {
@@ -61939,7 +61948,7 @@ class MSAGConsistency {
         }
         const numbers = new Set();
         for (const message of obj.getMessages()) {
-            // todo, get the right positions in xml file
+            // todo, get the right positions in xml file, and report the issue there
             if (!message.getNumber().match(/\d\d\d/)) {
                 const text = this.getDescription("Message number must be 3 digits: message " + message.getNumber());
                 const position = new position_1.Position(1, 1);
@@ -61962,6 +61971,25 @@ class MSAGConsistency {
             else {
                 numbers.add(num);
             }
+            if (this.getConfig().numericParamters === true) {
+                const placeholderCount = message.getPlaceholderCount();
+                if (placeholderCount > 4) {
+                    const text = `More than 4 placeholders in mesasge ${message.getNumber()}`;
+                    const position = new position_1.Position(1, 1);
+                    const issue = issue_1.Issue.atPosition(obj.getFiles()[0], position, text, this.getMetadata().key, this.conf.severity);
+                    issues.push(issue);
+                }
+                for (let i = 1; i <= placeholderCount; i++) {
+                    const placeholder = "&" + i;
+                    if (message.getMessage().includes(placeholder) === false) {
+                        const text = `Expected placeholder ${placeholder} in message ${message.getNumber()}`;
+                        const position = new position_1.Position(1, 1);
+                        const issue = issue_1.Issue.atPosition(obj.getFiles()[0], position, text, this.getMetadata().key, this.conf.severity);
+                        issues.push(issue);
+                        break;
+                    }
+                }
+            }
         }
         return issues;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.51",
+    "version": "2.102.53",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,14 +38,14 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.51",
+        "@abaplint/core": "^2.102.53",
         "@types/chai": "^4.3.6",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.3",
         "@types/mocha": "^10.0.2",
-        "@types/node": "^20.7.1",
+        "@types/node": "^20.8.0",
         "@types/progress": "^2.0.5",
-        "chai": "^4.3.9",
+        "chai": "^4.3.10",
         "chalk": "^5.3.0",
         "eslint": "^8.50.0",
         "glob": "^7.2.3",