@abaplint/transpiler-cli 2.13.59 → 2.13.61

package/build/bundle.js

@@ -4229,10 +4229,10 @@ exports.ClassName = ClassName;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Color = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
-const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/source.js");
+const simple_source3_1 = __webpack_require__(/*! ./simple_source3 */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/simple_source3.js");
 class Color extends combi_1.Expression {
     getRunnable() {
-        const eq = (0, combi_1.seq)("=", source_1.Source);
+        const eq = (0, combi_1.seq)("=", simple_source3_1.SimpleSource3);
         const integers = (0, combi_1.altPrio)("1", "2", "3", "4", "5", "6", "7");
         const texts = (0, combi_1.altPrio)("COL_BACKGROUND", "COL_HEADING", "COL_NORMAL", "COL_TOTAL", "COL_KEY", "COL_POSITIVE", "COL_NEGATIVE", "COL_GROUP");
         const value = (0, combi_1.alt)(eq, (0, combi_1.altPrio)("ON", "OFF", "COLOR OFF", (0, combi_1.altPrio)(integers, texts)));
@@ -9527,7 +9527,7 @@ const sql_field_1 = __webpack_require__(/*! ./sql_field */ "./node_modules/@abap
 const sql_grouping_sets_1 = __webpack_require__(/*! ./sql_grouping_sets */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_grouping_sets.js");
 class SQLGroupBy extends combi_1.Expression {
     getRunnable() {
-        const fieldName = (0, combi_1.alt)(sql_field_name_1.SQLFieldName, dynamic_1.Dynamic);
+        const fieldName = (0, combi_1.seq)((0, combi_1.stopBefore)("ORDER", "BY"), (0, combi_1.alt)(sql_field_name_1.SQLFieldName, dynamic_1.Dynamic));
         const expr = (0, combi_1.ver)(version_1.Release.v740sp02, sql_field_1.SQLField, { also: combi_1.AlsoIn.OpenABAP });
         const newGroup = (0, combi_1.ver)(version_1.Release.v740sp02, (0, combi_1.seq)("GROUP BY", expr, (0, combi_1.altPrio)((0, combi_1.plus)((0, combi_1.seq)(",", expr)), (0, combi_1.optPrio)((0, combi_1.plus)(fieldName)))), { also: combi_1.AlsoIn.OpenABAP });
         const old = (0, combi_1.seq)((0, combi_1.plus)((0, combi_1.seq)(fieldName, ",")), fieldName);
@@ -13129,7 +13129,7 @@ class ClassDefinition {
         const risk = (0, combi_1.seq)("RISK LEVEL", level);
         const time = (0, combi_1.altPrio)("LONG", "MEDIUM", "SHORT");
         const duration = (0, combi_1.seq)("DURATION", time);
-        const blah = (0, combi_1.per)(expressions_1.ClassGlobal, expressions_1.ClassFinal, "ABSTRACT", (0, combi_1.seq)("INHERITING FROM", expressions_1.SuperClassName), create, "FOR TESTING", risk, "SHARED MEMORY ENABLED", duration, (0, combi_1.ver)(version_1.Release.v754, (0, combi_1.seq)("FOR BEHAVIOR OF", expressions_1.BehaviorDefinitionName), { also: combi_1.AlsoIn.OpenABAP }), expressions_1.ClassFriends);
+        const blah = (0, combi_1.per)(expressions_1.ClassGlobal, expressions_1.ClassFinal, "ABSTRACT", (0, combi_1.seq)("INHERITING FROM", expressions_1.SuperClassName), create, "FOR TESTING", risk, "SHARED MEMORY ENABLED", duration, (0, combi_1.ver)(version_1.Release.v754, (0, combi_1.seq)("FOR BEHAVIOR OF", expressions_1.BehaviorDefinitionName), { also: combi_1.AlsoIn.OpenABAP }), (0, combi_1.ver)(version_1.Release.v758, (0, combi_1.seq)("FOR EVENTS OF", expressions_1.BehaviorDefinitionName)), expressions_1.ClassFriends);
         const def = (0, combi_1.seq)("DEFINITION", (0, combi_1.optPrio)(blah));
         return (0, combi_1.seq)("CLASS", expressions_1.ClassName, def);
     }
@@ -17098,6 +17098,7 @@ exports.MethodDef = void 0;
 const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
 const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
 const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/index.js");
 class MethodDef {
     getMatcher() {
         const def = (0, combi_1.ver)(version_1.Release.v740sp08, (0, combi_1.seq)("DEFAULT", (0, combi_1.altPrio)("FAIL", "IGNORE")), { also: combi_1.AlsoIn.OpenABAP });
@@ -17111,6 +17112,9 @@ class MethodDef {
         const modify = (0, combi_1.alt)((0, combi_1.seq)("FOR ACTION", expressions_1.TypeName, (0, combi_1.optPrio)(result)), (0, combi_1.seq)("FOR CREATE", (0, combi_1.alt)(expressions_1.TypeName, expressions_1.EntityAssociation)), (0, combi_1.seq)("FOR DELETE", expressions_1.TypeName), (0, combi_1.seq)("FOR UPDATE", expressions_1.TypeName));
         const forRead = (0, combi_1.seq)("FOR READ", (0, combi_1.alt)(expressions_1.TypeName, expressions_1.EntityAssociation), (0, combi_1.optPrio)(full), result, (0, combi_1.optPrio)(link));
         const forfunction = (0, combi_1.seq)("FOR FUNCTION", expressions_1.TypeName, result);
+        const entityEventReference = (0, combi_1.seq)("REFERENCE", (0, combi_1.tok)(tokens_1.ParenLeft), expressions_1.MethodParamName, (0, combi_1.tok)(tokens_1.ParenRightW));
+        const entityEventParameter = (0, combi_1.seq)((0, combi_1.altPrio)(entityEventReference, expressions_1.MethodParamName), "FOR", expressions_1.TypeName);
+        const entityEvent = (0, combi_1.seq)((0, combi_1.optPrio)("FINAL"), "FOR ENTITY EVENT", (0, combi_1.optPrio)("IMPORTING"), (0, combi_1.plus)(entityEventParameter));
         const behavior = (0, combi_1.altPrio)("DDL OBJECT OPTIONS CDS SESSION CLIENT REQUIRED", // todo, this is only from version something
         (0, combi_1.seq)("TABLE FUNCTION", expressions_1.NamespaceSimpleName), // todo, this is only from version something
         (0, combi_1.seq)("VALIDATE ON SAVE IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("MODIFY", (0, combi_1.opt)("IMPORTING"), (0, combi_1.plus)((0, combi_1.seq)(expressions_1.MethodParamName, modify))), (0, combi_1.seq)("PRECHECK IMPORTING", expressions_1.MethodParamName, modify), (0, combi_1.seq)("DETERMINATION", expressions_1.TypeName, "IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("VALIDATION", expressions_1.TypeName, "IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("NUMBERING IMPORTING", expressions_1.MethodParamName, modify), (0, combi_1.seq)("READ IMPORTING", expressions_1.MethodParamName, (0, combi_1.altPrio)(forRead, forfunction)), (0, combi_1.seq)("FEATURES IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.NamespaceSimpleName, "FOR", expressions_1.NamespaceSimpleName, result), (0, combi_1.seq)("BEHAVIOR IMPORTING", expressions_1.MethodParamName, "FOR CREATE", expressions_1.TypeName, expressions_1.MethodParamName, "FOR UPDATE", expressions_1.TypeName, expressions_1.MethodParamName, "FOR DELETE", expressions_1.TypeName), (0, combi_1.seq)("BEHAVIOR IMPORTING", expressions_1.MethodParamName, "FOR READ", expressions_1.TypeName, result), (0, combi_1.seq)("BEHAVIOR IMPORTING", expressions_1.MethodParamName, "FOR UPDATE", expressions_1.TypeName), (0, combi_1.seq)((0, combi_1.alt)("BEHAVIOR", "LOCK"), "IMPORTING", expressions_1.MethodParamName, "FOR LOCK", expressions_1.TypeName), (0, combi_1.seq)("DETERMINE", (0, combi_1.alt)("ON MODIFY", "ON SAVE"), "IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("GLOBAL AUTHORIZATION IMPORTING REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result), (0, combi_1.seq)("GLOBAL FEATURES IMPORTING REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result), (0, combi_1.seq)((0, combi_1.seq)((0, combi_1.opt)("INSTANCE"), "AUTHORIZATION IMPORTING"), expressions_1.MethodParamName, "REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result), (0, combi_1.seq)("INSTANCE FEATURES IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result));
@@ -17118,7 +17122,9 @@ class MethodDef {
         // AMDP OPTIONS clause blocked in KeyUser
         const amdp = (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.seq)("AMDP OPTIONS", (0, combi_1.optPrio)("READ-ONLY"), "CDS SESSION CLIENT", (0, combi_1.alt)("CURRENT", "DEPENDENT"), (0, combi_1.optPrio)(expressions_1.MethodDefImporting), (0, combi_1.optPrio)(expressions_1.MethodDefExporting), (0, combi_1.optPrio)(expressions_1.MethodDefRaising)));
         // FOR DDL OBJECT / FOR TABLE FUNCTION / FOR SCALAR FUNCTION blocked in KeyUser
-        const ret = (0, combi_1.seq)((0, combi_1.altPrio)("CLASS-METHODS", "METHODS"), expressions_1.MethodName, (0, combi_1.alt)((0, combi_1.seq)((0, combi_1.optPrio)(expressions_1.Abstract), (0, combi_1.optPrio)(def), expressions_1.EventHandler), parameters, testing, (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.seq)("FOR", behavior)), amdp, "NOT AT END OF MODE", (0, combi_1.optPrio)(expressions_1.Redefinition)));
+        const regular = (0, combi_1.seq)((0, combi_1.altPrio)("CLASS-METHODS", "METHODS"), expressions_1.MethodName, (0, combi_1.alt)((0, combi_1.seq)((0, combi_1.optPrio)(expressions_1.Abstract), (0, combi_1.optPrio)(def), expressions_1.EventHandler), parameters, testing, (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.seq)("FOR", behavior)), amdp, "NOT AT END OF MODE", (0, combi_1.optPrio)(expressions_1.Redefinition)));
+        // RAP event handlers are instance methods; CLASS-METHODS is not allowed.
+        const ret = (0, combi_1.alt)((0, combi_1.seq)("METHODS", expressions_1.MethodName, (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, entityEvent)), regular);
         return ret;
     }
 }
@@ -27118,6 +27124,31 @@ class ObjectOriented {
         }
         return ret;
     }
+    // does the supplied class have friendship with "friendName"?
+    // note that interfaces of the friend, and subclasses of the friend also have friendship
+    hasFriendship(cd, friendName) {
+        var _a;
+        const friends = cd.getFriends().map(f => f.toUpperCase());
+        const isFriend = (name) => friends.includes(name.toUpperCase()) || this.scope.isLocalFriend(cd.getName(), name);
+        if (isFriend(friendName)) {
+            return true;
+        }
+        const friendDefinition = this.scope.findClassDefinition(friendName);
+        if (friendDefinition === undefined) {
+            return false;
+        }
+        if (this.findInterfaces(friendDefinition).some(i => friends.includes(i.name.toUpperCase()))) {
+            return true;
+        }
+        let sup = friendDefinition.getSuperClass();
+        while (sup !== undefined) {
+            if (isFriend(sup)) {
+                return true;
+            }
+            sup = (_a = this.scope.findClassDefinition(sup)) === null || _a === void 0 ? void 0 : _a.getSuperClass();
+        }
+        return false;
+    }
     searchEvent(def, name) {
         if (def === undefined || name === undefined) {
             return undefined;
@@ -27126,6 +27157,11 @@ class ObjectOriented {
         if (found) {
             return found;
         }
+        if (name.includes("~")) {
+            // event of an implemented interface, ie. "if_name~event_name"
+            const [interfaceName, eventName] = name.split("~");
+            return this.searchEvent(this.scope.findInterfaceDefinition(interfaceName), eventName);
+        }
         for (const a of def.getAliases() || []) {
             if (a.getName().toUpperCase() === name.toUpperCase()) {
                 const comp = a.getComponent();
@@ -28021,6 +28057,12 @@ class TypeUtils {
         if (source2 instanceof basic_1.HexType && this.isCalculated(node2) && source1 instanceof basic_1.IntegerType) {
             return false;
         }
+        if (source1 instanceof basic_1.DateType && this.isCalculated(node2) && source2 instanceof basic_1.IntegerType) {
+            return false;
+        }
+        if (source2 instanceof basic_1.DateType && this.isCalculated(node1) && source1 instanceof basic_1.IntegerType) {
+            return false;
+        }
         return true;
     }
     structureContainsString(structure) {
@@ -28111,7 +28153,10 @@ class TypeUtils {
                 || source instanceof basic_1.DataType
                 || source instanceof basic_1.UnknownType;
         }
-        if (source instanceof basic_1.CharacterType) {
+        if (source instanceof cgeneric_type_1.CGenericType && target instanceof basic_1.StringType) {
+            return false;
+        }
+        else if (source instanceof basic_1.CharacterType) {
             if (target instanceof basic_1.CharacterType) {
                 if (((_b = source.getAbstractTypeData()) === null || _b === void 0 ? void 0 : _b.derivedFromConstant) === true) {
                     return source.getLength() <= target.getLength();
@@ -28154,6 +28199,11 @@ class TypeUtils {
                 return false;
             }
         }
+        else if (source instanceof basic_1.DateType || source instanceof basic_1.TimeType) {
+            if (target instanceof basic_1.CharacterType || target instanceof basic_1.StringType) {
+                return false;
+            }
+        }
         else if (source instanceof basic_1.StringType) {
             if (target instanceof basic_1.StructureType && this.structureContainsString(target)) {
                 return false;
@@ -28180,6 +28230,11 @@ class TypeUtils {
                 }
                 return false;
             }
+            else if (target instanceof basic_1.TableType
+                || target instanceof basic_1.DataReference
+                || target instanceof basic_1.ObjectReferenceType) {
+                return false;
+            }
             return true;
         }
         else if (source instanceof basic_1.StructureType) {
@@ -28204,7 +28259,8 @@ class TypeUtils {
             }
             else if (target instanceof basic_1.VoidType
                 || target instanceof basic_1.AnyType
-                || target instanceof basic_1.DataType) {
+                || target instanceof basic_1.DataType
+                || target instanceof basic_1.UnknownType) {
                 return true;
             }
             return false;
@@ -28215,7 +28271,9 @@ class TypeUtils {
             }
         }
         else if (source instanceof basic_1.IntegerType) {
-            if (target instanceof basic_1.StringType || target instanceof basic_1.DateType) {
+            if (target instanceof basic_1.StringType
+                || target instanceof basic_1.DateType
+                || target instanceof basic_1.CharacterType) {
                 return false;
             }
             else if (target instanceof basic_1.Integer8Type || target instanceof basic_1.PackedType) {
@@ -28293,6 +28351,29 @@ class TypeUtils {
         }
         return this.isAssignable(source, target);
     }
+    // Typing with the classic STRUCTURE addition, ie. "FORM foo USING bar STRUCTURE zkey",
+    // is not a compatibility check: ABAP only checks the fragment view up to the end of the
+    // formal structure, so the actual parameter is allowed to be longer. For TABLES parameters
+    // the table key is not part of the check at all.
+    isAssignableStructureTyping(source, target, node) {
+        if (source instanceof basic_1.StructureType && target instanceof basic_1.StructureType) {
+            const sourceComponents = source.getComponents();
+            const targetComponents = target.getComponents();
+            if (sourceComponents.length < targetComponents.length) {
+                return false;
+            }
+            for (let i = 0; i < targetComponents.length; i++) {
+                if (this.isAssignableStrict(sourceComponents[i].type, targetComponents[i].type) === false) {
+                    return false;
+                }
+            }
+            return true;
+        }
+        else if (source instanceof basic_1.TableType && target instanceof basic_1.TableType) {
+            return this.isAssignableStructureTyping(source.getRowType(), target.getRowType());
+        }
+        return this.isAssignableStrict(source, target, node);
+    }
     isAssignable(source, target) {
         if (source === undefined || target === undefined) {
             return true;
@@ -28301,6 +28382,9 @@ class TypeUtils {
             console.dir(source);
             console.dir(target);
         */
+        if (source instanceof basic_1.UTCLongType || target instanceof basic_1.UTCLongType) {
+            return this.isUTCLongAssignable(source, target);
+        }
         if (target instanceof basic_1.TableType) {
             if (target.isWithHeader()) {
                 return this.isAssignable(source, target.getRowType());
@@ -28452,6 +28536,33 @@ class TypeUtils {
         }
         return true;
     }
+    // utclong can only be converted to and from character like types
+    isUTCLongAssignable(source, target) {
+        if (this.isGenericOrVoid(source) || this.isGenericOrVoid(target)) {
+            return true;
+        }
+        else if (source instanceof basic_1.UTCLongType && target instanceof basic_1.UTCLongType) {
+            return true;
+        }
+        else if (source instanceof basic_1.UTCLongType) {
+            return this.isTextLike(target);
+        }
+        return this.isTextLike(source);
+    }
+    isGenericOrVoid(type) {
+        return type instanceof basic_1.VoidType
+            || type instanceof basic_1.AnyType
+            || type instanceof basic_1.DataType
+            || type instanceof basic_1.UnknownType;
+    }
+    isTextLike(type) {
+        return type instanceof basic_1.CharacterType
+            || type instanceof basic_1.StringType
+            || type instanceof basic_1.CLikeType
+            || type instanceof basic_1.CSequenceType
+            || type instanceof cgeneric_type_1.CGenericType
+            || type instanceof basic_1.SimpleType;
+    }
 }
 exports.TypeUtils = TypeUtils;
 //# sourceMappingURL=_type_utils.js.map
@@ -28534,6 +28645,7 @@ const types_1 = __webpack_require__(/*! ../types */ "./node_modules/@abaplint/co
 const expressions_1 = __webpack_require__(/*! ../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const _builtin_1 = __webpack_require__(/*! ./_builtin */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_builtin.js");
 const position_1 = __webpack_require__(/*! ../../position */ "./node_modules/@abaplint/core/build/src/position.js");
+const _syntax_input_1 = __webpack_require__(/*! ./_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 class BasicTypes {
     constructor(input) {
         this.input = input;
@@ -29016,7 +29128,7 @@ class BasicTypes {
             return Types.VoidType.get((_f = node.findFirstExpression(Expressions.TypeStructure)) === null || _f === void 0 ? void 0 : _f.concatTokens());
         }
         else if (text.startsWith("LIKE LINE OF ")) {
-            const name = (_g = node.findFirstExpression(Expressions.FieldChain)) === null || _g === void 0 ? void 0 : _g.concatTokens();
+            let name = (_g = node.findFirstExpression(Expressions.FieldChain)) === null || _g === void 0 ? void 0 : _g.concatTokens();
             let e = node.findFirstExpression(Expressions.Type);
             if (e === undefined) {
                 e = node.findFirstExpression(Expressions.FormParamType);
@@ -29024,19 +29136,32 @@ class BasicTypes {
             if (e === undefined) {
                 e = node.findFirstExpression(Expressions.FieldChain);
             }
+            if (e === undefined) {
+                // old style, eg "LIKE LINE OF tab OCCURS 0 WITH HEADER LINE", it is parsed as TypeName
+                e = typeName;
+                name = typeName === null || typeName === void 0 ? void 0 : typeName.concatTokens();
+            }
             const type = this.resolveLikeName(e, false);
+            let row = undefined;
             if (type === undefined) {
                 return new Types.UnknownType("Type error, could not resolve \"" + name + "\", parseType");
             }
             else if (type instanceof Types.TableType) {
-                return type.getRowType();
+                row = type.getRowType();
             }
             else if (type instanceof Types.VoidType) {
-                return type;
+                row = type;
             }
             else {
                 return new Types.UnknownType("Type error, not a table type " + name);
             }
+            if (this.isOccurs(node)) {
+                return new Types.TableType(row, {
+                    withHeader: text.includes(" WITH HEADER LINE"),
+                    keyType: Types.TableKeyType.default
+                }, qualifiedName);
+            }
+            return row;
         }
         else if (text.startsWith("LIKE REF TO ")) {
             const name = (_h = node.findFirstExpression(Expressions.FieldChain)) === null || _h === void 0 ? void 0 : _h.concatTokens();
@@ -29474,6 +29599,13 @@ class BasicTypes {
         }
         const chain = val.findFirstExpression(Expressions.SimpleFieldChain);
         if (chain) {
+            // todo, hardcoded for now, should be a generic check that the chain is a constant
+            const concat = chain.concatTokens().toUpperCase();
+            if (concat === "SY-DATUM" || concat === "SY-UZEIT") {
+                const message = "VALUE, " + concat.toLowerCase() + " is not a constant";
+                this.input.issues.push((0, _syntax_input_1.syntaxIssue)(this.input, chain.getFirstToken(), message));
+                return undefined;
+            }
             return this.resolveConstantValue(chain);
         }
         throw new Error("findValue, unexpected");
@@ -29539,6 +29671,51 @@ exports.BasicTypes = BasicTypes;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/_check_database_fields.js"
+/*!***************************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/_check_database_fields.js ***!
+  \***************************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.checkDatabaseFields = checkDatabaseFields;
+const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
+function checkDatabaseFields(fields, dbSources, input, token) {
+    if (dbSources.length > 1) {
+        return;
+    }
+    const first = dbSources[0];
+    if (first === undefined) {
+        // then its voided
+        return;
+    }
+    const type = first.parseType(input.scope.getRegistry());
+    if (type instanceof basic_1.VoidType || type instanceof basic_1.UnknownType) {
+        return;
+    }
+    if (!(type instanceof basic_1.StructureType)) {
+        const message = "checkFields, expected structure, " + type.constructor.name;
+        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, token, message));
+        return;
+    }
+    for (const field of fields) {
+        if (field === "*") {
+            continue;
+        }
+        if (/^[A-Z_]\w*$/i.test(field) && type.getComponentByName(field) === undefined) {
+            const message = `checkFields, field ${field} not found`;
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, token, message));
+            return;
+        }
+    }
+}
+//# sourceMappingURL=_check_database_fields.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/_check_offset_length.js"
 /*!*************************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/_check_offset_length.js ***!
@@ -30018,6 +30195,7 @@ const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/c
 const method_call_chain_1 = __webpack_require__(/*! ./method_call_chain */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/method_call_chain.js");
 const source_field_symbol_1 = __webpack_require__(/*! ./source_field_symbol */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source_field_symbol.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
+const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_type_utils.js");
 class Compare {
     static runSyntax(node, input) {
@@ -30035,7 +30213,15 @@ class Compare {
         if (node.findDirectExpression(Expressions.CompareOperator)
             && new _type_utils_1.TypeUtils(input.scope).isCompareable(sourceTypes[0], sourceTypes[1], sources[0], sources[1]) === false
             && sourceTypes.length === 2) {
-            const message = "Incompatible types for comparison";
+            let message = "Incompatible types for comparison";
+            if ((sourceTypes[0] instanceof basic_1.DateType
+                && sourceTypes[1] instanceof basic_1.IntegerType
+                && sources[1].findFirstExpression(Expressions.ArithOperator))
+                || (sourceTypes[1] instanceof basic_1.DateType
+                    && sourceTypes[0] instanceof basic_1.IntegerType
+                    && sources[0].findFirstExpression(Expressions.ArithOperator))) {
+                message = "Date cannot be compared with arithmetic result of type Integer";
+            }
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
         }
     }
@@ -31790,18 +31976,27 @@ class FormParam {
             else {
                 type = new basic_1.UnknownType("todo, FORM STRUCTURES typing");
             }
-            return new _typed_identifier_1.TypedIdentifier(nameToken, input.filename, type, ["form_parameter" /* IdentifierMeta.FormParameter */]);
+            return new _typed_identifier_1.TypedIdentifier(nameToken, input.filename, type, ["form_parameter" /* IdentifierMeta.FormParameter */, "form_parameter_structure" /* IdentifierMeta.FormParameterStructure */]);
         }
         if (node.getChildren().length === 1 && nameToken) {
             // untyped FORM parameter
             return new _typed_identifier_1.TypedIdentifier(nameToken, input.filename, basic_1.AnyType.get(), ["form_parameter" /* IdentifierMeta.FormParameter */]);
         }
         let bfound = new basic_types_1.BasicTypes(input).parseType(node);
-        const isTypeC = ((_c = node.findFirstExpression(expressions_1.TypeName)) === null || _c === void 0 ? void 0 : _c.concatTokens().toUpperCase()) === "C";
+        const typeName = (_c = node.findFirstExpression(expressions_1.TypeName)) === null || _c === void 0 ? void 0 : _c.concatTokens().toUpperCase();
         const hasExplicitLength = node.findFirstExpression(expressions_1.Length) !== undefined
             || node.findFirstExpression(expressions_1.ConstantFieldLength) !== undefined;
-        if (isTypeC && hasExplicitLength === false && bfound instanceof basic_1.CharacterType) {
-            bfound = basic_1.CGenericType.get();
+        // the length cannot be specified for FORM parameters, so these types are generic
+        if (hasExplicitLength === false) {
+            if (typeName === "C" && bfound instanceof basic_1.CharacterType) {
+                bfound = basic_1.CGenericType.get();
+            }
+            else if (typeName === "X" && bfound instanceof basic_1.HexType) {
+                bfound = basic_1.XGenericType.get();
+            }
+            else if (typeName === "P" && bfound instanceof basic_1.PackedType) {
+                bfound = basic_1.PGenericType.get();
+            }
         }
         if (nameToken && bfound) {
             return new _typed_identifier_1.TypedIdentifier(nameToken, input.filename, bfound, ["form_parameter" /* IdentifierMeta.FormParameter */]);
@@ -32693,6 +32888,8 @@ const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@a
 const component_name_1 = __webpack_require__(/*! ./component_name */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/component_name.js");
 const attribute_name_1 = __webpack_require__(/*! ./attribute_name */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/attribute_name.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
+const visibility_1 = __webpack_require__(/*! ../../4_file_information/visibility */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/visibility.js");
+const class_definition_1 = __webpack_require__(/*! ../../types/class_definition */ "./node_modules/@abaplint/core/build/src/abap/types/class_definition.js");
 class MethodCallChain {
     static runSyntax(node, input, targetType) {
         var _a, _b;
@@ -32741,6 +32938,12 @@ class MethodCallChain {
                         input.issues.push((0, _syntax_input_1.syntaxIssue)(input, methodToken, message));
                         return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                     }
+                    const notVisible = method ? this.checkVisibility(method, def, foundDef, input) : undefined;
+                    if (notVisible !== undefined) {
+                        const message = `Method "${methodName}" is ${notVisible} and cannot be accessed`;
+                        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, methodToken, message));
+                        return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
+                    }
                     const voidedName = context instanceof basic_1.VoidType ? context.getVoided() : undefined;
                     const extra = helper.methodReferenceExtras(foundDef, className || voidedName);
                     input.scope.addReference(methodToken, method, _reference_1.ReferenceType.MethodReference, input.filename, extra);
@@ -32783,6 +32986,47 @@ class MethodCallChain {
         return context;
     }
     //////////////////////////////////////
+    // returns the visibility as text if the method cannot be accessed from the current scope
+    static checkVisibility(method, def, foundDef, input) {
+        var _a, _b, _c;
+        const visibility = method.getVisibility();
+        if (visibility === visibility_1.Visibility.Public || foundDef === undefined) {
+            return undefined;
+        }
+        else if (!(foundDef instanceof class_definition_1.ClassDefinition)) {
+            // interface members are always public
+            return undefined;
+        }
+        const name = foundDef.getName().toUpperCase();
+        const enclosing = (_a = input.scope.getEnclosingClassName()) === null || _a === void 0 ? void 0 : _a.toUpperCase();
+        if (enclosing === undefined || enclosing === name) {
+            return undefined;
+        }
+        // friendship with the class used at the call site also gives access to inherited members
+        const helper = new _object_oriented_1.ObjectOriented(input.scope);
+        const visited = [];
+        let current = def instanceof class_definition_1.ClassDefinition ? def : foundDef;
+        while (current !== undefined && visited.includes(current.getName().toUpperCase()) === false) {
+            visited.push(current.getName().toUpperCase());
+            if (helper.hasFriendship(current, enclosing)) {
+                return undefined;
+            }
+            const sup = current.getSuperClass();
+            current = sup === undefined ? undefined : input.scope.findClassDefinition(sup);
+        }
+        if (visibility === visibility_1.Visibility.Protected) {
+            // subclasses can access protected members
+            let sup = (_b = input.scope.findClassDefinition(enclosing)) === null || _b === void 0 ? void 0 : _b.getSuperClass();
+            while (sup !== undefined) {
+                if (sup.toUpperCase() === name) {
+                    return undefined;
+                }
+                sup = (_c = input.scope.findClassDefinition(sup)) === null || _c === void 0 ? void 0 : _c.getSuperClass();
+            }
+            return "protected";
+        }
+        return "private";
+    }
     static findTop(first, input, targetType) {
         var _a;
         if (first.get() instanceof Expressions.ClassName) {
@@ -34179,6 +34423,7 @@ const dynamic_1 = __webpack_require__(/*! ./dynamic */ "./node_modules/@abaplint
 const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
+const _check_database_fields_1 = __webpack_require__(/*! ./_check_database_fields */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/_check_database_fields.js");
 const isSimple = /^\w+$/;
 class Select {
     static runSyntax(node, input, skipImplicitInto = false, selectLoop = false) {
@@ -34203,8 +34448,8 @@ class Select {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return;
         }
-        this.checkFields(fields, dbSources, input, node);
-        const intoExpression = this.handleInto(node, input, fields, dbSources);
+        (0, _check_database_fields_1.checkDatabaseFields)(fields.map(field => field.code), dbSources, input, node.getFirstToken());
+        const intoExpression = this.handleInto(node, input, fields, dbSources, from);
         const fae = node.findDirectExpression(Expressions.SQLForAllEntries);
         if (fae) {
             input.scope.push(_scope_type_1.ScopeType.OpenSQL, "SELECT", token.getStart(), input.filename);
@@ -34365,12 +34610,12 @@ class Select {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
         }
     }
-    static handleInto(node, input, fields, dbSources) {
+    static handleInto(node, input, fields, dbSources, from) {
         const intoTable = node.findDirectExpression(Expressions.SQLIntoTable);
         if (intoTable) {
             const inline = intoTable.findFirstExpression(Expressions.InlineData);
             if (inline) {
-                inline_data_1.InlineData.runSyntax(inline, input, this.buildTableType(fields, dbSources, input.scope));
+                inline_data_1.InlineData.runSyntax(inline, input, this.buildTableType(fields, dbSources, input.scope, from));
             }
             return intoTable;
         }
@@ -34430,35 +34675,6 @@ class Select {
         }
         return undefined;
     }
-    static checkFields(fields, dbSources, input, node) {
-        if (dbSources.length > 1) {
-            return;
-        }
-        const first = dbSources[0];
-        if (first === undefined) {
-            // then its voided
-            return;
-        }
-        const type = first.parseType(input.scope.getRegistry());
-        if (type instanceof basic_1.VoidType || type instanceof basic_1.UnknownType) {
-            return;
-        }
-        if (!(type instanceof basic_1.StructureType)) {
-            const message = "checkFields, expected structure, " + type.constructor.name;
-            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
-            return;
-        }
-        for (const field of fields) {
-            if (field.code === "*") {
-                continue;
-            }
-            if (isSimple.test(field.code) && type.getComponentByName(field.code) === undefined) {
-                const message = `checkFields, field ${field.code} not found`;
-                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
-                return;
-            }
-        }
-    }
     static countResultType(scope) {
         if ((0, version_1.releaseAtLeast)(scope.getRelease(), version_1.Release.v750)
             || scope.getOpenABAP()
@@ -34547,11 +34763,36 @@ class Select {
             return basic_1.VoidType.get("SELECT_todo13");
         }
     }
-    static buildTableType(fields, dbSources, scope) {
-        if (dbSources.length !== 1) {
+    static buildTableType(fields, dbSources, scope, from) {
+        var _a;
+        const tableOptions = { withHeader: false, keyType: basic_1.TableKeyType.empty };
+        if (dbSources.length > 1) {
+            const sourceInfos = this.buildDatabaseSourceInfos(from, dbSources);
+            if (fields.length === 1 && fields[0].code === "*") {
+                const components = [];
+                for (const info of sourceInfos) {
+                    const type = (_a = info.source) === null || _a === void 0 ? void 0 : _a.parseType(scope.getRegistry());
+                    if (!(type instanceof basic_1.StructureType)) {
+                        return basic_1.VoidType.get("SELECT_todo3");
+                    }
+                    components.push({ name: info.alias || info.name, type });
+                }
+                return new basic_1.TableType(new basic_1.StructureType(components), tableOptions, undefined);
+            }
+            const components = [];
+            for (const field of fields) {
+                const type = this.findFieldType(field.code, sourceInfos, scope);
+                if (type === undefined) {
+                    return basic_1.VoidType.get("SELECT_todo3");
+                }
+                const name = field.as || field.code.split("~").pop();
+                components.push({ name, type });
+            }
+            return new basic_1.TableType(new basic_1.StructureType(components), tableOptions, undefined);
+        }
+        else if (dbSources.length !== 1) {
             return basic_1.VoidType.get("SELECT_todo3");
         }
-        const tableOptions = { withHeader: false, keyType: basic_1.TableKeyType.empty };
         if (dbSources[0] === undefined) {
             // then its a voided table
             return new basic_1.TableType(basic_1.VoidType.get("SELECT_todo4"), tableOptions, undefined);
@@ -34577,6 +34818,39 @@ class Select {
         }
         return basic_1.VoidType.get("SELECT_todo7");
     }
+    static buildDatabaseSourceInfos(from, dbSources) {
+        const fromSources = from.findAllExpressions(Expressions.SQLFromSource);
+        return dbSources.map((source, index) => {
+            var _a, _b;
+            const fromSource = fromSources[index];
+            const name = ((_a = fromSource === null || fromSource === void 0 ? void 0 : fromSource.findFirstExpression(Expressions.DatabaseTable)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr().replace(/^\*/, "").toUpperCase()) || "";
+            const alias = ((_b = fromSource === null || fromSource === void 0 ? void 0 : fromSource.findDirectExpression(Expressions.SQLAsName)) === null || _b === void 0 ? void 0 : _b.concatTokens().toUpperCase()) || "";
+            return { source, name, alias };
+        });
+    }
+    static findFieldType(code, sourceInfos, scope) {
+        var _a;
+        const separator = code.indexOf("~");
+        const qualifier = separator >= 0 ? code.substring(0, separator) : undefined;
+        const fieldName = separator >= 0 ? code.substring(separator + 1) : code;
+        const candidates = qualifier === undefined
+            ? sourceInfos
+            : sourceInfos.filter(info => info.alias === qualifier || info.name === qualifier);
+        for (const candidate of candidates) {
+            let type = (_a = candidate.source) === null || _a === void 0 ? void 0 : _a.parseType(scope.getRegistry());
+            for (const component of fieldName.split("-")) {
+                if (!(type instanceof basic_1.StructureType)) {
+                    type = undefined;
+                    break;
+                }
+                type = type.getComponentByName(component);
+            }
+            if (type) {
+                return type;
+            }
+        }
+        return undefined;
+    }
     static findFields(node, input) {
         var _a, _b;
         let expr = undefined;
@@ -34590,7 +34864,7 @@ class Select {
             let code = field.concatTokens().toUpperCase();
             const as = ((_b = field.findDirectExpression(Expressions.SQLAsName)) === null || _b === void 0 ? void 0 : _b.concatTokens()) || "";
             if (as !== "") {
-                code = code.replace(" AS " + as, "");
+                code = code.replace(" AS " + as.toUpperCase(), "");
             }
             ret.push({ code, as, expression: field });
         }
@@ -35421,6 +35695,7 @@ const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@ab
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 const sql_set_op_group_1 = __webpack_require__(/*! ./sql_set_op_group */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_set_op_group.js");
 const sql_source_1 = __webpack_require__(/*! ./sql_source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js");
+const RANGE_COMPONENTS = ["SIGN", "OPTION", "LOW", "HIGH"];
 class SQLIn {
     static runSyntax(node, input) {
         const setop = node.findDirectExpression(Expressions.SQLSetOpGroup);
@@ -35440,6 +35715,12 @@ class SQLIn {
                     input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                     return;
                 }
+                if (intype instanceof basic_1.TableType && this.isRangeRow(intype.getRowType()) === false) {
+                    const name = insource.concatTokens().replace(/^@/, "").replace(/\[\]$/, "");
+                    const message = `row structure of ${name} is not correct`;
+                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                    return;
+                }
             }
             return;
         }
@@ -35450,6 +35731,16 @@ class SQLIn {
             sql_source_1.SQLSource.runSyntax(s, input);
         }
     }
+    // "IN itab" expects a ranges table, ie. the row must have SIGN, OPTION, LOW and HIGH
+    static isRangeRow(rowType) {
+        if (rowType instanceof basic_1.VoidType || rowType instanceof basic_1.UnknownType || rowType instanceof basic_1.AnyType) {
+            return true;
+        }
+        else if (!(rowType instanceof basic_1.StructureType)) {
+            return false;
+        }
+        return RANGE_COMPONENTS.every(c => rowType.getComponentByName(c) !== undefined);
+    }
 }
 exports.SQLIn = SQLIn;
 //# sourceMappingURL=sql_in.js.map
@@ -36346,6 +36637,7 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.ValueBody = void 0;
+const nodes_1 = __webpack_require__(/*! ../../nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
 const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
 const for_1 = __webpack_require__(/*! ./for */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/for.js");
 const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
@@ -36355,7 +36647,7 @@ const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@ab
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 class ValueBody {
     static runSyntax(node, input, targetType) {
-        var _a, _b;
+        var _a, _b, _c;
         if (node === undefined) {
             return targetType;
         }
@@ -36372,26 +36664,45 @@ class ValueBody {
             }
         }
         const fields = new Set();
-        for (const s of node.findDirectExpressions(Expressions.FieldAssignment)) {
-            field_assignment_1.FieldAssignment.runSyntax(s, input, targetType);
-            if (node.findDirectExpression(Expressions.ValueBodyLine) === undefined) {
-                // todo: refine, still needs to be checked when there are table lines
-                const fieldname = (_a = s.findDirectExpression(Expressions.FieldSub)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
+        const hasTableLines = node.findDirectExpression(Expressions.ValueBodyLine) !== undefined;
+        for (const child of node.getChildren()) {
+            if (!(child instanceof nodes_1.ExpressionNode)) {
+                continue;
+            }
+            else if (child.get() instanceof Expressions.FieldAssignment) {
+                const fieldname = (_a = child.findDirectExpression(Expressions.FieldSub)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
+                if (fieldname && hasTableLines === false && fields.has(fieldname)) {
+                    const message = "Duplicate field assignment";
+                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, child.getFirstToken(), message));
+                    return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
+                }
                 if (fieldname) {
-                    if (fields.has(fieldname)) {
+                    fields.add(fieldname);
+                }
+            }
+            else if (child.get() instanceof Expressions.ValueBodyLine) {
+                const rowFields = new Set(fields);
+                for (const assignment of child.findDirectExpressions(Expressions.FieldAssignment)) {
+                    const fieldname = (_b = assignment.findDirectExpression(Expressions.FieldSub)) === null || _b === void 0 ? void 0 : _b.concatTokens().toUpperCase();
+                    if (fieldname && rowFields.has(fieldname)) {
                         const message = "Duplicate field assignment";
-                        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, s.getFirstToken(), message));
+                        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, assignment.getFirstToken(), message));
                         return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                     }
-                    fields.add(fieldname);
+                    if (fieldname) {
+                        rowFields.add(fieldname);
+                    }
                 }
             }
         }
+        for (const s of node.findDirectExpressions(Expressions.FieldAssignment)) {
+            field_assignment_1.FieldAssignment.runSyntax(s, input, targetType);
+        }
         let type = undefined; // todo, this is only correct if there is a single source in the body
         for (const s of node.findDirectExpressions(Expressions.Source)) {
             type = source_1.Source.runSyntax(s, input, type);
         }
-        for (const s of ((_b = node.findDirectExpression(Expressions.ValueBase)) === null || _b === void 0 ? void 0 : _b.findDirectExpressions(Expressions.Source)) || []) {
+        for (const s of ((_c = node.findDirectExpression(Expressions.ValueBase)) === null || _c === void 0 ? void 0 : _c.findDirectExpressions(Expressions.Source)) || []) {
             type = source_1.Source.runSyntax(s, input, type);
         }
         for (const foo of node.findDirectExpressions(Expressions.ValueBodyLine)) {
@@ -38067,7 +38378,12 @@ const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modu
 const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_type_utils.js");
 class Catch {
     runSyntax(node, input) {
-        var _a;
+        var _a, _b, _c;
+        const target = node.findDirectExpression(Expressions.Target);
+        const inlineTarget = (_b = (_a = target === null || target === void 0 ? void 0 : target.findDirectExpression(Expressions.InlineData)) === null || _a === void 0 ? void 0 : _a.findFirstExpression(Expressions.TargetField)) === null || _b === void 0 ? void 0 : _b.getFirstToken();
+        if (inlineTarget && inlineTarget.getStr().length > 30) {
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, inlineTarget, "Inline exception name longer than 30 characters"));
+        }
         const names = new Set();
         for (const c of node.findDirectExpressions(Expressions.ClassName)) {
             const token = c.getFirstToken();
@@ -38091,9 +38407,8 @@ class Catch {
             }
             names.add(className);
         }
-        const target = node.findDirectExpression(Expressions.Target);
         if (target === null || target === void 0 ? void 0 : target.findDirectExpression(Expressions.InlineData)) {
-            const token = (_a = target.findFirstExpression(Expressions.TargetField)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+            const token = (_c = target.findFirstExpression(Expressions.TargetField)) === null || _c === void 0 ? void 0 : _c.getFirstToken();
             if (token) {
                 const classNames = Array.from(names);
                 const unknownClass = classNames.find(name => input.scope.findClassDefinition(name) === undefined);
@@ -39504,7 +39819,7 @@ class CreateObject {
         this.validateParameters(cdef, node, input);
     }
     static checkInstantiationAllowed(cdef, input) {
-        var _a, _b, _c;
+        var _a, _b;
         const createVis = cdef.getCreateVisibility();
         if (createVis === visibility_1.Visibility.Public) {
             return undefined;
@@ -39517,34 +39832,17 @@ class CreateObject {
         if (enclosingClass.toUpperCase() === cdef.getName().toUpperCase()) {
             return undefined;
         }
-        if (cdef.getFriends().some(f => f.toUpperCase() === enclosingClass.toUpperCase()) ||
-            input.scope.isLocalFriend(cdef.getName(), enclosingClass)) {
+        if (new _object_oriented_1.ObjectOriented(input.scope).hasFriendship(cdef, enclosingClass)) {
             return undefined;
         }
-        const enclosingDefinition = input.scope.findClassDefinition(enclosingClass);
-        if (enclosingDefinition !== undefined) {
-            const implementedInterfaces = new _object_oriented_1.ObjectOriented(input.scope).findInterfaces(enclosingDefinition);
-            if (cdef.getFriends().some(friend => implementedInterfaces.some(implemented => implemented.name.toUpperCase() === friend.toUpperCase()))) {
-                return undefined;
-            }
-        }
-        // subclasses of friends also have friendship
-        let enclosingSup = enclosingDefinition === null || enclosingDefinition === void 0 ? void 0 : enclosingDefinition.getSuperClass();
-        while (enclosingSup !== undefined) {
-            if (cdef.getFriends().some(f => f.toUpperCase() === enclosingSup.toUpperCase()) ||
-                input.scope.isLocalFriend(cdef.getName(), enclosingSup)) {
-                return undefined;
-            }
-            enclosingSup = (_a = input.scope.findClassDefinition(enclosingSup)) === null || _a === void 0 ? void 0 : _a.getSuperClass();
-        }
         if (createVis === visibility_1.Visibility.Protected) {
             // subclasses are also allowed
-            let sup = (_b = input.scope.findClassDefinition(enclosingClass)) === null || _b === void 0 ? void 0 : _b.getSuperClass();
+            let sup = (_a = input.scope.findClassDefinition(enclosingClass)) === null || _a === void 0 ? void 0 : _a.getSuperClass();
             while (sup !== undefined) {
                 if (sup.toUpperCase() === cdef.getName().toUpperCase()) {
                     return undefined;
                 }
-                sup = (_c = input.scope.findClassDefinition(sup)) === null || _c === void 0 ? void 0 : _c.getSuperClass();
+                sup = (_b = input.scope.findClassDefinition(sup)) === null || _b === void 0 ? void 0 : _b.getSuperClass();
             }
         }
         return cdef.getName() + " cannot be instantiated, class is defined as " +
@@ -39807,6 +40105,8 @@ const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expr
 const source_1 = __webpack_require__(/*! ../expressions/source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
 const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/dynamic.js");
 const database_table_1 = __webpack_require__(/*! ../expressions/database_table */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/database_table.js");
+const _check_database_fields_1 = __webpack_require__(/*! ../expressions/_check_database_fields */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/_check_database_fields.js");
+const sql_in_1 = __webpack_require__(/*! ../expressions/sql_in */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_in.js");
 class DeleteDatabase {
     runSyntax(node, input) {
         for (const s of node.findAllExpressions(Expressions.Source)) {
@@ -39818,9 +40118,22 @@ class DeleteDatabase {
         for (const d of node.findAllExpressions(Expressions.Dynamic)) {
             dynamic_1.Dynamic.runSyntax(d, input);
         }
+        for (const compare of node.findAllExpressions(Expressions.SQLCompare)) {
+            const sqlin = compare.findDirectExpression(Expressions.SQLIn);
+            if (sqlin !== undefined) {
+                sql_in_1.SQLIn.runSyntax(sqlin, input);
+            }
+        }
         const dbtab = node.findFirstExpression(Expressions.DatabaseTable);
         if (dbtab !== undefined) {
-            database_table_1.DatabaseTable.runSyntax(dbtab, input);
+            const dbSource = database_table_1.DatabaseTable.runSyntax(dbtab, input);
+            const fields = node.findAllExpressions(Expressions.SQLCompare)
+                .map(compare => { var _a; return (_a = compare.findDirectExpression(Expressions.SQLFieldName)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase(); })
+                .filter(field => field !== undefined);
+            for (const orderBy of node.findAllExpressions(Expressions.SQLOrderBy)) {
+                fields.push(...orderBy.findAllExpressions(Expressions.SQLFieldName).map(field => field.concatTokens().toUpperCase()));
+            }
+            (0, _check_database_fields_1.checkDatabaseFields)(fields, [dbSource], input, node.getFirstToken());
         }
     }
 }
@@ -40815,13 +41128,17 @@ const expressions_1 = __webpack_require__(/*! ../../2_statements/expressions */
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 class Form {
     runSyntax(node, input) {
-        var _a;
-        const name = (_a = node.findDirectExpression(expressions_1.FormName)) === null || _a === void 0 ? void 0 : _a.concatTokens();
-        if (name === undefined) {
+        const nameExpression = node.findDirectExpression(expressions_1.FormName);
+        const name = nameExpression === null || nameExpression === void 0 ? void 0 : nameExpression.concatTokens();
+        if (nameExpression === undefined || name === undefined) {
             const message = "Form, could not find name";
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return;
         }
+        if (name.length > 30) {
+            const message = "FORM name longer than 30 characters";
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, nameExpression.getFirstToken(), message));
+        }
         input.scope.push(_scope_type_1.ScopeType.Form, name, node.getFirstToken().getStart(), input.filename);
         const form = new form_definition_1.FormDefinition(node, input);
         input.scope.addList(form.getUsingParameters());
@@ -40881,7 +41198,7 @@ const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expr
 const source_1 = __webpack_require__(/*! ../expressions/source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
 class Format {
     runSyntax(node, input) {
-        for (const s of node.findAllExpressions(Expressions.Source)) {
+        for (const s of node.findAllExpressionsMulti([Expressions.Source, Expressions.SimpleSource3])) {
             source_1.Source.runSyntax(s, input);
         }
     }
@@ -43706,6 +44023,16 @@ class Parameter {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return;
         }
+        const radioGroup = node.findFirstExpression(Expressions.RadioGroupName);
+        if (radioGroup && radioGroup.concatTokens().length > 4) {
+            const message = "Radio button group name too long, " + radioGroup.concatTokens();
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, radioGroup.getFirstToken(), message));
+        }
+        if (this.hasUserCommand(node) && this.hasLength(node)) {
+            const message = "USER-COMMAND and LENGTH not possible together";
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+            return;
+        }
         const bfound = new basic_types_1.BasicTypes(input).parseType(node);
         if (bfound) {
             input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(nameToken, input.filename, bfound));
@@ -43717,6 +44044,30 @@ class Parameter {
         const magicToken = new tokens_1.Identifier(nameToken.getStart(), magicName);
         input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(magicToken, input.filename, basic_1.VoidType.get("PARAMETER-MAGIC")));
     }
+    // "USER-COMMAND" is lexed as three tokens
+    hasUserCommand(node) {
+        const tokens = node.getTokens();
+        for (let i = 0; i < tokens.length - 2; i++) {
+            if (tokens[i].getStr().toUpperCase() === "USER"
+                && tokens[i + 1].getStr() === "-"
+                && tokens[i + 2].getStr().toUpperCase() === "COMMAND") {
+                return true;
+            }
+        }
+        return false;
+    }
+    // "VISIBLE LENGTH" is not the same as "LENGTH"
+    hasLength(node) {
+        var _a;
+        const tokens = node.getTokens();
+        for (let i = 0; i < tokens.length; i++) {
+            if (tokens[i].getStr().toUpperCase() === "LENGTH"
+                && ((_a = tokens[i - 1]) === null || _a === void 0 ? void 0 : _a.getStr().toUpperCase()) !== "VISIBLE") {
+                return true;
+            }
+        }
+        return false;
+    }
 }
 exports.Parameter = Parameter;
 //# sourceMappingURL=parameter.js.map
@@ -43774,6 +44125,7 @@ const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_module
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 const assert_error_1 = __webpack_require__(/*! ../assert_error */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/assert_error.js");
 const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/dynamic.js");
+const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_type_utils.js");
 class Perform {
     runSyntax(node, input) {
         if (!(node.get() instanceof Statements.Perform)) {
@@ -43781,19 +44133,22 @@ class Perform {
         }
         ////////////////////////////
         // check parameters are defined
+        const changing = [];
         for (const c of node.findDirectExpressions(Expressions.PerformChanging)) {
             for (const s of c.findDirectExpressions(Expressions.Target)) {
-                target_1.Target.runSyntax(s, input);
+                changing.push({ node: s, type: target_1.Target.runSyntax(s, input) });
             }
         }
+        const tables = [];
         for (const t of node.findDirectExpressions(Expressions.PerformTables)) {
             for (const s of t.findDirectExpressions(Expressions.Source)) {
-                source_1.Source.runSyntax(s, input);
+                tables.push({ node: s, type: source_1.Source.runSyntax(s, input) });
             }
         }
+        const using = [];
         for (const u of node.findDirectExpressions(Expressions.PerformUsing)) {
             for (const s of u.findDirectExpressions(Expressions.SimpleSource3)) {
-                source_1.Source.runSyntax(s, input);
+                using.push({ node: s, type: source_1.Source.runSyntax(s, input) });
             }
         }
         ////////////////////////////
@@ -43818,7 +44173,40 @@ class Perform {
             return;
         }
         input.scope.addReference(expr.getFirstToken(), found, _reference_1.ReferenceType.FormReference, input.filename);
-        // todo, also check parameters match
+        ////////////////////////////
+        // check parameters match
+        // USING and CHANGING are interchangeable, the actual parameters form a single positional list
+        if (this.checkParameters(node, tables, found.getTablesParameters(), "TABLES", input) === false) {
+            return;
+        }
+        const formal = found.getUsingParameters().concat(found.getChangingParameters());
+        this.checkParameters(node, using.concat(changing), formal, "USING/CHANGING", input);
+    }
+    // returns false if an issue was reported
+    checkParameters(node, actual, formal, kind, input) {
+        if (actual.length !== formal.length) {
+            const message = `PERFORM, expected ${formal.length} ${kind} parameters, found ${actual.length}`;
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+            return false;
+        }
+        const typeUtils = new _type_utils_1.TypeUtils(input.scope);
+        for (let i = 0; i < actual.length; i++) {
+            const { node: source, type } = actual[i];
+            if (type === undefined) {
+                continue;
+            }
+            const parameter = formal[i];
+            const structureTyping = parameter.getMeta().includes("form_parameter_structure" /* IdentifierMeta.FormParameterStructure */);
+            const assignable = structureTyping
+                ? typeUtils.isAssignableStructureTyping(type, parameter.getType(), source)
+                : typeUtils.isAssignableStrict(type, parameter.getType(), source);
+            if (assignable === false) {
+                const message = `PERFORM parameter type not compatible, ${parameter.getName()}`;
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, source.getFirstToken(), message));
+                return false;
+            }
+        }
+        return true;
     }
 }
 exports.Perform = Perform;
@@ -46765,11 +47153,19 @@ const _typed_identifier_1 = __webpack_require__(/*! ../../types/_typed_identifie
 const identifier_1 = __webpack_require__(/*! ../../1_lexer/tokens/identifier */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/identifier.js");
 const database_table_1 = __webpack_require__(/*! ../expressions/database_table */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/database_table.js");
 const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/dynamic.js");
+const _check_database_fields_1 = __webpack_require__(/*! ../expressions/_check_database_fields */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/_check_database_fields.js");
 class UpdateDatabase {
     runSyntax(node, input) {
         const dbtab = node.findFirstExpression(Expressions.DatabaseTable);
         if (dbtab !== undefined) {
-            database_table_1.DatabaseTable.runSyntax(dbtab, input);
+            const dbSource = database_table_1.DatabaseTable.runSyntax(dbtab, input);
+            const fields = node.findAllExpressions(Expressions.SQLFieldAndValue)
+                .flatMap(fieldAndValue => fieldAndValue.findDirectExpressions(Expressions.SQLFieldName))
+                .map(field => field.concatTokens().toUpperCase());
+            fields.push(...node.findAllExpressions(Expressions.SQLCompare)
+                .map(compare => { var _a; return (_a = compare.findDirectExpression(Expressions.SQLFieldName)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase(); })
+                .filter(field => field !== undefined));
+            (0, _check_database_fields_1.checkDatabaseFields)(fields, [dbSource], input, node.getFirstToken());
         }
         const tableName = node.findDirectExpression(Expressions.DatabaseTable);
         const tokenName = tableName === null || tableName === void 0 ? void 0 : tableName.getFirstToken();
@@ -48043,6 +48439,7 @@ const include_type_1 = __webpack_require__(/*! ../statements/include_type */ "./
 const type_1 = __webpack_require__(/*! ../statements/type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/type.js");
 const Basic = __importStar(__webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js"));
 const _scope_type_1 = __webpack_require__(/*! ../_scope_type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_scope_type.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 class Types {
     runSyntax(node, input, qualifiedNamePrefix) {
         const name = node.findFirstExpression(Expressions.NamespaceSimpleName).getFirstToken();
@@ -48057,6 +48454,9 @@ class Types {
                 if (ctyp instanceof Statements.Type) {
                     const found = new type_1.Type().runSyntax(c, input, qualifiedNamePrefix + name.getStr() + "-");
                     if (found) {
+                        if (found.getName().length > 30) {
+                            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, found.getToken(), "Structure field name longer than 30 characters"));
+                        }
                         components.push({ name: found.getName(), type: found.getType() });
                     }
                 }
@@ -48073,6 +48473,9 @@ class Types {
             else if (c instanceof nodes_1.StructureNode && ctyp instanceof Structures.Types) {
                 const found = new Types().runSyntax(c, input, qualifiedNamePrefix + name.getStr() + "-");
                 if (found) {
+                    if (found.getName().length > 30) {
+                        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, found.getToken(), "Structure field name longer than 30 characters"));
+                    }
                     components.push({ name: found.getName(), type: found.getType() });
                 }
             }
@@ -51784,6 +52187,7 @@ const constants_1 = __webpack_require__(/*! ../5_syntax/structures/constants */
 const type_definitions_1 = __webpack_require__(/*! ./type_definitions */ "./node_modules/@abaplint/core/build/src/abap/types/type_definitions.js");
 const types_1 = __webpack_require__(/*! ../5_syntax/structures/types */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/structures/types.js");
 const type_1 = __webpack_require__(/*! ../5_syntax/statements/type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/type.js");
+const _syntax_input_1 = __webpack_require__(/*! ../5_syntax/_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 const _reference_1 = __webpack_require__(/*! ../5_syntax/_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const alias_1 = __webpack_require__(/*! ./alias */ "./node_modules/@abaplint/core/build/src/abap/types/alias.js");
 class Attributes {
@@ -52005,6 +52409,13 @@ class Attributes {
                 if (foundAttribute) {
                     input.scope.addNamedIdentifier(aliasName.getStr(), foundAttribute);
                 }
+                const component = compName.split("~")[1];
+                if (foundType === undefined
+                    && foundAttribute === undefined
+                    && this.existsInInterface(idef, component, input, []) === false) {
+                    const message = "Component " + component + " not found in interface " + idef.getName();
+                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, compToken, message));
+                }
             }
             else if (this.declaredInterfaces.includes(name.toUpperCase()) || input.scope.getDDIC().inErrorNamespace(name) === false) {
                 input.scope.addReference(compToken, undefined, _reference_1.ReferenceType.ObjectOrientedVoidReference, input.filename);
@@ -52014,6 +52425,33 @@ class Attributes {
             }
         }
     }
+    /** does the component exist in the interface, or in one of its aliases or included interfaces? */
+    existsInInterface(idef, component, input, visited) {
+        const upperName = idef.getName().toUpperCase();
+        if (visited.includes(upperName)) {
+            return false;
+        }
+        visited.push(upperName);
+        const upper = component.toUpperCase();
+        if (idef.getTypeDefinitions().getByName(component) !== undefined
+            || idef.getAttributes().findByName(component) !== undefined
+            || idef.getMethodDefinitions().getByName(component) !== undefined
+            || idef.getEvents().some(e => e.getName().toUpperCase() === upper)
+            || idef.getAliases().some(a => a.getName().toUpperCase() === upper)) {
+            return true;
+        }
+        for (const i of idef.getImplementing()) {
+            const sub = input.scope.findInterfaceDefinition(i.name);
+            if (sub === undefined) {
+                // voided or not found, assume the component exists
+                return true;
+            }
+            else if (this.existsInInterface(sub, component, input, visited)) {
+                return true;
+            }
+        }
+        return false;
+    }
     parseAttribute(node, visibility, input) {
         let found = undefined;
         const s = node.get();
@@ -52173,7 +52611,7 @@ class ClassDefinition extends _identifier_1.Identifier {
         this.checkEventNameConflicts(input);
         this.checkClassNameLength(input);
         this.checkMethodNameLength(input);
-        this.checkClassConstructorStatic(input);
+        this.checkClassConstructor(input);
     }
     getFriends() {
         return this.friends;
@@ -52243,11 +52681,17 @@ class ClassDefinition extends _identifier_1.Identifier {
             }
         }
     }
-    checkClassConstructorStatic(input) {
+    checkClassConstructor(input) {
         for (const m of this.methodDefs.getAll()) {
-            if (m.getName().toUpperCase() === "CLASS_CONSTRUCTOR" && m.isStatic() === false) {
+            if (m.getName().toUpperCase() !== "CLASS_CONSTRUCTOR") {
+                continue;
+            }
+            if (m.isStatic() === false) {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, m.getToken(), "CLASS_CONSTRUCTOR must be static"));
             }
+            else if (m.getVisibility() !== visibility_1.Visibility.Public) {
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, m.getToken(), "CLASS_CONSTRUCTOR must be declared in the public section"));
+            }
         }
     }
     checkInterfaceVisibility(input, node) {
@@ -52639,7 +53083,8 @@ class FormDefinition extends _identifier_1.Identifier {
                 else if (!(type instanceof basic_1.UnknownType) && !(type instanceof basic_1.VoidType)) {
                     type = new basic_1.UnknownType("FORM TABLES type must be table type");
                 }
-                ret.push(new _typed_identifier_1.TypedIdentifier(p.getToken(), input.filename, type, ["form_parameter" /* IdentifierMeta.FormParameter */]));
+                // keep the meta from FormParam, so STRUCTURE typing can be recognized later
+                ret.push(new _typed_identifier_1.TypedIdentifier(p.getToken(), input.filename, type, p.getMeta()));
             }
         }
         return ret;
@@ -53412,6 +53857,7 @@ const _object_oriented_1 = __webpack_require__(/*! ../5_syntax/_object_oriented
 const _reference_1 = __webpack_require__(/*! ../5_syntax/_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const identifier_1 = __webpack_require__(/*! ../1_lexer/tokens/identifier */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/identifier.js");
 const _scope_type_1 = __webpack_require__(/*! ../5_syntax/_scope_type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_scope_type.js");
+const _syntax_input_1 = __webpack_require__(/*! ../5_syntax/_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 // todo:
 // this.exceptions = [];
 // also consider RAISING vs EXCEPTIONS
@@ -53434,6 +53880,7 @@ class MethodParameters {
         input.scope.push(_scope_type_1.ScopeType.MethodDefinition, "method definition", node.getStart(), input.filename);
         try {
             this.parse(node, input, parentName, abstractMethod);
+            this.checkNameLengths(input);
             this.checkDuplicateNames();
         }
         finally {
@@ -53585,6 +54032,13 @@ class MethodParameters {
             names.add(name);
         }
     }
+    checkNameLengths(input) {
+        for (const parameter of this.getAll()) {
+            if (parameter.getName().length > 30) {
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, parameter.getToken(), "Method parameter name longer than 30 characters"));
+            }
+        }
+    }
     workaroundRAP(node, input, parentName) {
         const resultName = node.findExpressionAfterToken("RESULT");
         const concat = node.concatTokens().toUpperCase();
@@ -62875,6 +63329,7 @@ const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node
 const ddic_1 = __webpack_require__(/*! ../ddic */ "./node_modules/@abaplint/core/build/src/ddic.js");
 const Types = __importStar(__webpack_require__(/*! ../abap/types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js"));
 const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@abaplint/core/build/src/xml_utils.js");
+const domain_1 = __webpack_require__(/*! ./domain */ "./node_modules/@abaplint/core/build/src/objects/domain.js");
 class DataElement extends _abstract_object_1.AbstractObject {
     constructor() {
         super(...arguments);
@@ -62905,6 +63360,22 @@ class DataElement extends _abstract_object_1.AbstractObject {
         this.parse();
         return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.domname;
     }
+    /** parseType() cannot distinguish DEC, CURR, and QUAN because they all become PackedType.
+     * Return the original DDIC datatype instead, resolving domain-backed data elements when possible. */
+    getDataType(reg) {
+        var _a, _b;
+        this.parse();
+        if ((_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.datatype) {
+            return this.parsedXML.datatype;
+        }
+        if (reg && ((_b = this.parsedXML) === null || _b === void 0 ? void 0 : _b.refkind) === "D" && this.parsedXML.domname) {
+            const domain = reg.getObject("DOMA", this.parsedXML.domname);
+            if (domain instanceof domain_1.Domain) {
+                return domain.getDataType();
+            }
+        }
+        return undefined;
+    }
     getTexts() {
         var _a;
         this.parse();
@@ -63188,6 +63659,11 @@ class Domain extends _abstract_object_1.AbstractObject {
         var _a;
         return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.conversionExit;
     }
+    getDataType() {
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.datatype;
+    }
     getAllowedNaming() {
         return {
             maxLength: 30,
@@ -70119,7 +70595,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.23";
+        return "2.120.37";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -72783,6 +73259,379 @@ exports.CDSAssociationName = CDSAssociationName;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/cds_check_syntax.js"
+/*!*************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/cds_check_syntax.js ***!
+  \*************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.CDSCheckSyntax = exports.CDSCheckSyntaxConf = void 0;
+const nodes_1 = __webpack_require__(/*! ../abap/nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
+const basic_1 = __webpack_require__(/*! ../abap/types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const expressions_1 = __webpack_require__(/*! ../cds/expressions */ "./node_modules/@abaplint/core/build/src/cds/expressions/index.js");
+const ddic_1 = __webpack_require__(/*! ../ddic */ "./node_modules/@abaplint/core/build/src/ddic.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+class CDSCheckSyntaxConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.CDSCheckSyntaxConf = CDSCheckSyntaxConf;
+const KNOWN_QUAN_DATA_ELEMENTS = ["MENGE_D"];
+const KNOWN_CURR_DATA_ELEMENTS = ["BWERT", "DZWERT"];
+// Reserved names, cannot be used as element names, see DDIC table TRESE
+const RESERVED_ELEMENT_NAMES = ["BEGIN", "NUMBER", "POSITION"];
+const MAX_LABEL_LENGTH = 40;
+// Annotations which are not supported in CDS view entities
+const VIEW_ENTITY_FORBIDDEN_ANNOTATIONS = ["Semantics.unitOfMeasure"];
+class CDSCheckSyntax {
+    constructor() {
+        this.conf = new CDSCheckSyntaxConf();
+    }
+    getMetadata() {
+        return {
+            key: "cds_check_syntax",
+            title: "CDS Check Syntax",
+            shortDescription: "Checks CDS source, field, type, and semantic annotation references",
+            extendedInformation: `Performs semantic checks which require other DDIC objects to be available.
+
+Missing objects are only reported when their names match the configured errorNamespace.`,
+            tags: [_irule_1.RuleTag.Syntax],
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    initialize(reg) {
+        this.reg = reg;
+        return this;
+    }
+    run(object) {
+        if (!(object instanceof objects_1.DataDefinition)) {
+            return [];
+        }
+        const tree = object.getTree();
+        const file = object.findSourceFile();
+        if (tree === undefined || file === undefined) {
+            return [];
+        }
+        const issues = [];
+        const ddic = new ddic_1.DDIC(this.reg);
+        const sources = this.findSources(tree, ddic, issues, file);
+        this.checkRelations(tree, ddic, issues, file);
+        this.checkTypes(tree, ddic, issues, file);
+        this.checkRootProjection(tree, ddic, issues, file);
+        this.checkFieldAnnotations(tree, ddic, issues, file);
+        this.checkLabels(tree, issues, file);
+        this.checkViewEntityAnnotations(tree, issues, file);
+        this.checkReservedNames(tree, issues, file);
+        this.checkFields(tree, sources, object, issues, file);
+        return issues;
+    }
+    findSources(tree, ddic, issues, file) {
+        var _a;
+        const sources = [];
+        for (const node of tree.findAllExpressionsRecursive(expressions_1.CDSSource)) {
+            const prefixed = node.findDirectExpression(expressions_1.CDSPrefixedName);
+            const entityNode = prefixed === null || prefixed === void 0 ? void 0 : prefixed.findDirectExpression(expressions_1.CDSName);
+            if (entityNode === undefined) {
+                continue;
+            }
+            const entity = this.name(entityNode);
+            const asName = (_a = node.findDirectExpression(expressions_1.CDSAs)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(expressions_1.CDSName);
+            const bareAlias = node.findDirectExpression(expressions_1.CDSName);
+            const alias = asName === undefined ? bareAlias : asName;
+            const lookup = ddic.lookupTableOrView(entity);
+            if (lookup.type instanceof basic_1.UnknownType && lookup.object === undefined && this.reg.inErrorNamespace(entity)) {
+                issues.push(issue_1.Issue.atToken(file, entityNode.getFirstToken(), `CDS source "${entity}" not found`, this.getMetadata().key, this.conf.severity));
+            }
+            if (lookup.type instanceof basic_1.StructureType || lookup.type instanceof basic_1.UnknownType || lookup.type instanceof basic_1.VoidType) {
+                const names = [entity.toUpperCase()];
+                if (alias) {
+                    names.push(this.name(alias).toUpperCase());
+                }
+                sources.push({ entity, names, type: lookup.type });
+            }
+        }
+        return sources;
+    }
+    checkRelations(tree, ddic, issues, file) {
+        for (const relation of tree.findAllExpressionsRecursive(expressions_1.CDSRelation)) {
+            const directTokens = relation.getDirectTokens();
+            if (directTokens.length === 0) {
+                continue;
+            }
+            const entity = directTokens.map(t => t.getStr()).join("");
+            const lookup = ddic.lookupTableOrView(entity);
+            if (lookup.type instanceof basic_1.UnknownType && lookup.object === undefined && this.reg.inErrorNamespace(entity)) {
+                issues.push(issue_1.Issue.atToken(file, directTokens[0], `CDS source "${entity}" not found`, this.getMetadata().key, this.conf.severity));
+            }
+        }
+    }
+    checkTypes(tree, ddic, issues, file) {
+        for (const typeNode of tree.findAllExpressionsRecursive(expressions_1.CDSType)) {
+            const typeName = typeNode.findDirectExpressions(expressions_1.CDSName).map(n => this.name(n)).join(".");
+            if (typeName === "" || typeName.toUpperCase().startsWith("ABAP.")) {
+                continue;
+            }
+            const lookup = ddic.lookup(typeName);
+            if (lookup.type instanceof basic_1.UnknownType && lookup.object === undefined && this.reg.inErrorNamespace(typeName)) {
+                issues.push(issue_1.Issue.atToken(file, typeNode.getFirstToken(), `CDS type "${typeName}" not found`, this.getMetadata().key, this.conf.severity));
+            }
+        }
+    }
+    checkRootProjection(tree, ddic, issues, file) {
+        var _a;
+        const projection = tree.findFirstExpression(expressions_1.CDSDefineProjection);
+        const rootToken = projection === null || projection === void 0 ? void 0 : projection.findDirectTokenByText("ROOT");
+        if (projection === undefined || rootToken === undefined) {
+            return;
+        }
+        const names = projection.findDirectExpressions(expressions_1.CDSName);
+        const projectedEntityNode = names[1];
+        if (projectedEntityNode === undefined) {
+            return;
+        }
+        const projectedEntityName = this.name(projectedEntityNode);
+        const projectedEntity = ddic.lookupTableOrView(projectedEntityName).object;
+        if (projectedEntity instanceof objects_1.DataDefinition
+            && ((_a = projectedEntity.getTree()) === null || _a === void 0 ? void 0 : _a.findDirectTokenByText("ROOT")) === undefined) {
+            const message = `ROOT keyword not valid since ${projectedEntityName} is not a root property`;
+            issues.push(issue_1.Issue.atToken(file, rootToken, message, this.getMetadata().key, this.conf.severity));
+        }
+    }
+    checkFieldAnnotations(tree, ddic, issues, file) {
+        const definition = tree.findFirstExpression(expressions_1.CDSDefineAbstract);
+        if (definition === undefined) {
+            return;
+        }
+        let definitionNameSeen = false;
+        let fieldName;
+        let annotations = [];
+        const fields = [];
+        for (const child of definition.getChildren()) {
+            if (child instanceof nodes_1.TokenNode) {
+                if (child.get().getStr() === ";") {
+                    fieldName = undefined;
+                    annotations = [];
+                }
+                continue;
+            }
+            else if (child.get() instanceof expressions_1.CDSAnnotation) {
+                annotations.push(child);
+            }
+            else if (child.get() instanceof expressions_1.CDSName) {
+                if (definitionNameSeen === false) {
+                    definitionNameSeen = true;
+                    annotations = [];
+                }
+                else {
+                    fieldName = child;
+                }
+            }
+            else if (child.get() instanceof expressions_1.CDSType && fieldName !== undefined) {
+                const semanticType = this.determineSemanticType(child, ddic);
+                const normalizedAnnotations = annotations.map(a => a.concatTokens().replace(/\s/g, ""));
+                fields.push({ name: this.name(fieldName), node: fieldName, annotations: normalizedAnnotations, semanticType });
+                fieldName = undefined;
+                annotations = [];
+            }
+        }
+        const reportedCompanionFields = new Set();
+        for (const field of fields) {
+            if (field.semanticType === undefined) {
+                continue;
+            }
+            const quantity = field.semanticType === "QUAN";
+            const referenceAnnotation = quantity ? "@Semantics.quantity.unitOfMeasure" : "@Semantics.amount.currencyCode";
+            const companionAnnotation = quantity ? "@Semantics.unitOfMeasure: true" : "@Semantics.currencyCode: true";
+            const reference = this.findSemanticReference(field.annotations, field.semanticType);
+            if (reference === undefined) {
+                const kind = quantity ? "quantity" : "amount";
+                const message = `CDS ${kind} field "${field.name}" requires ${referenceAnnotation}`;
+                issues.push(issue_1.Issue.atToken(file, field.node.getFirstToken(), message, this.getMetadata().key, this.conf.severity));
+                continue;
+            }
+            const companion = fields.find(f => f.name.toUpperCase() === reference.toUpperCase());
+            if (companion === undefined) {
+                const kind = quantity ? "unit" : "currency";
+                const message = `CDS field "${field.name}" references missing ${kind} field "${reference}"`;
+                issues.push(issue_1.Issue.atToken(file, field.node.getFirstToken(), message, this.getMetadata().key, this.conf.severity));
+                continue;
+            }
+            const companionKey = field.semanticType + ":" + companion.name.toUpperCase();
+            if (reportedCompanionFields.has(companionKey) === false
+                && this.hasCompanionAnnotation(companion.annotations, field.semanticType) === false) {
+                const message = `CDS field "${companion.name}" requires ${companionAnnotation}`;
+                issues.push(issue_1.Issue.atToken(file, companion.node.getFirstToken(), message, this.getMetadata().key, this.conf.severity));
+                reportedCompanionFields.add(companionKey);
+            }
+        }
+    }
+    determineSemanticType(typeNode, ddic) {
+        var _a;
+        const typeName = typeNode.findDirectExpressions(expressions_1.CDSName).map(n => this.name(n)).join(".");
+        const upper = typeName.toUpperCase();
+        if (upper === "ABAP.QUAN" || KNOWN_QUAN_DATA_ELEMENTS.includes(upper)) {
+            return "QUAN";
+        }
+        else if (upper === "ABAP.CURR" || KNOWN_CURR_DATA_ELEMENTS.includes(upper)) {
+            return "CURR";
+        }
+        const object = ddic.lookup(typeName).object;
+        if (object instanceof objects_1.DataElement) {
+            const dataType = (_a = object.getDataType(this.reg)) === null || _a === void 0 ? void 0 : _a.toUpperCase();
+            if (dataType === "QUAN" || dataType === "CURR") {
+                return dataType;
+            }
+        }
+        return undefined;
+    }
+    findSemanticReference(annotations, type) {
+        const expression = type === "QUAN"
+            ? /(?:SEMANTICS\.QUANTITY|SEMANTICS:\{QUANTITY:\{)\.?UNITOFMEASURE:'([^']+)'/i
+            : /(?:SEMANTICS\.AMOUNT|SEMANTICS:\{AMOUNT:\{)\.?CURRENCYCODE:'([^']+)'/i;
+        for (const annotation of annotations) {
+            const match = annotation.match(expression);
+            if (match) {
+                return match[1];
+            }
+        }
+        return undefined;
+    }
+    hasCompanionAnnotation(annotations, type) {
+        if (type === "QUAN") {
+            return annotations.some(a => a.toUpperCase().includes("@SEMANTICS.UNITOFMEASURE:TRUE")
+                || a.toUpperCase().includes("@SEMANTICS:{UNITOFMEASURE:TRUE"));
+        }
+        else {
+            return annotations.some(a => a.toUpperCase().includes("@SEMANTICS.CURRENCYCODE:TRUE")
+                || a.toUpperCase().includes("@SEMANTICS:{CURRENCYCODE:TRUE"));
+        }
+    }
+    checkLabels(tree, issues, file) {
+        for (const annotation of tree.findAllExpressionsRecursive(expressions_1.CDSAnnotation)) {
+            const concatenated = annotation.concatTokens();
+            if (/@\s*<?\s*EndUserText/i.test(concatenated) === false) {
+                continue;
+            }
+            const expression = /label\s*:\s*'((?:[^']|'')*)'/gi;
+            let match = expression.exec(concatenated);
+            while (match !== null) {
+                const label = match[1].replace(/''/g, "'");
+                if (label.length > MAX_LABEL_LENGTH) {
+                    const message = `CDS label is ${label.length} characters, maximum is ${MAX_LABEL_LENGTH}`;
+                    issues.push(issue_1.Issue.atToken(file, annotation.getFirstToken(), message, this.getMetadata().key, this.conf.severity));
+                }
+                match = expression.exec(concatenated);
+            }
+        }
+    }
+    checkViewEntityAnnotations(tree, issues, file) {
+        const view = tree.findFirstExpression(expressions_1.CDSDefineView);
+        if (view === undefined || view.findDirectTokenByText("ENTITY") === undefined) {
+            return;
+        }
+        for (const annotation of tree.findAllExpressionsRecursive(expressions_1.CDSAnnotation)) {
+            const normalized = annotation.concatTokens().replace(/\s/g, "").toUpperCase().replace(/^@</, "@");
+            for (const forbidden of VIEW_ENTITY_FORBIDDEN_ANNOTATIONS) {
+                const upper = forbidden.toUpperCase();
+                if (normalized.startsWith("@" + upper) === false
+                    && normalized.startsWith("@" + upper.replace(".", ":{")) === false) {
+                    continue;
+                }
+                const message = `Annotation ${forbidden} is not allowed in view entities`;
+                issues.push(issue_1.Issue.atToken(file, annotation.getFirstToken(), message, this.getMetadata().key, this.conf.severity));
+            }
+        }
+    }
+    checkReservedNames(tree, issues, file) {
+        var _a, _b;
+        for (const element of tree.findAllExpressionsRecursive(expressions_1.CDSElement)) {
+            if (element.findDirectTokenByText("INCLUDE") !== undefined) {
+                continue;
+            }
+            let nameNode = (_a = element.findDirectExpression(expressions_1.CDSAs)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(expressions_1.CDSName);
+            if (nameNode === undefined) {
+                const names = ((_b = element.findDirectExpression(expressions_1.CDSPrefixedName)) === null || _b === void 0 ? void 0 : _b.findDirectExpressions(expressions_1.CDSName)) || [];
+                nameNode = names[names.length - 1];
+            }
+            if (nameNode === undefined) {
+                continue;
+            }
+            const name = this.name(nameNode);
+            if (RESERVED_ELEMENT_NAMES.includes(name.toUpperCase())) {
+                issues.push(issue_1.Issue.atToken(file, nameNode.getFirstToken(), `CDS element name "${name}" is reserved`, this.getMetadata().key, this.conf.severity));
+            }
+        }
+    }
+    checkFields(tree, sources, object, issues, file) {
+        var _a;
+        const associationNames = new Set();
+        for (const association of ((_a = object.getParsedData()) === null || _a === void 0 ? void 0 : _a.associations) || []) {
+            associationNames.add(association.name.toUpperCase());
+            if (association.as) {
+                associationNames.add(association.as.toUpperCase());
+            }
+        }
+        for (const element of tree.findAllExpressionsRecursive(expressions_1.CDSElement)) {
+            const prefixed = element.findDirectExpression(expressions_1.CDSPrefixedName);
+            if (prefixed === undefined) {
+                continue;
+            }
+            const names = prefixed.findDirectExpressions(expressions_1.CDSName);
+            if (names.length === 0) {
+                continue;
+            }
+            const first = this.name(names[0]);
+            if (first === "*" || first.startsWith("$") || first.startsWith("#") || associationNames.has(first.toUpperCase())) {
+                continue;
+            }
+            if (names.length > 1) {
+                const matchingSources = sources.filter(s => s.names.includes(first.toUpperCase()));
+                if (matchingSources.length !== 1) {
+                    continue;
+                }
+                const source = matchingSources[0];
+                const sourceType = source.type;
+                if (!(sourceType instanceof basic_1.StructureType)) {
+                    continue;
+                }
+                const field = this.name(names[1]);
+                if (field !== "*" && sourceType.getComponentByName(field) === undefined) {
+                    issues.push(issue_1.Issue.atToken(file, names[1].getFirstToken(), `CDS field "${field}" not found in "${source.entity}"`, this.getMetadata().key, this.conf.severity));
+                }
+                continue;
+            }
+            // Do not guess across joins or when a dependency is absent.
+            if (sources.length !== 1) {
+                continue;
+            }
+            const source = sources[0];
+            const sourceType = source.type;
+            if (!(sourceType instanceof basic_1.StructureType)) {
+                continue;
+            }
+            if (sourceType.getComponentByName(first) === undefined) {
+                issues.push(issue_1.Issue.atToken(file, names[0].getFirstToken(), `CDS field "${first}" not found in "${source.entity}"`, this.getMetadata().key, this.conf.severity));
+            }
+        }
+    }
+    name(node) {
+        return node.concatTokens().replace(/ /g, "");
+    }
+}
+exports.CDSCheckSyntax = CDSCheckSyntax;
+//# sourceMappingURL=cds_check_syntax.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/cds_comment_style.js"
 /*!**************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/cds_comment_style.js ***!
@@ -72950,7 +73799,8 @@ class CDSFieldOrder {
     }
     getAssociationNames(tree) {
         const names = new Set();
-        for (const assoc of tree.findAllExpressions(expressions_1.CDSAssociation)) {
+        const found = [...tree.findAllExpressions(expressions_1.CDSAssociation), ...tree.findAllExpressions(expressions_1.CDSComposition)];
+        for (const assoc of found) {
             const relation = assoc.findDirectExpression(expressions_1.CDSRelation);
             if (relation === undefined) {
                 continue;
@@ -73068,7 +73918,6 @@ const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/c
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
 const expressions_1 = __webpack_require__(/*! ../cds/expressions */ "./node_modules/@abaplint/core/build/src/cds/expressions/index.js");
-const cds_name_1 = __webpack_require__(/*! ../cds/expressions/cds_name */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_name.js");
 class CDSNamingConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -73140,7 +73989,7 @@ https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/ee6ff9b281d8448f96b4fe6c89f2bdc8
         if (file === undefined) {
             return [];
         }
-        const name = this.getCDSName(tree);
+        const name = o.getDefinitionName();
         if (name === undefined) {
             return [];
         }
@@ -73157,13 +74006,6 @@ https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/ee6ff9b281d8448f96b4fe6c89f2bdc8
         }
         return [];
     }
-    getCDSName(tree) {
-        const nameNodes = tree.findDirectExpressions(cds_name_1.CDSName);
-        if (nameNodes.length > 0) {
-            return nameNodes[0].getFirstToken().getStr();
-        }
-        return undefined;
-    }
     getAllowedPrefixes(tree) {
         const root = tree.get();
         if (root instanceof expressions_1.CDSExtendView) {
@@ -75185,6 +76027,7 @@ class CloudTypes {
             || obj instanceof Objects.BehaviorDefinition
             || obj instanceof Objects.BusinessCatalog
             || obj instanceof Objects.BusinessCatalogAppAssignment
+            || obj instanceof Objects.BusinessConfigurationMaintenanceObject
             || obj instanceof Objects.CDSMetadataExtension
             || obj instanceof Objects.Class
             || obj instanceof Objects.CDSType
@@ -75201,11 +76044,13 @@ class CloudTypes {
             || obj instanceof Objects.EventBinding
             || obj instanceof Objects.EventConsumer
             || obj instanceof Objects.FunctionGroup
+            || obj instanceof Objects.GatewayODataGroupAndAssignment
             || obj instanceof Objects.HttpService
             || obj instanceof Objects.IAMApp
             || obj instanceof Objects.InboundService
             || obj instanceof Objects.Interface
             || obj instanceof Objects.LockObject
+            || obj instanceof Objects.MaintenanceAndTransportObject
             || obj instanceof Objects.MessageClass
             || obj instanceof Objects.NumberRange
             || obj instanceof Objects.OutboundService
@@ -83877,6 +84722,7 @@ __exportStar(__webpack_require__(/*! ./begin_single_include */ "./node_modules/@
 __exportStar(__webpack_require__(/*! ./call_transaction_authority_check */ "./node_modules/@abaplint/core/build/src/rules/call_transaction_authority_check.js"), exports);
 __exportStar(__webpack_require__(/*! ./catch_and_raise */ "./node_modules/@abaplint/core/build/src/rules/catch_and_raise.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_association_name */ "./node_modules/@abaplint/core/build/src/rules/cds_association_name.js"), exports);
+__exportStar(__webpack_require__(/*! ./cds_check_syntax */ "./node_modules/@abaplint/core/build/src/rules/cds_check_syntax.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_comment_style */ "./node_modules/@abaplint/core/build/src/rules/cds_comment_style.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_field_order */ "./node_modules/@abaplint/core/build/src/rules/cds_field_order.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_legacy_view */ "./node_modules/@abaplint/core/build/src/rules/cds_legacy_view.js"), exports);
@@ -83970,6 +84816,7 @@ __exportStar(__webpack_require__(/*! ./newline_between_methods */ "./node_module
 __exportStar(__webpack_require__(/*! ./no_aliases */ "./node_modules/@abaplint/core/build/src/rules/no_aliases.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_chained_assignment */ "./node_modules/@abaplint/core/build/src/rules/no_chained_assignment.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_comments_between_methods */ "./node_modules/@abaplint/core/build/src/rules/no_comments_between_methods.js"), exports);
+__exportStar(__webpack_require__(/*! ./no_dynamic_stuff */ "./node_modules/@abaplint/core/build/src/rules/no_dynamic_stuff.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_external_form_calls */ "./node_modules/@abaplint/core/build/src/rules/no_external_form_calls.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_inline_in_optional_branches */ "./node_modules/@abaplint/core/build/src/rules/no_inline_in_optional_branches.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_macros */ "./node_modules/@abaplint/core/build/src/rules/no_macros.js"), exports);
@@ -84049,6 +84896,7 @@ __exportStar(__webpack_require__(/*! ./when_others_last */ "./node_modules/@abap
 __exportStar(__webpack_require__(/*! ./whitespace_end */ "./node_modules/@abaplint/core/build/src/rules/whitespace_end.js"), exports);
 __exportStar(__webpack_require__(/*! ./xml_consistency */ "./node_modules/@abaplint/core/build/src/rules/xml_consistency.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_exclamation_escape */ "./node_modules/@abaplint/core/build/src/rules/no_exclamation_escape.js"), exports);
+__exportStar(__webpack_require__(/*! ./xml_bom */ "./node_modules/@abaplint/core/build/src/rules/xml_bom.js"), exports);
 __exportStar(__webpack_require__(/*! ./no_exclamation_escape */ "./node_modules/@abaplint/core/build/src/rules/no_exclamation_escape.js"), exports);
 //# sourceMappingURL=index.js.map
 
@@ -88082,6 +88930,250 @@ exports.NoCommentsBetweenMethods = NoCommentsBetweenMethods;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/no_dynamic_stuff.js"
+/*!*************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/no_dynamic_stuff.js ***!
+  \*************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.NoDynamicStuff = exports.NoDynamicStuffConf = void 0;
+const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js"));
+const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
+const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+class NoDynamicStuffConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        /** Detects dynamic method calls, ie. CALL METHOD (name), SET HANDLER and CALL BADI */
+        this.callMethod = true;
+        /** Detects dynamic CALL FUNCTION, ie. the function module name is not a literal */
+        this.callFunction = true;
+        /** Detects CALL DATABASE PROCEDURE, the procedure name is always dynamic */
+        this.callDatabaseProcedure = true;
+        /** Detects dynamic CALL TRANSFORMATION */
+        this.callTransformation = true;
+        /** Detects dynamic CALL TRANSACTION, ie. the transaction code is not a literal */
+        this.callTransaction = true;
+        /** Detects dynamic PERFORM */
+        this.perform = true;
+        /** Detects dynamic SUBMIT */
+        this.submit = true;
+        /** Detects dynamic CREATE OBJECT */
+        this.createObject = true;
+        /** Detects dynamic CREATE DATA */
+        this.createData = true;
+        /** Detects dynamic GET BADI */
+        this.getBadi = true;
+        /** Detects dynamic ASSIGN, including ASSIGN COMPONENT */
+        this.assign = true;
+        /** Detects dynamic internal table access, ie. dynamic WHERE, keys, SORT BY and TRANSPORTING */
+        this.internalTable = true;
+        /** Detects dynamic EXPORT and IMPORT */
+        this.exportImport = true;
+    }
+}
+exports.NoDynamicStuffConf = NoDynamicStuffConf;
+class NoDynamicStuff extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new NoDynamicStuffConf();
+    }
+    getMetadata() {
+        return {
+            key: "no_dynamic_stuff",
+            title: "No dynamic stuff",
+            shortDescription: `Detects dynamic calls and other dynamic language constructs`,
+            extendedInformation: `Dynamic constructs cannot be checked statically, they are not found by
+where-used lists and refactorings, and they can introduce injection vulnerabilities.
+
+Dynamic tokens are also reported when containing a literal, eg. CALL METHOD go_calendar->('RESET_DAY_INFO'),
+the syntax check does not resolve these either, so they behave like any other dynamic token.
+
+Dynamic SQL is reported by rule dangerous_statement.`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Security],
+            badExample: `CALL METHOD (lv_class)=>(lv_method).
+CALL METHOD go_calendar->('RESET_DAY_INFO').
+CALL FUNCTION lv_function_module.
+CREATE OBJECT ref TYPE (lv_class).
+ASSIGN (lv_name) TO <fs>.
+SORT tab BY (lv_field).`,
+            goodExample: `cl_class=>method( ).
+go_calendar->reset_day_info( ).
+CALL FUNCTION 'ZFOO'.
+CREATE OBJECT ref TYPE cl_class.
+ASSIGN foo TO <fs>.
+SORT tab BY field.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        for (const statementNode of file.getStatements()) {
+            const found = this.check(statementNode);
+            if (found !== undefined) {
+                issues.push(issue_1.Issue.atStatement(file, statementNode, "Dynamic " + found, this.getMetadata().key, this.conf.severity));
+            }
+        }
+        return issues;
+    }
+    check(node) {
+        const statement = node.get();
+        // note that MethodSource is also used by SET HANDLER and CALL BADI
+        if (this.conf.callMethod === true && this.dynamicMethodSource(node) === true) {
+            return "method call";
+        }
+        if (statement instanceof Statements.CallFunction) {
+            if (this.conf.callFunction === true
+                && this.notLiteral(node.findDirectExpression(Expressions.FunctionName))) {
+                return "CALL FUNCTION";
+            }
+        }
+        else if (statement instanceof Statements.CallDatabase) {
+            if (this.conf.callDatabaseProcedure === true) {
+                return "CALL DATABASE PROCEDURE";
+            }
+        }
+        else if (statement instanceof Statements.CallTransformation) {
+            if (this.conf.callTransformation === true && this.hasDynamic(node)) {
+                return "CALL TRANSFORMATION";
+            }
+        }
+        else if (statement instanceof Statements.CallTransaction) {
+            // the transaction code is a Source, ie. there is no parenthesized dynamic variant
+            if (this.conf.callTransaction === true
+                && this.notLiteral(node.findDirectExpression(Expressions.Source))) {
+                return "CALL TRANSACTION";
+            }
+        }
+        else if (statement instanceof Statements.Perform) {
+            if (this.conf.perform === true && this.hasDynamic(node)) {
+                return "PERFORM";
+            }
+        }
+        else if (statement instanceof Statements.Submit) {
+            if (this.conf.submit === true && this.hasDynamic(node)) {
+                return "SUBMIT";
+            }
+        }
+        else if (statement instanceof Statements.CreateObject) {
+            if (this.conf.createObject === true && this.hasDynamic(node)) {
+                return "CREATE OBJECT";
+            }
+        }
+        else if (statement instanceof Statements.CreateData) {
+            if (this.conf.createData === true && this.hasDynamic(node)) {
+                return "CREATE DATA";
+            }
+        }
+        else if (statement instanceof Statements.GetBadi) {
+            if (this.conf.getBadi === true && this.hasDynamic(node)) {
+                return "GET BADI";
+            }
+        }
+        else if (statement instanceof Statements.Assign
+            || statement instanceof Statements.AssignLocalCopy) {
+            if (this.conf.assign === true
+                && (this.hasDynamic(node) || this.dynamicAssignComponent(node))) {
+                return "ASSIGN";
+            }
+        }
+        else if (statement instanceof Statements.Export) {
+            if (this.conf.exportImport === true && this.hasDynamic(node)) {
+                return "EXPORT";
+            }
+        }
+        else if (statement instanceof Statements.Import) {
+            if (this.conf.exportImport === true && this.hasDynamic(node)) {
+                return "IMPORT";
+            }
+        }
+        else if (statement instanceof Statements.At
+            || statement instanceof Statements.DeleteInternal
+            || statement instanceof Statements.InsertInternal
+            || statement instanceof Statements.Loop
+            || statement instanceof Statements.ModifyInternal
+            || statement instanceof Statements.ReadTable
+            || statement instanceof Statements.Sort
+            || statement instanceof Statements.SortDataset) {
+            if (this.conf.internalTable === true && this.hasDynamic(node)) {
+                return "internal table access";
+            }
+        }
+        return undefined;
+    }
+    dynamicMethodSource(node) {
+        for (const source of node.findAllExpressions(Expressions.MethodSource)) {
+            // Dynamic is always a direct child of MethodSource, dynamics further down
+            // are parameters of the method call and not part of the method name
+            if (source.findDirectExpression(Expressions.Dynamic) !== undefined) {
+                return true;
+            }
+        }
+        return false;
+    }
+    dynamicAssignComponent(node) {
+        var _a;
+        const component = (_a = node.findDirectExpression(Expressions.AssignSource)) === null || _a === void 0 ? void 0 : _a.findExpressionAfterToken("COMPONENT");
+        return component !== undefined && this.isLiteral(component) === false;
+    }
+    hasDynamic(node) {
+        return node.findFirstExpression(Expressions.Dynamic) !== undefined;
+    }
+    notLiteral(node) {
+        return node !== undefined && this.isLiteral(node) === false;
+    }
+    isLiteral(node) {
+        return node.findDirectExpression(Expressions.Constant) !== undefined;
+    }
+}
+exports.NoDynamicStuff = NoDynamicStuff;
+//# sourceMappingURL=no_dynamic_stuff.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/no_exclamation_escape.js"
 /*!******************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/no_exclamation_escape.js ***!
@@ -99275,6 +100367,64 @@ exports.WhitespaceEnd = WhitespaceEnd;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/xml_bom.js"
+/*!****************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/xml_bom.js ***!
+  \****************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.XMLBOM = exports.XMLBOMConf = void 0;
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+class XMLBOMConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.XMLBOMConf = XMLBOMConf;
+class XMLBOM {
+    constructor() {
+        this.conf = new XMLBOMConf();
+    }
+    getMetadata() {
+        return {
+            key: "xml_bom",
+            title: "XML UTF-8 byte order mark",
+            shortDescription: "Checks that XML files start with a UTF-8 byte order mark",
+            extendedInformation: "Checks that each XML file has a UTF-8 byte order mark (BOM) at its beginning.",
+            tags: [_irule_1.RuleTag.Syntax, _irule_1.RuleTag.Quickfix],
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    initialize(_reg) {
+        return this;
+    }
+    run(obj) {
+        const file = obj.getXMLFile();
+        if (file === undefined || file.getRaw().startsWith("\uFEFF")) {
+            return [];
+        }
+        return [this.createIssue(file)];
+    }
+    createIssue(file) {
+        const start = new position_1.Position(1, 1);
+        const fix = edit_helper_1.EditHelper.insertAt(file, start, "\uFEFF");
+        return issue_1.Issue.atRange(file, start, start, "XML file must start with a UTF-8 byte order mark", this.getMetadata().key, this.conf.severity, fix);
+    }
+}
+exports.XMLBOM = XMLBOM;
+//# sourceMappingURL=xml_bom.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/xml_consistency.js"
 /*!************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/xml_consistency.js ***!
@@ -100007,6 +101157,10 @@ exports.DECLARATION_STUFF = [
     Statements.DataBegin,
     Statements.Constant,
     Statements.ConstantBegin,
+    Statements.Static,
+    Statements.StaticBegin,
+    Statements.FieldSymbol,
+    Statements.Ranges,
     Statements.Tables,
     Statements.Include, // this is not super correct, but anyhow
     Statements.Parameter,
@@ -105776,8 +106930,13 @@ class SQLCondTranspiler {
     sqlSource(source, traversal, filename, table) {
         let ret = "";
         const simple = source.findDirectExpression(abaplint.Expressions.SimpleSource3);
+        const hostExpression = source.findDirectExpression(abaplint.Expressions.Source);
         const alias = source.findDirectExpression(abaplint.Expressions.SQLAliasField);
-        if (simple && simple.findDirectExpression(abaplint.Expressions.Constant) === undefined) {
+        if (source.getFirstToken().getStr() === "@" && hostExpression) {
+            const code = new source_1.SourceTranspiler(true).transpile(hostExpression, traversal).getCode();
+            ret += "'\" + " + code + " + \"'";
+        }
+        else if (simple && simple.findDirectExpression(abaplint.Expressions.Constant) === undefined) {
             ret += "'\" + " + new simple_source3_1.SimpleSource3Transpiler(true).transpile(simple, traversal).getCode() + " + \"'";
         }
         else if (alias) {
@@ -109873,7 +111032,35 @@ class AuthorityCheckTranspiler {
     transpile(node, traversal) {
         const lookup = traversal.lookupClassOrInterface("KERNEL_AUTHORITY_CHECK", node.getFirstToken());
         const options = [];
-        // todo
+        const object = node.findExpressionAfterToken("OBJECT");
+        if (object) {
+            options.push("object: " + traversal.traverse(object).getCode());
+        }
+        const user = node.findExpressionAfterToken("USER");
+        if (user) {
+            options.push("user: " + traversal.traverse(user).getCode());
+        }
+        const fields = [];
+        const children = node.getChildren();
+        for (let i = 0; i < children.length; i++) {
+            if (children[i].concatTokens().toUpperCase() !== "ID") {
+                continue;
+            }
+            const id = children[i + 1];
+            const addition = children[i + 2]?.concatTokens().toUpperCase();
+            if (id === undefined) {
+                continue;
+            }
+            const field = ["id: " + traversal.traverse(id).getCode()];
+            if (addition === "FIELD" && children[i + 3] !== undefined) {
+                field.push("field: " + traversal.traverse(children[i + 3]).getCode());
+            }
+            else if (addition === "DUMMY") {
+                field.push("dummy: true");
+            }
+            fields.push("{" + field.join(", ") + "}");
+        }
+        options.push("fields: [" + fields.join(", ") + "]");
         const call = `await ${lookup}.call({${options.join(",")}});`;
         return new chunk_1.Chunk().append(`if (${lookup} === undefined) throw new Error("AuthorityCheck, kernel class missing");\n${call}`, node, traversal);
     }
@@ -113787,6 +114974,7 @@ __exportStar(__webpack_require__(/*! ./generate_subroutine */ "./node_modules/@a
 __exportStar(__webpack_require__(/*! ./set_screen */ "./node_modules/@abaplint/transpiler/build/src/statements/set_screen.js"), exports);
 __exportStar(__webpack_require__(/*! ./set_titlebar */ "./node_modules/@abaplint/transpiler/build/src/statements/set_titlebar.js"), exports);
 __exportStar(__webpack_require__(/*! ./shift */ "./node_modules/@abaplint/transpiler/build/src/statements/shift.js"), exports);
+__exportStar(__webpack_require__(/*! ./skip */ "./node_modules/@abaplint/transpiler/build/src/statements/skip.js"), exports);
 __exportStar(__webpack_require__(/*! ./sort_dataset */ "./node_modules/@abaplint/transpiler/build/src/statements/sort_dataset.js"), exports);
 __exportStar(__webpack_require__(/*! ./sort */ "./node_modules/@abaplint/transpiler/build/src/statements/sort.js"), exports);
 __exportStar(__webpack_require__(/*! ./split */ "./node_modules/@abaplint/transpiler/build/src/statements/split.js"), exports);
@@ -116548,7 +117736,10 @@ class SelectTranspiler {
         else if (node.findFirstExpression(abaplint.Expressions.SQLIntoStructure)) {
             target = traversal.traverse(node.findFirstExpression(abaplint.Expressions.SQLIntoStructure)).getCode();
         }
-        let select = "SELECT ";
+        const tokens = node.getTokens();
+        const isDistinct = tokens[0]?.getStr().toUpperCase() === "SELECT"
+            && tokens[1]?.getStr().toUpperCase() === "DISTINCT";
+        let select = isDistinct ? "SELECT DISTINCT " : "SELECT ";
         const fieldList = node.findFirstExpression(abaplint.Expressions.SQLFieldList)
             || node.findFirstExpression(abaplint.Expressions.SQLFieldListLoop);
         if (fieldList === undefined) {
@@ -117207,6 +118398,68 @@ exports.ShiftTranspiler = ShiftTranspiler;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/transpiler/build/src/statements/skip.js"
+/*!************************************************************************!*\
+  !*** ./node_modules/@abaplint/transpiler/build/src/statements/skip.js ***!
+  \************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SkipTranspiler = void 0;
+const abaplint = __importStar(__webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js"));
+const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
+class SkipTranspiler {
+    transpile(node, traversal) {
+        const source = node.findDirectExpression(abaplint.Expressions.Source);
+        if (source === undefined) {
+            return new chunk_1.Chunk().append("abap.statements.skip();", node, traversal);
+        }
+        const option = node.concatTokens().toUpperCase().includes("TO LINE") ? "toLine" : "lines";
+        return new chunk_1.Chunk().append("abap.statements.skip({" + option + ": ", node, traversal)
+            .appendChunk(traversal.traverse(source))
+            .append("});", node.getLastToken(), traversal);
+    }
+}
+exports.SkipTranspiler = SkipTranspiler;
+//# sourceMappingURL=skip.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/transpiler/build/src/statements/sort.js"
 /*!************************************************************************!*\
   !*** ./node_modules/@abaplint/transpiler/build/src/statements/sort.js ***!
@@ -119460,6 +120713,7 @@ const traversal_1 = __webpack_require__(/*! ../traversal */ "./node_modules/@aba
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
 const transpile_types_1 = __webpack_require__(/*! ../transpile_types */ "./node_modules/@abaplint/transpiler/build/src/transpile_types.js");
 const unique_identifier_1 = __webpack_require__(/*! ../unique_identifier */ "./node_modules/@abaplint/transpiler/build/src/unique_identifier.js");
+const inline_1 = __webpack_require__(/*! ../inline */ "./node_modules/@abaplint/transpiler/build/src/inline.js");
 class FunctionModuleTranspiler {
     transpile(node, traversal) {
         const chunk = new chunk_1.Chunk();
@@ -119472,6 +120726,7 @@ class FunctionModuleTranspiler {
                 }
                 chunk.append(`async function ${traversal_1.Traversal.escapeNamespace(name)}(INPUT) {\n`, c, traversal);
                 chunk.appendString(this.findSignature(traversal, name, c));
+                chunk.appendString(inline_1.InlineDeclarations.buildDeclarations(node, traversal));
             }
             else if (c.get() instanceof abaplint.Statements.EndFunction) {
                 chunk.append("}\n", c, traversal);