@abaplint/cli 2.120.1 → 2.120.3

package/build/cli.js

@@ -26953,6 +26953,7 @@ const basic_1 = __webpack_require__(/*! ../types/basic */ "../core/build/src/aba
 const ddic_1 = __webpack_require__(/*! ../../ddic */ "../core/build/src/ddic.js");
 const _object_oriented_1 = __webpack_require__(/*! ./_object_oriented */ "../core/build/src/abap/5_syntax/_object_oriented.js");
 const _reference_1 = __webpack_require__(/*! ./_reference */ "../core/build/src/abap/5_syntax/_reference.js");
+const tokens_1 = __webpack_require__(/*! ../1_lexer/tokens */ "../core/build/src/abap/1_lexer/tokens/index.js");
 class Procedural {
     constructor(reg, scope) {
         this.scope = scope;
@@ -27132,7 +27133,8 @@ class Procedural {
                 continue;
             }
             else {
-                const type = new _typed_identifier_1.TypedIdentifier(nameToken, filename, found);
+                const token = new tokens_1.Identifier(nameToken.getStart(), param.name);
+                const type = new _typed_identifier_1.TypedIdentifier(token, filename, found, this.functionModuleParameterMeta(param));
                 if (ignoreIfAlreadyExists === true) {
                     const exists = this.scope.findVariable(param.name);
                     if (exists === undefined) {
@@ -27146,6 +27148,25 @@ class Procedural {
             }
         }
     }
+    functionModuleParameterMeta(param) {
+        const ret = [];
+        if (param.direction === types_1.FunctionModuleParameterDirection.importing) {
+            ret.push("function_module_importing" /* IdentifierMeta.FunctionModuleImporting */);
+        }
+        else if (param.direction === types_1.FunctionModuleParameterDirection.exporting) {
+            ret.push("function_module_exporting" /* IdentifierMeta.FunctionModuleExporting */);
+        }
+        else if (param.direction === types_1.FunctionModuleParameterDirection.changing) {
+            ret.push("function_module_changing" /* IdentifierMeta.FunctionModuleChanging */);
+        }
+        else if (param.direction === types_1.FunctionModuleParameterDirection.tables) {
+            ret.push("function_module_tables" /* IdentifierMeta.FunctionModuleTables */);
+        }
+        if (param.passByValue) {
+            ret.push("pass_by_value" /* IdentifierMeta.PassByValue */);
+        }
+        return ret;
+    }
 }
 exports.Procedural = Procedural;
 //# sourceMappingURL=_procedural.js.map
@@ -34057,6 +34078,7 @@ class Source {
                         else {
                             this.addIfInferred(node, input, foundType);
                         }
+                        this.traverseRemainingChildren(children, input);
                         return foundType ? foundType : bodyType;
                     }
                 case "SWITCH":
@@ -37127,8 +37149,10 @@ const inline_data_1 = __webpack_require__(/*! ../expressions/inline_data */ "../
 const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "../core/build/src/abap/5_syntax/expressions/dynamic.js");
 class CallTransformation {
     runSyntax(node, input) {
+        const resultParameters = node.findExpressionAfterToken("RESULT");
+        const resultSources = new Set((resultParameters === null || resultParameters === void 0 ? void 0 : resultParameters.findAllExpressions(Expressions.SimpleSource3)) || []);
         for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
-            source_1.Source.runSyntax(s, input);
+            source_1.Source.runSyntax(s, input, undefined, resultSources.has(s));
         }
         for (const d of node.findAllExpressions(Expressions.Dynamic)) {
             dynamic_1.Dynamic.runSyntax(d, input);
@@ -51701,6 +51725,7 @@ class FunctionModuleDefinition {
                     type: param.TYP || param.DBFIELD,
                     optional: param.OPTIONAL === "X",
                     defaultValue: param.DEFAULT,
+                    passByValue: param.REFERENCE !== "X",
                 });
             }
         }
@@ -51715,6 +51740,7 @@ class FunctionModuleDefinition {
                     type: param.TYP || param.DBFIELD,
                     optional: param.OPTIONAL === "X",
                     defaultValue: param.DEFAULT,
+                    passByValue: param.REFERENCE !== "X",
                 });
             }
         }
