@abaplint/core 2.120.73 → 2.120.75

package/build/src/rules/inline_packed_computation.js

@@ -0,0 +1,220 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.InlinePackedComputation = exports.InlinePackedComputationConf = void 0;
+const Expressions = __importStar(require("../abap/2_statements/expressions"));
+const Statements = __importStar(require("../abap/2_statements/statements"));
+const _reference_1 = require("../abap/5_syntax/_reference");
+const constant_1 = require("../abap/5_syntax/expressions/constant");
+const syntax_1 = require("../abap/5_syntax/syntax");
+const nodes_1 = require("../abap/nodes");
+const types_1 = require("../abap/types");
+const _typed_identifier_1 = require("../abap/types/_typed_identifier");
+const basic_1 = require("../abap/types/basic");
+const issue_1 = require("../issue");
+const _abap_object_1 = require("../objects/_abap_object");
+const _basic_rule_config_1 = require("./_basic_rule_config");
+class InlinePackedComputationConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.InlinePackedComputationConf = InlinePackedComputationConf;
+class InlinePackedComputation {
+    constructor() {
+        this.conf = new InlinePackedComputationConf();
+    }
+    getMetadata() {
+        return {
+            key: "inline_packed_computation",
+            title: "Inline declaration from a packed computation",
+            shortDescription: `Finds inline declarations whose value is an arithmetic expression with calculation type p`,
+            extendedInformation: `The result of a computation with calculation type p has no length and number of decimals
+that a declaration could take over, so DATA( ) or FINAL( ) gets the type p LENGTH 8 DECIMALS 0 and the
+syntax check warns "For the result of a computation with type P, the type P(8,0) is used here implicitly".
+Decimals of the operands are lost, and so are digits beyond the length.
+
+Reported when at least one operand is packed and all other operands are of type i, int8 or p. Operands
+of other types, generic or unknown types, and the operator ** are not reported.
+
+Declare the variable with the operands' type, or name the type with CONV.`,
+            tags: [],
+            badExample: `DATA a TYPE p LENGTH 8 DECIMALS 2.
+DATA b TYPE p LENGTH 8 DECIMALS 2.
+DATA(diff) = a - b.`,
+            goodExample: `DATA a TYPE p LENGTH 8 DECIMALS 2.
+DATA b TYPE p LENGTH 8 DECIMALS 2.
+DATA diff LIKE a.
+diff = a - b.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    initialize(reg) {
+        this.reg = reg;
+        return this;
+    }
+    run(obj) {
+        var _a, _b;
+        if (!(obj instanceof _abap_object_1.ABAPObject)) {
+            return [];
+        }
+        const syntax = new syntax_1.SyntaxLogic(this.reg, obj).run();
+        if (syntax.issues.length > 0) {
+            return [];
+        }
+        const issues = [];
+        for (const file of obj.getABAPFiles()) {
+            for (const statement of file.getStatements()) {
+                if (!(statement.get() instanceof Statements.Move)) {
+                    continue;
+                }
+                const inline = (_a = statement.findDirectExpression(Expressions.Target)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(Expressions.InlineData);
+                const name = (_b = inline === null || inline === void 0 ? void 0 : inline.findDirectExpression(Expressions.TargetField)) === null || _b === void 0 ? void 0 : _b.getFirstToken();
+                const source = statement.findDirectExpression(Expressions.Source);
+                if (name === undefined
+                    || source === undefined
+                    || source.findDirectExpression(Expressions.ArithOperator) === undefined) {
+                    continue;
+                }
+                const scope = syntax.spaghetti.lookupPosition(name.getStart(), file.getFilename());
+                if (scope === undefined) {
+                    continue;
+                }
+                const operands = this.operandTypes(source, scope);
+                if (operands === undefined || operands.some(t => t instanceof basic_1.PackedType) === false) {
+                    continue;
+                }
+                const message = `"${name.getStr()}" gets type p LENGTH 8 DECIMALS 0 from this packed computation, ` +
+                    `declare it with the operands' type or use CONV`;
+                issues.push(issue_1.Issue.atToken(file, name, message, this.getMetadata().key, this.conf.severity));
+            }
+        }
+        return issues;
+    }
+    /** The types of the operands of an arithmetic expression, undefined when an operand is not of type
+     *  i, int8 or p, or is not known */
+    operandTypes(source, scope) {
+        const ret = [];
+        for (const child of source.getChildren()) {
+            let found = undefined;
+            if (child instanceof nodes_1.TokenNode) {
+                // parentheses and a sign, anything else is a constructor expression or a string operator
+                if (["(", ")", "+", "-"].includes(child.getFirstToken().getStr()) === false) {
+                    return undefined;
+                }
+                continue;
+            }
+            else if (child.get() instanceof Expressions.ArithOperator) {
+                if (["+", "-", "*", "/", "DIV", "MOD"].includes(child.concatTokens().toUpperCase()) === false) {
+                    return undefined;
+                }
+                continue;
+            }
+            else if (child.get() instanceof Expressions.Source) {
+                const nested = this.operandTypes(child, scope);
+                if (nested === undefined) {
+                    return undefined;
+                }
+                ret.push(...nested);
+                continue;
+            }
+            else if (child.get() instanceof Expressions.FieldChain) {
+                found = this.fieldChainType(child, scope);
+            }
+            else if (child.get() instanceof Expressions.MethodCallChain) {
+                found = this.methodCallType(child, scope);
+            }
+            else if (child.get() instanceof Expressions.Constant) {
+                found = constant_1.Constant.runSyntax(child);
+            }
+            if (found === undefined
+                || found.isGeneric()
+                || !(found instanceof basic_1.PackedType || found instanceof basic_1.IntegerType || found instanceof basic_1.Integer8Type)) {
+                return undefined;
+            }
+            ret.push(found);
+        }
+        return ret;
+    }
+    methodCallType(chain, scope) {
+        var _a, _b, _c;
+        const last = chain.getLastChild();
+        if (!(last instanceof nodes_1.ExpressionNode) || !(last.get() instanceof Expressions.MethodCall)) {
+            return undefined;
+        }
+        const name = (_a = last.findDirectExpression(Expressions.MethodName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+        if (name === undefined) {
+            return undefined;
+        }
+        const method = (_b = scope.getData().references.find(reference => reference.referenceType === _reference_1.ReferenceType.MethodReference
+            && reference.position.getStart().equals(name.getStart()))) === null || _b === void 0 ? void 0 : _b.resolved;
+        if (!(method instanceof types_1.MethodDefinition)) {
+            return undefined;
+        }
+        return (_c = method.getParameters().getReturning()) === null || _c === void 0 ? void 0 : _c.getType();
+    }
+    fieldChainType(fieldChain, scope) {
+        var _a;
+        const operand = (_a = scope.getData().references.find(reference => reference.referenceType === _reference_1.ReferenceType.DataReadReference
+            && reference.position.getStart().equals(fieldChain.getFirstToken().getStart()))) === null || _a === void 0 ? void 0 : _a.resolved;
+        if (!(operand instanceof _typed_identifier_1.TypedIdentifier)) {
+            return undefined;
+        }
+        let type = operand.getType();
+        for (const child of fieldChain.getChildren()) {
+            if (!(child instanceof nodes_1.ExpressionNode)
+                || child.get() instanceof Expressions.SourceField) {
+                continue;
+            }
+            else if (child.get() instanceof Expressions.ComponentName) {
+                if (type instanceof basic_1.TableType) {
+                    type = type.getRowType();
+                }
+                if (!(type instanceof basic_1.StructureType)) {
+                    return undefined;
+                }
+                type = type.getComponentByName(child.concatTokens());
+            }
+            else {
+                return undefined;
+            }
+        }
+        return type;
+    }
+}
+exports.InlinePackedComputation = InlinePackedComputation;
+//# sourceMappingURL=inline_packed_computation.js.map

package/build/src/rules/intf_referencing_clas.js

@@ -20,6 +20,7 @@ exports.IntfReferencingClasConf = IntfReferencingClasConf;
 class IntfReferencingClas {
     constructor() {
         this.conf = new IntfReferencingClasConf();
+        this.allow = [];
     }
     getMetadata() {
         return {
@@ -40,6 +41,7 @@ class IntfReferencingClas {
     }
     setConfig(conf) {
         this.conf = conf;
+        this.allow = (this.conf.allow || []).map(reg => new RegExp(reg, "i"));
     }
     initialize(reg) {
         this.reg = reg;
@@ -65,7 +67,7 @@ class IntfReferencingClas {
                 if (found && ddic.isException(found.getClassDefinition(), found)) {
                     continue;
                 }
-                else if (this.getConfig().allow.some(reg => new RegExp(reg, "i").test(r.extra.ooName))) {
+                else if (this.allow.some(reg => reg.test(r.extra.ooName))) {
                     continue;
                 }
                 ret.push(issue_1.Issue.atIdentifier(r.position, message + r.extra.ooName, this.getMetadata().key, this.conf.severity));

package/build/src/rules/line_only_punc.js

@@ -57,7 +57,7 @@ https://docs.abapopenchecks.org/checks/16/`,
             }
         }
         const rows = file.getRawRows();
-        const reg = new RegExp("^\\)?\\. *(\\\".*)?$");
+        const reg = /^\)?\. *(".*)?$/;
         for (let i = 0; i < rows.length; i++) {
             if (reg.exec(rows[i].trim())) {
                 const column = rows[i].indexOf(")") >= 0 ? rows[i].indexOf(")") + 1 : rows[i].indexOf(".") + 1;

package/build/src/rules/local_class_naming.js

@@ -23,6 +23,7 @@ class LocalClassNaming extends _abap_rule_1.ABAPRule {
     constructor() {
         super(...arguments);
         this.conf = new LocalClassNamingConf();
+        this.regex = this.compile();
     }
     getMetadata() {
         return {
@@ -46,6 +47,15 @@ ENDCLASS.`,
     }
     setConfig(conf) {
         this.conf = conf;
+        this.regex = this.compile();
+    }
+    compile() {
+        return {
+            local: new RegExp(this.conf.local, "i"),
+            exception: new RegExp(this.conf.exception, "i"),
+            test: new RegExp(this.conf.test, "i"),
+            ignorePatterns: name_validator_1.NameValidator.compileIgnorePatterns(this.conf),
+        };
     }
     runParsed(file, obj) {
         const issues = [];
@@ -59,20 +69,23 @@ ENDCLASS.`,
             }
             const className = classDef.name;
             let expected;
+            let regex;
             if (classDef.isForTesting) {
                 expected = this.conf.test;
+                regex = this.regex.test;
             }
             else if (ddic.isException(classDef, obj)) {
                 expected = this.conf.exception;
+                regex = this.regex.exception;
             }
             else {
                 expected = this.conf.local;
+                regex = this.regex.local;
             }
             if (expected === undefined || expected.length === 0) {
                 continue;
             }
-            const regex = new RegExp(expected, "i");
-            if (name_validator_1.NameValidator.violatesRule(className, regex, this.conf)) {
+            if (name_validator_1.NameValidator.violatesRule(className, regex, this.conf, this.regex.ignorePatterns)) {
                 issues.push(issue_1.Issue.atIdentifier(classDef.identifier, this.getDescription(expected, className), this.getMetadata().key, this.conf.severity));
             }
         }

package/build/src/rules/local_variable_names.js

@@ -58,6 +58,7 @@ class LocalVariableNames extends _abap_rule_1.ABAPRule {
     constructor() {
         super(...arguments);
         this.conf = new LocalVariableNamesConf();
+        this.regex = this.compile();
     }
     getMetadata() {
         return {
@@ -85,6 +86,15 @@ ENDFORM.`,
     }
     setConfig(conf) {
         this.conf = conf;
+        this.regex = this.compile();
+    }
+    compile() {
+        return {
+            expectedData: new RegExp(this.conf.expectedData, "i"),
+            expectedConstant: new RegExp(this.conf.expectedConstant, "i"),
+            expectedFS: new RegExp(this.conf.expectedFS, "i"),
+            ignorePatterns: name_validator_1.NameValidator.compileIgnorePatterns(this.conf),
+        };
     }
     runParsed(file) {
         const ret = [];
@@ -118,7 +128,7 @@ ENDFORM.`,
             const found = dat.findFirstExpression(Expressions.DefinitionName);
             if (found) {
                 const token = found.getFirstToken();
-                ret = ret.concat(this.checkName(token, file, this.conf.expectedData));
+                ret = ret.concat(this.checkName(token, file, this.conf.expectedData, this.regex.expectedData));
             }
         }
         // inline data
@@ -126,7 +136,7 @@ ENDFORM.`,
             const found = dat.findFirstExpression(Expressions.TargetField);
             if (found) {
                 const token = found.getFirstToken();
-                ret = ret.concat(this.checkName(token, file, this.conf.expectedData));
+                ret = ret.concat(this.checkName(token, file, this.conf.expectedData, this.regex.expectedData));
             }
         }
         // data structures, data begin of, first level
@@ -137,14 +147,14 @@ ENDFORM.`,
             const found = stat === null || stat === void 0 ? void 0 : stat.findFirstExpression(Expressions.DefinitionName);
             if (found) {
                 const token = found.getFirstToken();
-                ret = ret.concat(this.checkName(token, file, this.conf.expectedData));
+                ret = ret.concat(this.checkName(token, file, this.conf.expectedData, this.regex.expectedData));
             }
         }
         for (const fieldsymbol of structure.findAllStatements(Statements.FieldSymbol)) {
             const found = fieldsymbol.findFirstExpression(Expressions.FieldSymbol);
             if (found) {
                 const token = found.getFirstToken();
-                ret = ret.concat(this.checkName(token, file, this.conf.expectedFS));
+                ret = ret.concat(this.checkName(token, file, this.conf.expectedFS, this.regex.expectedFS));
             }
         }
         // inline field symbols
@@ -152,7 +162,7 @@ ENDFORM.`,
             const found = fieldsymbol.findFirstExpression(Expressions.TargetFieldSymbol);
             if (found) {
                 const token = found.getFirstToken();
-                ret = ret.concat(this.checkName(token, file, this.conf.expectedFS));
+                ret = ret.concat(this.checkName(token, file, this.conf.expectedFS, this.regex.expectedFS));
             }
         }
         const constants = structure.findAllStatements(Statements.Constant);
@@ -164,16 +174,15 @@ ENDFORM.`,
             const found = constant.findFirstExpression(Expressions.DefinitionName);
             if (found) {
                 const token = found.getFirstToken();
-                ret = ret.concat(this.checkName(token, file, this.conf.expectedConstant));
+                ret = ret.concat(this.checkName(token, file, this.conf.expectedConstant, this.regex.expectedConstant));
             }
         }
         return ret;
     }
-    checkName(token, file, expected) {
+    checkName(token, file, expected, regex) {
         const ret = [];
-        const regex = new RegExp(expected, "i");
         const name = token.getStr();
-        if (name_validator_1.NameValidator.violatesRule(name, regex, this.conf)) {
+        if (name_validator_1.NameValidator.violatesRule(name, regex, this.conf, this.regex.ignorePatterns)) {
             const message = this.getDescription(expected, name);
             const issue = issue_1.Issue.atToken(file, token, message, this.getMetadata().key, this.conf.severity);
             ret.push(issue);

package/build/src/rules/macro_naming.js

@@ -53,6 +53,7 @@ class MacroNaming extends _abap_rule_1.ABAPRule {
     constructor() {
         super(...arguments);
         this.conf = new MacroNamingConf();
+        this.regex = new RegExp(this.conf.pattern, "i");
     }
     getMetadata() {
         return {
@@ -72,10 +73,10 @@ END-OF-DEFINITION.`,
     }
     setConfig(conf) {
         this.conf = conf;
+        this.regex = new RegExp(this.conf.pattern, "i");
     }
     runParsed(file, obj) {
         const issues = [];
-        const testRegex = new RegExp(this.conf.pattern, "i");
         if (obj instanceof objects_1.TypePool) {
             return [];
         }
@@ -88,7 +89,7 @@ END-OF-DEFINITION.`,
                 continue;
             }
             const token = expr.getFirstToken();
-            if (testRegex.exec(token.getStr())) {
+            if (this.regex.exec(token.getStr())) {
                 continue;
             }
             else {

package/build/src/rules/method_parameter_names.js

@@ -27,6 +27,7 @@ exports.MethodParameterNamesConf = MethodParameterNamesConf;
 class MethodParameterNames {
     constructor() {
         this.conf = new MethodParameterNamesConf();
+        this.regex = this.compile();
     }
     getMetadata() {
         return {
@@ -52,6 +53,16 @@ class MethodParameterNames {
     }
     setConfig(conf) {
         this.conf = conf;
+        this.regex = this.compile();
+    }
+    compile() {
+        return {
+            importing: new RegExp(this.conf.importing, "i"),
+            returning: new RegExp(this.conf.returning, "i"),
+            changing: new RegExp(this.conf.changing, "i"),
+            exporting: new RegExp(this.conf.exporting, "i"),
+            ignorePatterns: name_validator_1.NameValidator.compileIgnorePatterns(this.conf),
+        };
     }
     initialize(reg) {
         this.reg = reg;
@@ -91,16 +102,16 @@ class MethodParameterNames {
         for (const p of method.parameters) {
             switch (p.direction) {
                 case _abap_file_information_1.MethodParameterDirection.Importing:
-                    ret = ret.concat(this.checkParameter(p, this.conf.importing));
+                    ret = ret.concat(this.checkParameter(p, this.conf.importing, this.regex.importing));
                     break;
                 case _abap_file_information_1.MethodParameterDirection.Exporting:
-                    ret = ret.concat(this.checkParameter(p, this.conf.exporting));
+                    ret = ret.concat(this.checkParameter(p, this.conf.exporting, this.regex.exporting));
                     break;
                 case _abap_file_information_1.MethodParameterDirection.Changing:
-                    ret = ret.concat(this.checkParameter(p, this.conf.changing));
+                    ret = ret.concat(this.checkParameter(p, this.conf.changing, this.regex.changing));
                     break;
                 case _abap_file_information_1.MethodParameterDirection.Returning:
-                    ret = ret.concat(this.checkParameter(p, this.conf.returning));
+                    ret = ret.concat(this.checkParameter(p, this.conf.returning, this.regex.returning));
                     break;
                 default:
                     break;
@@ -108,11 +119,10 @@ class MethodParameterNames {
         }
         return ret;
     }
-    checkParameter(param, expected) {
+    checkParameter(param, expected, regex) {
         const ret = [];
-        const regex = new RegExp(expected, "i");
         const name = param.name;
-        if (name_validator_1.NameValidator.violatesRule(name, regex, this.conf)) {
+        if (name_validator_1.NameValidator.violatesRule(name, regex, this.conf, this.regex.ignorePatterns)) {
             const message = this.getDescription(expected, name);
             const issue = issue_1.Issue.atIdentifier(param.identifier, message, this.getMetadata().key, this.conf.severity);
             ret.push(issue);

package/build/src/rules/modify_only_own_db_tables.js

@@ -53,6 +53,7 @@ exports.ModifyOnlyOwnDBTablesConf = ModifyOnlyOwnDBTablesConf;
 class ModifyOnlyOwnDBTables {
     constructor() {
         this.conf = new ModifyOnlyOwnDBTablesConf();
+        this.ownTables = new RegExp(this.conf.ownTables, "i");
     }
     getMetadata() {
         return {
@@ -70,6 +71,7 @@ class ModifyOnlyOwnDBTables {
     }
     setConfig(conf) {
         this.conf = conf;
+        this.ownTables = new RegExp(this.conf.ownTables, "i");
     }
     initialize(reg) {
         this.reg = reg;
@@ -87,7 +89,6 @@ class ModifyOnlyOwnDBTables {
             if (struc === undefined) {
                 return [];
             }
-            const regExp = new RegExp(this.getConfig().ownTables, "i");
             for (const s of file.getStatements()) {
                 const g = s.get();
                 if (g instanceof Statements.DeleteDatabase
@@ -105,7 +106,7 @@ class ModifyOnlyOwnDBTables {
                         continue;
                     }
                     const concat = databaseTable.concatTokens().toUpperCase();
-                    if (regExp.test(concat) === false) {
+                    if (this.ownTables.test(concat) === false) {
                         // must contain a ReferenceType.TableVoidReference or a ReferenceType.TableReference if its a dependency
                         if (spaghetti === undefined) {
                             spaghetti = new syntax_1.SyntaxLogic(this.reg, obj).run().spaghetti;

package/build/src/rules/no_exclamation_escape.js

@@ -7,6 +7,31 @@ const _basic_rule_config_1 = require("./_basic_rule_config");
 const _irule_1 = require("./_irule");
 const edit_helper_1 = require("../edit_helper");
 const tokens_1 = require("../abap/1_lexer/tokens");
+const artifacts_1 = require("../abap/artifacts");
+const combi_1 = require("../abap/2_statements/combi");
+let keywords = undefined;
+/** every word of the statement and expression grammar, upper case */
+function getKeywords() {
+    if (keywords === undefined) {
+        const list = [];
+        for (const stat of artifacts_1.ArtifactsABAP.getStatements()) {
+            list.push(...combi_1.Combi.listKeywords(stat.getMatcher()));
+        }
+        for (const expr of artifacts_1.ArtifactsABAP.getExpressions()) {
+            list.push(...combi_1.Combi.listKeywords(new expr().getRunnable()));
+        }
+        keywords = new Set();
+        for (const words of list) {
+            // a WordSequence lists eg. "PREFERRED PARAMETER" or "IS-INITIAL" as one entry
+            for (const word of words.replace(/-/g, " ").split(" ")) {
+                if (word !== "") {
+                    keywords.add(word.toUpperCase());
+                }
+            }
+        }
+    }
+    return keywords;
+}
 class NoExclamationEscapeConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.NoExclamationEscapeConf = NoExclamationEscapeConf;
@@ -20,8 +45,10 @@ class NoExclamationEscape extends _abap_rule_1.ABAPRule {
             key: "no_exclamation_escape",
             title: "No exclamation escape",
             shortDescription: `Detects and removes exclamation marks (!) used to escape identifiers`,
-            // eslint-disable-next-line max-len
-            extendedInformation: `Exclamation marks are not needed when the identifier is not a reserved keyword, or could be avoided by renaming.`,
+            extendedInformation: `Exclamation marks are not needed when the identifier is not a keyword, or could be avoided by renaming.
+
+An identifier named like an ABAP keyword is allowed to be escaped, eg. a method parameter "!default",
+which the system otherwise reads as the DEFAULT addition of the parameter before it.`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
             badExample: "methods CONVERT changing !CO_sdf type ref to ZCL_sdf optional.",
             goodExample: "methods CONVERT changing CO_sdf type ref to ZCL_sdf optional.",
@@ -40,6 +67,9 @@ class NoExclamationEscape extends _abap_rule_1.ABAPRule {
                 const str = token.getStr();
                 if (str.startsWith("!")) {
                     const replacement = str.substring(1);
+                    if (getKeywords().has(replacement.toUpperCase())) {
+                        continue;
+                    }
                     const fix = edit_helper_1.EditHelper.replaceToken(file, token, replacement);
                     issues.push(issue_1.Issue.atToken(file, token, "Do not use exclamation mark to escape identifiers", this.getMetadata().key, this.conf.severity, fix));
                 }

package/build/src/rules/no_prefixes.js

@@ -71,6 +71,7 @@ class NoPrefixes extends _abap_rule_1.ABAPRule {
     constructor() {
         super(...arguments);
         this.conf = new NoPrefixesConf();
+        this.regex = this.compile();
     }
     getMetadata() {
         return {
@@ -94,6 +95,18 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/sub-sections/AvoidEncodi
     }
     setConfig(conf) {
         this.conf = conf;
+        this.regex = this.compile();
+    }
+    compile() {
+        const compile = (pattern) => pattern !== undefined && pattern !== "" ? new RegExp(pattern, "i") : undefined;
+        return {
+            data: compile(this.conf.data),
+            statics: compile(this.conf.statics),
+            fieldSymbols: compile(this.conf.fieldSymbols),
+            constants: compile(this.conf.constants),
+            types: compile(this.conf.types),
+            methodParameters: compile(this.conf.methodParameters),
+        };
     }
     runParsed(file) {
         const ret = [];
@@ -107,38 +120,38 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/sub-sections/AvoidEncodi
         if (max === undefined || max < 1) {
             max = 10;
         }
-        if (config.data !== undefined && config.data !== "") {
-            ret.push(...this.checkData(structure, new RegExp(config.data, "i"), file));
+        if (this.regex.data !== undefined) {
+            ret.push(...this.checkData(structure, this.regex.data, file));
         }
         if (ret.length >= max) {
             return ret;
         }
-        if (config.statics !== undefined && config.statics !== "") {
-            ret.push(...this.checkStatics(structure, new RegExp(config.statics, "i"), file));
+        if (this.regex.statics !== undefined) {
+            ret.push(...this.checkStatics(structure, this.regex.statics, file));
         }
         if (ret.length >= max) {
             return ret;
         }
-        if (config.fieldSymbols !== undefined && config.fieldSymbols !== "") {
-            ret.push(...this.checkFieldSymbols(structure, new RegExp(config.fieldSymbols, "i"), file));
+        if (this.regex.fieldSymbols !== undefined) {
+            ret.push(...this.checkFieldSymbols(structure, this.regex.fieldSymbols, file));
         }
         if (ret.length >= max) {
             return ret;
         }
-        if (config.constants !== undefined && config.constants !== "") {
-            ret.push(...this.checkConstants(structure, new RegExp(config.constants, "i"), file));
+        if (this.regex.constants !== undefined) {
+            ret.push(...this.checkConstants(structure, this.regex.constants, file));
         }
         if (ret.length >= max) {
             return ret;
         }
-        if (config.types !== undefined && config.types !== "") {
-            ret.push(...this.checkTypes(structure, new RegExp(config.types, "i"), file));
+        if (this.regex.types !== undefined) {
+            ret.push(...this.checkTypes(structure, this.regex.types, file));
         }
         if (ret.length >= max) {
             return ret;
         }
-        if (config.methodParameters !== undefined && config.methodParameters !== "") {
-            ret.push(...this.checkMethodParameters(structure, new RegExp(config.methodParameters, "i"), file));
+        if (this.regex.methodParameters !== undefined) {
+            ret.push(...this.checkMethodParameters(structure, this.regex.methodParameters, file));
         }
         return ret;
     }