@abaplint/core 2.79.8 → 2.79.9

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

@@ -249,6 +249,7 @@ class BuiltIn {
         components.push({ name: "index", type: new basic_1.IntegerType() });
         components.push({ name: "langu", type: new basic_1.CharacterType(1) });
         components.push({ name: "ldbpg", type: new basic_1.CharacterType(40) });
+        components.push({ name: "pdest", type: new basic_1.CharacterType(4) });
         components.push({ name: "lilli", type: new basic_1.IntegerType() });
         components.push({ name: "linct", type: new basic_1.IntegerType() });
         components.push({ name: "linno", type: new basic_1.IntegerType() });

package/build/src/objects/rename/rename_data_element.js

@@ -0,0 +1,24 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.RenameDataElement = void 0;
+const __1 = require("..");
+const renamer_helper_1 = require("./renamer_helper");
+class RenameDataElement {
+    constructor(reg) {
+        this.reg = reg;
+    }
+    buildEdits(obj, oldName, newName) {
+        if (!(obj instanceof __1.DataElement)) {
+            throw new Error("RenameDataElement, not a data element");
+        }
+        let changes = [];
+        const helper = new renamer_helper_1.RenamerHelper(this.reg);
+        changes = changes.concat(helper.buildXMLFileEdits(obj, "ROLLNAME", oldName, newName));
+        changes = changes.concat(helper.renameFiles(obj, oldName, newName));
+        return {
+            documentChanges: changes,
+        };
+    }
+}
+exports.RenameDataElement = RenameDataElement;
+//# sourceMappingURL=rename_data_element.js.map

package/build/src/objects/rename/rename_domain.js

@@ -0,0 +1,24 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.RenameDomain = void 0;
+const __1 = require("..");
+const renamer_helper_1 = require("./renamer_helper");
+class RenameDomain {
+    constructor(reg) {
+        this.reg = reg;
+    }
+    buildEdits(obj, oldName, newName) {
+        if (!(obj instanceof __1.Domain)) {
+            throw new Error("RenameDomain, not a domain");
+        }
+        let changes = [];
+        const helper = new renamer_helper_1.RenamerHelper(this.reg);
+        changes = changes.concat(helper.buildXMLFileEdits(obj, "DOMNAME", oldName, newName));
+        changes = changes.concat(helper.renameFiles(obj, oldName, newName));
+        return {
+            documentChanges: changes,
+        };
+    }
+}
+exports.RenameDomain = RenameDomain;
+//# sourceMappingURL=rename_domain.js.map

package/build/src/objects/rename/renamer.js

@@ -3,6 +3,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.Renamer = void 0;
 const vscode_languageserver_types_1 = require("vscode-languageserver-types");
 const memory_file_1 = require("../../files/memory_file");
+const rename_data_element_1 = require("./rename_data_element");
+const rename_domain_1 = require("./rename_domain");
 const rename_global_class_1 = require("./rename_global_class");
 const rename_global_interface_1 = require("./rename_global_interface");
 class Renamer {
@@ -40,6 +42,10 @@ class Renamer {
         switch (type) {
             case "CLAS":
                 return new rename_global_class_1.RenameGlobalClass(this.reg);
+            case "DTEL":
+                return new rename_data_element_1.RenameDataElement(this.reg);
+            case "DOMA":
+                return new rename_domain_1.RenameDomain(this.reg);
             case "INTF":
                 return new rename_global_interface_1.RenameGlobalInterface(this.reg);
             default:

package/build/src/objects/rename/renamer_helper.js

@@ -35,11 +35,12 @@ class RenamerHelper {
         }
         const tag = xmlTag.toUpperCase();
         const search = "<" + tag + ">" + oldName.toUpperCase() + "</" + tag + ">";
+        const length = tag.length + 2;
         const rows = xml.getRawRows();
         for (let i = 0; i < rows.length; i++) {
             const index = rows[i].indexOf(search);
             if (index >= 0) {
-                const range = vscode_languageserver_types_1.Range.create(i, index + 9, i, index + oldName.length + 9);
+                const range = vscode_languageserver_types_1.Range.create(i, index + length, i, index + oldName.length + length);
                 changes.push(vscode_languageserver_types_1.TextDocumentEdit.create({ uri: xml.getFilename(), version: 1 }, [vscode_languageserver_types_1.TextEdit.replace(range, newName.toUpperCase())]));
             }
         }

package/build/src/registry.js

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

package/build/src/rules/constant_classes.js

@@ -7,6 +7,11 @@ const _irule_1 = require("./_irule");
 const __1 = require("..");
 /** Checks that constants classes are in sync with domain fixed values */
 class ConstantClassesConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        /** Specify a list of domain-class pairs which will be validated */
+        this.mapping = [];
+    }
 }
 exports.ConstantClassesConf = ConstantClassesConf;
 class ConstantClasses {

package/build/src/rules/exit_or_check.js

@@ -26,11 +26,9 @@ class ExitOrCheck extends _abap_rule_1.ABAPRule {
             title: "Find EXIT or CHECK outside loops",
             shortDescription: `Detects usages of EXIT or CHECK statements outside of loops.
 Use RETURN to leave procesing blocks instead.`,
-            extendedInformation: `
-https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenleave_processing_blocks.htm
+            extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenleave_processing_blocks.htm
 https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abapcheck_processing_blocks.htm
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#check-vs-return
-`,
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#check-vs-return`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
         };
     }
@@ -44,19 +42,20 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#check-vs-re
         const issues = [];
         const stack = [];
         for (const statement of file.getStatements()) {
-            if (statement.get() instanceof Statements.Loop
-                || statement.get() instanceof Statements.While
-                || statement.get() instanceof Statements.SelectLoop
-                || statement.get() instanceof Statements.Do) {
+            const get = statement.get();
+            if (get instanceof Statements.Loop
+                || get instanceof Statements.While
+                || get instanceof Statements.SelectLoop
+                || get instanceof Statements.Do) {
                 stack.push(statement);
             }
-            else if (statement.get() instanceof Statements.EndLoop
-                || statement.get() instanceof Statements.EndWhile
-                || statement.get() instanceof Statements.EndSelect
-                || statement.get() instanceof Statements.EndDo) {
+            else if (get instanceof Statements.EndLoop
+                || get instanceof Statements.EndWhile
+                || get instanceof Statements.EndSelect
+                || get instanceof Statements.EndDo) {
                 stack.pop();
             }
-            else if (this.conf.allowCheck === false && statement.get() instanceof Statements.Check && stack.length === 0) {
+            else if (this.conf.allowCheck === false && get instanceof Statements.Check && stack.length === 0) {
                 const message = "CHECK is not allowed outside of loops";
                 let tokensString = statement.concatTokens();
                 tokensString = tokensString.slice(statement.getFirstToken().getEnd().getCol());
@@ -65,7 +64,7 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#check-vs-re
                 const issue = issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, fix);
                 issues.push(issue);
             }
-            else if (this.conf.allowExit === false && statement.get() instanceof Statements.Exit && stack.length === 0) {
+            else if (this.conf.allowExit === false && get instanceof Statements.Exit && stack.length === 0) {
                 const message = "EXIT is not allowed outside of loops";
                 const fix = edit_helper_1.EditHelper.replaceToken(file, statement.getFirstToken(), "RETURN");
                 const issue = issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, fix);

package/build/src/rules/unused_variables.js

@@ -186,12 +186,10 @@ class UnusedVariables {
         if (statement === undefined) {
             return undefined;
         }
-        else if (statement.get() instanceof Statements.FunctionModule) {
-            return undefined; // function module parameters are not part of the code
-        }
-        else {
+        else if (statement.get() instanceof Statements.Data) {
             return edit_helper_1.EditHelper.deleteStatement(file, statement);
         }
+        return undefined;
     }
 }
 exports.UnusedVariables = UnusedVariables;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.79.8",
+    "version": "2.79.9",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -54,21 +54,21 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@microsoft/api-extractor": "^7.18.10",
+        "@microsoft/api-extractor": "^7.18.11",
         "@types/chai": "^4.2.22",
         "@types/mocha": "^9.0.0",
-        "@types/node": "^16.9.6",
+        "@types/node": "^16.10.2",
         "chai": "^4.3.4",
         "eslint": "^7.32.0",
         "madge": "^5.0.1",
-        "mocha": "^9.1.1",
+        "mocha": "^9.1.2",
         "c8": "^7.9.0",
         "source-map-support": "^0.5.20",
         "ts-json-schema-generator": "^0.95.0",
         "typescript": "^4.4.3"
     },
     "dependencies": {
-        "fast-xml-parser": "^3.20.0",
+        "fast-xml-parser": "^3.20.3",
         "json5": "^2.2.0",
         "vscode-languageserver-protocol": "^3.16.0",
         "vscode-languageserver-types": "^3.16.0"