@@ -51729,6 +51755,7 @@ class FunctionModuleDefinition {
                     type: param.TYP || param.DBFIELD,
                     optional: true,
                     defaultValue: undefined,
+                    passByValue: param.REFERENCE !== "X",
                 });
             }
         }
@@ -51744,6 +51771,7 @@ class FunctionModuleDefinition {
                     type: param.DBSTRUCT || param.TYP,
                     optional: param.OPTIONAL === "X",
                     defaultValue: undefined,
+                    passByValue: false,
                 });
             }
         }
@@ -52596,14 +52624,14 @@ class MethodParameters {
         }
     }
     add(target, source, input, meta, abstractMethod) {
-        var _a, _b;
+        var _a;
         for (const opt of source.findAllExpressions(Expressions.MethodParamOptional)) {
             const p = opt.findDirectExpression(Expressions.MethodParam);
             if (p === undefined) {
                 continue;
             }
             const extraMeta = [];
-            if (p.getFirstToken().getStr().toUpperCase() === "VALUE" && ((_a = p.getChildren()[1]) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr()) === "(") {
+            if (this.isPassByValue(p)) {
                 extraMeta.push("pass_by_value" /* IdentifierMeta.PassByValue */);
             }
             else if (meta.includes("importing" /* IdentifierMeta.MethodImporting */)) {
@@ -52622,7 +52650,7 @@ class MethodParameters {
             else if (opt.findFirstExpression(Expressions.Default)) {
                 const name = target[target.length - 1].getName().toUpperCase();
                 this.optional.push(name);
-                const val = (_b = opt.findFirstExpression(Expressions.Default)) === null || _b === void 0 ? void 0 : _b.getLastChild();
+                const val = (_a = opt.findFirstExpression(Expressions.Default)) === null || _a === void 0 ? void 0 : _a.getLastChild();
                 if (val && val instanceof nodes_1.ExpressionNode) {
                     this.defaults[name] = val;
                 }
@@ -52633,9 +52661,15 @@ class MethodParameters {
         }
         const params = source.findAllExpressions(Expressions.MethodParam);
         for (const param of params) {
-            target.push(method_param_1.MethodParam.runSyntax(param, input, meta));
+            const extraMeta = this.isPassByValue(param) ? ["pass_by_value" /* IdentifierMeta.PassByValue */] : [];
+            target.push(method_param_1.MethodParam.runSyntax(param, input, [...meta, ...extraMeta]));
         }
     }
+    isPassByValue(param) {
+        var _a;
+        return param.getFirstToken().getStr().toUpperCase() === "VALUE"
+            && ((_a = param.getChildren()[1]) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr()) === "(";
+    }
 }
 exports.MethodParameters = MethodParameters;
 //# sourceMappingURL=method_parameters.js.map
@@ -59509,10 +59543,10 @@ class ABAPObject extends _abstract_object_1.AbstractObject {
         return undefined;
     }
     getMainABAPFile() {
-        // todo, uris, https://github.com/abaplint/abaplint/issues/673
         const search = this.getName().replace(/\//g, "#").toLowerCase() + "." + this.getType().toLowerCase() + ".abap";
+        const encodedSearch = search.replace(/#/g, "%23");
         for (const file of this.getABAPFiles()) {
-            if (file.getFilename().endsWith(search)) {
+            if (file.getFilename().endsWith(search) || file.getFilename().endsWith(encodedSearch)) {
                 return file;
             }
         }
@@ -68936,7 +68970,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.1";
+        return "2.120.3";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -73647,39 +73681,6 @@ exports.ClassicExceptionsOverlap = ClassicExceptionsOverlap;
 
 "use strict";
 
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.ClearExportingParameters = exports.ClearExportingParametersConf = void 0;
 const issue_1 = __webpack_require__(/*! ../issue */ "../core/build/src/issue.js");
@@ -73688,12 +73689,11 @@ const _irule_1 = __webpack_require__(/*! ./_irule */ "../core/build/src/rules/_i
 const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "../core/build/src/abap/5_syntax/syntax.js");
 const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "../core/build/src/objects/_abap_object.js");
 const _scope_type_1 = __webpack_require__(/*! ../abap/5_syntax/_scope_type */ "../core/build/src/abap/5_syntax/_scope_type.js");
+const _typed_identifier_1 = __webpack_require__(/*! ../abap/types/_typed_identifier */ "../core/build/src/abap/types/_typed_identifier.js");
 const objects_1 = __webpack_require__(/*! ../objects */ "../core/build/src/objects/index.js");
 const _reference_1 = __webpack_require__(/*! ../abap/5_syntax/_reference */ "../core/build/src/abap/5_syntax/_reference.js");
 const basic_1 = __webpack_require__(/*! ../abap/types/basic */ "../core/build/src/abap/types/basic/index.js");
 const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "../core/build/src/abap/2_statements/statements/_statement.js");
-const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "../core/build/src/edit_helper.js");
-const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
 class ClearExportingParametersConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -73726,7 +73726,7 @@ Reading and writing the parameter in the same statement (e.g. "ev_result = ev_re
 as the source is evaluated before the parameter is assigned.
 Objects containing parser or syntax errors are not reported.
 
-Only class and interface method implementations are checked, function module parameters are currently not supported.`,
+Class and interface method implementations and function modules are checked.`,
             tags: [_irule_1.RuleTag.Styleguide],
             badExample: `CLASS lcl DEFINITION.
   PUBLIC SECTION.
@@ -73782,19 +73782,20 @@ ENDCLASS.`,
         return issues;
     }
     traverse(node, obj, issues) {
-        if (node.getIdentifier().stype === _scope_type_1.ScopeType.Method) {
-            this.checkMethod(node, obj, issues);
+        if (node.getIdentifier().stype === _scope_type_1.ScopeType.Method
+            || node.getIdentifier().stype === _scope_type_1.ScopeType.FunctionModule) {
+            this.checkProcedure(node, obj, issues);
         }
         for (const child of node.getChildren()) {
             this.traverse(child, obj, issues);
         }
     }
-    checkMethod(node, obj, issues) {
-        const parameters = this.findExportingByReference(node, obj);
+    checkProcedure(node, obj, issues) {
+        const references = this.collectReferences(node);
+        const parameters = this.findExportingByReference(node, references);
         if (parameters.length === 0) {
             return;
         }
-        const references = this.collectReferences(node);
         for (const parameter of parameters) {
             const issue = this.checkParameter(parameter, references, obj);
             if (issue !== undefined) {
@@ -73802,52 +73803,36 @@ ENDCLASS.`,
             }
         }
     }
-    findExportingByReference(node, obj) {
+    findExportingByReference(node, references) {
         var _a;
         const ret = [];
-        const vars = node.getData().vars;
-        for (const name in vars) {
-            const parameter = vars[name];
+        const candidates = new Set(Object.values(node.getData().vars));
+        for (const reference of references) {
+            if (reference.resolved instanceof _typed_identifier_1.TypedIdentifier) {
+                candidates.add(reference.resolved);
+            }
+        }
+        for (const parameter of candidates) {
             const meta = parameter.getMeta();
-            if (meta.includes("exporting" /* IdentifierMeta.MethodExporting */) === false
+            const exporting = meta.includes("exporting" /* IdentifierMeta.MethodExporting */)
+                || meta.includes("function_module_exporting" /* IdentifierMeta.FunctionModuleExporting */);
+            if (exporting === false
                 || meta.includes("pass_by_value" /* IdentifierMeta.PassByValue */) === true) {
                 continue;
             }
-            else if ((_a = this.conf.skipNames) === null || _a === void 0 ? void 0 : _a.some(s => s.toUpperCase() === name.toUpperCase())) {
+            else if ((_a = this.conf.skipNames) === null || _a === void 0 ? void 0 : _a.some(s => s.toUpperCase() === parameter.getName().toUpperCase())) {
                 continue;
             }
             const type = parameter.getType();
             if (type instanceof basic_1.VoidType || type instanceof basic_1.UnknownType) {
                 continue; // e.g. RAP magic parameters, or unresolved types
             }
-            else if (this.isPassByValue(parameter, obj) === true) {
-                continue; // VALUE(..) parameters are always initialized
-            }
             ret.push(parameter);
         }
         return ret;
     }
-    isPassByValue(parameter, obj) {
-        var _a;
-        // the PassByValue meta is not set for EXPORTING parameters, so determine it from the definition
-        const file = obj.getABAPFileByName(parameter.getFilename());
-        if (file === undefined) {
-            return false;
-        }
-        const statement = edit_helper_1.EditHelper.findStatement(parameter.getToken(), file);
-        if (statement === undefined) {
-            return false;
-        }
-        for (const param of statement.findAllExpressions(Expressions.MethodParam)) {
-            const nameToken = (_a = param.findFirstExpression(Expressions.MethodParamName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
-            if (nameToken !== undefined && nameToken.getStart().equals(parameter.getStart())) {
-                return param.getFirstToken().getStr().toUpperCase() === "VALUE";
-            }
-        }
-        return false;
-    }
     collectReferences(node) {
-        // methods do not nest, so all descendant scopes (FOR, LET, ...) belong to this method
+        // procedures do not nest, so all descendant scopes (FOR, LET, ...) belong to this procedure
         const ret = [...node.getData().references];
         for (const child of node.getChildren()) {
             ret.push(...this.collectReferences(child));
@@ -73858,7 +73843,9 @@ ENDCLASS.`,
         let firstRead = undefined;
         let earliestWrite = undefined;
         for (const reference of references) {
-            if (reference.resolved === undefined || parameter.equals(reference.resolved) === false) {
+            if (reference.resolved === undefined
+                || parameter.equals(reference.resolved) === false
+                || parameter.getName().toUpperCase() !== reference.resolved.getName().toUpperCase()) {
                 continue;
             }
             const pos = reference.position.getStart();
@@ -73868,8 +73855,8 @@ ENDCLASS.`,
                 }
             }
             else if (reference.referenceType === _reference_1.ReferenceType.DataWriteReference) {
-                if (earliestWrite === undefined || pos.isBefore(earliestWrite)) {
-                    earliestWrite = pos;
+                if (earliestWrite === undefined || pos.isBefore(earliestWrite.position.getStart())) {
+                    earliestWrite = reference;
                 }
             }
         }
@@ -73877,7 +73864,7 @@ ENDCLASS.`,
             return undefined; // never read, no hazard
         }
         // the parameter is safe only if it is written in a statement strictly before the first read
-        const readStatement = this.statementStart(firstRead.position.getStart(), obj);
+        const readStatement = this.statementStart(firstRead, obj);
         const writeStatement = earliestWrite === undefined ? undefined : this.statementStart(earliestWrite, obj);
         if (writeStatement !== undefined && readStatement !== undefined && writeStatement.isBefore(readStatement)) {
             return undefined;
@@ -73885,14 +73872,14 @@ ENDCLASS.`,
         const message = `EXPORTING parameter "${parameter.getName().toLowerCase()}" read before it is cleared or assigned`;
         return issue_1.Issue.atIdentifier(firstRead.position, message, this.getMetadata().key, this.conf.severity);
     }
-    statementStart(pos, obj) {
-        for (const file of obj.getABAPFiles()) {
-            for (const statement of file.getStatements()) {
-                const start = statement.getStart();
-                const end = statement.getEnd();
-                if (pos.equals(start) || (pos.isAfter(start) && pos.isBefore(end))) {
-                    return start;
-                }
+    statementStart(reference, obj) {
+        const file = obj.getABAPFileByName(reference.position.getFilename());
+        for (const statement of (file === null || file === void 0 ? void 0 : file.getStatements()) || []) {
+            const pos = reference.position.getStart();
+            const start = statement.getStart();
+            const end = statement.getEnd();
+            if (pos.equals(start) || (pos.isAfter(start) && pos.isBefore(end))) {
+                return start;
             }
         }
         return undefined;
@@ -93060,7 +93047,8 @@ ENDCLASS.`,
             }
             for (const v in vars) {
                 const id = vars[v];
-                if (id.getMeta().includes("pass_by_value" /* IdentifierMeta.PassByValue */) === false) {
+                if (id.getMeta().includes("importing" /* IdentifierMeta.MethodImporting */) === false
+                    || id.getMeta().includes("pass_by_value" /* IdentifierMeta.PassByValue */) === false) {
                     continue;
                 }
                 else if (this.reg.isFileDependency(id.getFilename()) === true) {

package/package.json

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