@abaplint/core 2.91.7 → 2.91.10

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

@@ -8,7 +8,7 @@ class GetLocale {
     getMatcher() {
         const country = (0, combi_1.seq)("COUNTRY", expressions_1.Target);
         const modifier = (0, combi_1.seq)("MODIFIER", expressions_1.Target);
-        const ret = (0, combi_1.seq)("GET LOCALE LANGUAGE", expressions_1.Target, country, (0, combi_1.opt)(modifier));
+        const ret = (0, combi_1.seq)("GET LOCALE LANGUAGE", expressions_1.Target, country, modifier);
         return (0, combi_1.verNot)(version_1.Version.Cloud, ret);
     }
 }

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

@@ -27,6 +27,11 @@ class ValueBody {
         for (const s of node.findDirectExpressions(Expressions.Source)) {
             type = new source_1.Source().runSyntax(s, scope, filename);
         }
+        for (const l of node.findDirectExpressions(Expressions.ValueBodyLines)) {
+            for (const s of l.findDirectExpressions(Expressions.Source)) {
+                new source_1.Source().runSyntax(s, scope, filename);
+            }
+        }
         if (scoped === true) {
             scope.pop(node.getLastToken().getEnd());
         }

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

@@ -141,6 +141,9 @@ class MethodParameters {
             this.add(this.importing, importing, scope, ["importing" /* IdentifierMeta.MethodImporting */]);
             if (importing.concatTokens().toUpperCase().includes(" PREFERRED PARAMETER")) {
                 this.preferred = importing.getLastToken().getStr().toUpperCase();
+                if (this.preferred.startsWith("!")) {
+                    this.preferred = this.preferred.substring(1);
+                }
             }
         }
         const exporting = node.findFirstExpression(Expressions.MethodDefExporting);

package/build/src/registry.js

@@ -68,7 +68,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.91.7";
+        return "2.91.10";
     }
     getDDICReferences() {
         return this.references;

package/build/src/rules/max_one_statement.js

@@ -23,6 +23,8 @@ class MaxOneStatement extends _abap_rule_1.ABAPRule {
             shortDescription: `Checks that each line contains only a single statement.`,
             extendedInformation: `Does not report empty statements, use rule empty_statement for detecting empty statements.
 
+Does not report anything for chained statements.
+
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#no-more-than-one-statement-per-line
 https://docs.abapopenchecks.org/checks/11/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],

package/build/src/rules/unknown_types.js

@@ -99,7 +99,7 @@ class UnknownTypes {
             }
         }
         for (const v of nodeData.idefs) {
-            const found = this.checkMethodParameters(v);
+            const found = this.checkParameters(v);
             if (found) {
                 const message = "Contains unknown, " + found.found;
                 ret.push(issue_1.Issue.atIdentifier(found.id, message, this.getMetadata().key, this.conf.severity));
@@ -107,7 +107,7 @@ class UnknownTypes {
         }
         for (const name in nodeData.cdefs) {
             const v = nodeData.cdefs[name];
-            const found = this.checkMethodParameters(v);
+            const found = this.checkParameters(v);
             if (found) {
                 const message = "Contains unknown, " + found.found;
                 ret.push(issue_1.Issue.atIdentifier(found.id, message, this.getMetadata().key, this.conf.severity));
@@ -118,7 +118,7 @@ class UnknownTypes {
         }
         return ret;
     }
-    checkMethodParameters(idef) {
+    checkParameters(idef) {
         var _a;
         for (const m of ((_a = idef.getMethodDefinitions()) === null || _a === void 0 ? void 0 : _a.getAll()) || []) {
             for (const p of m.getParameters().getAll()) {
@@ -128,6 +128,14 @@ class UnknownTypes {
                 }
             }
         }
+        for (const e of idef.getEvents() || []) {
+            for (const p of e.getParameters()) {
+                const found = this.containsUnknown(p.getType());
+                if (found) {
+                    return { id: p, found };
+                }
+            }
+        }
         return undefined;
     }
     containsUnknown(type) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.91.7",
+    "version": "2.91.10",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -45,21 +45,21 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@microsoft/api-extractor": "^7.28.3",
+        "@microsoft/api-extractor": "^7.28.4",
         "@types/chai": "^4.3.1",
         "@types/mocha": "^9.1.1",
-        "@types/node": "^18.0.3",
+        "@types/node": "^18.0.6",
         "chai": "^4.3.6",
-        "eslint": "^8.19.0",
+        "eslint": "^8.20.0",
         "mocha": "^10.0.0",
-        "c8": "^7.11.3",
+        "c8": "^7.12.0",
         "source-map-support": "^0.5.21",
         "ts-json-schema-generator": "^1.0.0",
         "typescript": "^4.7.4"
     },
     "dependencies": {
-        "fast-xml-parser": "^4.0.8",
+        "fast-xml-parser": "^4.0.9",
         "json5": "^2.2.1",
-        "vscode-languageserver-types": "^3.17.1"
+        "vscode-languageserver-types": "^3.17.2"
     }
 }