@abaplint/core 2.120.12 → 2.120.14

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

@@ -1020,7 +1020,7 @@ class BasicTypes {
             }
         }
         if (val === undefined) {
-            return 1;
+            return undefined;
         }
         const intExpr = val.findFirstExpression(Expressions.Integer);
         if (intExpr) {

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

@@ -84,9 +84,11 @@ class Target {
                 else if (context instanceof basic_1.TableType && context.isWithHeader() && context.getRowType() instanceof unknown_type_1.UnknownType) {
                     return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                 }
+                else if (context instanceof basic_1.TableType && context.isWithHeader() && context.getRowType() instanceof basic_1.VoidType) {
+                    return context.getRowType();
+                }
                 else if (!(context instanceof basic_1.StructureType)
                     && !(context instanceof basic_1.TableType && context.isWithHeader() && context.getRowType() instanceof basic_1.StructureType)
-                    && !(context instanceof basic_1.TableType && context.isWithHeader() && context.getRowType() instanceof basic_1.VoidType)
                     && !(context instanceof basic_1.VoidType)) {
                     const message = "Not a structure, target, " + (context === null || context === void 0 ? void 0 : context.constructor.name) + ", " + current.concatTokens();
                     input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));

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

@@ -54,7 +54,7 @@ class Type {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return new _typed_identifier_1.TypedIdentifier(found.getToken(), input.filename, basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey));
             }
-            if (input.scope.isGlobalOO() && found.getType() instanceof basic_1.PackedType) {
+            if (input.scope.isAnyOO() && found.getType() instanceof basic_1.PackedType) {
                 const concat = node.concatTokens().toUpperCase();
                 if ((concat.includes(" TYPE P ") || concat.includes(" TYPE P."))
                     && concat.includes(" DECIMALS ") === false) {

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

@@ -70,7 +70,7 @@ class Write {
         if (target) {
             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)`;
+                const message = `"${target.concatTokens()}" must be a character-like field (data type C, N, D, or T, got "${targetType === null || targetType === void 0 ? void 0 : targetType.constructor.name}")`;
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, target.getFirstToken(), message));
             }
         }

package/build/src/ddic.js

@@ -147,7 +147,7 @@ class DDIC {
                     return new Types.PackedType(length, 0, { qualifiedName: qualifiedName });
                 }
                 else {
-                    return new Types.PackedType(1, 0, { qualifiedName: qualifiedName });
+                    return new Types.PackedType(8, 0, { qualifiedName: qualifiedName });
                 }
             case "C":
                 if (length) {
@@ -375,14 +375,14 @@ class DDIC {
             case "DF16_DEC": // 1 <= len <= 31
             case "DF34_DEC": // 1 <= len <= 31
             case "CURR": // 1 <= len <= 31
-            case "QUAN": // 1 <= len <= 31
+            case "QUAN": { // 1 <= len <= 31
                 if (input.length === undefined) {
                     return new Types.UnknownType(input.text + " unknown length, " + input.infoText, input.infoText);
                 }
-                else if (input.decimals === undefined) {
-                    return new Types.PackedType(parseInt(input.length, 10), 0, extra);
-                }
-                return new Types.PackedType(parseInt(input.length, 10), parseInt(input.decimals, 10), extra);
+                const packedLength = Math.ceil((parseInt(input.length, 10) + 1) / 2);
+                const decimals = input.decimals === undefined ? 0 : parseInt(input.decimals, 10);
+                return new Types.PackedType(packedLength, decimals, extra);
+            }
             case "ACCP":
                 return new Types.CharacterType(6, extra); // YYYYMM
             case "LANG":

package/build/src/registry.js

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

package/package.json

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