@abaplint/transpiler-cli 2.7.8 → 2.7.10

package/build/bundle.js

@@ -17971,7 +17971,7 @@ class ABAPFileInformation {
         }
     }
     parseClasses(structure) {
-        var _a, _b, _c;
+        var _a;
         for (const found of structure.findAllStructures(Structures.ClassDefinition)) {
             const className = found.findFirstStatement(Statements.ClassDefinition).findFirstExpression(Expressions.ClassName).getFirstToken();
             const methods = this.parseMethodDefinition(found.findFirstStructure(Structures.PublicSection), visibility_1.Visibility.Public);
@@ -17990,8 +17990,26 @@ class ABAPFileInformation {
             const containsGlobal = found.findFirstExpression(Expressions.ClassGlobal);
             const cdef = found.findFirstStatement(Statements.ClassDefinition);
             const concat = (cdef === null || cdef === void 0 ? void 0 : cdef.concatTokens().toUpperCase()) || "";
-            const duration = (_b = cdef === null || cdef === void 0 ? void 0 : cdef.findExpressionAfterToken("DURATION")) === null || _b === void 0 ? void 0 : _b.concatTokens().toUpperCase();
-            const riskLevel = (_c = cdef === null || cdef === void 0 ? void 0 : cdef.findExpressionAfterToken("LEVEL")) === null || _c === void 0 ? void 0 : _c.concatTokens().toUpperCase();
+            let riskLevel;
+            if (concat.includes("RISK LEVEL CRITICAL")) {
+                riskLevel = _abap_file_information_1.RiskLevel.critical;
+            }
+            else if (concat.includes("RISK LEVEL DANGEROUS")) {
+                riskLevel = _abap_file_information_1.RiskLevel.dangerous;
+            }
+            else if (concat.includes("RISK LEVEL HARMLESS")) {
+                riskLevel = _abap_file_information_1.RiskLevel.harmless;
+            }
+            let duration;
+            if (concat.includes("DURATION SHORT")) {
+                duration = _abap_file_information_1.Duration.short;
+            }
+            else if (concat.includes("DURATION LONG")) {
+                duration = _abap_file_information_1.Duration.long;
+            }
+            else if (concat.includes("DURATION MEDIUM")) {
+                duration = _abap_file_information_1.Duration.medium;
+            }
             this.classes.push({
                 name: className.getStr(),
                 identifier: new _identifier_1.Identifier(className, this.filename),
@@ -24959,7 +24977,6 @@ const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/c
 const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_type_utils.js");
 class StringTemplate {
     runSyntax(node, scope, filename) {
-        var _a;
         const typeUtils = new _type_utils_1.TypeUtils(scope);
         for (const templateSource of node.findAllExpressions(Expressions.StringTemplateSource)) {
             const s = templateSource.findDirectExpression(Expressions.Source);
@@ -24970,9 +24987,22 @@ class StringTemplate {
             else if (typeUtils.isCharLike(type) === false && typeUtils.isHexLike(type) === false) {
                 throw new Error("Not character like, " + type.constructor.name);
             }
-            for (const formatSource of ((_a = templateSource.findDirectExpression(Expressions.StringTemplateFormatting)) === null || _a === void 0 ? void 0 : _a.findAllExpressions(Expressions.Source)) || []) {
+            const format = templateSource.findDirectExpression(Expressions.StringTemplateFormatting);
+            const formatConcat = format === null || format === void 0 ? void 0 : format.concatTokens();
+            for (const formatSource of (format === null || format === void 0 ? void 0 : format.findAllExpressions(Expressions.Source)) || []) {
                 new source_1.Source().runSyntax(formatSource, scope, filename);
             }
+            if ((formatConcat === null || formatConcat === void 0 ? void 0 : formatConcat.includes("ALPHA = "))
+                && !(type instanceof basic_1.UnknownType)
+                && !(type instanceof basic_1.VoidType)
+                && !(type instanceof basic_1.StringType)
+                && !(type instanceof basic_1.CLikeType)
+                && !(type instanceof basic_1.CharacterType)
+                && !(type instanceof basic_1.NumericGenericType)
+                && !(type instanceof basic_1.NumericType)
+                && !(type instanceof basic_1.AnyType)) {
+                throw new Error("Cannot apply ALPHA to this type");
+            }
         }
         return new basic_1.StringType({ qualifiedName: "STRING" });
     }
@@ -28845,45 +28875,56 @@ const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@a
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 const message_source_1 = __webpack_require__(/*! ../expressions/message_source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/message_source.js");
 const raise_with_1 = __webpack_require__(/*! ../expressions/raise_with */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/raise_with.js");
+const _object_oriented_1 = __webpack_require__(/*! ../_object_oriented */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_object_oriented.js");
+const method_parameters_1 = __webpack_require__(/*! ../expressions/method_parameters */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/method_parameters.js");
 class Raise {
     runSyntax(node, scope, filename) {
         // todo
-        var _a, _b, _c;
+        var _a, _b, _c, _d, _e;
+        const helper = new _object_oriented_1.ObjectOriented(scope);
+        let method;
         const classTok = (_a = node.findDirectExpression(Expressions.ClassName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
-        const classNam = classTok === null || classTok === void 0 ? void 0 : classTok.getStr();
-        if (classNam) {
-            const found = scope.existsObject(classNam);
+        const className = classTok === null || classTok === void 0 ? void 0 : classTok.getStr();
+        if (className) {
+            const found = scope.existsObject(className);
             if (found.found === true && found.id) {
                 scope.addReference(classTok, found.id, found.type, filename);
+                const def = scope.findObjectDefinition(className);
+                method = (_b = helper.searchMethodName(def, "CONSTRUCTOR")) === null || _b === void 0 ? void 0 : _b.method;
             }
-            else if (scope.getDDIC().inErrorNamespace(classNam) === false) {
-                const extra = { ooName: classNam, ooType: "Void" };
+            else if (scope.getDDIC().inErrorNamespace(className) === false) {
+                const extra = { ooName: className, ooType: "Void" };
                 scope.addReference(classTok, undefined, _reference_1.ReferenceType.ObjectOrientedVoidReference, filename, extra);
+                method = new basic_1.VoidType(className);
             }
             else {
-                throw new Error("RAISE, unknown class " + classNam);
+                throw new Error("RAISE, unknown class " + className);
+            }
+            if (method === undefined) {
+                method = new basic_1.VoidType(className);
             }
         }
-        let prev = "";
-        for (const c of node.getChildren()) {
-            if (c instanceof nodes_1.ExpressionNode
-                && (c.get() instanceof Expressions.SimpleSource2 || c.get() instanceof Expressions.Source)) {
-                const type = new source_1.Source().runSyntax(c, scope, filename);
-                if (prev === "EXCEPTION"
-                    && type
-                    && !(type instanceof basic_1.VoidType)
-                    && !(type instanceof basic_1.ObjectReferenceType)) {
-                    throw new Error("RAISE EXCEPTION, must be object reference, got " + type.constructor.name);
-                }
+        const c = node.findExpressionAfterToken("EXCEPTION");
+        if (c instanceof nodes_1.ExpressionNode && (c.get() instanceof Expressions.SimpleSource2 || c.get() instanceof Expressions.Source)) {
+            const type = new source_1.Source().runSyntax(c, scope, filename);
+            if (type instanceof basic_1.VoidType) {
+                method = type;
             }
-            prev = c.concatTokens().toUpperCase();
+            else if (type instanceof basic_1.ObjectReferenceType) {
+                const def = scope.findObjectDefinition(type.getIdentifierName());
+                method = (_c = helper.searchMethodName(def, "CONSTRUCTOR")) === null || _c === void 0 ? void 0 : _c.method;
+            }
+            else if (type !== undefined) {
+                throw new Error("RAISE EXCEPTION, must be object reference, got " + type.constructor.name);
+            }
+        }
+        if (method === undefined) {
+            method = new basic_1.VoidType("Exception");
         }
-        // todo, check parameters vs constructor
+        // check parameters vs constructor
         const param = node.findDirectExpression(Expressions.ParameterListS);
         if (param) {
-            for (const s of param.findAllExpressions(Expressions.Source)) {
-                new source_1.Source().runSyntax(s, scope, filename);
-            }
+            new method_parameters_1.MethodParameters().checkExporting(param, scope, method, filename, true);
         }
         for (const s of node.findDirectExpressions(Expressions.RaiseWith)) {
             new raise_with_1.RaiseWith().runSyntax(s, scope, filename);
@@ -28897,8 +28938,8 @@ class Raise {
         for (const s of node.findDirectExpressions(Expressions.MessageSource)) {
             new message_source_1.MessageSource().runSyntax(s, scope, filename);
         }
-        const id = (_b = node.findExpressionAfterToken("ID")) === null || _b === void 0 ? void 0 : _b.concatTokens();
-        const number = (_c = node.findDirectExpression(Expressions.MessageNumber)) === null || _c === void 0 ? void 0 : _c.concatTokens();
+        const id = (_d = node.findExpressionAfterToken("ID")) === null || _d === void 0 ? void 0 : _d.concatTokens();
+        const number = (_e = node.findDirectExpression(Expressions.MessageNumber)) === null || _e === void 0 ? void 0 : _e.concatTokens();
         if ((id === null || id === void 0 ? void 0 : id.startsWith("'")) && number) {
             const messageClass = id.substring(1, id.length - 1).toUpperCase();
             scope.getMSAGReferences().addUsing(filename, node.getFirstToken(), messageClass, number);
@@ -47045,7 +47086,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.101.2";
+        return "2.101.6";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -59478,14 +59519,18 @@ class ModifyOnlyOwnDBTables {
                     }
                     const concat = databaseTable.concatTokens().toUpperCase();
                     if (regExp.test(concat) === false) {
-                        // must contain a ReferenceType.TableVoidReference
+                        // must contain a ReferenceType.TableVoidReference or a ReferenceType.TableReference if its a dependency
                         if (spaghetti === undefined) {
                             spaghetti = new syntax_1.SyntaxLogic(this.reg, obj).run().spaghetti;
                         }
                         const start = databaseTable.getFirstToken().getStart();
                         const scope = spaghetti.lookupPosition(start, file.getFilename());
-                        const found = scope === null || scope === void 0 ? void 0 : scope.findTableVoidReference(start);
-                        if (found) {
+                        const found1 = scope === null || scope === void 0 ? void 0 : scope.findTableVoidReference(start);
+                        if (found1) {
+                            output.push(issue_1.Issue.atStatement(file, s, this.getMetadata().title, this.getMetadata().key, this.getConfig().severity));
+                        }
+                        const found2 = scope === null || scope === void 0 ? void 0 : scope.findTableReference(start);
+                        if (found2) {
                             output.push(issue_1.Issue.atStatement(file, s, this.getMetadata().title, this.getMetadata().key, this.getConfig().severity));
                         }
                     }
@@ -79751,10 +79796,9 @@ run().then(() => {
                 if (a.obj.getName() < b.obj.getName()) {
                     ret = -1;
                 }
-                if (a.obj.getName() > b.obj.getName()) {
+                else if (a.obj.getName() > b.obj.getName()) {
                     ret = 1;
                 }
-                ret = 0;
             }
             return ret;
         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.7.8",
+  "version": "2.7.10",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -26,14 +26,14 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.7.8",
+    "@abaplint/transpiler": "^2.7.10",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",
     "@types/node": "^20.2.3",
-    "@abaplint/core": "^2.101.2",
+    "@abaplint/core": "^2.101.6",
     "progress": "^2.0.3",
-    "webpack": "^5.83.1",
+    "webpack": "^5.84.0",
     "webpack-cli": "^5.1.1",
     "typescript": "^5.0.4"
   }