@abaplint/cli 2.102.6 → 2.102.8

package/build/cli.js

@@ -19249,7 +19249,12 @@ class BuiltInMethod extends _identifier_1.Identifier {
         this.row = row;
     }
     getRequiredParameters() {
-        return [];
+        const ret = [];
+        for (const i in this.method.mandatory) {
+            const id = new tokens_1.Identifier(new position_1.Position(this.row, 1), i);
+            ret.push(new _typed_identifier_1.TypedIdentifier(id, BuiltIn.filename, this.method.mandatory[i]));
+        }
+        return ret;
     }
     getOptional() {
         throw new Error("BuiltInMethod->Method not implemented.");
@@ -27133,7 +27138,7 @@ class Assign {
         const theSource = sources[sources.length - 1];
         let sourceType = new source_1.Source().runSyntax(theSource, scope, filename);
         if (sourceType === undefined || ((_b = node.findDirectExpression(Expressions.AssignSource)) === null || _b === void 0 ? void 0 : _b.findDirectExpression(Expressions.Dynamic))) {
-            sourceType = new basic_1.VoidType("DynamicAssign");
+            sourceType = new basic_1.AnyType();
         }
         for (const d of ((_c = node.findDirectExpression(Expressions.AssignSource)) === null || _c === void 0 ? void 0 : _c.findAllExpressions(Expressions.Dynamic)) || []) {
             new dynamic_1.Dynamic().runSyntax(d, scope, filename);
@@ -27640,10 +27645,15 @@ const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ ".
 const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 class ClassLocalFriends {
     runSyntax(node, scope, filename) {
-        const found = node.findDirectExpression(Expressions.ClassName);
+        const classNames = node.findAllExpressions(Expressions.ClassName);
+        const found = classNames[0];
         if (found) {
             const token = found.getFirstToken();
             const name = token.getStr();
+            if (scope.getParentObj().getType() === "CLAS"
+                && name.toUpperCase() !== scope.getParentObj().getName().toUpperCase()) {
+                throw new Error(`Befriending must be ` + scope.getParentObj().getName().toUpperCase());
+            }
             const def = scope.findClassDefinition(name);
             if (def) {
                 scope.addReference(token, def, _reference_1.ReferenceType.ObjectOrientedReference, filename);
@@ -27652,6 +27662,14 @@ class ClassLocalFriends {
                 throw new Error(`Class ${name.toUpperCase()} not found`);
             }
         }
+        for (let i = 1; i < classNames.length; i++) {
+            const className = classNames[i].concatTokens();
+            // make sure to check also DEFINITION DEFERRED
+            const found = scope.existsObject(className);
+            if (found.found === false) {
+                throw new Error(`Class ${className.toUpperCase()} not found`);
+            }
+        }
     }
 }
 exports.ClassLocalFriends = ClassLocalFriends;
@@ -47353,46 +47371,45 @@ class TableType extends _abstract_object_1.AbstractObject {
         this.parsedXML = undefined;
         super.setDirty();
     }
-    buildTableOptions() {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k;
-        let primaryKey = undefined;
-        if (((_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.accessmode) === "S") {
-            primaryKey = {
-                isUnique: ((_b = this.parsedXML) === null || _b === void 0 ? void 0 : _b.keykind) === "U",
-                type: basic_1.TableAccessType.sorted,
-                keyFields: [],
-                name: "primary_key",
-            };
+    buildPrimaryKey() {
+        var _a, _b, _c, _d, _e;
+        const primaryKey = {
+            isUnique: ((_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.keykind) === "U",
+            type: basic_1.TableAccessType.standard,
+            keyFields: [],
+            name: "primary_key",
+        };
+        if (((_b = this.parsedXML) === null || _b === void 0 ? void 0 : _b.accessmode) === "S") {
+            primaryKey.type = basic_1.TableAccessType.sorted;
         }
         else if (((_c = this.parsedXML) === null || _c === void 0 ? void 0 : _c.accessmode) === "H") {
-            primaryKey = {
-                isUnique: ((_d = this.parsedXML) === null || _d === void 0 ? void 0 : _d.keykind) === "U",
-                type: basic_1.TableAccessType.hashed,
-                keyFields: [],
-                name: "primary_key",
-            };
+            primaryKey.type = basic_1.TableAccessType.hashed;
         }
-        for (const f of ((_e = this.parsedXML) === null || _e === void 0 ? void 0 : _e.dd42v) || []) {
+        for (const f of ((_d = this.parsedXML) === null || _d === void 0 ? void 0 : _d.dd42v) || []) {
             if (f.keyname === "") {
-                primaryKey === null || primaryKey === void 0 ? void 0 : primaryKey.keyFields.push(f.keyfield);
+                primaryKey.keyFields.push(f.keyfield);
             }
         }
-        if (((_f = this.parsedXML) === null || _f === void 0 ? void 0 : _f.keydef) === "T" && (primaryKey === null || primaryKey === void 0 ? void 0 : primaryKey.keyFields.length) === 0) {
-            primaryKey === null || primaryKey === void 0 ? void 0 : primaryKey.keyFields.push("table_line");
+        if (((_e = this.parsedXML) === null || _e === void 0 ? void 0 : _e.keydef) === "T" && primaryKey.keyFields.length === 0) {
+            primaryKey.keyFields.push("table_line");
         }
+        return primaryKey;
+    }
+    buildTableOptions() {
+        var _a, _b, _c, _d;
         let keyType = Types.TableKeyType.user;
-        if (((_g = this.parsedXML) === null || _g === void 0 ? void 0 : _g.keydef) === "D") {
+        if (((_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.keydef) === "D") {
             keyType = Types.TableKeyType.default;
         }
         const tableOptions = {
             withHeader: false,
             keyType: keyType,
-            primaryKey: primaryKey,
+            primaryKey: this.buildPrimaryKey(),
             secondary: [],
         };
-        for (const k of ((_h = this.parsedXML) === null || _h === void 0 ? void 0 : _h.dd43v) || []) {
+        for (const k of ((_b = this.parsedXML) === null || _b === void 0 ? void 0 : _b.dd43v) || []) {
             const fields = [];
-            for (const f of ((_j = this.parsedXML) === null || _j === void 0 ? void 0 : _j.dd42v) || []) {
+            for (const f of ((_c = this.parsedXML) === null || _c === void 0 ? void 0 : _c.dd42v) || []) {
                 if (f.keyname === k.keyname) {
                     fields.push(f.keyfield);
                 }
@@ -47408,7 +47425,7 @@ class TableType extends _abstract_object_1.AbstractObject {
                 default:
                     break;
             }
-            (_k = tableOptions.secondary) === null || _k === void 0 ? void 0 : _k.push({
+            (_d = tableOptions.secondary) === null || _d === void 0 ? void 0 : _d.push({
                 name: k.keyname,
                 type: accessType,
                 keyFields: fields,
@@ -48539,7 +48556,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.6";
+        return "2.102.8";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -52458,6 +52475,9 @@ class CyclomaticComplexity {
     }
     setConfig(conf) {
         this.conf = conf;
+        if (this.conf.max === undefined) {
+            this.conf.max = new CyclomaticComplexityConf().max;
+        }
     }
     initialize(_reg) {
         return this;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.6",
+    "version": "2.102.8",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,16 +38,16 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.6",
+        "@abaplint/core": "^2.102.8",
         "@types/chai": "^4.3.5",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.2",
         "@types/mocha": "^10.0.1",
-        "@types/node": "^20.4.4",
+        "@types/node": "^20.4.5",
         "@types/progress": "^2.0.5",
         "chai": "^4.3.7",
         "chalk": "^5.3.0",
-        "eslint": "^8.45.0",
+        "eslint": "^8.46.0",
         "glob": "^7.2.3",
         "json5": "^2.2.3",
         "memfs": "^4.2.0",