@abaplint/core 2.113.201 → 2.113.203

package/build/src/abap/1_lexer/lexer.js

@@ -6,19 +6,17 @@ const virtual_position_1 = require("../../virtual_position");
 const tokens_1 = require("./tokens");
 const lexer_buffer_1 = require("./lexer_buffer");
 const lexer_stream_1 = require("./lexer_stream");
+const ModeNormal = 1;
+const ModePing = 2;
+const ModeStr = 3;
+const ModeTemplate = 4;
+const ModeComment = 5;
+const ModePragma = 6;
 class Lexer {
-    constructor() {
-        this.ModeNormal = 1;
-        this.ModePing = 2;
-        this.ModeStr = 3;
-        this.ModeTemplate = 4;
-        this.ModeComment = 5;
-        this.ModePragma = 6;
-    }
     run(file, virtual) {
         this.virtual = virtual;
         this.tokens = [];
-        this.m = this.ModeNormal;
+        this.m = ModeNormal;
         this.process(file.getRaw());
         return { file, tokens: this.tokens };
     }
@@ -44,13 +42,13 @@ class Lexer {
                 pos = new virtual_position_1.VirtualPosition(this.virtual, pos.getRow(), pos.getCol());
             }
             let tok = undefined;
-            if (this.m === this.ModeComment) {
+            if (this.m === ModeComment) {
                 tok = new tokens_1.Comment(pos, s);
             }
-            else if (this.m === this.ModePing || this.m === this.ModeStr) {
+            else if (this.m === ModePing || this.m === ModeStr) {
                 tok = new tokens_1.StringToken(pos, s);
             }
-            else if (this.m === this.ModeTemplate) {
+            else if (this.m === ModeTemplate) {
                 const first = s.charAt(0);
                 const last = s.charAt(s.length - 1);
                 if (first === "|" && last === "|") {
@@ -243,35 +241,35 @@ class Lexer {
             const buf = this.buffer.add(current);
             const ahead = this.stream.nextChar();
             const aahead = this.stream.nextNextChar();
-            if (this.m === this.ModeNormal) {
+            if (this.m === ModeNormal) {
                 if (splits[ahead]) {
                     this.add();
                 }
                 else if (ahead === "'") {
                     // start string
                     this.add();
-                    this.m = this.ModeStr;
+                    this.m = ModeStr;
                 }
                 else if (ahead === "|" || ahead === "}") {
                     // start template
                     this.add();
-                    this.m = this.ModeTemplate;
+                    this.m = ModeTemplate;
                 }
                 else if (ahead === "`") {
                     // start ping
                     this.add();
-                    this.m = this.ModePing;
+                    this.m = ModePing;
                 }
                 else if (aahead === "##") {
                     // start pragma
                     this.add();
-                    this.m = this.ModePragma;
+                    this.m = ModePragma;
                 }
                 else if (ahead === "\""
                     || (ahead === "*" && current === "\n")) {
                     // start comment
                     this.add();
-                    this.m = this.ModeComment;
+                    this.m = ModeComment;
                 }
                 else if (ahead === "@" && buf.trim().length === 0) {
                     this.add();
@@ -292,12 +290,12 @@ class Lexer {
                     this.add();
                 }
             }
-            else if (this.m === this.ModePragma && (ahead === "," || ahead === ":" || ahead === "." || ahead === " " || ahead === "\n")) {
+            else if (this.m === ModePragma && (ahead === "," || ahead === ":" || ahead === "." || ahead === " " || ahead === "\n")) {
                 // end of pragma
                 this.add();
-                this.m = this.ModeNormal;
+                this.m = ModeNormal;
             }
-            else if (this.m === this.ModePing
+            else if (this.m === ModePing
                 && buf.length > 1
                 && current === "`"
                 && aahead !== "``"
@@ -306,26 +304,26 @@ class Lexer {
                 // end of ping
                 this.add();
                 if (ahead === `"`) {
-                    this.m = this.ModeComment;
+                    this.m = ModeComment;
                 }
                 else {
-                    this.m = this.ModeNormal;
+                    this.m = ModeNormal;
                 }
             }
-            else if (this.m === this.ModeTemplate
+            else if (this.m === ModeTemplate
                 && buf.length > 1
                 && (current === "|" || current === "{")
                 && (this.stream.prevChar() !== "\\" || this.stream.prevPrevChar() === "\\\\")) {
                 // end of template
                 this.add();
-                this.m = this.ModeNormal;
+                this.m = ModeNormal;
             }
-            else if (this.m === this.ModeTemplate
+            else if (this.m === ModeTemplate
                 && ahead === "}"
                 && current !== "\\") {
                 this.add();
             }
-            else if (this.m === this.ModeStr
+            else if (this.m === ModeStr
                 && current === "'"
                 && buf.length > 1
                 && aahead !== "''"
@@ -334,17 +332,17 @@ class Lexer {
                 // end of string
                 this.add();
                 if (ahead === "\"") {
-                    this.m = this.ModeComment;
+                    this.m = ModeComment;
                 }
                 else {
-                    this.m = this.ModeNormal;
+                    this.m = ModeNormal;
                 }
             }
-            else if (ahead === "\n" && this.m !== this.ModeTemplate) {
+            else if (ahead === "\n" && this.m !== ModeTemplate) {
                 this.add();
-                this.m = this.ModeNormal;
+                this.m = ModeNormal;
             }
-            else if (this.m === this.ModeTemplate && current === "\n") {
+            else if (this.m === ModeTemplate && current === "\n") {
                 this.add();
             }
             if (!this.stream.advance()) {

package/build/src/objects/table.js

@@ -42,7 +42,7 @@ class Table extends _abstract_object_1.AbstractObject {
     }
     getAllowedNaming() {
         let length = 30;
-        const regex = /^((\/[A-Z_\d]{3,8}\/)|[a-zA-Z0-9]{3})\w+$/;
+        const regex = /^((\/[A-Z_\d]{3,8}\/)|[a-zA-Z0-9]{3}|CI_)\w+$/;
         if (this.getTableCategory() === TableCategory.Transparent) {
             length = 16;
         }

package/build/src/registry.js

@@ -67,7 +67,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.201";
+        return "2.113.203";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/check_comments.js

@@ -11,6 +11,7 @@ class CheckCommentsConf extends _basic_rule_config_1.BasicRuleConfig {
         super(...arguments);
         /** Allows the use of end-of-line comments. */
         this.allowEndOfLine = false;
+        this.maxIssuesPerFile = 10;
     }
 }
 exports.CheckCommentsConf = CheckCommentsConf;
@@ -56,6 +57,10 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#put-comment
         if (this.conf.allowEndOfLine === true) {
             return [];
         }
+        let max = this.getConfig().maxIssuesPerFile;
+        if (max === undefined || max < 1) {
+            max = 10;
+        }
         const commentRows = [];
         for (let i = 0; i < rows.length; i++) {
             const row = rows[i];
@@ -72,6 +77,9 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#put-comment
                     continue;
                 }
                 issues.push(issue_1.Issue.atStatement(file, statement, this.getDescription(IssueType.EndOfLine), this.getMetadata().key, this.conf.severity));
+                if (issues.length >= max) {
+                    break;
+                }
             }
         }
         return issues;

package/build/src/rules/indentation.js

@@ -26,6 +26,7 @@ class IndentationConf extends _basic_rule_config_1.BasicRuleConfig {
         this.globalClassSkipFirst = false;
         this.ignoreGlobalClassDefinition = false;
         this.ignoreGlobalInterface = false;
+        this.maxIssuesPerFile = 10;
     }
 }
 exports.IndentationConf = IndentationConf;
@@ -68,7 +69,10 @@ ENDCLASS.`,
     }
     runParsed(file, obj) {
         var _a, _b;
-        const MAX_ISSUES = 100;
+        let max = this.getConfig().maxIssuesPerFile;
+        if (max === undefined || max < 1) {
+            max = 10;
+        }
         let skip = false;
         if (file.getStructure() === undefined) {
             return []; // syntax error in file
@@ -140,7 +144,7 @@ ENDCLASS.`,
                 const message = "Indentation problem, expected " + expected + " spaces";
                 const issue = issue_1.Issue.atPosition(file, position, message, this.getMetadata().key, this.conf.severity, fix);
                 ret.push(issue);
-                if (ret.length >= MAX_ISSUES) {
+                if (ret.length >= max) {
                     break;
                 }
             }

package/build/src/rules/keyword_case.js

@@ -33,6 +33,7 @@ class KeywordCaseConf extends _basic_rule_config_1.BasicRuleConfig {
         this.ignoreGlobalClassBoundaries = false;
         /** A list of keywords to be ignored */
         this.ignoreKeywords = [];
+        this.maxIssuesPerFile = 10;
     }
 }
 exports.KeywordCaseConf = KeywordCaseConf;
@@ -143,6 +144,10 @@ class KeywordCase extends _abap_rule_1.ABAPRule {
                 return [];
             }
         }
+        let max = this.getConfig().maxIssuesPerFile;
+        if (max === undefined || max < 1) {
+            max = 10;
+        }
         const skip = new Skip(this.getConfig());
         let prev = undefined;
         for (const statement of file.getStatements()) {
@@ -165,6 +170,9 @@ class KeywordCase extends _abap_rule_1.ABAPRule {
                 }
                 prev = result[0].token;
             }
+            if (issues.length >= max) {
+                break;
+            }
         }
         return issues;
     }

package/build/src/rules/line_length.js

@@ -10,6 +10,7 @@ class LineLengthConf extends _basic_rule_config_1.BasicRuleConfig {
         super(...arguments);
         /** Maximum line length in characters, trailing whitespace ignored */
         this.length = 120;
+        this.maxIssuesPerFile = 10;
     }
 }
 exports.LineLengthConf = LineLengthConf;
@@ -38,6 +39,10 @@ https://docs.abapopenchecks.org/checks/04/`,
         const issues = [];
         // maximum line length in abap files
         const maxLineLength = 255;
+        let max = this.getConfig().maxIssuesPerFile;
+        if (max === undefined || max < 1) {
+            max = 10;
+        }
         const array = file.getRawRows();
         for (let rowIndex = 0; rowIndex < array.length; rowIndex++) {
             const row = array[rowIndex].replace("\r", "");
@@ -49,6 +54,9 @@ https://docs.abapopenchecks.org/checks/04/`,
                 const message = `Reduce line length to max ${this.conf.length}, currently ${row.length}`;
                 issues.push(issue_1.Issue.atRow(file, rowIndex + 1, message, this.getMetadata().key, this.conf.severity));
             }
+            if (issues.length >= max) {
+                break;
+            }
         }
         return issues;
     }

package/build/src/rules/no_prefixes.js

@@ -23,6 +23,7 @@ class NoPrefixesConf extends _basic_rule_config_1.BasicRuleConfig {
         /** importing, exporting, returning and changing parameters, case insensitive regex */
         this.methodParameters = "^[ICER].?_";
         this.allowIsPrefixBoolean = true;
+        this.maxIssuesPerFile = 10;
         // todo, public localClass: string = "";
         // todo, public localInterface: string = "";
         // todo, public functionModuleParameters: string = "";
@@ -69,21 +70,40 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/sub-sections/AvoidEncodi
             // syntax error, skip
             return [];
         }
+        let max = config.maxIssuesPerFile;
+        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 (ret.length >= max) {
+            return ret;
+        }
         if (config.statics !== undefined && config.statics !== "") {
             ret.push(...this.checkStatics(structure, new RegExp(config.statics, "i"), 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 (ret.length >= max) {
+            return ret;
+        }
         if (config.constants !== undefined && config.constants !== "") {
             ret.push(...this.checkConstants(structure, new RegExp(config.constants, "i"), 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 (ret.length >= max) {
+            return ret;
+        }
         if (config.methodParameters !== undefined && config.methodParameters !== "") {
             ret.push(...this.checkMethodParameters(structure, new RegExp(config.methodParameters, "i"), file));
         }

package/build/src/rules/unnecessary_chaining.js

@@ -8,6 +8,10 @@ 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 {
+    constructor() {
+        super(...arguments);
+        this.maxIssuesPerFile = 10;
+    }
 }
 exports.UnnecessaryChainingConf = UnnecessaryChainingConf;
 class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
@@ -34,6 +38,10 @@ class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
     }
     runParsed(file) {
         const issues = [];
+        let max = this.getConfig().maxIssuesPerFile;
+        if (max === undefined || max < 1) {
+            max = 10;
+        }
         const statements = file.getStatements();
         for (let i = 0; i < statements.length; i++) {
             const colon = statements[i].getColon();
@@ -64,6 +72,9 @@ class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
             const message = "Unnecessary chaining";
             const issue = issue_1.Issue.atToken(file, colon, message, this.getMetadata().key, this.conf.severity, fix);
             issues.push(issue);
+            if (issues.length >= max) {
+                break;
+            }
         }
         return issues;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.113.201",
+    "version": "2.113.203",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",