@abaplint/core 2.82.10 → 2.82.14

package/build/abaplint.d.ts

@@ -324,6 +324,7 @@ declare namespace BasicTypes {
         CSequenceType,
         DataReference,
         DateType,
+        DecFloatType,
         DecFloat16Type,
         DecFloat34Type,
         FloatType,
@@ -878,10 +879,7 @@ declare class ConcatenatedConstant extends Expression {
 
 declare class ConceptsOfPackage extends AbstractObject {
     getType(): string;
-    getAllowedNaming(): {
-        maxLength: number;
-        allowNamespace: boolean;
-    };
+    getAllowedNaming(): IAllowedNaming;
     getDescription(): string | undefined;
 }
 
@@ -1241,6 +1239,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;
 }
@@ -2339,12 +2344,15 @@ declare interface IAliases {
     getAll(): readonly Alias[];
 }
 
+declare interface IAllowedNaming {
+    maxLength: number;
+    allowNamespace: boolean;
+    customRegex?: RegExp;
+}
+
 declare interface IArtifact {
     getType(): string;
-    getAllowedNaming(): {
-        maxLength: number;
-        allowNamespace: boolean;
-    };
+    getAllowedNaming(): IAllowedNaming;
 }
 
 declare interface IAttributes {
@@ -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;
@@ -3600,10 +3608,7 @@ declare class Multiply implements IStatement {
 
 declare class Namespace extends AbstractObject {
     getType(): string;
-    getAllowedNaming(): {
-        maxLength: number;
-        allowNamespace: boolean;
-    };
+    getAllowedNaming(): IAllowedNaming;
     getDescription(): string | undefined;
 }
 
@@ -5903,10 +5908,7 @@ declare class WebDynproComponentConfiguration extends AbstractObject {
 
 declare class WebMIME extends AbstractObject {
     getType(): string;
-    getAllowedNaming(): {
-        maxLength: number;
-        allowNamespace: boolean;
-    };
+    getAllowedNaming(): IAllowedNaming;
     getDescription(): string | undefined;
 }
 

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/5_syntax/statements/get_badi.js

@@ -7,7 +7,7 @@ const target_1 = require("../expressions/target");
 const dynamic_1 = require("../expressions/dynamic");
 class GetBadi {
     runSyntax(node, scope, filename) {
-        for (const s of node.findDirectExpressions(Expressions.Source)) {
+        for (const s of node.findAllExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(s, scope, filename);
         }
         for (const t of node.findDirectExpressions(Expressions.Target)) {

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

@@ -10,6 +10,7 @@ class ConceptsOfPackage extends _abstract_object_1.AbstractObject {
         return {
             maxLength: 30,
             allowNamespace: true,
+            customRegex: new RegExp(/^(\/[A-Z_\d]{3,8}\/)?[A-Z_-\d<> ]+$/i),
         };
     }
     getDescription() {

package/build/src/objects/namespace.js

@@ -10,6 +10,7 @@ class Namespace extends _abstract_object_1.AbstractObject {
         return {
             maxLength: 10,
             allowNamespace: true,
+            customRegex: new RegExp(/^\/[A-Z_\d]{3,8}\/$/i),
         };
     }
     getDescription() {

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/objects/web_mime.js

@@ -10,6 +10,7 @@ class WebMIME extends _abstract_object_1.AbstractObject {
         return {
             maxLength: 40,
             allowNamespace: true,
+            customRegex: new RegExp(/^[A-Z_-\d/<> ]+$/i),
         };
     }
     getDescription() {

package/build/src/registry.js

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

package/build/src/rules/allowed_object_naming.js

@@ -30,19 +30,20 @@ class AllowedObjectNaming {
     }
     run(obj) {
         const allowed = obj.getAllowedNaming();
+        const name = obj.getName();
         let message = "";
-        if (obj.getName().length > allowed.maxLength) {
+        if (name.length > allowed.maxLength) {
             message = "Name exceeds max length";
         }
-        else if (allowed.allowNamespace === false && obj.getName().indexOf("/") >= 0) {
+        else if (allowed.allowNamespace === false && name.indexOf("/") >= 0) {
             message = "Namespace not allowed for object type";
         }
-        else if (obj.getType() === "NSPC") {
-            if (obj.getName().match(/^\/[A-Z_\d]{3,8}\/$/i) === null) {
+        else if (allowed.customRegex !== undefined) {
+            if (name.match(allowed.customRegex) === null) {
                 message = "Name not allowed";
             }
         }
-        else if (obj.getName().match(/^(\/[A-Z_\d]{3,8}\/)?[A-Z_-\d<> ]+$/i) === null) {
+        else if (name.match(/^(\/[A-Z_\d]{3,8}\/)?[A-Z_\d<> ]+$/i) === null) {
             message = "Name not allowed";
         }
         if (message.length > 0) {

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/build/src/rules/unnecessary_chaining.js

@@ -5,6 +5,7 @@ const issue_1 = require("../issue");
 const _abap_rule_1 = require("./_abap_rule");
 const _basic_rule_config_1 = require("./_basic_rule_config");
 const _irule_1 = require("./_irule");
+const edit_helper_1 = require("../edit_helper");
 class UnnecessaryChainingConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.UnnecessaryChainingConf = UnnecessaryChainingConf;
@@ -19,7 +20,7 @@ class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
             title: "Unnecessary Chaining",
             shortDescription: `Find unnecessary chaining, all statements are checked`,
             extendedInformation: ``,
-            tags: [_irule_1.RuleTag.SingleFile],
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
             badExample: `WRITE: bar.`,
             goodExample: `WRITE bar.`,
         };
@@ -47,8 +48,9 @@ class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
             else if (prev !== undefined && colon.getStart().equals(prev.getStart())) {
                 continue;
             }
+            const fix = edit_helper_1.EditHelper.deleteRange(file, colon.getStart(), colon.getEnd());
             const message = "Unnecessary chaining";
-            const issue = issue_1.Issue.atStatement(file, statements[i], message, this.getMetadata().key);
+            const issue = issue_1.Issue.atStatement(file, statements[i], message, this.getMetadata().key, this.conf.severity, fix);
             issues.push(issue);
         }
         return issues;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.82.10",
+    "version": "2.82.14",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -45,12 +45,12 @@
     },
     "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",
         "chai": "^4.3.4",
-        "eslint": "^8.3.0",
+        "eslint": "^8.4.0",
         "mocha": "^9.1.3",
         "c8": "^7.10.0",
         "source-map-support": "^0.5.21",