@abaplint/cli 2.119.2 → 2.119.4

package/build/cli.js

@@ -23531,6 +23531,8 @@ var ScopeType;
 (function (ScopeType) {
     ScopeType["BuiltIn"] = "_builtin";
     ScopeType["Dummy"] = "_dummy";
+    // definitions inside this one are local
+    ScopeType["SelectionEvent"] = "selection_event";
     ScopeType["Global"] = "_global";
     ScopeType["Program"] = "_program";
     ScopeType["TypePool"] = "_type_pool";
@@ -32802,6 +32804,42 @@ exports.GetParameter = GetParameter;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_pf_status.js"
+/*!*****************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_pf_status.js ***!
+  \*****************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.GetPFStatus = void 0;
+const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const source_1 = __webpack_require__(/*! ../expressions/source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
+const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js");
+const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const inline_data_1 = __webpack_require__(/*! ../expressions/inline_data */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/inline_data.js");
+class GetPFStatus {
+    runSyntax(node, input) {
+        for (const s of node.findDirectExpressions(Expressions.Source)) {
+            source_1.Source.runSyntax(s, input);
+        }
+        for (const t of node.findDirectExpressions(Expressions.Target)) {
+            const inline = t === null || t === void 0 ? void 0 : t.findDirectExpression(Expressions.InlineData);
+            if (inline) {
+                inline_data_1.InlineData.runSyntax(inline, input, basic_1.VoidType.get("GET_PF_STATUS"));
+            }
+            else {
+                target_1.Target.runSyntax(t, input);
+            }
+        }
+    }
+}
+exports.GetPFStatus = GetPFStatus;
+//# sourceMappingURL=get_pf_status.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_reference.js"
 /*!*****************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_reference.js ***!
@@ -36619,6 +36657,7 @@ const read_report_1 = __webpack_require__(/*! ./statements/read_report */ "./nod
 const authority_check_1 = __webpack_require__(/*! ./statements/authority_check */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/authority_check.js");
 const insert_report_1 = __webpack_require__(/*! ./statements/insert_report */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/insert_report.js");
 const get_reference_1 = __webpack_require__(/*! ./statements/get_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_reference.js");
+const get_pf_status_1 = __webpack_require__(/*! ./statements/get_pf_status */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_pf_status.js");
 const insert_database_1 = __webpack_require__(/*! ./statements/insert_database */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/insert_database.js");
 const delete_database_1 = __webpack_require__(/*! ./statements/delete_database */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/delete_database.js");
 const import_dynpro_1 = __webpack_require__(/*! ./statements/import_dynpro */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/import_dynpro.js");
@@ -36687,6 +36726,7 @@ const commit_entities_1 = __webpack_require__(/*! ./statements/commit_entities *
 const _syntax_input_1 = __webpack_require__(/*! ./_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 const assert_error_1 = __webpack_require__(/*! ./assert_error */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/assert_error.js");
 const field_group_1 = __webpack_require__(/*! ./statements/field_group */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/field_group.js");
+const stuff_1 = __webpack_require__(/*! ../../stuff */ "./node_modules/@abaplint/core/build/src/stuff.js");
 // -----------------------------------
 const map = {};
 function addToMap(handler) {
@@ -36772,6 +36812,7 @@ if (Object.keys(map).length === 0) {
     addToMap(new if_1.If());
     addToMap(new log_point_1.LogPoint());
     addToMap(new while_1.While());
+    addToMap(new get_pf_status_1.GetPFStatus());
     addToMap(new with_1.With());
     addToMap(new with_loop_1.WithLoop());
     addToMap(new call_transformation_1.CallTransformation());
@@ -36940,6 +36981,9 @@ class SyntaxLogic {
     updateScopeStructure(node) {
         const filename = this.currentFile.getFilename();
         const stru = node.get();
+        if (this.scope.getType() === _scope_type_1.ScopeType.SelectionEvent && this.isSelectionEventBoundaryStructure(stru)) {
+            this.scope.pop(node.getFirstToken().getStart());
+        }
         const input = {
             scope: this.scope,
             filename,
@@ -36979,6 +37023,7 @@ class SyntaxLogic {
     updateScopeStatement(node) {
         const filename = this.currentFile.getFilename();
         const s = node.get();
+        this.updateSelectionEventScope(node);
         const input = {
             scope: this.scope,
             filename,
@@ -37024,6 +37069,27 @@ class SyntaxLogic {
             }
         }
     }
+    updateSelectionEventScope(node) {
+        const statement = node.get();
+        if (this.scope.getType() === _scope_type_1.ScopeType.SelectionEvent && this.isSelectionEventBoundary(statement)) {
+            this.scope.pop(node.getFirstToken().getStart());
+        }
+        if (this.opensSelectionEventScope(statement)) {
+            this.scope.push(_scope_type_1.ScopeType.SelectionEvent, statement.constructor.name, node.getFirstToken().getStart(), this.currentFile.getFilename());
+        }
+    }
+    opensSelectionEventScope(statement) {
+        return statement instanceof Statements.AtSelectionScreen;
+    }
+    isSelectionEventBoundary(statement) {
+        return stuff_1.SELECTION_EVENTS.some(event => statement instanceof event)
+            || statement instanceof Statements.Form
+            || statement instanceof Statements.FunctionModule;
+    }
+    isSelectionEventBoundaryStructure(structure) {
+        return structure instanceof Structures.ClassDefinition
+            || structure instanceof Structures.Interface;
+    }
 }
 exports.SyntaxLogic = SyntaxLogic;
 //# sourceMappingURL=syntax.js.map
@@ -40348,7 +40414,7 @@ class ClassDefinition extends _identifier_1.Identifier {
                 input.scope.addReference(token, undefined, _reference_1.ReferenceType.ObjectOrientedVoidReference, input.filename);
             }
             else {
-                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, token, name.toUpperCase() + " does not exist"));
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, token, "Friend class " + name.toUpperCase() + " does not exist"));
             }
             result.push(name);
         }
@@ -56149,7 +56215,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.119.2";
+        return "2.119.4";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -68685,6 +68751,7 @@ const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
 const Expressions = __webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 class KeepSingleParameterCallsOnOneLineConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -68704,7 +68771,7 @@ class KeepSingleParameterCallsOnOneLine extends _abap_rule_1.ABAPRule {
             title: "Keep single parameters on one line",
             shortDescription: `Keep single parameter calls on one line`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-single-parameter-calls-on-one-line`,
-            tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
             badExample: `call_method(\n  2 ).`,
             goodExample: `call_method( 2 ).`,
         };
@@ -68780,7 +68847,8 @@ class KeepSingleParameterCallsOnOneLine extends _abap_rule_1.ABAPRule {
                 }
             }
             const message = "Keep single parameter on one line";
-            return [issue_1.Issue.atToken(file, c.getFirstToken(), message, this.getMetadata().key, this.conf.severity)];
+            const fix = edit_helper_1.EditHelper.replaceRange(file, c.getFirstToken().getStart(), c.getLastToken().getEnd(), c.concatTokens());
+            return [issue_1.Issue.atToken(file, c.getFirstToken(), message, this.getMetadata().key, this.conf.severity, fix)];
         }
         return [];
     }
@@ -69871,6 +69939,7 @@ const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/co
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const Objects = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
 const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 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 position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
@@ -69888,6 +69957,7 @@ class MainFileContents {
             key: "main_file_contents",
             title: "Main file contents",
             shortDescription: `Checks related to report declarations.`,
+            tags: [_irule_1.RuleTag.Syntax],
             extendedInformation: `Does not run if the target version is Cloud
 
 * PROGs must begin with "REPORT <name>." or "PROGRAM <name>.

package/package.json

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