@abaplint/cli 2.120.2 → 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
@@ -51704,6 +51725,7 @@ class FunctionModuleDefinition {
                     type: param.TYP || param.DBFIELD,
                     optional: param.OPTIONAL === "X",
                     defaultValue: param.DEFAULT,
+                    passByValue: param.REFERENCE !== "X",
                 });
             }
         }
@@ -51718,6 +51740,7 @@ class FunctionModuleDefinition {
                     type: param.TYP || param.DBFIELD,
                     optional: param.OPTIONAL === "X",
                     defaultValue: param.DEFAULT,
+                    passByValue: param.REFERENCE !== "X",
                 });
             }
         }
@@ -51732,6 +51755,7 @@ class FunctionModuleDefinition {
                     type: param.TYP || param.DBFIELD,
                     optional: true,
                     defaultValue: undefined,
+                    passByValue: param.REFERENCE !== "X",
                 });
             }
         }
@@ -51747,6 +51771,7 @@ class FunctionModuleDefinition {
                     type: param.DBSTRUCT || param.TYP,
                     optional: param.OPTIONAL === "X",
                     defaultValue: undefined,
+                    passByValue: false,
                 });
             }
         }
@@ -52599,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 */)) {
@@ -52625,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;
                 }
@@ -52636,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
@@ -59512,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;
             }
         }
@@ -68939,7 +68970,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.2";
+        return "2.120.3";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -73650,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");
@@ -73691,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);
@@ -73729,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.
@@ -73785,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) {
@@ -73805,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));
@@ -73861,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();
@@ -93063,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.2",
+    "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.2",
+        "@abaplint/core": "^2.120.3",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",