@abaplint/cli 2.113.0 → 2.113.2

package/build/cli.js

@@ -35224,6 +35224,8 @@ exports.DECLARATION_STUFF = [
     Statements.Data,
     Statements.DataBegin,
     Statements.Constant,
+    Statements.Tables,
+    Statements.Include, // this is not super correct, but anyhow
     Statements.Parameter,
     Statements.SelectionScreen,
     Statements.ConstantBegin,
@@ -44890,6 +44892,43 @@ exports.AbstractObject = AbstractObject;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/objects/_dynpros.js":
+/*!*******************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/objects/_dynpros.js ***!
+  \*******************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.parseDynpros = parseDynpros;
+const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@abaplint/core/build/src/xml_utils.js");
+function parseDynpros(parsed) {
+    var _a, _b, _c, _d;
+    const dynpros = [];
+    const xmlDynpros = (_c = (_b = (_a = parsed.abapGit) === null || _a === void 0 ? void 0 : _a["asx:abap"]) === null || _b === void 0 ? void 0 : _b["asx:values"]) === null || _c === void 0 ? void 0 : _c.DYNPROS;
+    if (xmlDynpros !== undefined) {
+        for (const d of (0, xml_utils_1.xmlToArray)(xmlDynpros.item)) {
+            const fields = [];
+            for (const f of (0, xml_utils_1.xmlToArray)((_d = d.FIELDS) === null || _d === void 0 ? void 0 : _d.RPY_DYFATC)) {
+                fields.push({
+                    name: f.NAME,
+                    type: f.TYPE,
+                    length: f.LENGTH,
+                });
+            }
+            dynpros.push({
+                number: d.HEADER.SCREEN,
+                fields: fields,
+            });
+        }
+    }
+    return dynpros;
+}
+//# sourceMappingURL=_dynpros.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/_unknown_object.js":
 /*!**************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/_unknown_object.js ***!
@@ -47293,18 +47332,23 @@ const _abap_object_1 = __webpack_require__(/*! ./_abap_object */ "./node_modules
 const types_1 = __webpack_require__(/*! ../abap/types */ "./node_modules/@abaplint/core/build/src/abap/types/index.js");
 const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@abaplint/core/build/src/xml_utils.js");
 const fast_xml_parser_1 = __webpack_require__(/*! fast-xml-parser */ "./node_modules/fast-xml-parser/src/fxp.js");
+const _dynpros_1 = __webpack_require__(/*! ./_dynpros */ "./node_modules/@abaplint/core/build/src/objects/_dynpros.js");
 class FunctionGroup extends _abap_object_1.ABAPObject {
     constructor() {
         super(...arguments);
         this.includes = undefined;
         this.modules = undefined;
+        this.description = undefined;
+        this.dynpros = undefined;
     }
     getType() {
         return "FUGR";
     }
     getDescription() {
-        // todo
-        return undefined;
+        if (this.description === undefined) {
+            this.parseXML();
+        }
+        return this.description;
     }
     setDirty() {
         super.setDirty();
@@ -47317,6 +47361,12 @@ class FunctionGroup extends _abap_object_1.ABAPObject {
             allowNamespace: true,
         };
     }
+    getDynpros() {
+        if (this.dynpros === undefined) {
+            this.parseXML();
+        }
+        return this.dynpros || [];
+    }
     getSequencedFiles() {
         const main = this.getMainABAPFile();
         if (main === undefined) {
@@ -47418,22 +47468,24 @@ class FunctionGroup extends _abap_object_1.ABAPObject {
     }
     /////////////////////////////////
     parseXML() {
-        var _a, _b;
+        var _a, _b, _c;
         this.includes = [];
         this.modules = [];
         const parsed = this.parseRaw2();
         if (parsed === undefined) {
             return;
         }
+        this.description = (_a = parsed.abapGit["asx:abap"]["asx:values"]) === null || _a === void 0 ? void 0 : _a.AREAT;
+        this.dynpros = (0, _dynpros_1.parseDynpros)(parsed);
         // INCLUDES
-        const includes = (_a = parsed.abapGit["asx:abap"]["asx:values"]) === null || _a === void 0 ? void 0 : _a.INCLUDES;
+        const includes = (_b = parsed.abapGit["asx:abap"]["asx:values"]) === null || _b === void 0 ? void 0 : _b.INCLUDES;
         if (includes !== undefined) {
             for (const i of (0, xml_utils_1.xmlToArray)(includes.SOBJ_NAME)) {
                 this.includes.push(i);
             }
         }
         // FUNCTION MODULES
-        const functions = (_b = parsed.abapGit["asx:abap"]["asx:values"]) === null || _b === void 0 ? void 0 : _b.FUNCTIONS;
+        const functions = (_c = parsed.abapGit["asx:abap"]["asx:values"]) === null || _c === void 0 ? void 0 : _c.FUNCTIONS;
         for (const module of (0, xml_utils_1.xmlToArray)(functions === null || functions === void 0 ? void 0 : functions.item)) {
             this.modules.push(new types_1.FunctionModuleDefinition(module));
         }
@@ -49792,7 +49844,7 @@ exports.PersonalizationObject = PersonalizationObject;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Program = void 0;
 const _abap_object_1 = __webpack_require__(/*! ./_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
-const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@abaplint/core/build/src/xml_utils.js");
+const _dynpros_1 = __webpack_require__(/*! ./_dynpros */ "./node_modules/@abaplint/core/build/src/objects/_dynpros.js");
 class Program extends _abap_object_1.ABAPObject {
     getType() {
         return "PROG";
@@ -49828,11 +49880,10 @@ class Program extends _abap_object_1.ABAPObject {
     }
     getDynpros() {
         this.parseXML();
-        return this.parsedXML.dynpros;
+        return this.parsedXML.dynpros || [];
     }
     ////////////////////////////
     parseXML() {
-        var _a, _b, _c, _d;
         if (this.parsedXML !== undefined) {
             return;
         }
@@ -49846,24 +49897,7 @@ class Program extends _abap_object_1.ABAPObject {
             };
             return;
         }
-        const dynpros = [];
-        const xmlDynpros = (_c = (_b = (_a = parsed.abapGit) === null || _a === void 0 ? void 0 : _a["asx:abap"]) === null || _b === void 0 ? void 0 : _b["asx:values"]) === null || _c === void 0 ? void 0 : _c.DYNPROS;
-        if (xmlDynpros !== undefined) {
-            for (const d of (0, xml_utils_1.xmlToArray)(xmlDynpros.item)) {
-                const fields = [];
-                for (const f of (0, xml_utils_1.xmlToArray)((_d = d.FIELDS) === null || _d === void 0 ? void 0 : _d.RPY_DYFATC)) {
-                    fields.push({
-                        name: f.NAME,
-                        type: f.TYPE,
-                        length: f.LENGTH,
-                    });
-                }
-                dynpros.push({
-                    number: d.HEADER.SCREEN,
-                    fields: fields,
-                });
-            }
-        }
+        const dynpros = (0, _dynpros_1.parseDynpros)(parsed);
         this.parsedXML = {
             isInclude: file ? file.getRaw().includes("<SUBC>I</SUBC>") : false,
             isModulePool: file ? file.getRaw().includes("<SUBC>M</SUBC>") : false,
@@ -52640,7 +52674,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.0";
+        return "2.113.2";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -53308,25 +53342,25 @@ class AddTestAttributes extends _abap_rule_1.ABAPRule {
             title: "Add test attributes for tests classes with test methods",
             shortDescription: `Add test attributes DURATION and RISK LEVEL for tests classes with test methods`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `CLASS ltcl_test DEFINITION FINAL FOR TESTING.
+            badExample: `CLASS ltcl_test1 DEFINITION FINAL FOR TESTING.
   PUBLIC SECTION.
   PROTECTED SECTION.
   PRIVATE SECTION.
     METHODS test FOR TESTING RAISING cx_static_check.
 ENDCLASS.
 
-CLASS ltcl_test IMPLEMENTATION.
+CLASS ltcl_test1 IMPLEMENTATION.
   METHOD test.
   ENDMETHOD.
 ENDCLASS.`,
-            goodExample: `CLASS ltcl_test DEFINITION FINAL FOR TESTING DURATION SHORT RISK LEVEL HARMLESS.
+            goodExample: `CLASS ltcl_test2 DEFINITION FINAL FOR TESTING DURATION SHORT RISK LEVEL HARMLESS.
   PUBLIC SECTION.
   PROTECTED SECTION.
   PRIVATE SECTION.
     METHODS test FOR TESTING RAISING cx_static_check.
 ENDCLASS.
 
-CLASS ltcl_test IMPLEMENTATION.
+CLASS ltcl_test2 IMPLEMENTATION.
   METHOD test.
   ENDMETHOD.
 ENDCLASS.`,
@@ -60838,12 +60872,11 @@ class EmptyEvent extends _abap_rule_1.ABAPRule {
             shortDescription: `Empty selection screen or list processing event block`,
             extendedInformation: ``,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `REPORT zfoo.
-START-OF-SELECTION.
-  PERFORM sdf.
-  COMMIT WORK.
+            badExample: `
+INITIALIZATION.
+  WRITE 'hello'.
 END-OF-SELECTION.`,
-            goodExample: `REPORT zfoo.
+            goodExample: `
 START-OF-SELECTION.
   PERFORM sdf.
   COMMIT WORK.`,
@@ -60868,6 +60901,9 @@ START-OF-SELECTION.
         let children = [];
         for (const s of stru.getChildren() || []) {
             if (selection_events_1.SELECTION_EVENTS.some(f => s.get() instanceof f)) {
+                if (currentEvent !== undefined && children.length === 0) {
+                    issues.push(issue_1.Issue.atStatement(file, currentEvent, "Empty event", this.getMetadata().key, this.getConfig().severity));
+                }
                 children = [];
                 currentEvent = s;
             }
@@ -60880,7 +60916,11 @@ START-OF-SELECTION.
                 children.push(s);
             }
             else {
+                if (currentEvent !== undefined && children.length === 0) {
+                    issues.push(issue_1.Issue.atStatement(file, currentEvent, "Empty event", this.getMetadata().key, this.getConfig().severity));
+                }
                 children = [];
+                currentEvent = undefined;
             }
         }
         if (currentEvent !== undefined && children.length === 0) {
@@ -63297,7 +63337,7 @@ https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapstart-of-selectio
             tags: [_irule_1.RuleTag.SingleFile],
             badExample: `REPORT zfoo.
 WRITE 'hello'.`,
-            goodExample: `REPORT zfoo.
+            goodExample: `
 START-OF-SELECTION.
   WRITE 'hello'.`,
         };

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.0",
+    "version": "2.113.2",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.0",
+        "@abaplint/core": "^2.113.2",
         "@types/chai": "^4.3.19",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",