@abaplint/cli 2.120.5 → 2.120.6

package/build/cli.js

@@ -31419,6 +31419,10 @@ class InlineData {
     static runSyntax(node, input, type) {
         var _a;
         const token = (_a = node.findFirstExpression(Expressions.TargetField)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+        if (token && token.getStr().length > 30) {
+            const message = "DATA name too long, " + token.getStr();
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, token, message));
+        }
         if (token && type) {
             if (type instanceof basic_1.CSequenceType || type instanceof basic_1.CLikeType) {
                 type = basic_1.StringType.get();
@@ -31590,10 +31594,16 @@ const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expr
 const _typed_identifier_1 = __webpack_require__(/*! ../../types/_typed_identifier */ "../core/build/src/abap/types/_typed_identifier.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "../core/build/src/abap/types/basic/index.js");
 const _reference_1 = __webpack_require__(/*! ../_reference */ "../core/build/src/abap/5_syntax/_reference.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
 class InlineFS {
     static runSyntax(node, input, type) {
         var _a;
         const token = (_a = node.findFirstExpression(Expressions.TargetFieldSymbol)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+        const fs = node.findFirstExpression(Expressions.FieldSymbol);
+        if (fs && fs.concatTokens().length > 30) {
+            const message = "FIELD-SYMBOLS name too long, " + fs.concatTokens();
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, fs.getFirstToken(), message));
+        }
         if (token && type) {
             const identifier = new _typed_identifier_1.TypedIdentifier(token, input.filename, type, ["inline" /* IdentifierMeta.InlineDefinition */]);
             input.scope.addIdentifier(identifier);
@@ -36728,6 +36738,75 @@ exports.Assign = Assign;
 
 /***/ },
 
+/***/ "../core/build/src/abap/5_syntax/statements/at_selection_screen.js"
+/*!*************************************************************************!*\
+  !*** ../core/build/src/abap/5_syntax/statements/at_selection_screen.js ***!
+  \*************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"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.AtSelectionScreen = void 0;
+const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
+const _reference_1 = __webpack_require__(/*! ../_reference */ "../core/build/src/abap/5_syntax/_reference.js");
+const source_field_1 = __webpack_require__(/*! ../expressions/source_field */ "../core/build/src/abap/5_syntax/expressions/source_field.js");
+class AtSelectionScreen {
+    runSyntax(node, input) {
+        // "AT SELECTION-SCREEN ON <field>", "ON VALUE-REQUEST FOR <field>",
+        // "ON HELP-REQUEST FOR <field>" and "ON END OF <field>" all reference
+        // a parameter or select-option, register these as read references so
+        // they are not reported as unused.
+        const fields = [
+            ...node.findDirectExpressions(Expressions.FieldSub),
+            ...node.findDirectExpressions(Expressions.Field),
+        ];
+        for (const field of fields) {
+            const token = field.getFirstToken();
+            if (input.scope.findVariable(token.getStr()) !== undefined) {
+                source_field_1.SourceField.runSyntax(field, input, _reference_1.ReferenceType.DataReadReference, false);
+            }
+        }
+    }
+}
+exports.AtSelectionScreen = AtSelectionScreen;
+//# sourceMappingURL=at_selection_screen.js.map
+
+/***/ },
+
 /***/ "../core/build/src/abap/5_syntax/statements/authority_check.js"
 /*!*********************************************************************!*\
   !*** ../core/build/src/abap/5_syntax/statements/authority_check.js ***!
@@ -39915,14 +39994,19 @@ const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expr
 const _typed_identifier_1 = __webpack_require__(/*! ../../types/_typed_identifier */ "../core/build/src/abap/types/_typed_identifier.js");
 const basic_types_1 = __webpack_require__(/*! ../basic_types */ "../core/build/src/abap/5_syntax/basic_types.js");
 const unknown_type_1 = __webpack_require__(/*! ../../types/basic/unknown_type */ "../core/build/src/abap/types/basic/unknown_type.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "../core/build/src/abap/types/basic/index.js");
 class FieldSymbol {
     runSyntax(node, input) {
-        var _a;
-        const fsname = (_a = node.findFirstExpression(Expressions.FieldSymbol)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
-        if (fsname === undefined) {
+        const fs = node.findFirstExpression(Expressions.FieldSymbol);
+        const fsname = fs === null || fs === void 0 ? void 0 : fs.getFirstToken();
+        if (fs === undefined || fsname === undefined) {
             return;
         }
+        if (fs.concatTokens().length > 30) {
+            const message = "FIELD-SYMBOLS name too long, " + fs.concatTokens();
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, fsname, message));
+        }
         if (node.getChildren().length === 5) {
             // no type specified
             input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(fsname, input.filename, basic_1.VoidType.get("FS-SIMPLE")));
@@ -47325,6 +47409,7 @@ const if_1 = __webpack_require__(/*! ./statements/if */ "../core/build/src/abap/
 const else_if_1 = __webpack_require__(/*! ./statements/else_if */ "../core/build/src/abap/5_syntax/statements/else_if.js");
 const append_1 = __webpack_require__(/*! ./statements/append */ "../core/build/src/abap/5_syntax/statements/append.js");
 const selection_screen_1 = __webpack_require__(/*! ./statements/selection_screen */ "../core/build/src/abap/5_syntax/statements/selection_screen.js");
+const at_selection_screen_1 = __webpack_require__(/*! ./statements/at_selection_screen */ "../core/build/src/abap/5_syntax/statements/at_selection_screen.js");
 const ranges_1 = __webpack_require__(/*! ./statements/ranges */ "../core/build/src/abap/5_syntax/statements/ranges.js");
 const write_1 = __webpack_require__(/*! ./statements/write */ "../core/build/src/abap/5_syntax/statements/write.js");
 const case_1 = __webpack_require__(/*! ./statements/case */ "../core/build/src/abap/5_syntax/statements/case.js");
@@ -47532,6 +47617,7 @@ if (Object.keys(map).length === 0) {
     addToMap(new authority_check_1.AuthorityCheck());
     addToMap(new insert_report_1.InsertReport());
     addToMap(new selection_screen_1.SelectionScreen());
+    addToMap(new at_selection_screen_1.AtSelectionScreen());
     addToMap(new ranges_1.Ranges());
     addToMap(new add_1.Add());
     addToMap(new raise_event_1.RaiseEvent());
@@ -51231,6 +51317,7 @@ class ClassDefinition extends _identifier_1.Identifier {
             this.createVisibilityValue = visibility_1.Visibility.Public;
         }
         // perform checks after everything has been initialized
+        this.checkInterfaceVisibility(input, node);
         this.checkMethodsFromSuperClasses(input);
         this.checkMethodNameLength(input);
         this.checkClassConstructorStatic(input);
@@ -51304,6 +51391,17 @@ class ClassDefinition extends _identifier_1.Identifier {
             }
         }
     }
+    checkInterfaceVisibility(input, node) {
+        const sections = [
+            node.findDirectStructure(Structures.ProtectedSection),
+            node.findDirectStructure(Structures.PrivateSection),
+        ];
+        for (const section of sections) {
+            for (const statement of (section === null || section === void 0 ? void 0 : section.findAllStatements(Statements.InterfaceDef)) || []) {
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, statement.getFirstToken(), "INTERFACES can only be declared in public sections"));
+            }
+        }
+    }
     checkMethodsFromSuperClasses(input) {
         var _a;
         const scope = input.scope;
@@ -69066,7 +69164,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.5";
+        return "2.120.6";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

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