@abaplint/core 2.101.4 → 2.101.6

package/build/src/abap/5_syntax/expressions/string_template.js

@@ -7,7 +7,6 @@ const source_1 = require("./source");
 const _type_utils_1 = require("../_type_utils");
 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);
@@ -18,9 +17,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" });
     }

package/build/src/abap/5_syntax/statements/raise.js

@@ -33,6 +33,9 @@ class Raise {
             else {
                 throw new Error("RAISE, unknown class " + className);
             }
+            if (method === undefined) {
+                method = new basic_1.VoidType(className);
+            }
         }
         const c = node.findExpressionAfterToken("EXCEPTION");
         if (c instanceof nodes_1.ExpressionNode && (c.get() instanceof Expressions.SimpleSource2 || c.get() instanceof Expressions.Source)) {
@@ -48,6 +51,9 @@ class Raise {
                 throw new Error("RAISE EXCEPTION, must be object reference, got " + type.constructor.name);
             }
         }
+        if (method === undefined) {
+            method = new basic_1.VoidType("Exception");
+        }
         // check parameters vs constructor
         const param = node.findDirectExpression(Expressions.ParameterListS);
         if (param) {

package/build/src/registry.js

@@ -65,7 +65,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.101.4";
+        return "2.101.6";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/modify_only_own_db_tables.js

@@ -71,14 +71,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));
                         }
                     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.101.4",
+    "version": "2.101.6",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",