@abaplint/core 2.91.27 → 2.91.28

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

@@ -6,6 +6,7 @@ const basic_types_1 = require("../basic_types");
 const Expressions = require("../../2_statements/expressions");
 const basic_1 = require("../../types/basic");
 const _scope_type_1 = require("../_scope_type");
+const type_table_key_1 = require("./type_table_key");
 class TypeTable {
     runSyntax(node, scope, filename, qualifiedNamePrefix) {
         // todo, input is currently the statement, but should be the expression?
@@ -25,10 +26,16 @@ class TypeTable {
                 qualifiedName = scope.getName() + "=>" + qualifiedName;
             }
         }
-        const type = new basic_types_1.BasicTypes(filename, scope).parseTable(node, qualifiedName);
+        let type = new basic_types_1.BasicTypes(filename, scope).parseTable(node, qualifiedName);
         if (type === undefined) {
             return new _typed_identifier_1.TypedIdentifier(name, filename, new basic_1.UnknownType("TableType, fallback"));
         }
+        for (const tt of node.findAllExpressions(Expressions.TypeTableKey)) {
+            const error = new type_table_key_1.TypeTableKey().runSyntax(tt, type);
+            if (error) {
+                type = error;
+            }
+        }
         return new _typed_identifier_1.TypedIdentifier(name, filename, type);
     }
 }

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

@@ -0,0 +1,25 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.TypeTableKey = void 0;
+const Expressions = require("../../2_statements/expressions");
+const basic_1 = require("../../types/basic");
+class TypeTableKey {
+    runSyntax(node, type) {
+        if (type instanceof basic_1.TableType) {
+            const rowType = type.getRowType();
+            if (rowType instanceof basic_1.StructureType) {
+                for (const c of node.findAllExpressions(Expressions.FieldSub)) {
+                    const concat = c.concatTokens();
+                    if (concat.includes("-") === false // todo, properly check sub fields
+                        && rowType.getComponentByName(concat) === undefined
+                        && concat.toUpperCase() !== "TABLE_LINE") {
+                        return new basic_1.UnknownType(`Field ${concat} not part of structure`);
+                    }
+                }
+            }
+        }
+        return undefined;
+    }
+}
+exports.TypeTableKey = TypeTableKey;
+//# sourceMappingURL=type_table_key.js.map

package/build/src/registry.js

@@ -68,7 +68,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.91.27";
+        return "2.91.28";
     }
     getDDICReferences() {
         return this.references;

package/build/src/rules/downport.js

@@ -916,6 +916,7 @@ IF sy-subrc = 0.
   <${uniqueFS}>-${groupCountName} = <${uniqueFS}>-${groupCountName} + 1.
   INSERT ${loopTargetName}->* INTO TABLE <${uniqueFS}>-items.
 ELSE.\n`;
+        code += `  CLEAR ${uniqueName}.\n`;
         for (const c of group.findAllExpressions(Expressions.LoopGroupByComponent)) {
             code += `  ${uniqueName}-${c.concatTokens().replace("GROUP SIZE", "1")}.\n`;
         }

package/package.json

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