@abaplint/core 2.82.11 → 2.82.15

package/build/abaplint.d.ts

@@ -879,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;
 }
 
@@ -2347,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 {
@@ -3150,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;
@@ -3608,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;
 }
 
@@ -5911,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/5_syntax/structures/type_enum.js

@@ -11,7 +11,7 @@ const enum_type_1 = require("../../types/basic/enum_type");
 class TypeEnum {
     runSyntax(node, scope, filename) {
         var _a;
-        const values = [];
+        let values = [];
         const types = [];
         if (!(node.get() instanceof Structures.TypeEnum)) {
             throw new Error("TypeEnum, unexpected type");
@@ -61,6 +61,7 @@ class TypeEnum {
                     type: r.getType(),
                 });
             }
+            values = [];
             const id = new _typed_identifier_1.TypedIdentifier(stru.getFirstToken(), filename, new basic_1.StructureType(components), ["enum" /* Enum */]);
             values.push(id);
         }

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/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/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.11";
+        return "2.82.15";
     }
     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/unnecessary_chaining.js

@@ -5,6 +5,8 @@ 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");
+const _statement_1 = require("../abap/2_statements/statements/_statement");
 class UnnecessaryChainingConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.UnnecessaryChainingConf = UnnecessaryChainingConf;
@@ -19,7 +21,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.`,
         };
@@ -31,7 +33,7 @@ class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
         this.conf = conf;
     }
     runParsed(file) {
-        var _a, _b;
+        var _a;
         const issues = [];
         const statements = file.getStatements();
         for (let i = 0; i < statements.length; i++) {
@@ -39,16 +41,24 @@ class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
             if (colon === undefined) {
                 continue;
             }
-            const next = (_a = statements[i + 1]) === null || _a === void 0 ? void 0 : _a.getColon();
-            const prev = (_b = statements[i - 1]) === null || _b === void 0 ? void 0 : _b.getColon();
+            // find the next non Comment statement
+            let j = 1;
+            let nextStatement = statements[i + j];
+            while ((nextStatement === null || nextStatement === void 0 ? void 0 : nextStatement.get()) instanceof _statement_1.Comment) {
+                j++;
+                nextStatement = statements[i + j];
+            }
+            const next = nextStatement === null || nextStatement === void 0 ? void 0 : nextStatement.getColon();
+            const prev = (_a = statements[i - 1]) === null || _a === void 0 ? void 0 : _a.getColon();
             if (next !== undefined && colon.getStart().equals(next.getStart())) {
                 continue;
             }
             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.11",
+    "version": "2.82.15",
     "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.21",
         "@types/chai": "^4.2.22",
         "@types/mocha": "^9.0.0",
-        "@types/node": "^16.11.11",
+        "@types/node": "^16.11.12",
         "chai": "^4.3.4",
-        "eslint": "^8.3.0",
+        "eslint": "^8.4.1",
         "mocha": "^9.1.3",
         "c8": "^7.10.0",
         "source-map-support": "^0.5.21",