@abaplint/cli 2.102.57 → 2.102.59

package/build/cli.js

@@ -25734,6 +25734,7 @@ const sql_for_all_entries_1 = __webpack_require__(/*! ./sql_for_all_entries */ "
 const _scope_type_1 = __webpack_require__(/*! ../_scope_type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_scope_type.js");
 const sql_source_1 = __webpack_require__(/*! ./sql_source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js");
 const sql_compare_1 = __webpack_require__(/*! ./sql_compare */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_compare.js");
+const sql_order_by_1 = __webpack_require__(/*! ./sql_order_by */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_order_by.js");
 const isSimple = /^\w+$/;
 class Select {
     runSyntax(node, scope, filename, skipImplicitInto = false) {
@@ -25787,6 +25788,9 @@ class Select {
         for (const s of node.findAllExpressions(Expressions.SQLCompare)) {
             new sql_compare_1.SQLCompare().runSyntax(s, scope, filename, dbSources);
         }
+        for (const s of node.findDirectExpressions(Expressions.SQLOrderBy)) {
+            new sql_order_by_1.SQLOrderBy().runSyntax(s, scope, filename);
+        }
         if (scope.getType() === _scope_type_1.ScopeType.OpenSQL) {
             scope.pop(node.getLastToken().getEnd());
         }
@@ -26446,6 +26450,31 @@ exports.SQLFrom = SQLFrom;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_order_by.js":
+/*!*****************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_order_by.js ***!
+  \*****************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SQLOrderBy = void 0;
+const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const dynamic_1 = __webpack_require__(/*! ./dynamic */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/dynamic.js");
+class SQLOrderBy {
+    runSyntax(node, scope, filename) {
+        const dyn = node.findDirectExpression(Expressions.Dynamic);
+        if (dyn) {
+            new dynamic_1.Dynamic().runSyntax(dyn, scope, filename);
+        }
+    }
+}
+exports.SQLOrderBy = SQLOrderBy;
+//# sourceMappingURL=sql_order_by.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js":
 /*!***************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js ***!
@@ -37044,6 +37073,9 @@ class InterfaceDefinition extends _identifier_1.Identifier {
             }
         }
         this.methodDefinitions = new method_definitions_1.MethodDefinitions(this.node, this.filename, scope);
+        if (this.methodDefinitions.getByName("CONSTRUCTOR") !== undefined) {
+            throw new Error("Interfaces cannot have constructor methods");
+        }
         const events = this.node.findAllStatements(Statements.Events);
         for (const e of events) {
             this.events.push(new event_definition_1.EventDefinition(e, visibility_1.Visibility.Public, this.filename, scope));
@@ -46959,6 +46991,56 @@ exports.RenameGlobalInterface = RenameGlobalInterface;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/objects/rename/rename_message_class.js":
+/*!**************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/objects/rename/rename_message_class.js ***!
+  \**************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.RenameMessageClass = void 0;
+const vscode_languageserver_types_1 = __webpack_require__(/*! vscode-languageserver-types */ "./node_modules/vscode-languageserver-types/lib/umd/main.js");
+const __1 = __webpack_require__(/*! .. */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+const renamer_helper_1 = __webpack_require__(/*! ./renamer_helper */ "./node_modules/@abaplint/core/build/src/objects/rename/renamer_helper.js");
+class RenameMessageClass {
+    constructor(reg) {
+        this.reg = reg;
+    }
+    buildEdits(obj, oldName, newName) {
+        if (!(obj instanceof __1.MessageClass)) {
+            throw new Error("RenameMessageClass, not a message class");
+        }
+        let changes = [];
+        const helper = new renamer_helper_1.RenamerHelper(this.reg);
+        changes = changes.concat(helper.buildXMLFileEdits(obj, "ARBGB", oldName, newName));
+        changes = changes.concat(helper.renameFiles(obj, oldName, newName));
+        for (const message of obj.getMessages()) {
+            for (const ref of this.reg.getMSAGReferences().listByMessage(obj.getName(), message.getNumber())) {
+                const file = this.reg.getFileByName(ref.filename);
+                if (file === undefined) {
+                    continue;
+                }
+                const rows = file.getRawRows();
+                const i = ref.token.getRow() - 1;
+                const index = rows[i].indexOf(oldName.toLowerCase());
+                if (index >= 0) {
+                    const range = vscode_languageserver_types_1.Range.create(i, index, i, index + oldName.length);
+                    changes.push(vscode_languageserver_types_1.TextDocumentEdit.create({ uri: file.getFilename(), version: 1 }, [vscode_languageserver_types_1.TextEdit.replace(range, newName.toLowerCase())]));
+                }
+            }
+        }
+        return {
+            documentChanges: changes,
+        };
+    }
+}
+exports.RenameMessageClass = RenameMessageClass;
+//# sourceMappingURL=rename_message_class.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/rename/rename_table.js":
 /*!******************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/rename/rename_table.js ***!
@@ -47051,6 +47133,7 @@ const rename_global_class_1 = __webpack_require__(/*! ./rename_global_class */ "
 const rename_global_interface_1 = __webpack_require__(/*! ./rename_global_interface */ "./node_modules/@abaplint/core/build/src/objects/rename/rename_global_interface.js");
 const rename_table_1 = __webpack_require__(/*! ./rename_table */ "./node_modules/@abaplint/core/build/src/objects/rename/rename_table.js");
 const rename_table_type_1 = __webpack_require__(/*! ./rename_table_type */ "./node_modules/@abaplint/core/build/src/objects/rename/rename_table_type.js");
+const rename_message_class_1 = __webpack_require__(/*! ./rename_message_class */ "./node_modules/@abaplint/core/build/src/objects/rename/rename_message_class.js");
 class Renamer {
     constructor(reg) {
         this.reg = reg;
@@ -47098,6 +47181,8 @@ class Renamer {
                 return new rename_table_type_1.RenameTableType(this.reg);
             case "INTF":
                 return new rename_global_interface_1.RenameGlobalInterface(this.reg);
+            case "MSAG":
+                return new rename_message_class_1.RenameMessageClass(this.reg);
             default:
                 throw new Error("Renaming of " + type + " not yet supported");
         }
@@ -49306,7 +49391,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.57";
+        return "2.102.59";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -65766,6 +65851,10 @@ const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "./node_modu
 const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
 const basic_1 = __webpack_require__(/*! ../abap/types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 class SelectAddOrderByConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        this.skipForAllEntries = false;
+    }
 }
 exports.SelectAddOrderByConf = SelectAddOrderByConf;
 class SelectAddOrderBy {
@@ -65797,7 +65886,7 @@ If the target is a sorted/hashed table, no issue is reported`,
         this.conf = conf;
     }
     run(obj) {
-        var _a;
+        var _a, _b;
         const issues = [];
         if (!(obj instanceof _abap_object_1.ABAPObject) || obj.getType() === "INTF") {
             return [];
@@ -65811,13 +65900,16 @@ If the target is a sorted/hashed table, no issue is reported`,
             const selects = stru.findAllStatements(Statements.Select);
             selects.push(...stru.findAllStatements(Statements.SelectLoop));
             for (const s of selects) {
-                const c = s.concatTokens().toUpperCase();
-                if (c.startsWith("SELECT SINGLE ")) {
+                const concat = s.concatTokens().toUpperCase();
+                if (concat.startsWith("SELECT SINGLE ")) {
+                    continue;
+                }
+                else if (((_a = this.getConfig()) === null || _a === void 0 ? void 0 : _a.skipForAllEntries) === true && concat.includes(" FOR ALL ENTRIES ")) {
                     continue;
                 }
                 // skip COUNT(*)
                 const list = s.findAllExpressions(Expressions.SQLField);
-                if (list.length === 1 && ((_a = list[0].getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.SQLAggregation) {
+                if (list.length === 1 && ((_b = list[0].getFirstChild()) === null || _b === void 0 ? void 0 : _b.get()) instanceof Expressions.SQLAggregation) {
                     continue;
                 }
                 else if (s.findFirstExpression(Expressions.SQLOrderBy)) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.57",
+    "version": "2.102.59",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,12 +38,12 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.57",
-        "@types/chai": "^4.3.7",
+        "@abaplint/core": "^2.102.59",
+        "@types/chai": "^4.3.8",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.3",
         "@types/mocha": "^10.0.2",
-        "@types/node": "^20.8.4",
+        "@types/node": "^20.8.6",
         "@types/progress": "^2.0.5",
         "chai": "^4.3.10",
         "chalk": "^5.3.0",
@@ -55,7 +55,7 @@
         "mocha": "^10.2.0",
         "progress": "^2.0.3",
         "typescript": "^5.2.2",
-        "webpack": "^5.88.2",
+        "webpack": "^5.89.0",
         "webpack-cli": "^5.1.4",
         "xml-js": "^1.6.11"
     },