@abaplint/core 2.113.50 → 2.113.52

package/build/src/abap/2_statements/statements/selectionscreen.js

@@ -32,7 +32,7 @@ class SelectionScreen {
         const posIntegers = (0, combi_1.regex)(/^(0?[1-9]|[1234567][0-9]|8[0-3])$/);
         const pos = (0, combi_1.seq)("POSITION", (0, combi_1.altPrio)(posIntegers, posSymbols));
         const incl = (0, combi_1.seq)("INCLUDE BLOCKS", expressions_1.BlockName);
-        const tabbed = (0, combi_1.seq)("BEGIN OF TABBED BLOCK", expressions_1.InlineField, "FOR", expressions_1.Integer, "LINES", (0, combi_1.optPrio)("NO INTERVALS"));
+        const tabbed = (0, combi_1.seq)("BEGIN OF TABBED BLOCK", expressions_1.BlockName, "FOR", expressions_1.Integer, "LINES", (0, combi_1.optPrio)("NO INTERVALS"));
         const uline = (0, combi_1.seq)("ULINE", (0, combi_1.opt)(position));
         const param = (0, combi_1.seq)("INCLUDE PARAMETERS", expressions_1.Field);
         const iso = (0, combi_1.seq)("INCLUDE SELECT-OPTIONS", expressions_1.Field);

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

@@ -35,7 +35,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) {
@@ -67,7 +67,7 @@ class ObjectOriented {
         return undefined;
     }
     findMethodViaAlias(methodName, def) {
-        for (const a of def.getAliases()) {
+        for (const a of def.getAliases() || []) {
             if (a.getName().toUpperCase() === methodName.toUpperCase()) {
                 const comp = a.getComponent();
                 const res = this.findMethodInInterface(comp.split("~")[0], comp.split("~")[1]);
@@ -150,9 +150,12 @@ class ObjectOriented {
             return undefined;
         }
         const upper = name.toUpperCase();
-        for (const a of def.getAttributes().getAll()) {
-            if (a.getName().toUpperCase() === upper) {
-                return a;
+        const attr = def.getAttributes();
+        if (attr) {
+            for (const a of attr.getAll()) {
+                if (a.getName().toUpperCase() === upper) {
+                    return a;
+                }
             }
         }
         for (const a of def.getAliases() || []) {
@@ -266,7 +269,11 @@ class ObjectOriented {
         return { method: undefined, def: undefined };
     }
     findMethod(def, methodName) {
-        for (const method of def.getMethodDefinitions().getAll()) {
+        const defs = def.getMethodDefinitions();
+        if (defs === undefined) {
+            return undefined;
+        }
+        for (const method of defs.getAll()) {
             if (method.getName().toUpperCase() === methodName.toUpperCase()) {
                 if (method.isRedefinition()) {
                     return this.findMethodInSuper(def, methodName);

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

@@ -29,7 +29,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;
         }
@@ -208,7 +208,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);

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

@@ -112,7 +112,7 @@ class Source {
                         const foundType = this.determineType(node, input, targetType);
                         const bodyType = new conv_body_1.ConvBody().runSyntax(node.findDirectExpression(Expressions.ConvBody), input);
                         if (new _type_utils_1.TypeUtils(input.scope).isAssignable(foundType, bodyType) === false) {
-                            const message = "CONV: Types not compatible";
+                            const message = `CONV: Types not compatible, ${foundType === null || foundType === void 0 ? void 0 : foundType.constructor.name}, ${bodyType === null || bodyType === void 0 ? void 0 : bodyType.constructor.name}`;
                             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                             return new basic_1.VoidType(_syntax_input_1.CheckSyntaxKey);
                         }

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

@@ -20,8 +20,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) {
@@ -34,10 +37,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);
     }

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

@@ -7,30 +7,35 @@ const basic_1 = require("../../types/basic");
 const _syntax_input_1 = require("../_syntax_input");
 class SelectionScreen {
     runSyntax(node, input) {
-        const field = node.findFirstExpression(Expressions.InlineField);
-        if (field === undefined) {
+        const blockNode = node.findFirstExpression(Expressions.BlockName);
+        const blockToken = blockNode === null || blockNode === void 0 ? void 0 : blockNode.getFirstToken();
+        const blockName = blockNode === null || blockNode === void 0 ? void 0 : blockNode.concatTokens();
+        if (blockName !== undefined && blockName.length > 16) {
+            const message = "SELECTION-SCREEN block name too long, " + blockName;
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return;
         }
-        if (field.getFirstToken().getStr().length > 8) {
+        const field = node.findFirstExpression(Expressions.InlineField);
+        if (field !== undefined && field.getFirstToken().getStr().length > 8) {
             const message = "SELECTION-SCREEN name too long, " + field.getFirstToken().getStr();
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, field.getFirstToken(), message));
             return;
         }
-        const name = field.getFirstToken();
+        const fieldName = field === null || field === void 0 ? void 0 : field.getFirstToken();
         const concat = node.concatTokens().toUpperCase();
-        if (concat.includes("BEGIN OF TABBED BLOCK")) {
+        if (concat.includes("BEGIN OF TABBED BLOCK") && blockToken) {
             const type = new basic_1.StructureType([
                 { name: "PROG", type: new basic_1.CharacterType(40) },
                 { name: "DYNNR", type: new basic_1.CharacterType(4) },
                 { name: "ACTIVETAB", type: new basic_1.CharacterType(132) },
             ]);
-            input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(name, input.filename, type, ["selection_screen_tab" /* IdentifierMeta.SelectionScreenTab */]));
+            input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(blockToken, input.filename, type, ["selection_screen_tab" /* IdentifierMeta.SelectionScreenTab */]));
         }
-        else if (concat.startsWith("SELECTION-SCREEN TAB")) {
-            input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(name, input.filename, new basic_1.CharacterType(83), ["selection_screen_tab" /* IdentifierMeta.SelectionScreenTab */]));
+        else if (concat.startsWith("SELECTION-SCREEN TAB") && fieldName) {
+            input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(fieldName, input.filename, new basic_1.CharacterType(83), ["selection_screen_tab" /* IdentifierMeta.SelectionScreenTab */]));
         }
-        else {
-            input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(name, input.filename, new basic_1.CharacterType(83)));
+        else if (fieldName) {
+            input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(fieldName, input.filename, new basic_1.CharacterType(83)));
         }
     }
 }

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

@@ -7,7 +7,7 @@ const cds_as_1 = require("./cds_as");
 const cds_cast_1 = require("./cds_cast");
 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;

package/build/src/registry.js

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

package/build/src/rules/selection_screen_naming.js

@@ -95,7 +95,11 @@ class SelectionScreenNaming extends _abap_rule_1.ABAPRule {
             return statNode.findFirstExpression(expressions_1.FieldSub);
         }
         else if (statNode.get() instanceof statements_1.SelectionScreen) {
-            return statNode.findFirstExpression(expressions_1.InlineField);
+            let ret = statNode.findFirstExpression(expressions_1.InlineField);
+            if (ret === undefined && statNode.concatTokens().toUpperCase().includes(" BEGIN OF TABBED BLOCK")) {
+                ret = statNode.findFirstExpression(expressions_1.BlockName);
+            }
+            return ret;
         }
         else {
             return undefined;

package/package.json

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