@abaplint/cli 2.114.9 → 2.115.0

package/build/cli.js

@@ -31175,6 +31175,11 @@ class Concatenate {
                     input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                     return;
                 }
+                if (linesMode === true && byteMode === true && type instanceof basic_1.TableType && type.getRowType() instanceof basic_1.StructureType) {
+                    const message = "Source row type must not be a structure in BYTE mode";
+                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                    return;
+                }
             }
         }
         for (const s of node.findDirectExpressions(Expressions.SimpleSource3)) {
@@ -53460,6 +53465,10 @@ class Table extends _abstract_object_1.AbstractObject {
             && this.parsedData.dataClass === "USER3") {
             return new Types.UnknownType("Data class = USER3 not allowed in cloud");
         }
+        if (this.getTableCategory() === TableCategory.Transparent
+            && this.listKeys(reg).length === 0) {
+            return new Types.UnknownType("Table " + this.getName() + " has no key fields");
+        }
         if (this.parsedType) {
             return this.parsedType;
         }
@@ -54477,6 +54486,7 @@ exports.WebDynproComponentConfiguration = WebDynproComponentConfiguration;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.WebMIME = void 0;
+const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@abaplint/core/build/src/xml_utils.js");
 const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
 class WebMIME extends _abstract_object_1.AbstractObject {
     getType() {
@@ -54494,6 +54504,16 @@ class WebMIME extends _abstract_object_1.AbstractObject {
         this.parse();
         return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.description;
     }
+    getParameter(name) {
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.params[name.toLowerCase()];
+    }
+    getParameters() {
+        var _a, _b;
+        this.parse();
+        return (_b = (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.params) !== null && _b !== void 0 ? _b : {};
+    }
     setDirty() {
         this.parsedXML = undefined;
         super.setDirty();
@@ -54507,11 +54527,12 @@ class WebMIME extends _abstract_object_1.AbstractObject {
         return undefined;
     }
     parse() {
+        var _a;
         if (this.parsedXML) {
             return { updated: false, runtime: 0 };
         }
         const start = Date.now();
-        this.parsedXML = {};
+        this.parsedXML = { params: {} };
         const parsed = super.parseRaw2();
         if (parsed === undefined
             || parsed.abapGit === undefined
@@ -54519,6 +54540,9 @@ class WebMIME extends _abstract_object_1.AbstractObject {
             return { updated: false, runtime: 0 };
         }
         this.parsedXML.description = parsed.abapGit["asx:abap"]["asx:values"].TEXT;
+        for (const param of (0, xml_utils_1.xmlToArray)((_a = parsed.abapGit["asx:abap"]["asx:values"].PARAMS) === null || _a === void 0 ? void 0 : _a.WWWPARAMS)) {
+            this.parsedXML.params[param.NAME.toLowerCase()] = param.VALUE;
+        }
         const end = Date.now();
         return { updated: true, runtime: end - start };
     }
@@ -55050,7 +55074,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.114.9";
+        return "2.115.0";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -65494,6 +65518,76 @@ exports.IdenticalFormNames = IdenticalFormNames;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/identical_move.js":
+/*!***********************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/identical_move.js ***!
+  \***********************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.IdenticalMove = exports.IdenticalMoveConf = void 0;
+const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const Expressions = __webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+class IdenticalMoveConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.IdenticalMoveConf = IdenticalMoveConf;
+class IdenticalMove extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new IdenticalMoveConf();
+    }
+    getMetadata() {
+        return {
+            key: "identical_move",
+            title: "Identical move",
+            shortDescription: `Moving the same value from left to right or right to left is redundant.`,
+            tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `DATA lv_value TYPE i.
+lv_value = lv_value.`,
+            goodExample: `DATA lv_value TYPE i.
+lv_value = 5.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file, obj) {
+        var _a, _b;
+        const issues = [];
+        if (!(obj instanceof objects_1.Class)) {
+            return [];
+        }
+        else if (file !== obj.getMainABAPFile()) {
+            return [];
+        }
+        for (const statement of file.getStatements()) {
+            const statementType = statement.get();
+            if (statementType instanceof Statements.Move) {
+                const source = (_a = statement.findDirectExpression(Expressions.Source)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
+                const target = (_b = statement.findDirectExpression(Expressions.Target)) === null || _b === void 0 ? void 0 : _b.concatTokens().toUpperCase();
+                if (source === target && source !== undefined) {
+                    issues.push(issue_1.Issue.atStatement(file, statement, "Comment between methods in global class implementation", this.getMetadata().key, this.conf.severity));
+                }
+            }
+        }
+        return issues;
+    }
+}
+exports.IdenticalMove = IdenticalMove;
+//# sourceMappingURL=identical_move.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/if_in_if.js":
 /*!*****************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/if_in_if.js ***!
@@ -66427,6 +66521,7 @@ __exportStar(__webpack_require__(/*! ./intf_referencing_clas */ "./node_modules/
 __exportStar(__webpack_require__(/*! ./invalid_table_index */ "./node_modules/@abaplint/core/build/src/rules/invalid_table_index.js"), exports);
 __exportStar(__webpack_require__(/*! ./keep_single_parameter_on_one_line */ "./node_modules/@abaplint/core/build/src/rules/keep_single_parameter_on_one_line.js"), exports);
 __exportStar(__webpack_require__(/*! ./keyword_case */ "./node_modules/@abaplint/core/build/src/rules/keyword_case.js"), exports);
+__exportStar(__webpack_require__(/*! ./identical_move */ "./node_modules/@abaplint/core/build/src/rules/identical_move.js"), exports);
 __exportStar(__webpack_require__(/*! ./line_break_multiple_parameters */ "./node_modules/@abaplint/core/build/src/rules/line_break_multiple_parameters.js"), exports);
 __exportStar(__webpack_require__(/*! ./line_break_style */ "./node_modules/@abaplint/core/build/src/rules/line_break_style.js"), exports);
 __exportStar(__webpack_require__(/*! ./line_length */ "./node_modules/@abaplint/core/build/src/rules/line_length.js"), exports);
@@ -66452,6 +66547,7 @@ __exportStar(__webpack_require__(/*! ./nesting */ "./node_modules/@abaplint/core
 __exportStar(__webpack_require__(/*! ./newline_between_methods */ "./node_modules/@abaplint/core/build/src/rules/newline_between_methods.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_aliases */ "./node_modules/@abaplint/core/build/src/rules/no_aliases.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_chained_assignment */ "./node_modules/@abaplint/core/build/src/rules/no_chained_assignment.js"), exports);
+__exportStar(__webpack_require__(/*! ./no_comments_between_methods */ "./node_modules/@abaplint/core/build/src/rules/no_comments_between_methods.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_external_form_calls */ "./node_modules/@abaplint/core/build/src/rules/no_external_form_calls.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_inline_in_optional_branches */ "./node_modules/@abaplint/core/build/src/rules/no_inline_in_optional_branches.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_prefixes */ "./node_modules/@abaplint/core/build/src/rules/no_prefixes.js"), exports);
@@ -69781,6 +69877,81 @@ exports.NoChainedAssignment = NoChainedAssignment;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/no_comments_between_methods.js":
+/*!************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/no_comments_between_methods.js ***!
+  \************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.NoCommentsBetweenMethods = exports.NoCommentsBetweenMethodsConf = void 0;
+const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
+class NoCommentsBetweenMethodsConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.NoCommentsBetweenMethodsConf = NoCommentsBetweenMethodsConf;
+class NoCommentsBetweenMethods extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new NoCommentsBetweenMethodsConf();
+    }
+    getMetadata() {
+        return {
+            key: "no_comments_between_methods",
+            title: "No comments between methods in global classes",
+            shortDescription: `Its not possible to have comments between methods in global classes.`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Syntax],
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file, obj) {
+        const issues = [];
+        if (!(obj instanceof objects_1.Class)) {
+            return [];
+        }
+        else if (file !== obj.getMainABAPFile()) {
+            return [];
+        }
+        let inMethod = false;
+        let inClassImpl = false;
+        for (const statement of file.getStatements()) {
+            const statementType = statement.get();
+            if (statementType instanceof Statements.ClassImplementation) {
+                inClassImpl = true;
+            }
+            else if (statementType instanceof Statements.EndClass) {
+                inClassImpl = false;
+            }
+            else if (statementType instanceof Statements.MethodImplementation) {
+                inMethod = true;
+            }
+            else if (statementType instanceof Statements.EndMethod) {
+                inMethod = false;
+            }
+            else if (inClassImpl === true && inMethod === false && statementType instanceof _statement_1.Comment) {
+                issues.push(issue_1.Issue.atStatement(file, statement, "Comment between methods in global class implementation", this.getMetadata().key, this.conf.severity));
+            }
+        }
+        return issues;
+    }
+}
+exports.NoCommentsBetweenMethods = NoCommentsBetweenMethods;
+//# sourceMappingURL=no_comments_between_methods.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/no_external_form_calls.js":
 /*!*******************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/no_external_form_calls.js ***!
@@ -75680,7 +75851,7 @@ class UncaughtException extends _abap_rule_1.ABAPRule {
         return this.issues;
     }
     traverse(n, file) {
-        var _a, _b, _c, _d, _e, _f;
+        var _a, _b, _c, _d, _e;
         const get = n.get();
         if (get instanceof Structures.ClassDefinition
             || get instanceof Structures.Interface) {
@@ -75695,13 +75866,14 @@ class UncaughtException extends _abap_rule_1.ABAPRule {
                     this.traverse(c, file);
                 }
                 this.sinked = previous;
-                for (const c of ((_b = n.findDirectStructure(Structures.Catch)) === null || _b === void 0 ? void 0 : _b.getChildren()) || []) {
-                    this.traverse(c, file);
+                for (const catchStructure of n.findDirectStructures(Structures.Catch)) {
+                    for (const c of catchStructure.getChildren()) {
+                        this.traverse(c, file);
+                    }
                 }
-                for (const c of ((_c = n.findDirectStructure(Structures.Cleanup)) === null || _c === void 0 ? void 0 : _c.getChildren()) || []) {
+                for (const c of ((_b = n.findDirectStructure(Structures.Cleanup)) === null || _b === void 0 ? void 0 : _b.getChildren()) || []) {
                     this.traverse(c, file);
                 }
-                return;
             }
             else {
                 for (const c of n.getChildren()) {
@@ -75730,10 +75902,10 @@ class UncaughtException extends _abap_rule_1.ABAPRule {
                 let name = undefined;
                 const concat = n.concatTokens().toUpperCase();
                 if (concat.startsWith("RAISE EXCEPTION TYPE ")) {
-                    name = (_d = n.findFirstExpression(Expressions.ClassName)) === null || _d === void 0 ? void 0 : _d.getFirstToken().getStr().toUpperCase();
+                    name = (_c = n.findFirstExpression(Expressions.ClassName)) === null || _c === void 0 ? void 0 : _c.getFirstToken().getStr().toUpperCase();
                 }
                 else if (concat.startsWith("RAISE EXCEPTION NEW ")) {
-                    name = (_f = (_e = n.findFirstExpression(Expressions.NewObject)) === null || _e === void 0 ? void 0 : _e.findFirstExpression(Expressions.TypeNameOrInfer)) === null || _f === void 0 ? void 0 : _f.getFirstToken().getStr().toUpperCase();
+                    name = (_e = (_d = n.findFirstExpression(Expressions.NewObject)) === null || _d === void 0 ? void 0 : _d.findFirstExpression(Expressions.TypeNameOrInfer)) === null || _e === void 0 ? void 0 : _e.getFirstToken().getStr().toUpperCase();
                     // todo: else its a normal Source, infer the type from it
                 }
                 this.check(name, n, file);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.114.9",
+    "version": "2.115.0",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.114.9",
+        "@abaplint/core": "^2.115.0",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",