@abaplint/cli 2.113.130 → 2.113.132

package/build/cli.js

@@ -14204,6 +14204,7 @@ __exportStar(__webpack_require__(/*! ./modify_entities */ "./node_modules/@abapl
 __exportStar(__webpack_require__(/*! ./read_entities */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/read_entities.js"), exports);
 __exportStar(__webpack_require__(/*! ./rollback_entities */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/rollback_entities.js"), exports);
 __exportStar(__webpack_require__(/*! ./summary */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/summary.js"), exports);
+__exportStar(__webpack_require__(/*! ./input */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/input.js"), exports);
 __exportStar(__webpack_require__(/*! ./on_change */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/on_change.js"), exports);
 __exportStar(__webpack_require__(/*! ./at_user_command */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/at_user_command.js"), exports);
 __exportStar(__webpack_require__(/*! ./position */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/position.js"), exports);
@@ -14485,6 +14486,28 @@ exports.Initialization = Initialization;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/input.js":
+/*!*************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/2_statements/statements/input.js ***!
+  \*************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.Input = void 0;
+const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
+class Input {
+    getMatcher() {
+        return (0, combi_1.verNot)(version_1.Version.Cloud, "INPUT");
+    }
+}
+exports.Input = Input;
+//# sourceMappingURL=input.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/insert_database.js":
 /*!***********************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/2_statements/statements/insert_database.js ***!
@@ -24290,6 +24313,9 @@ class BasicTypes {
             if (found && this.isOccurs(node)) {
                 found = new Types.TableType(found, { withHeader: text.includes("WITH HEADER LINE"), keyType: Types.TableKeyType.default }, qualifiedName);
             }
+            else if (text.endsWith(" WITH HEADER LINE") && found instanceof Types.TableType) {
+                found = new Types.TableType(found.getRowType(), { withHeader: true, keyType: Types.TableKeyType.default }, qualifiedName);
+            }
         }
         else if (text.startsWith("TYPE LINE OF ")) {
             const sub = node.findFirstExpression(Expressions.TypeName);
@@ -27853,10 +27879,12 @@ class Select {
         var _a, _b;
         let expr = undefined;
         const ret = [];
-        expr = node.findFirstExpression(Expressions.SQLFieldList);
-        if (expr === undefined) {
+        if (node.get() instanceof Expressions.SelectLoop) {
             expr = node.findFirstExpression(Expressions.SQLFieldListLoop);
         }
+        else {
+            expr = node.findFirstExpression(Expressions.SQLFieldList);
+        }
         if (((_a = expr === null || expr === void 0 ? void 0 : expr.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.Dynamic) {
             dynamic_1.Dynamic.runSyntax(expr.getFirstChild(), input);
         }
@@ -33349,7 +33377,9 @@ class ReadTable {
             }
             const type = source_1.Source.runSyntax(s, input);
             if (s === afterKey) {
-                if (type instanceof basic_1.StringType || type instanceof basic_1.TableType || type instanceof basic_1.ObjectReferenceType) {
+                if (type instanceof basic_1.StringType
+                    || (type instanceof basic_1.TableType && type.isWithHeader() === false)
+                    || type instanceof basic_1.ObjectReferenceType) {
                     const message = "Key cannot be string or table or reference";
                     input.issues.push((0, _syntax_input_1.syntaxIssue)(input, s.getFirstToken(), message));
                     return;
@@ -35020,7 +35050,7 @@ class Data {
             else if (c instanceof nodes_1.StatementNode && ctyp instanceof Statements.Type) {
                 const found = new type_1.Type().runSyntax(c, input);
                 if (found) {
-                    input.scope.addIdentifier(found);
+                    input.scope.addType(found);
                 }
             }
             else if (c instanceof nodes_1.StructureNode && ctyp instanceof Structures.Types) {
@@ -49828,6 +49858,7 @@ __exportStar(__webpack_require__(/*! ./object_characteristic */ "./node_modules/
 __exportStar(__webpack_require__(/*! ./outbound_service */ "./node_modules/@abaplint/core/build/src/objects/outbound_service.js"), exports);
 __exportStar(__webpack_require__(/*! ./package_interface */ "./node_modules/@abaplint/core/build/src/objects/package_interface.js"), exports);
 __exportStar(__webpack_require__(/*! ./package */ "./node_modules/@abaplint/core/build/src/objects/package.js"), exports);
+__exportStar(__webpack_require__(/*! ./page_format */ "./node_modules/@abaplint/core/build/src/objects/page_format.js"), exports);
 __exportStar(__webpack_require__(/*! ./parameter */ "./node_modules/@abaplint/core/build/src/objects/parameter.js"), exports);
 __exportStar(__webpack_require__(/*! ./personalization_object */ "./node_modules/@abaplint/core/build/src/objects/personalization_object.js"), exports);
 __exportStar(__webpack_require__(/*! ./program */ "./node_modules/@abaplint/core/build/src/objects/program.js"), exports);
@@ -51360,6 +51391,37 @@ exports.PackageInterface = PackageInterface;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/objects/page_format.js":
+/*!**********************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/objects/page_format.js ***!
+  \**********************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.PageFormat = void 0;
+const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
+class PageFormat extends _abstract_object_1.AbstractObject {
+    getType() {
+        return "SPPF";
+    }
+    getAllowedNaming() {
+        return {
+            maxLength: 30,
+            allowNamespace: true,
+        };
+    }
+    getDescription() {
+        // todo
+        return undefined;
+    }
+}
+exports.PageFormat = PageFormat;
+//# sourceMappingURL=page_format.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/parameter.js":
 /*!********************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/parameter.js ***!
@@ -54354,7 +54416,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.130";
+        return "2.113.132";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -67326,6 +67388,7 @@ ENDIF.
         return issues;
     }
     analyze(file, cond) {
+        var _a, _b;
         const issues = [];
         let comparator = "";
         let found = false;
@@ -67351,6 +67414,17 @@ ENDIF.
                     found = true; // dont report for the simple case that contains quick fixes
                 }
             }
+            else if (c instanceof nodes_1.ExpressionNode
+                && c.get() instanceof Expressions.Compare
+                && ((_a = c.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.Source
+                && c.getChildren().length === 3) {
+                const concat = (_b = c.getFirstChild()) === null || _b === void 0 ? void 0 : _b.concatTokens();
+                if ((concat === null || concat === void 0 ? void 0 : concat.startsWith("(")) && concat.endsWith(")")) {
+                    const message = "Parentheses can be removed";
+                    const issue = issue_1.Issue.atToken(file, c.getFirstToken(), message, this.getMetadata().key, this.conf.severity);
+                    issues.push(issue);
+                }
+            }
             if (comparator === "") {
                 comparator = current;
             }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.130",
+    "version": "2.113.132",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,11 +38,11 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.130",
+        "@abaplint/core": "^2.113.132",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.0.3",
+        "@types/node": "^24.0.4",
         "@types/progress": "^2.0.7",
         "chai": "^4.5.0",
         "p-limit": "^3.1.0",
@@ -52,7 +52,7 @@
         "json5": "^2.2.3",
         "memfs": "^4.17.2",
         "minimist": "^1.2.8",
-        "mocha": "^11.6.0",
+        "mocha": "^11.7.1",
         "progress": "^2.0.3",
         "typescript": "^5.8.3",
         "webpack": "^5.99.9",