@abaplint/cli 2.106.0 → 2.106.1

package/build/cli.js

@@ -28321,8 +28321,15 @@ class Append {
             else if (targetType instanceof basic_1.VoidType) {
                 rowType = targetType;
             }
-            const sourceType = new source_1.Source().runSyntax(source, scope, filename, rowType);
+            let sourceType = new source_1.Source().runSyntax(source, scope, filename, rowType);
             if (node.findDirectTokenByText("LINES")) {
+                // hmm, checking only the row types are compatible will not check the table type, e.g. sorted or hashed
+                if (sourceType instanceof basic_1.TableType) {
+                    sourceType = sourceType.getRowType();
+                }
+                if (targetType instanceof basic_1.TableType) {
+                    targetType = targetType.getRowType();
+                }
                 if (new _type_utils_1.TypeUtils(scope).isAssignable(sourceType, targetType) === false) {
                     throw new Error("Incompatible types");
                 }
@@ -31079,7 +31086,8 @@ class ModifyInternal {
             new source_1.Source().runSyntax(s, scope, filename);
         }
         // there is only one
-        const targetExpression = node.findFirstExpression(Expressions.Target);
+        const target = node.findFirstExpression(Expressions.Target);
+        const targetExpression = target;
         if (targetExpression) {
             // it might be a dynamic target
             const targetType = new target_1.Target().runSyntax(targetExpression, scope, filename);
@@ -31096,13 +31104,19 @@ class ModifyInternal {
                     throw new Error("Table does not have header line");
                 }
             }
+            else if (targetType instanceof basic_1.StructureType) {
+                // it might originate from a TABLES statement
+                if (target.concatTokens().toUpperCase() !== targetType.getDDICName()) {
+                    throw new Error("Not an internal table");
+                }
+            }
             else {
                 throw new Error("Not an internal table");
             }
         }
-        const target = node.findDirectExpression(Expressions.FSTarget);
-        if (target) {
-            new fstarget_1.FSTarget().runSyntax(target, scope, filename, undefined);
+        const fstarget = node.findDirectExpression(Expressions.FSTarget);
+        if (fstarget) {
+            new fstarget_1.FSTarget().runSyntax(fstarget, scope, filename, undefined);
         }
         for (const t of node.findDirectExpressions(Expressions.ComponentCond)) {
             new component_cond_1.ComponentCond().runSyntax(t, scope, filename);
@@ -51369,7 +51383,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.106.0";
+        return "2.106.1";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.106.0",
+    "version": "2.106.1",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.106.0",
+        "@abaplint/core": "^2.106.1",
         "@types/chai": "^4.3.12",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",