@abaplint/core 2.113.131 → 2.113.132

package/build/abaplint.d.ts

@@ -3660,6 +3660,10 @@ export declare interface INode {
     getLastToken(): Token;
 }
 
+declare class Input implements IStatement {
+    getMatcher(): IStatementRunnable;
+}
+
 declare class InsertDatabase implements IStatement {
     getMatcher(): IStatementRunnable;
 }
@@ -5015,6 +5019,7 @@ declare namespace Objects {
         OutboundService,
         PackageInterface,
         Package,
+        PageFormat,
         Parameter,
         PersonalizationObject,
         Program,
@@ -5130,6 +5135,15 @@ declare class PackedType extends AbstractType {
     toCDS(): string;
 }
 
+declare class PageFormat extends AbstractObject {
+    getType(): string;
+    getAllowedNaming(): {
+        maxLength: number;
+        allowNamespace: boolean;
+    };
+    getDescription(): string | undefined;
+}
+
 declare class Parameter extends AbstractObject {
     getType(): string;
     getAllowedNaming(): {
@@ -6345,6 +6359,7 @@ declare namespace Statements {
         ReadEntities,
         RollbackEntities,
         Summary,
+        Input,
         OnChange_2 as OnChange,
         AtUserCommand,
         Position_2 as Position,

package/build/src/abap/2_statements/statements/index.js

@@ -105,6 +105,7 @@ __exportStar(require("./modify_entities"), exports);
 __exportStar(require("./read_entities"), exports);
 __exportStar(require("./rollback_entities"), exports);
 __exportStar(require("./summary"), exports);
+__exportStar(require("./input"), exports);
 __exportStar(require("./on_change"), exports);
 __exportStar(require("./at_user_command"), exports);
 __exportStar(require("./position"), exports);

package/build/src/abap/2_statements/statements/input.js

@@ -0,0 +1,12 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.Input = void 0;
+const combi_1 = require("../combi");
+const version_1 = require("../../../version");
+class Input {
+    getMatcher() {
+        return (0, combi_1.verNot)(version_1.Version.Cloud, "INPUT");
+    }
+}
+exports.Input = Input;
+//# sourceMappingURL=input.js.map

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

@@ -513,6 +513,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);

package/build/src/abap/5_syntax/expressions/select.js

@@ -252,10 +252,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);
         }

package/build/src/abap/5_syntax/statements/read_table.js

@@ -62,7 +62,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;

package/build/src/objects/index.js

@@ -136,6 +136,7 @@ __exportStar(require("./object_characteristic"), exports);
 __exportStar(require("./outbound_service"), exports);
 __exportStar(require("./package_interface"), exports);
 __exportStar(require("./package"), exports);
+__exportStar(require("./page_format"), exports);
 __exportStar(require("./parameter"), exports);
 __exportStar(require("./personalization_object"), exports);
 __exportStar(require("./program"), exports);

package/build/src/objects/page_format.js

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.PageFormat = void 0;
+const _abstract_object_1 = require("./_abstract_object");
+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

package/build/src/registry.js

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

package/build/src/rules/many_parentheses.js

@@ -116,6 +116,7 @@ ENDIF.
         return issues;
     }
     analyze(file, cond) {
+        var _a, _b;
         const issues = [];
         let comparator = "";
         let found = false;
@@ -141,6 +142,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/core",
-    "version": "2.113.131",
+    "version": "2.113.132",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -53,10 +53,10 @@
         "@microsoft/api-extractor": "^7.52.8",
         "@types/chai": "^4.3.20",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.0.1",
+        "@types/node": "^24.0.4",
         "chai": "^4.5.0",
         "eslint": "^9.29.0",
-        "mocha": "^11.6.0",
+        "mocha": "^11.7.1",
         "c8": "^10.1.3",
         "source-map-support": "^0.5.21",
         "ts-json-schema-generator": "^2.4.0",