@abaplint/cli 2.102.58 → 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.58";
+        return "2.102.59";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.58",
+    "version": "2.102.59",
     "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.58",
+        "@abaplint/core": "^2.102.59",
         "@types/chai": "^4.3.8",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.3",