@abaplint/core 2.82.7 → 2.82.11

package/build/abaplint.d.ts

@@ -324,6 +324,7 @@ declare namespace BasicTypes {
         CSequenceType,
         DataReference,
         DateType,
+        DecFloatType,
         DecFloat16Type,
         DecFloat34Type,
         FloatType,
@@ -1241,6 +1242,13 @@ declare class DecFloat34Type extends AbstractType {
     containsVoid(): boolean;
 }
 
+declare class DecFloatType extends AbstractType {
+    toText(): string;
+    isGeneric(): boolean;
+    toABAP(): string;
+    containsVoid(): boolean;
+}
+
 declare class Decimals extends Expression {
     getRunnable(): IStatementRunnable;
 }

package/build/src/abap/5_syntax/basic_types.js

@@ -524,39 +524,52 @@ class BasicTypes {
         return foundType;
     }
     resolveConstantValue(expr) {
-        var _a;
+        var _a, _b;
         if (!(expr.get() instanceof Expressions.SimpleFieldChain)) {
             throw new Error("resolveConstantValue");
         }
-        const first = expr.getFirstChild();
-        if (first.get() instanceof Expressions.Field) {
-            const token = first.getFirstToken();
-            const name = token.getStr();
-            const found = this.scope.findVariable(name);
+        const firstNode = expr.getFirstChild();
+        const firstToken = firstNode.getFirstToken();
+        const firstName = firstToken.getStr();
+        if (firstNode.get() instanceof Expressions.Field) {
+            const found = this.scope.findVariable(firstName);
             const val = found === null || found === void 0 ? void 0 : found.getValue();
             if (typeof val === "string") {
-                this.scope.addReference(token, found, _reference_1.ReferenceType.DataReadReference, this.filename);
+                this.scope.addReference(firstToken, found, _reference_1.ReferenceType.DataReadReference, this.filename);
                 return val;
             }
             return undefined;
         }
-        else if (first.get() instanceof Expressions.ClassName) {
-            const name = first.getFirstToken().getStr();
-            const obj = this.scope.findObjectDefinition(name);
+        else if (firstNode.get() instanceof Expressions.ClassName
+            && firstName.toLowerCase() === this.scope.getName().toLowerCase()
+            && (this.scope.getType() === _scope_type_1.ScopeType.Interface
+                || this.scope.getType() === _scope_type_1.ScopeType.ClassDefinition)) {
+            const children = expr.getChildren();
+            const token = (_a = children[2]) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+            const found = this.scope.findVariable(token.getStr());
+            const val = found === null || found === void 0 ? void 0 : found.getValue();
+            if (typeof val === "string") {
+                this.scope.addReference(firstToken, found, _reference_1.ReferenceType.DataReadReference, this.filename);
+                return val;
+            }
+            return undefined;
+        }
+        else if (firstNode.get() instanceof Expressions.ClassName) {
+            const obj = this.scope.findObjectDefinition(firstName);
             if (obj === undefined) {
-                if (this.scope.existsObject(name).found === true) {
+                if (this.scope.existsObject(firstName).found === true) {
                     return undefined;
                 }
-                else if (this.scope.getDDIC().inErrorNamespace(name) === true) {
-                    throw new Error("resolveConstantValue, not found: " + name);
+                else if (this.scope.getDDIC().inErrorNamespace(firstName) === true) {
+                    throw new Error("resolveConstantValue, not found: " + firstName);
                 }
                 else {
-                    this.scope.addReference(first.getFirstToken(), undefined, _reference_1.ReferenceType.ObjectOrientedVoidReference, this.filename, { ooName: name.toUpperCase() });
+                    this.scope.addReference(firstNode.getFirstToken(), undefined, _reference_1.ReferenceType.ObjectOrientedVoidReference, this.filename, { ooName: firstName.toUpperCase() });
                     return undefined;
                 }
             }
             const children = expr.getChildren();
-            const token = (_a = children[2]) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+            const token = (_b = children[2]) === null || _b === void 0 ? void 0 : _b.getFirstToken();
             const attr = token.getStr();
             const c = new _object_oriented_1.ObjectOriented(this.scope).searchConstantName(obj, attr);
             if (c instanceof class_constant_1.ClassConstant) {

package/build/src/abap/types/basic/decfloat_type.js

@@ -0,0 +1,20 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.DecFloatType = void 0;
+const _abstract_type_1 = require("./_abstract_type");
+class DecFloatType extends _abstract_type_1.AbstractType {
+    toText() {
+        return "```decfloat```";
+    }
+    isGeneric() {
+        return true;
+    }
+    toABAP() {
+        return "decfloat";
+    }
+    containsVoid() {
+        return false;
+    }
+}
+exports.DecFloatType = DecFloatType;
+//# sourceMappingURL=decfloat_type.js.map

package/build/src/abap/types/basic/index.js

@@ -16,6 +16,7 @@ __exportStar(require("./clike_type"), exports);
 __exportStar(require("./csequence_type"), exports);
 __exportStar(require("./data_reference_type"), exports);
 __exportStar(require("./date_type"), exports);
+__exportStar(require("./decfloat_type"), exports);
 __exportStar(require("./decfloat16_type"), exports);
 __exportStar(require("./decfloat34_type"), exports);
 __exportStar(require("./float_type"), exports);

package/build/src/ddic.js

@@ -57,6 +57,8 @@ class DDIC {
                 return new Types.XSequenceType();
             case "CLIKE":
                 return new Types.CLikeType();
+            case "DECFLOAT":
+                return new Types.DecFloatType();
             case "ANY":
                 return new Types.AnyType();
             case "SIMPLE":

package/build/src/objects/table_type.js

@@ -4,6 +4,7 @@ exports.TableType = void 0;
 const _abstract_object_1 = require("./_abstract_object");
 const Types = require("../abap/types/basic");
 const ddic_1 = require("../ddic");
+const basic_1 = require("../abap/types/basic");
 class TableType extends _abstract_object_1.AbstractObject {
     constructor() {
         super(...arguments);
@@ -55,6 +56,12 @@ class TableType extends _abstract_object_1.AbstractObject {
                 references.push({ object: lookup.object });
             }
         }
+        else if (this.parsedXML.rowkind === "R" && this.parsedXML.rowtype === "OBJECT") {
+            type = new Types.TableType(new basic_1.GenericObjectReferenceType(), { withHeader: false }, this.getName());
+        }
+        else if (this.parsedXML.rowkind === "R" && this.parsedXML.rowtype === "DATA") {
+            type = new Types.TableType(new basic_1.DataReference(new basic_1.AnyType()), { withHeader: false }, this.getName());
+        }
         else if (this.parsedXML.rowkind === "R" && this.parsedXML.rowtype !== undefined) {
             const lookup = ddic.lookupObject(this.parsedXML.rowtype);
             type = new Types.TableType(lookup.type, { withHeader: false }, this.getName());

package/build/src/registry.js

@@ -68,7 +68,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.82.7";
+        return "2.82.11";
     }
     getDDICReferences() {
         return this.references;

package/build/src/rules/unknown_types.js

@@ -40,9 +40,27 @@ class UnknownTypes {
             return [];
         }
         const spaghetti = new syntax_1.SyntaxLogic(this.reg, obj).run().spaghetti;
-        return this.traverse(spaghetti.getTop());
+        const found = this.traverse(spaghetti.getTop());
+        return this.removeDuplicates(found);
     }
     /////////////////////
+    removeDuplicates(list) {
+        const deduplicated = [];
+        for (const result of list) {
+            let cont = false;
+            for (const d of deduplicated) {
+                if (result.getStart().equals(d.getStart())) {
+                    cont = true;
+                    break;
+                }
+            }
+            if (cont === true) {
+                continue;
+            }
+            deduplicated.push(result);
+        }
+        return deduplicated;
+    }
     traverse(node) {
         var _a;
         const ret = [];
@@ -73,7 +91,14 @@ class UnknownTypes {
             }
         }
         for (const v of node.getData().idefs) {
-            const found = this.checkInterface(v);
+            const found = this.checkMethodParameters(v);
+            if (found) {
+                const message = "Contains unknown, " + found.found;
+                ret.push(issue_1.Issue.atIdentifier(found.id, message, this.getMetadata().key, this.conf.severity));
+            }
+        }
+        for (const v of node.getData().cdefs) {
+            const found = this.checkMethodParameters(v);
             if (found) {
                 const message = "Contains unknown, " + found.found;
                 ret.push(issue_1.Issue.atIdentifier(found.id, message, this.getMetadata().key, this.conf.severity));
@@ -84,7 +109,7 @@ class UnknownTypes {
         }
         return ret;
     }
-    checkInterface(idef) {
+    checkMethodParameters(idef) {
         var _a;
         for (const m of ((_a = idef.getMethodDefinitions()) === null || _a === void 0 ? void 0 : _a.getAll()) || []) {
             for (const p of m.getParameters().getAll()) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.82.7",
+    "version": "2.82.11",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",