@abaplint/cli 2.113.49 → 2.113.51

package/build/cli.js

@@ -21957,7 +21957,7 @@ class ObjectOriented {
         return ret;
     }
     addAliasedAttributes(classDefinition) {
-        for (const alias of classDefinition.getAliases()) {
+        for (const alias of classDefinition.getAliases() || []) {
             const comp = alias.getComponent();
             const idef = this.scope.findInterfaceDefinition(comp.split("~")[0]);
             if (idef) {
@@ -26953,7 +26953,7 @@ class Select {
         const fields = this.findFields(node, input);
         if (fields.length === 0
             && node.findDirectExpression(Expressions.SQLFieldListLoop) === undefined) {
-            const message = `fields missing`;
+            const message = `SELECT: fields missing`;
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return;
         }
@@ -27132,7 +27132,7 @@ class Select {
         const ret = [];
         expr = node.findFirstExpression(Expressions.SQLFieldList);
         if (expr === undefined) {
-            expr = node.findDirectExpression(Expressions.SQLFieldListLoop);
+            expr = node.findFirstExpression(Expressions.SQLFieldListLoop);
         }
         if (((_a = expr === null || expr === void 0 ? void 0 : expr.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.Dynamic) {
             new dynamic_1.Dynamic().runSyntax(expr.getFirstChild(), input);
@@ -29584,8 +29584,11 @@ class ClassImplementation {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return;
         }
-        for (const t of classDefinition.getTypeDefinitions().getAll()) {
-            input.scope.addType(t.type);
+        const types = classDefinition.getTypeDefinitions();
+        if (types !== undefined) {
+            for (const t of types.getAll()) {
+                input.scope.addType(t.type);
+            }
         }
         const sup = input.scope.findClassDefinition(classDefinition.getSuperClass());
         if (sup) {
@@ -29598,10 +29601,12 @@ class ClassImplementation {
         input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(new tokens_1.Identifier(new position_1.Position(1, 1), "me"), _builtin_1.BuiltIn.filename, new basic_1.ObjectReferenceType(classDefinition)));
         helper.addAliasedAttributes(classDefinition); // todo, this is not correct, take care of instance vs static
         const classAttributes = classDefinition.getAttributes();
-        input.scope.addList(classAttributes.getConstants());
-        input.scope.addList(classAttributes.getStatic());
-        for (const i of classAttributes.getInstance()) {
-            input.scope.addExtraLikeType(i);
+        if (classAttributes !== undefined) {
+            input.scope.addList(classAttributes.getConstants());
+            input.scope.addList(classAttributes.getStatic());
+            for (const i of classAttributes.getInstance()) {
+                input.scope.addExtraLikeType(i);
+            }
         }
         helper.fromSuperClassesAndInterfaces(classDefinition);
     }
@@ -40270,7 +40275,8 @@ class CDSArithmetics extends combi_1.Expression {
         const operatorValue = (0, combi_1.seq)(operator, val);
         const paren = (0, combi_1.seq)("(", val, (0, combi_1.plus)(operatorValue), ")");
         const noParen = (0, combi_1.seq)(val, (0, combi_1.plus)(operatorValue));
-        return (0, combi_1.altPrio)(paren, noParen);
+        // todo: this is pretty bad, it needs a rewrite
+        return (0, combi_1.altPrio)((0, combi_1.seq)(paren, (0, combi_1.plus)(operatorValue)), paren, noParen);
     }
 }
 exports.CDSArithmetics = CDSArithmetics;
@@ -40560,7 +40566,7 @@ const cds_as_1 = __webpack_require__(/*! ./cds_as */ "./node_modules/@abaplint/c
 const cds_cast_1 = __webpack_require__(/*! ./cds_cast */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_cast.js");
 class CDSElement extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.seq)((0, combi_1.starPrio)(_1.CDSAnnotation), (0, combi_1.optPrio)("KEY"), (0, combi_1.altPrio)(_1.CDSAggregate, _1.CDSString, _1.CDSFunction, _1.CDSArithmetics, cds_cast_1.CDSCast, _1.CDSCase, (0, combi_1.seq)(_1.CDSName, ": REDIRECTED TO", (0, combi_1.opt)((0, combi_1.alt)("PARENT", "COMPOSITION CHILD")), _1.CDSName), _1.CDSPrefixedName, (0, combi_1.regex)(/^\d+$/)), (0, combi_1.opt)(cds_as_1.CDSAs));
+        return (0, combi_1.seq)((0, combi_1.starPrio)(_1.CDSAnnotation), (0, combi_1.optPrio)("KEY"), (0, combi_1.altPrio)(_1.CDSAggregate, _1.CDSString, _1.CDSArithmetics, _1.CDSFunction, cds_cast_1.CDSCast, _1.CDSCase, (0, combi_1.seq)(_1.CDSName, ": REDIRECTED TO", (0, combi_1.opt)((0, combi_1.alt)("PARENT", "COMPOSITION CHILD")), _1.CDSName), _1.CDSPrefixedName, (0, combi_1.regex)(/^\d+$/)), (0, combi_1.opt)(cds_as_1.CDSAs));
     }
 }
 exports.CDSElement = CDSElement;
@@ -53108,7 +53114,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.49";
+        return "2.113.51";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

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