@abaplint/core 2.120.11 → 2.120.12

package/build/src/abap/5_syntax/_type_utils.js

@@ -79,6 +79,16 @@ class TypeUtils {
         }
         return false;
     }
+    isCharLikeField(type) {
+        return type instanceof basic_1.CharacterType
+            || type instanceof basic_1.NumericType
+            || type instanceof basic_1.DateType
+            || type instanceof basic_1.TimeType
+            || type instanceof basic_1.CLikeType
+            || type instanceof basic_1.AnyType
+            || type instanceof basic_1.UnknownType
+            || type instanceof basic_1.VoidType;
+    }
     isCharLike(type) {
         if (type === undefined) {
             return false;

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

@@ -106,7 +106,11 @@ class ValueBody {
                 field_assignment_1.FieldAssignment.runSyntax(s, input, rowType);
             }
             for (const s of foo.findDirectExpressions(Expressions.Source)) {
-                source_1.Source.runSyntax(s, input, rowType);
+                const sourceType = source_1.Source.runSyntax(s, input, rowType);
+                if (rowType instanceof basic_1.StringType && sourceType instanceof basic_1.CharacterType) {
+                    const message = "VALUE, source type CharacterType not compatible with StringType";
+                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, s.getFirstToken(), message));
+                }
             }
         }
         if (letScoped === true) {

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

@@ -68,7 +68,11 @@ class Write {
         }
         const target = node.findDirectExpression(Expressions.Target);
         if (target) {
-            target_1.Target.runSyntax(target, input);
+            const targetType = target_1.Target.runSyntax(target, input);
+            if (new _type_utils_1.TypeUtils(input.scope).isCharLikeField(targetType) === false) {
+                const message = `"${target.getFirstToken().getStr()}" must be a character-like field (data type C, N, D, or T)`;
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, target.getFirstToken(), message));
+            }
         }
     }
 }

package/build/src/registry.js

@@ -75,7 +75,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.11";
+        return "2.120.12";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/no_mandt_in_database_operations.js

@@ -104,7 +104,11 @@ class NoMandtInDatabaseOperations extends _abap_rule_1.ABAPRule {
     }
     findMandtInCondition(statement) {
         for (const condition of statement.findAllExpressions(Expressions.SQLCond)) {
-            for (const field of condition.findAllExpressions(Expressions.SQLFieldName)) {
+            const fields = condition.findAllExpressionsMulti([
+                Expressions.SQLFieldName,
+                Expressions.SQLAliasField,
+            ]);
+            for (const field of fields) {
                 const name = field.concatTokens().toUpperCase();
                 if (name === "MANDT" || name.endsWith("~MANDT")) {
                     return field;

package/package.json

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