@abaplint/core 2.82.9 → 2.82.13

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;
 }
@@ -3142,7 +3150,7 @@ export declare class Issue {
     static atRow(file: IFile, row: number, message: string, key: string, severity?: Severity): Issue;
     static atStatement(file: IFile, statement: StatementNode, message: string, key: string, severity?: Severity, fix?: IEdit): Issue;
     static atPosition(file: IFile, start: Position, message: string, key: string, severity?: Severity, fix?: IEdit): Issue;
-    static atRowRange(file: IFile, row: number, startCol: number, endCol: number, message: string, key: string, severity?: Severity): Issue;
+    static atRowRange(file: IFile, row: number, startCol: number, endCol: number, message: string, key: string, severity?: Severity, fix?: IEdit): Issue;
     static atRange(file: IFile, start: Position, end: Position, message: string, key: string, severity?: Severity, fix?: IEdit): Issue;
     static atToken(file: IFile, token: Token, message: string, key: string, severity?: Severity, fix?: IEdit): Issue;
     static atIdentifier(identifier: Identifier, message: string, key: string, severity?: Severity, fix?: IEdit): Issue;

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

@@ -44,6 +44,18 @@ class ObjectOriented {
             }
         }
     }
+    addAliasedTypes(aliases) {
+        for (const alias of aliases.getAll()) {
+            const comp = alias.getComponent();
+            const idef = this.scope.findInterfaceDefinition(comp.split("~")[0]);
+            if (idef) {
+                const found = idef.getTypeDefinitions().getByName(comp.split("~")[1]);
+                if (found) {
+                    this.scope.addTypeNamed(alias.getName(), found);
+                }
+            }
+        }
+    }
     findMethodInInterface(interfaceName, methodName) {
         const idef = this.scope.findInterfaceDefinition(interfaceName);
         if (idef) {

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/abap/types/class_definition.js

@@ -32,6 +32,7 @@ class ClassDefinition extends _identifier_1.Identifier {
         this.parse(filename, scope);
         const helper = new _object_oriented_1.ObjectOriented(scope);
         helper.fromSuperClassesAndInterfaces(this);
+        helper.addAliasedTypes(this.aliases);
         this.attributes = new class_attributes_1.Attributes(this.node, this.filename, scope);
         this.types = this.attributes.getTypes();
         this.methodDefs = new method_definitions_1.MethodDefinitions(this.node, this.filename, scope);

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/issue.js

@@ -48,7 +48,7 @@ class Issue {
             severity,
         });
     }
-    static atRowRange(file, row, startCol, endCol, message, key, severity) {
+    static atRowRange(file, row, startCol, endCol, message, key, severity, fix) {
         const start = new position_1.Position(row, startCol);
         const end = new position_1.Position(row, endCol);
         severity = severity !== null && severity !== void 0 ? severity : severity_1.Severity.Error;
@@ -58,6 +58,7 @@ class Issue {
             key,
             start,
             end,
+            fix,
             severity,
         });
     }

package/build/src/objects/table_type.js

@@ -59,6 +59,9 @@ class TableType extends _abstract_object_1.AbstractObject {
         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.9";
+        return "2.82.13";
     }
     getDDICReferences() {
         return this.references;

package/build/src/rules/space_before_colon.js

@@ -1,6 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SpaceBeforeColon = exports.SpaceBeforeColonConf = void 0;
+const position_1 = require("../position");
+const edit_helper_1 = require("../edit_helper");
 const issue_1 = require("../issue");
 const _abap_rule_1 = require("./_abap_rule");
 const _basic_rule_config_1 = require("./_basic_rule_config");
@@ -19,7 +21,7 @@ class SpaceBeforeColon extends _abap_rule_1.ABAPRule {
             title: "Space before colon",
             shortDescription: `Checks that there are no spaces in front of colons in chained statements.`,
             extendedInformation: `https://docs.abapopenchecks.org/checks/80/`,
-            tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.SingleFile],
+            tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
             badExample: `DATA : foo TYPE string.`,
             goodExample: `DATA: foo TYPE string.`,
         };
@@ -44,8 +46,10 @@ class SpaceBeforeColon extends _abap_rule_1.ABAPRule {
             else if (token.getStr() === ":"
                 && prev.getRow() === token.getRow()
                 && prev.getCol() + prev.getStr().length < token.getCol()) {
-                const issue = issue_1.Issue.atRowRange(file, token.getRow(), prev.getEnd().getCol(), token.getStart().getCol(), this.getMessage(), this.getMetadata().key, this.conf.severity);
-                //        const issue = Issue.atToken(file, token, this.getMessage(), this.getMetadata().key);
+                const start = new position_1.Position(token.getRow(), prev.getEnd().getCol());
+                const end = new position_1.Position(token.getRow(), token.getStart().getCol());
+                const fix = edit_helper_1.EditHelper.deleteRange(file, start, end);
+                const issue = issue_1.Issue.atRowRange(file, start.getRow(), start.getCol(), end.getCol(), this.getMessage(), this.getMetadata().key, this.conf.severity, fix);
                 issues.push(issue);
             }
             prev = token;

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.9",
+    "version": "2.82.13",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -45,7 +45,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@microsoft/api-extractor": "^7.18.19",
+        "@microsoft/api-extractor": "^7.18.20",
         "@types/chai": "^4.2.22",
         "@types/mocha": "^9.0.0",
         "@types/node": "^16.11.11",