@abaplint/cli 2.102.4 → 2.102.6

package/build/cli.js

@@ -21739,6 +21739,13 @@ class TypeUtils {
                 return false;
             }
         }
+        else if (source instanceof basic_1.ObjectReferenceType) {
+            if (target instanceof basic_1.XSequenceType
+                || target instanceof basic_1.IntegerType
+                || target instanceof basic_1.XStringType) {
+                return false;
+            }
+        }
         else if (source instanceof basic_1.XStringType) {
             if (target instanceof basic_1.CLikeType
                 || target instanceof basic_1.IntegerType
@@ -28174,16 +28181,16 @@ const data_definition_1 = __webpack_require__(/*! ../expressions/data_definition
 const unknown_type_1 = __webpack_require__(/*! ../../types/basic/unknown_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/unknown_type.js");
 class Data {
     runSyntax(node, scope, filename) {
+        const name = node.findFirstExpression(Expressions.DefinitionName);
         const dd = node.findFirstExpression(Expressions.DataDefinition);
         if (dd) {
             const id = new data_definition_1.DataDefinition().runSyntax(dd, scope, filename);
             if ((id === null || id === void 0 ? void 0 : id.getType().isGeneric()) === true
                 && (id === null || id === void 0 ? void 0 : id.getType().containsVoid()) === false) {
-                throw new Error("DATA definition cannot be generic");
+                throw new Error("DATA definition cannot be generic, " + name);
             }
             return id;
         }
-        const name = node.findFirstExpression(Expressions.DefinitionName);
         if (name) {
             return new _typed_identifier_1.TypedIdentifier(name.getFirstToken(), filename, new unknown_type_1.UnknownType("data, fallback"));
         }
@@ -34746,10 +34753,10 @@ class FloatingPointType extends _abstract_type_1.AbstractType {
         return this.length;
     }
     toText() {
-        return "```n LENGTH " + this.getLength() + "```";
+        return "```f LENGTH " + this.getLength() + "```";
     }
     toABAP() {
-        return "n LENGTH " + this.getLength();
+        return "f LENGTH " + this.getLength();
     }
     isGeneric() {
         return false;
@@ -47347,7 +47354,7 @@ class TableType extends _abstract_object_1.AbstractObject {
         super.setDirty();
     }
     buildTableOptions() {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j;
+        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 = {
@@ -47356,34 +47363,36 @@ class TableType extends _abstract_object_1.AbstractObject {
                 keyFields: [],
                 name: "primary_key",
             };
-            for (const f of ((_c = this.parsedXML) === null || _c === void 0 ? void 0 : _c.dd42v) || []) {
-                if (f.keyname === "") {
-                    primaryKey.keyFields.push(f.keyfield);
-                }
-            }
         }
-        else if (((_d = this.parsedXML) === null || _d === void 0 ? void 0 : _d.accessmode) === "H") {
+        else if (((_c = this.parsedXML) === null || _c === void 0 ? void 0 : _c.accessmode) === "H") {
             primaryKey = {
-                isUnique: ((_e = this.parsedXML) === null || _e === void 0 ? void 0 : _e.keykind) === "U",
+                isUnique: ((_d = this.parsedXML) === null || _d === void 0 ? void 0 : _d.keykind) === "U",
                 type: basic_1.TableAccessType.hashed,
                 keyFields: [],
                 name: "primary_key",
             };
-            for (const f of ((_f = this.parsedXML) === null || _f === void 0 ? void 0 : _f.dd42v) || []) {
-                if (f.keyname === "") {
-                    primaryKey.keyFields.push(f.keyfield);
-                }
+        }
+        for (const f of ((_e = this.parsedXML) === null || _e === void 0 ? void 0 : _e.dd42v) || []) {
+            if (f.keyname === "") {
+                primaryKey === null || primaryKey === void 0 ? void 0 : 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");
+        }
+        let keyType = Types.TableKeyType.user;
+        if (((_g = this.parsedXML) === null || _g === void 0 ? void 0 : _g.keydef) === "D") {
+            keyType = Types.TableKeyType.default;
+        }
         const tableOptions = {
             withHeader: false,
-            keyType: Types.TableKeyType.user,
+            keyType: keyType,
             primaryKey: primaryKey,
             secondary: [],
         };
-        for (const k of ((_g = this.parsedXML) === null || _g === void 0 ? void 0 : _g.dd43v) || []) {
+        for (const k of ((_h = this.parsedXML) === null || _h === void 0 ? void 0 : _h.dd43v) || []) {
             const fields = [];
-            for (const f of ((_h = this.parsedXML) === null || _h === void 0 ? void 0 : _h.dd42v) || []) {
+            for (const f of ((_j = this.parsedXML) === null || _j === void 0 ? void 0 : _j.dd42v) || []) {
                 if (f.keyname === k.keyname) {
                     fields.push(f.keyfield);
                 }
@@ -47399,7 +47408,7 @@ class TableType extends _abstract_object_1.AbstractObject {
                 default:
                     break;
             }
-            (_j = tableOptions.secondary) === null || _j === void 0 ? void 0 : _j.push({
+            (_k = tableOptions.secondary) === null || _k === void 0 ? void 0 : _k.push({
                 name: k.keyname,
                 type: accessType,
                 keyFields: fields,
@@ -47490,6 +47499,7 @@ class TableType extends _abstract_object_1.AbstractObject {
         this.parsedXML.accessmode = dd40v.ACCESSMODE;
         this.parsedXML.keykind = dd40v.KEYKIND;
         this.parsedXML.ddtext = dd40v.DDTEXT;
+        this.parsedXML.keydef = dd40v.KEYDEF;
         for (const x of (0, xml_utils_1.xmlToArray)((_a = values.DD42V) === null || _a === void 0 ? void 0 : _a.DD42V)) {
             this.parsedXML.dd42v.push({
                 keyname: x.SECKEYNAME || "",
@@ -48529,7 +48539,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.4";
+        return "2.102.6";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.4",
+    "version": "2.102.6",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.4",
+        "@abaplint/core": "^2.102.6",
         "@types/chai": "^4.3.5",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.2",