@abaplint/cli 2.116.1 → 2.117.0

package/build/cli.js

@@ -52638,6 +52638,10 @@ class Program extends _abap_object_1.ABAPObject {
         this.parseXML();
         return this.parsedXML.dynpros || [];
     }
+    getSelectionTexts() {
+        this.parseXML();
+        return this.parsedXML.selectionTexts;
+    }
     ////////////////////////////
     parseXML() {
         var _a, _b, _c;
@@ -52652,14 +52656,19 @@ class Program extends _abap_object_1.ABAPObject {
                 isModulePool: false,
                 description: undefined,
                 dynpros: [],
+                selectionTexts: {},
             };
             return;
         }
         let description = "";
+        const selectionTexts = {};
         for (const t of (0, xml_utils_1.xmlToArray)((_c = (_b = (_a = parsed.abapGit) === null || _a === void 0 ? void 0 : _a["asx:abap"]["asx:values"]) === null || _b === void 0 ? void 0 : _b.TPOOL) === null || _c === void 0 ? void 0 : _c.item)) {
             if ((t === null || t === void 0 ? void 0 : t.ID) === "R") {
                 description = t.ENTRY ? (0, xml_utils_1.unescape)(t.ENTRY) : "";
             }
+            else if ((t === null || t === void 0 ? void 0 : t.ID) === "S" && t.KEY !== undefined) {
+                selectionTexts[t.KEY.toUpperCase()] = t.ENTRY ? (0, xml_utils_1.unescape)(t.ENTRY) : "";
+            }
         }
         const dynpros = (0, _dynpros_1.parseDynpros)(parsed);
         this.parsedXML = {
@@ -52667,6 +52676,7 @@ class Program extends _abap_object_1.ABAPObject {
             isModulePool: file ? file.getRaw().includes("<SUBC>M</SUBC>") : false,
             dynpros: dynpros,
             description: description,
+            selectionTexts: selectionTexts,
         };
     }
 }
@@ -55795,7 +55805,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.116.1";
+        return "2.117.0";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -67450,6 +67460,7 @@ __exportStar(__webpack_require__(/*! ./select_add_order_by */ "./node_modules/@a
 __exportStar(__webpack_require__(/*! ./select_performance */ "./node_modules/@abaplint/core/build/src/rules/select_performance.js"), exports);
 __exportStar(__webpack_require__(/*! ./select_single_full_key */ "./node_modules/@abaplint/core/build/src/rules/select_single_full_key.js"), exports);
 __exportStar(__webpack_require__(/*! ./selection_screen_naming */ "./node_modules/@abaplint/core/build/src/rules/selection_screen_naming.js"), exports);
+__exportStar(__webpack_require__(/*! ./selection_screen_texts_missing */ "./node_modules/@abaplint/core/build/src/rules/selection_screen_texts_missing.js"), exports);
 __exportStar(__webpack_require__(/*! ./sequential_blank */ "./node_modules/@abaplint/core/build/src/rules/sequential_blank.js"), exports);
 __exportStar(__webpack_require__(/*! ./short_case */ "./node_modules/@abaplint/core/build/src/rules/short_case.js"), exports);
 __exportStar(__webpack_require__(/*! ./sicf_consistency */ "./node_modules/@abaplint/core/build/src/rules/sicf_consistency.js"), exports);
@@ -75029,6 +75040,95 @@ exports.SelectionScreenNaming = SelectionScreenNaming;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/selection_screen_texts_missing.js"
+/*!***************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/selection_screen_texts_missing.js ***!
+  \***************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SelectionScreenTextsMissing = exports.SelectionScreenTextsMissingConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const statements_1 = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const expressions_1 = __webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+class SelectionScreenTextsMissingConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.SelectionScreenTextsMissingConf = SelectionScreenTextsMissingConf;
+class SelectionScreenTextsMissing {
+    constructor() {
+        this.conf = new SelectionScreenTextsMissingConf();
+    }
+    getMetadata() {
+        return {
+            key: "selection_screen_texts_missing",
+            title: "Selection screen texts missing",
+            shortDescription: `Checks that selection screen parameters and select-options have selection texts`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    initialize(reg) {
+        this.reg = reg;
+        return this;
+    }
+    run(obj) {
+        if (!(obj instanceof objects_1.Program)) {
+            return [];
+        }
+        if (obj.isInclude()) {
+            return [];
+        }
+        const selTexts = obj.getSelectionTexts();
+        const output = [];
+        const checked = new Set();
+        this.checkFile(obj.getMainABAPFile(), selTexts, output, checked);
+        return output;
+    }
+    checkFile(file, selTexts, output, checked) {
+        if (file === undefined) {
+            return;
+        }
+        if (checked.has(file.getFilename())) {
+            return;
+        }
+        checked.add(file.getFilename());
+        for (const stat of file.getStatements()) {
+            const s = stat.get();
+            if (s instanceof statements_1.Parameter || s instanceof statements_1.SelectOption) {
+                const fieldNode = stat.findFirstExpression(expressions_1.FieldSub);
+                if (fieldNode) {
+                    const fieldName = fieldNode.getFirstToken().getStr().toUpperCase();
+                    if (selTexts[fieldName] === undefined) {
+                        output.push(issue_1.Issue.atToken(file, fieldNode.getFirstToken(), `Selection text missing for "${fieldName}"`, this.getMetadata().key, this.conf.severity));
+                    }
+                }
+            }
+            else if (s instanceof statements_1.Include) {
+                const nameNode = stat.findFirstExpression(expressions_1.IncludeName);
+                if (nameNode) {
+                    const inclName = nameNode.getFirstToken().getStr().toUpperCase();
+                    const inclObj = this.reg.getObject("PROG", inclName);
+                    if (inclObj) {
+                        this.checkFile(inclObj.getMainABAPFile(), selTexts, output, checked);
+                    }
+                }
+            }
+        }
+    }
+}
+exports.SelectionScreenTextsMissing = SelectionScreenTextsMissing;
+//# sourceMappingURL=selection_screen_texts_missing.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/sequential_blank.js"
 /*!*************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/sequential_blank.js ***!

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.116.1",
+    "version": "2.117.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.116.1",
+        "@abaplint/core": "^2.117.0",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",