@abaplint/core 2.113.215 → 2.113.217

package/build/src/abap/2_statements/expressions/association_name.js

@@ -4,7 +4,7 @@ exports.AssociationName = void 0;
 const combi_1 = require("../combi");
 class AssociationName extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.regex)(/^\\_[\w]+$/);
+        return (0, combi_1.regex)(/^(\\_[\w]+)+$/);
     }
 }
 exports.AssociationName = AssociationName;

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

@@ -50,8 +50,10 @@ class FieldChain {
             }
             else if (current.get() instanceof tokens_1.Dash) {
                 if (context instanceof basic_1.UnknownType) {
+                    /*
                     const message = "Not a structure, type unknown, FieldChain";
-                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, current.getFirstToken(), message));
+                    input.issues.push(syntaxIssue(input, current.getFirstToken(), message));
+                    */
                     return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                 }
                 else if (!(context instanceof basic_1.StructureType)

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

@@ -12,9 +12,10 @@ const component_cond_1 = require("../expressions/component_cond");
 const dynamic_1 = require("../expressions/dynamic");
 const loop_group_by_1 = require("../expressions/loop_group_by");
 const _syntax_input_1 = require("../_syntax_input");
+const version_1 = require("../../../version");
 class Loop {
     runSyntax(node, input) {
-        var _a;
+        var _a, _b;
         const loopTarget = node.findDirectExpression(Expressions.LoopTarget);
         let target = loopTarget === null || loopTarget === void 0 ? void 0 : loopTarget.findDirectExpression(Expressions.Target);
         const targetType = target ? target_1.Target.runSyntax(target, input) : undefined;
@@ -64,6 +65,7 @@ class Loop {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
         }
         const targetConcat = loopTarget === null || loopTarget === void 0 ? void 0 : loopTarget.concatTokens().toUpperCase();
+        const topType = sourceType; // todo: refactor topType vs sourceType vs rowType
         if (sourceType instanceof basic_1.TableType) {
             rowType = sourceType.getRowType();
             sourceType = rowType;
@@ -71,6 +73,10 @@ class Loop {
                 sourceType = new basic_1.DataReference(sourceType);
             }
         }
+        const cond = node.findDirectExpression(Expressions.ComponentCond);
+        if (cond !== undefined) {
+            component_cond_1.ComponentCond.runSyntax(cond, input, rowType);
+        }
         if (targetConcat
             && targetConcat.startsWith("TRANSPORTING ")
             && node.findDirectTokenByText("WHERE") === undefined) {
@@ -78,6 +84,32 @@ class Loop {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return;
         }
+        if (node.findDirectTokenByText("USING") !== undefined
+            && cond !== undefined
+            && topType instanceof basic_1.TableType) {
+            // https://github.com/abap2xlsx/abap2xlsx/issues/1341
+            const keyName = node.findExpressionAfterToken("KEY");
+            let key = undefined;
+            if ((keyName === null || keyName === void 0 ? void 0 : keyName.get()) instanceof Expressions.SimpleName) {
+                // it might be dynamic, in that case we cannot check anything
+                key = (_b = topType.getOptions().secondary) === null || _b === void 0 ? void 0 : _b.find(k => k.name.toUpperCase() === keyName.getFirstToken().getStr().toUpperCase());
+                if (key === undefined) {
+                    const message = "Key " + (keyName === null || keyName === void 0 ? void 0 : keyName.concatTokens()) + " not found in table type";
+                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                    return;
+                }
+                if (input.scope.getRegistry().getConfig().getVersion() <= version_1.Version.v740sp02) {
+                    const compares = cond.findAllExpressionsRecursive(Expressions.ComponentCompare).map(c => c.concatTokens().toUpperCase());
+                    for (const keyField of key.keyFields) {
+                        if (compares.find(c => c === keyField.toUpperCase() + " IS INITIAL") !== undefined) {
+                            const message = "Loop, key check with IS INITIAL cannot optimized before 7.40 SP02";
+                            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                            return;
+                        }
+                    }
+                }
+            }
+        }
         const inline = target === null || target === void 0 ? void 0 : target.findDirectExpression(Expressions.InlineData);
         if (inline) {
             inline_data_1.InlineData.runSyntax(inline, input, sourceType);
@@ -98,9 +130,6 @@ class Loop {
                 fstarget_1.FSTarget.runSyntax(fstarget, input, sourceType);
             }
         }
-        for (const t of node.findDirectExpressions(Expressions.ComponentCond)) {
-            component_cond_1.ComponentCond.runSyntax(t, input, rowType);
-        }
         for (const t of node.findDirectExpressions(Expressions.Dynamic)) {
             dynamic_1.Dynamic.runSyntax(t, input);
         }

package/build/src/cds/expressions/cds_aggregate.js

@@ -7,7 +7,7 @@ class CDSAggregate extends combi_1.Expression {
     getRunnable() {
         const name = (0, combi_1.seq)(_1.CDSName, (0, combi_1.star)((0, combi_1.seq)(".", _1.CDSName)));
         const value = (0, combi_1.alt)(name, "*", _1.CDSCast, _1.CDSCase, _1.CDSFunction);
-        return (0, combi_1.seq)((0, combi_1.alt)("MAX", "MIN", "SUM", "AVG", "COUNT"), "(", (0, combi_1.opt)("DISTINCT"), value, ")");
+        return (0, combi_1.seq)((0, combi_1.altPrio)("MAX", "MIN", "SUM", "AVG", "COUNT"), "(", (0, combi_1.opt)("DISTINCT"), value, ")");
     }
 }
 exports.CDSAggregate = CDSAggregate;

package/build/src/cds/expressions/cds_annotation_array.js

@@ -7,7 +7,7 @@ const cds_annotation_simple_1 = require("./cds_annotation_simple");
 class CDSAnnotationArray extends combi_1.Expression {
     getRunnable() {
         const value = (0, combi_1.alt)(cds_annotation_simple_1.CDSAnnotationSimple, _1.CDSAnnotationObject, CDSAnnotationArray);
-        const valueList = (0, combi_1.seq)("[", value, (0, combi_1.star)((0, combi_1.seq)(",", value)), "]");
+        const valueList = (0, combi_1.seq)("[", value, (0, combi_1.starPrio)((0, combi_1.seq)(",", value)), "]");
         return valueList;
     }
 }

package/build/src/cds/expressions/cds_case.js

@@ -6,7 +6,7 @@ const combi_1 = require("../../abap/2_statements/combi");
 class CDSCase extends combi_1.Expression {
     getRunnable() {
         const name = (0, combi_1.seq)(_1.CDSName, (0, combi_1.starPrio)((0, combi_1.seq)(".", _1.CDSName)));
-        const value = (0, combi_1.altPrio)(_1.CDSFunction, _1.CDSString, CDSCase, _1.CDSCast, _1.CDSArithmetics, name);
+        const value = (0, combi_1.altPrio)(_1.CDSString, CDSCase, _1.CDSCast, _1.CDSArithmetics, _1.CDSFunction, name);
         const thenValue = (0, combi_1.altPrio)((0, combi_1.seq)("(", value, ")"), value);
         const simple = (0, combi_1.seq)((0, combi_1.altPrio)(_1.CDSFunction, name), (0, combi_1.plusPrio)((0, combi_1.seq)("WHEN", value, "THEN", thenValue)));
         const complex = (0, combi_1.plusPrio)((0, combi_1.seq)("WHEN", _1.CDSCondition, "THEN", thenValue));

package/build/src/cds/expressions/cds_define_custom.js

@@ -9,7 +9,7 @@ class CDSDefineCustom extends combi_1.Expression {
     getRunnable() {
         const field = (0, combi_1.seq)((0, combi_1.opt)((0, combi_1.str)("KEY")), cds_name_1.CDSName, ":", cds_type_1.CDSType, ";");
         const compsiOrAssoci = (0, combi_1.seq)(cds_name_1.CDSName, ":", (0, combi_1.alt)(_1.CDSComposition, _1.CDSAssociation), ";");
-        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.str)("DEFINE"), (0, combi_1.opt)((0, combi_1.str)("ROOT")), (0, combi_1.str)("CUSTOM ENTITY"), (0, combi_1.opt)(_1.CDSWithParameters), cds_name_1.CDSName, (0, combi_1.str)("{"), (0, combi_1.plus)((0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.alt)(field, compsiOrAssoci))), (0, combi_1.str)("}"), (0, combi_1.opt)(";"));
+        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.str)("DEFINE"), (0, combi_1.opt)((0, combi_1.str)("ROOT")), (0, combi_1.str)("CUSTOM ENTITY"), cds_name_1.CDSName, (0, combi_1.opt)(_1.CDSWithParameters), (0, combi_1.str)("{"), (0, combi_1.plus)((0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.alt)(field, compsiOrAssoci))), (0, combi_1.str)("}"), (0, combi_1.opt)(";"));
     }
 }
 exports.CDSDefineCustom = CDSDefineCustom;

package/build/src/cds/expressions/cds_element.js

@@ -7,9 +7,9 @@ const cds_as_1 = require("./cds_as");
 const cds_cast_1 = require("./cds_cast");
 class CDSElement extends combi_1.Expression {
     getRunnable() {
-        const redirected = (0, combi_1.seq)(": REDIRECTED TO", (0, combi_1.opt)((0, combi_1.alt)("PARENT", "COMPOSITION CHILD")), _1.CDSName);
+        const redirected = (0, combi_1.seq)(": REDIRECTED TO", (0, combi_1.optPrio)((0, combi_1.altPrio)("PARENT", "COMPOSITION CHILD")), _1.CDSName);
         const colonThing = (0, combi_1.seq)(":", (0, combi_1.alt)(_1.CDSName, _1.CDSType, "LOCALIZED"));
-        return (0, combi_1.seq)((0, combi_1.starPrio)(_1.CDSAnnotation), (0, combi_1.optPrio)((0, combi_1.altPrio)("KEY", "VIRTUAL")), (0, combi_1.altPrio)(_1.CDSAggregate, _1.CDSString, _1.CDSArithmetics, _1.CDSFunction, cds_cast_1.CDSCast, _1.CDSCase, (0, combi_1.seq)("(", _1.CDSCase, ")"), (0, combi_1.seq)(_1.CDSPrefixedName, (0, combi_1.opt)(cds_as_1.CDSAs), (0, combi_1.opt)((0, combi_1.alt)(redirected, colonThing))), _1.CDSInteger), (0, combi_1.opt)(cds_as_1.CDSAs));
+        return (0, combi_1.seq)((0, combi_1.starPrio)(_1.CDSAnnotation), (0, combi_1.optPrio)((0, combi_1.altPrio)("KEY", "VIRTUAL")), (0, combi_1.altPrio)(_1.CDSAggregate, _1.CDSString, _1.CDSArithmetics, _1.CDSFunction, cds_cast_1.CDSCast, _1.CDSCase, (0, combi_1.seq)("(", _1.CDSCase, ")"), (0, combi_1.seq)(_1.CDSPrefixedName, (0, combi_1.optPrio)(cds_as_1.CDSAs), (0, combi_1.optPrio)((0, combi_1.altPrio)(redirected, colonThing))), _1.CDSInteger), (0, combi_1.optPrio)(cds_as_1.CDSAs));
     }
 }
 exports.CDSElement = CDSElement;

package/build/src/registry.js

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

package/package.json

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