@abaplint/cli 2.105.19 → 2.105.21

package/build/cli.js

@@ -22318,6 +22318,7 @@ class TypeUtils {
         }
         else if (type instanceof basic_1.StringType
             || type instanceof basic_1.AnyType
+            || type instanceof basic_1.DataType
             || type instanceof basic_1.CharacterType
             || type instanceof basic_1.SimpleType
             || type instanceof cgeneric_type_1.CGenericType
@@ -22366,6 +22367,7 @@ class TypeUtils {
             || type instanceof basic_1.CSequenceType
             || type instanceof cgeneric_type_1.CGenericType
             || type instanceof basic_1.DateType
+            || type instanceof basic_1.DataType
             || type instanceof basic_1.CLikeType
             || type instanceof basic_1.PackedType
             || type instanceof basic_1.TimeType) {
@@ -22596,7 +22598,9 @@ class TypeUtils {
             else if (target instanceof basic_1.CLikeType) {
                 return this.isCharLikeStrict(source);
             }
-            else if (target instanceof basic_1.VoidType || target instanceof basic_1.AnyType) {
+            else if (target instanceof basic_1.VoidType
+                || target instanceof basic_1.AnyType
+                || target instanceof basic_1.DataType) {
                 return true;
             }
             return false;
@@ -22658,6 +22662,7 @@ class TypeUtils {
             }
             if (source instanceof basic_1.VoidType
                 || source instanceof basic_1.AnyType
+                || source instanceof basic_1.DataType
                 || source instanceof basic_1.UnknownType) {
                 return true;
             }
@@ -22702,6 +22707,7 @@ class TypeUtils {
             if (source instanceof basic_1.ObjectReferenceType
                 || source instanceof basic_1.GenericObjectReferenceType
                 || source instanceof basic_1.VoidType
+                || source instanceof basic_1.DataType
                 || source instanceof basic_1.AnyType
                 || source instanceof basic_1.UnknownType) {
                 return true;
@@ -22711,6 +22717,7 @@ class TypeUtils {
         else if (target instanceof basic_1.DataReference) {
             if (source instanceof basic_1.DataReference
                 || source instanceof basic_1.VoidType
+                || source instanceof basic_1.DataType
                 || source instanceof basic_1.AnyType
                 || source instanceof basic_1.UnknownType) {
                 return true;
@@ -22723,6 +22730,7 @@ class TypeUtils {
             }
             else if (source instanceof basic_1.VoidType
                 || source instanceof basic_1.AnyType
+                || source instanceof basic_1.DataType
                 || source instanceof basic_1.UnknownType) {
                 return true;
             }
@@ -23619,7 +23627,7 @@ class BasicTypes {
             return new Types.DataReference(found);
         }
         else if (chain.concatTokens().toUpperCase() === "DATA") {
-            return new Types.DataReference(new Types.AnyType());
+            return new Types.DataReference(new Types.DataType());
         }
         if (this.scope.isBadiDef(name) === true) {
             return new Types.VoidType(name);
@@ -24517,6 +24525,7 @@ class Dereference {
     runSyntax(type) {
         if (type instanceof basic_1.VoidType
             || type instanceof basic_1.AnyType
+            || type instanceof basic_1.DataType
             || type === undefined
             || type instanceof basic_1.UnknownType) {
             return type;
@@ -26266,7 +26275,7 @@ class NewObject {
             new method_parameters_1.MethodParameters().checkExporting(parameters, scope, method, filename);
         }
         else if (requiredParameters.length > 0) {
-            throw new Error(`constructor parameter "${requiredParameters[0].getName()}" must be supplied` + name);
+            throw new Error(`constructor parameter "${requiredParameters[0].getName()}" must be supplied, ` + name);
         }
     }
     defaultImportingType(method) {
@@ -29387,6 +29396,7 @@ class CreateObject {
                 }
                 else if (!(found instanceof basic_1.ObjectReferenceType)
                     && !(found instanceof basic_1.AnyType)
+                    && !(found instanceof basic_1.DataType)
                     && !(found instanceof basic_1.GenericObjectReferenceType)) {
                     throw new Error("Target must be an object reference, " + t.concatTokens());
                 }
@@ -30708,6 +30718,7 @@ class Loop {
         }
         else if (!(sourceType instanceof basic_1.TableType)
             && !(sourceType instanceof basic_1.AnyType)
+            && !(sourceType instanceof basic_1.DataType)
             && !(sourceType instanceof basic_1.VoidType)
             && concat.startsWith("LOOP AT GROUP ") === false) {
             throw new Error("Loop, not a table type");
@@ -35925,6 +35936,7 @@ exports.CSequenceType = CSequenceType;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.DataReference = void 0;
 const _abstract_type_1 = __webpack_require__(/*! ./_abstract_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/_abstract_type.js");
+const any_type_1 = __webpack_require__(/*! ./any_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/any_type.js");
 class DataReference extends _abstract_type_1.AbstractType {
     constructor(type, qualifiedName) {
         super({ qualifiedName: qualifiedName });
@@ -35944,7 +35956,9 @@ class DataReference extends _abstract_type_1.AbstractType {
         return "REF TO " + this.type.toABAP();
     }
     isGeneric() {
-        // a DATA definition can be "REF TO data", so its not generic
+        if (this.type instanceof any_type_1.AnyType) {
+            return true;
+        }
         return false;
     }
     containsVoid() {
@@ -35959,6 +35973,39 @@ exports.DataReference = DataReference;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/types/basic/data_type.js":
+/*!*****************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/types/basic/data_type.js ***!
+  \*****************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.DataType = void 0;
+const _abstract_type_1 = __webpack_require__(/*! ./_abstract_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/_abstract_type.js");
+class DataType extends _abstract_type_1.AbstractType {
+    toText() {
+        return "```data```";
+    }
+    toABAP() {
+        return "data";
+    }
+    isGeneric() {
+        return true;
+    }
+    containsVoid() {
+        return false;
+    }
+    toCDS() {
+        return "abap.TODO_DATA";
+    }
+}
+exports.DataType = DataType;
+//# sourceMappingURL=data_type.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/types/basic/date_type.js":
 /*!*****************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/types/basic/date_type.js ***!
@@ -36309,6 +36356,7 @@ __exportStar(__webpack_require__(/*! ./character_type */ "./node_modules/@abapli
 __exportStar(__webpack_require__(/*! ./clike_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/clike_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./csequence_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/csequence_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./data_reference_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/data_reference_type.js"), exports);
+__exportStar(__webpack_require__(/*! ./data_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/data_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./date_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/date_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./decfloat_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/decfloat_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./decfloat16_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/decfloat16_type.js"), exports);
@@ -39950,7 +39998,7 @@ class DDIC {
             case "TABLE":
                 return new Types.TableType(new Types.AnyType(), { withHeader: false, keyType: Types.TableKeyType.default });
             case "DATA":
-                return new Types.AnyType({ qualifiedName: qualifiedName });
+                return new Types.DataType({ qualifiedName: qualifiedName });
             case "NUMERIC":
                 return new Types.NumericGenericType({ qualifiedName: qualifiedName });
             case "UTCLONG": // todo, take version into account
@@ -50177,7 +50225,7 @@ class TableType extends _abstract_object_1.AbstractObject {
             type = new Types.TableType(new basic_1.GenericObjectReferenceType(), tableOptions, this.getName());
         }
         else if (this.parsedXML.rowkind === "R" && this.parsedXML.rowtype === "DATA") {
-            type = new Types.TableType(new basic_1.DataReference(new basic_1.AnyType()), tableOptions, this.getName());
+            type = new Types.TableType(new basic_1.DataReference(new Types.DataType()), tableOptions, this.getName());
         }
         else if (this.parsedXML.rowkind === "R" && this.parsedXML.rowtype !== undefined) {
             const lookup = ddic.lookupObject(this.parsedXML.rowtype);
@@ -51249,7 +51297,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.105.19";
+        return "2.105.21";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -62052,6 +62100,11 @@ class KeepSingleParameterCallsOnOneLine extends _abap_rule_1.ABAPRule {
     }
     isSingleParameter(c) {
         if (c.findDirectExpression(Expressions.Source)) {
+            for (const params of c.findAllExpressions(Expressions.ParameterListS)) {
+                if (params.getChildren().length > 1) {
+                    return false;
+                }
+            }
             return true;
         }
         const list = c.findDirectExpression(Expressions.ParameterListS);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.105.19",
+    "version": "2.105.21",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,24 +38,24 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.105.19",
+        "@abaplint/core": "^2.105.21",
         "@types/chai": "^4.3.11",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.6",
-        "@types/node": "^20.11.19",
+        "@types/node": "^20.11.20",
         "@types/progress": "^2.0.7",
         "chai": "^4.4.1",
         "chalk": "^5.3.0",
         "eslint": "^8.56.0",
         "glob": "^7.2.3",
         "json5": "^2.2.3",
-        "memfs": "^4.7.6",
+        "memfs": "^4.7.7",
         "minimist": "^1.2.8",
         "mocha": "^10.3.0",
         "progress": "^2.0.3",
         "typescript": "^5.3.3",
-        "webpack": "^5.90.2",
+        "webpack": "^5.90.3",
         "webpack-cli": "^5.1.4",
         "xml-js": "^1.6.11"
     },