@abaplint/cli 2.102.70 → 2.103.1

package/build/cli.js

@@ -37191,13 +37191,14 @@ class ClassDefinition extends _identifier_1.Identifier {
         while (sup !== undefined) {
             const cdef = scope.findClassDefinition(sup);
             for (const m of ((_a = cdef === null || cdef === void 0 ? void 0 : cdef.getMethodDefinitions()) === null || _a === void 0 ? void 0 : _a.getAll()) || []) {
+                const name = m.getName().toUpperCase();
                 if (m.getVisibility() === visibility_1.Visibility.Private) {
                     continue;
                 }
-                else if (m.getName().toUpperCase() === "CONSTRUCTOR") {
+                else if (name === "CONSTRUCTOR" || name === "CLASS_CONSTRUCTOR") {
                     continue;
                 }
-                names.add(m.getName().toUpperCase());
+                names.add(name);
             }
             for (const a of (cdef === null || cdef === void 0 ? void 0 : cdef.getAliases().getAll()) || []) {
                 names.add(a.getName().toUpperCase());
@@ -47327,6 +47328,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");
 class Program extends _abap_object_1.ABAPObject {
     getType() {
         return "PROG";
@@ -47360,14 +47362,49 @@ class Program extends _abap_object_1.ABAPObject {
         this.parseXML();
         return this.parsedXML.isModulePool;
     }
+    getDynpros() {
+        this.parseXML();
+        return this.parsedXML.dynpros;
+    }
+    ////////////////////////////
     parseXML() {
-        if (this.parsedXML === undefined) {
-            const file = this.getXMLFile();
+        var _a, _b, _c, _d;
+        if (this.parsedXML !== undefined) {
+            return;
+        }
+        const file = this.getXMLFile();
+        const parsed = this.parseRaw2();
+        if (parsed === undefined) {
             this.parsedXML = {
-                isInclude: file ? file.getRaw().includes("<SUBC>I</SUBC>") : false,
-                isModulePool: file ? file.getRaw().includes("<SUBC>M</SUBC>") : false,
+                isInclude: false,
+                isModulePool: false,
+                dynpros: [],
             };
+            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,
+                });
+            }
         }
+        this.parsedXML = {
+            isInclude: file ? file.getRaw().includes("<SUBC>I</SUBC>") : false,
+            isModulePool: file ? file.getRaw().includes("<SUBC>M</SUBC>") : false,
+            dynpros: dynpros,
+        };
     }
 }
 exports.Program = Program;
@@ -50698,7 +50735,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.70";
+        return "2.103.1";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -58207,6 +58244,71 @@ exports.Downport = Downport;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/dynpro_checks.js":
+/*!**********************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/dynpro_checks.js ***!
+  \**********************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.DynproChecks = exports.DynproChecksConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
+class DynproChecksConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.DynproChecksConf = DynproChecksConf;
+class DynproChecks {
+    constructor() {
+        this.conf = new DynproChecksConf();
+    }
+    getMetadata() {
+        return {
+            key: "dynpro_checks",
+            title: "Dynpro Checks",
+            shortDescription: `Various Dynpro checks`,
+            extendedInformation: `* Check length of PUSH elements less than 132`,
+            tags: [_irule_1.RuleTag.Syntax],
+        };
+    }
+    initialize(_reg) {
+        return this;
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    run(obj) {
+        const ret = [];
+        if (!(obj instanceof objects_1.Program)) {
+            return [];
+        }
+        const file = obj.getXMLFile();
+        if (file === undefined) {
+            return [];
+        }
+        for (const dynpro of obj.getDynpros()) {
+            for (const field of dynpro.fields) {
+                if (field.type === "PUSH" && field.length > 132) {
+                    const message = `Screen ${dynpro.number}, field ${field.name} LENGTH longer than 132`;
+                    ret.push(issue_1.Issue.atPosition(file, new position_1.Position(1, 1), message, this.getMetadata().key, this.getConfig().severity));
+                }
+            }
+        }
+        return ret;
+    }
+}
+exports.DynproChecks = DynproChecks;
+//# sourceMappingURL=dynpro_checks.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/easy_to_find_messages.js":
 /*!******************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/easy_to_find_messages.js ***!
@@ -60924,6 +61026,7 @@ __exportStar(__webpack_require__(/*! ./definitions_top */ "./node_modules/@abapl
 __exportStar(__webpack_require__(/*! ./description_empty */ "./node_modules/@abaplint/core/build/src/rules/description_empty.js"), exports);
 __exportStar(__webpack_require__(/*! ./double_space */ "./node_modules/@abaplint/core/build/src/rules/double_space.js"), exports);
 __exportStar(__webpack_require__(/*! ./downport */ "./node_modules/@abaplint/core/build/src/rules/downport.js"), exports);
+__exportStar(__webpack_require__(/*! ./dynpro_checks */ "./node_modules/@abaplint/core/build/src/rules/dynpro_checks.js"), exports);
 __exportStar(__webpack_require__(/*! ./easy_to_find_messages */ "./node_modules/@abaplint/core/build/src/rules/easy_to_find_messages.js"), exports);
 __exportStar(__webpack_require__(/*! ./empty_line_in_statement */ "./node_modules/@abaplint/core/build/src/rules/empty_line_in_statement.js"), exports);
 __exportStar(__webpack_require__(/*! ./empty_statement */ "./node_modules/@abaplint/core/build/src/rules/empty_statement.js"), exports);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.70",
+    "version": "2.103.1",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.70",
+        "@abaplint/core": "^2.103.1",
         "@types/chai": "^4.3.9",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.4",