@abaplint/transpiler-cli 2.13.1 → 2.13.3

package/build/bundle.js

@@ -14070,7 +14070,7 @@ const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/co
 const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 class MethodDef {
     getMatcher() {
-        const def = (0, combi_1.ver)(version_1.Version.v740sp08, (0, combi_1.seq)("DEFAULT", (0, combi_1.altPrio)("FAIL", "IGNORE")));
+        const def = (0, combi_1.ver)(version_1.Version.v740sp08, (0, combi_1.seq)("DEFAULT", (0, combi_1.altPrio)("FAIL", "IGNORE")), version_1.Version.OpenABAP);
         const parameters = (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.altPrio)("FINAL", def, expressions_1.Abstract)), (0, combi_1.optPrio)(expressions_1.MethodDefImporting), (0, combi_1.optPrio)(expressions_1.MethodDefExporting), (0, combi_1.optPrio)(expressions_1.MethodDefChanging), (0, combi_1.optPrio)(expressions_1.MethodDefReturning), (0, combi_1.optPrio)((0, combi_1.altPrio)(expressions_1.MethodDefRaising, expressions_1.MethodDefExceptions)));
         const testing = (0, combi_1.seq)((0, combi_1.optPrio)(expressions_1.Abstract), "FOR TESTING", (0, combi_1.optPrio)((0, combi_1.altPrio)(expressions_1.MethodDefRaising, expressions_1.MethodDefExceptions)));
         const result = (0, combi_1.seq)("RESULT", expressions_1.MethodParamName);
@@ -15217,7 +15217,9 @@ class ReadTable {
         const key = (0, combi_1.seq)((0, combi_1.altPrio)("WITH KEY", "WITH TABLE KEY"), (0, combi_1.alt)(expressions_1.ComponentCompareSimple, components, (0, combi_1.seq)((0, combi_1.optPrio)("="), expressions_1.Source)));
         const using = (0, combi_1.seq)("USING KEY", (0, combi_1.alt)(expressions_1.Field, expressions_1.Dynamic));
         const from = (0, combi_1.seq)("FROM", expressions_1.Source);
-        const perm = (0, combi_1.per)((0, combi_1.alt)(index, key, from), expressions_1.ReadTableTarget, using, comparing, "CASTING", (0, combi_1.seq)("TRANSPORTING", (0, combi_1.altPrio)("ALL FIELDS", "NO FIELDS", transporting_fields_1.TransportingFields)), "BINARY SEARCH");
+        const transporting = (0, combi_1.seq)("TRANSPORTING", (0, combi_1.altPrio)("ALL FIELDS", "NO FIELDS", transporting_fields_1.TransportingFields));
+        const common = [expressions_1.ReadTableTarget, comparing, "CASTING", transporting, "BINARY SEARCH"];
+        const perm = (0, combi_1.alt)((0, combi_1.per)((0, combi_1.alt)(index, from), using, ...common), (0, combi_1.per)(key, ...common), (0, combi_1.per)(...common));
         return (0, combi_1.seq)("READ TABLE", (0, combi_1.alt)(expressions_1.SimpleSource2, (0, combi_1.ver)(version_1.Version.v740sp02, expressions_1.Source, version_1.Version.OpenABAP)), (0, combi_1.opt)(perm));
     }
 }
@@ -16555,6 +16557,8 @@ class Submit {
         const exporting = (0, combi_1.str)("EXPORTING LIST TO MEMORY");
         const spool = (0, combi_1.seq)("SPOOL PARAMETERS", expressions_1.Source);
         const archive = (0, combi_1.seq)("ARCHIVE PARAMETERS", expressions_1.Source);
+        const archiveMode = (0, combi_1.seq)("ARCHIVE MODE", expressions_1.Source);
+        const receiver = (0, combi_1.seq)("RECEIVER", expressions_1.Source);
         const lineSize = (0, combi_1.seq)("LINE-SIZE", expressions_1.Source);
         const lineCount = (0, combi_1.seq)("LINE-COUNT", expressions_1.Source);
         const user = (0, combi_1.seq)("USER", expressions_1.Source);
@@ -16573,7 +16577,7 @@ class Submit {
         const imm = (0, combi_1.seq)("IMMEDIATELY", expressions_1.Source);
         const dest = (0, combi_1.seq)("DESTINATION", expressions_1.Source);
         const language = (0, combi_1.seq)("LANGUAGE", expressions_1.Source);
-        const perm = (0, combi_1.per)((0, combi_1.plus)(awith), selectionTable, (0, combi_1.plus)(awith), spool, lineSize, lineCount, archive, user, sset, ssetp, keep, cover, imm, layout, dest, coverText, listName, language, free, newList, uss, copies, datasetExpiration, "TO SAP-SPOOL", "WITHOUT SPOOL DYNPRO", "VIA SELECTION-SCREEN", exporting, expressions_1.AndReturn, job);
+        const perm = (0, combi_1.per)((0, combi_1.plus)(awith), selectionTable, (0, combi_1.plus)(awith), (0, combi_1.plus)(awith), spool, lineSize, lineCount, archive, archiveMode, receiver, user, sset, ssetp, keep, cover, imm, layout, dest, coverText, listName, language, free, newList, uss, copies, datasetExpiration, "TO SAP-SPOOL", "WITHOUT SPOOL DYNPRO", "VIA SELECTION-SCREEN", exporting, expressions_1.AndReturn, job);
         const ret = (0, combi_1.seq)("SUBMIT", prog, (0, combi_1.opt)(perm));
         return (0, combi_1.verNot)(version_1.Version.Cloud, ret);
     }
@@ -17066,7 +17070,7 @@ const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@a
 class TypeEnum {
     getMatcher() {
         // it is also possible to define without Value, this is covered by normal type
-        const ret = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("TYPES", expressions_1.NamespaceSimpleName, expressions_1.Value));
+        const ret = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("TYPES", expressions_1.NamespaceSimpleName, expressions_1.Value), version_1.Version.OpenABAP);
         return ret;
     }
 }
@@ -17093,7 +17097,7 @@ class TypeEnumBegin {
         const structure = (0, combi_1.seq)("STRUCTURE", expressions_1.NamespaceSimpleName);
         const base = (0, combi_1.seq)("BASE TYPE", expressions_1.NamespaceSimpleName);
         const em = (0, combi_1.seq)("ENUM", expressions_1.NamespaceSimpleName, (0, combi_1.opt)(structure), (0, combi_1.opt)(base));
-        const ret = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("TYPES", "BEGIN OF", em));
+        const ret = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("TYPES", "BEGIN OF", em), version_1.Version.OpenABAP);
         return ret;
     }
 }
@@ -17118,7 +17122,7 @@ const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@a
 class TypeEnumEnd {
     getMatcher() {
         const structure = (0, combi_1.seq)("STRUCTURE", expressions_1.NamespaceSimpleName);
-        const ret = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("TYPES", "END OF", "ENUM", expressions_1.NamespaceSimpleName, (0, combi_1.opt)(structure)));
+        const ret = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("TYPES", "END OF", "ENUM", expressions_1.NamespaceSimpleName, (0, combi_1.opt)(structure)), version_1.Version.OpenABAP);
         return ret;
     }
 }
@@ -20543,7 +20547,7 @@ BuiltIn.methods = {
     "CONCAT_LINES_OF": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "table": new basic_1.TableType(basic_1.AnyType.get(), { withHeader: false, keyType: basic_1.TableKeyType.default }),
+            "table": new basic_1.TableType(basic_1.AnyType.get(), { withHeader: false, keyType: basic_1.TableKeyType.user }),
         },
         optional: {
             "sep": basic_1.CLikeType.get(),
@@ -20841,7 +20845,7 @@ BuiltIn.methods = {
     "LINES": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.TableType(basic_1.AnyType.get(), { withHeader: false, keyType: basic_1.TableKeyType.default }),
+            "val": new basic_1.TableType(basic_1.AnyType.get(), { withHeader: false, keyType: basic_1.TableKeyType.user }),
         },
         return: basic_1.IntegerType.get(),
     },
@@ -21286,6 +21290,7 @@ class CurrentScope {
         }
     }
     constructor(reg, obj) {
+        this.localFriends = new Map();
         this.current = undefined;
         this.parentObj = obj;
         this.reg = reg;
@@ -21709,6 +21714,27 @@ class CurrentScope {
         }
         return false;
     }
+    addLocalFriend(className, friendName) {
+        var _a;
+        const key = className.toUpperCase();
+        const list = (_a = this.localFriends.get(key)) !== null && _a !== void 0 ? _a : [];
+        list.push(friendName.toUpperCase());
+        this.localFriends.set(key, list);
+    }
+    isLocalFriend(className, friendName) {
+        var _a;
+        return ((_a = this.localFriends.get(className.toUpperCase())) === null || _a === void 0 ? void 0 : _a.includes(friendName.toUpperCase())) === true;
+    }
+    getEnclosingClassName() {
+        let curr = this.current;
+        while (curr !== undefined) {
+            if (curr.getIdentifier().stype === _scope_type_1.ScopeType.ClassImplementation) {
+                return curr.getIdentifier().sname;
+            }
+            curr = curr.getParent();
+        }
+        return undefined;
+    }
     isGlobalOO() {
         return this.parentObj.getType() === "INTF" || this.parentObj.getType() === "CLAS";
     }
@@ -22462,6 +22488,8 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.TypeUtils = void 0;
 const types_1 = __webpack_require__(/*! ../types */ "./node_modules/@abaplint/core/build/src/abap/types/index.js");
 const basic_1 = __webpack_require__(/*! ../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const table_type_1 = __webpack_require__(/*! ../types/basic/table_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/table_type.js");
+const enum_type_1 = __webpack_require__(/*! ../types/basic/enum_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/enum_type.js");
 const cgeneric_type_1 = __webpack_require__(/*! ../types/basic/cgeneric_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/cgeneric_type.js");
 const Expressions = __webpack_require__(/*! ../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 // todo: refactor to static? for performance
@@ -22538,7 +22566,8 @@ class TypeUtils {
             || type instanceof basic_1.DataType
             || type instanceof basic_1.CLikeType
             || type instanceof basic_1.PackedType
-            || type instanceof basic_1.TimeType) {
+            || type instanceof basic_1.TimeType
+            || type instanceof enum_type_1.EnumType) {
             return true;
         }
         return false;
@@ -22888,6 +22917,17 @@ class TypeUtils {
         }
         else if (source instanceof basic_1.TableType) {
             if (target instanceof basic_1.TableType) {
+                const sourceKeyType = source.getOptions().keyType;
+                const targetKeyType = target.getOptions().keyType;
+                if (sourceKeyType !== targetKeyType
+                    && sourceKeyType !== table_type_1.TableKeyType.user
+                    && targetKeyType !== table_type_1.TableKeyType.user) {
+                    return false;
+                }
+                if (sourceKeyType === table_type_1.TableKeyType.empty && targetKeyType !== table_type_1.TableKeyType.empty
+                    && !(target.getRowType() instanceof basic_1.AnyType)) {
+                    return false;
+                }
                 return this.isAssignableStrict(source.getRowType(), target.getRowType());
             }
             else if (target instanceof basic_1.UnknownType
@@ -23616,7 +23656,7 @@ class BasicTypes {
             || text === "TYPE HASHED TABLE"
             || text === "TYPE INDEX TABLE"
             || text === "TYPE ANY TABLE") {
-            return new Types.TableType(Types.AnyType.get(), { withHeader: node.concatTokens().toUpperCase().includes("WITH HEADER LINE"), keyType: Types.TableKeyType.default });
+            return new Types.TableType(Types.AnyType.get(), { withHeader: node.concatTokens().toUpperCase().includes("WITH HEADER LINE"), keyType: Types.TableKeyType.user });
         }
         else if (text.startsWith("LIKE ")) {
             let sub = node.findFirstExpression(Expressions.Type);
@@ -26462,7 +26502,7 @@ class MethodParameters {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
             }
-            else if (item.targetType && new _type_utils_1.TypeUtils(input.scope).isAssignable(parameterType, item.targetType) === false) {
+            else if (item.targetType && new _type_utils_1.TypeUtils(input.scope).isAssignableStrict(parameterType, item.targetType) === false) {
                 const message = "Method parameter type not compatible, " + item.name;
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
@@ -26490,7 +26530,7 @@ class MethodParameters {
                 }
                 parameterType = parameter.getType();
             }
-            if (item.targetType && new _type_utils_1.TypeUtils(input.scope).isAssignable(parameterType, item.targetType) === false) {
+            if (item.targetType && new _type_utils_1.TypeUtils(input.scope).isAssignableStrict(parameterType, item.targetType) === false) {
                 const message = "Method parameter type not compatible, " + item.name;
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
@@ -26805,6 +26845,7 @@ const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/
 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 _typed_identifier_1 = __webpack_require__(/*! ../../types/_typed_identifier */ "./node_modules/@abaplint/core/build/src/abap/types/_typed_identifier.js");
+const create_object_1 = __webpack_require__(/*! ../statements/create_object */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/create_object.js");
 class NewObject {
     static runSyntax(node, input, targetType) {
         let ret = undefined;
@@ -26838,6 +26879,13 @@ class NewObject {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
             }
+            if (clas) {
+                const err = create_object_1.CreateObject.checkInstantiationAllowed(clas, input);
+                if (err) {
+                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), err));
+                    return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
+                }
+            }
         }
         else if (typeName === "#" && targetType) {
             ret = targetType;
@@ -26859,6 +26907,13 @@ class NewObject {
                     input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                     return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                 }
+                if (clas) {
+                    const err = create_object_1.CreateObject.checkInstantiationAllowed(clas, input);
+                    if (err) {
+                        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), err));
+                        return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
+                    }
+                }
                 ret = objref;
             }
         }
@@ -27415,16 +27470,17 @@ class Select {
         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 basic_1.VoidType.get("SELECT_todo4");
+            return new basic_1.TableType(basic_1.VoidType.get("SELECT_todo4"), tableOptions, undefined);
         }
         const dbType = dbSources[0].parseType(scope.getRegistry());
         if (!(dbType instanceof basic_1.StructureType)) {
             return basic_1.VoidType.get("SELECT_todo5");
         }
         if (fields.length === 1 && fields[0].code === "*") {
-            return new basic_1.TableType(dbType, { withHeader: false, keyType: basic_1.TableKeyType.default }, undefined);
+            return new basic_1.TableType(dbType, tableOptions, undefined);
         }
         const allFieldsSimple = fields.every(f => isSimple.test(f.code));
         if (allFieldsSimple === true) {
@@ -27436,7 +27492,7 @@ class Select {
                 }
                 components.push({ name: field.code, type });
             }
-            return new basic_1.TableType(new basic_1.StructureType(components), { withHeader: false, keyType: basic_1.TableKeyType.default }, undefined);
+            return new basic_1.TableType(new basic_1.StructureType(components), tableOptions, undefined);
         }
         return basic_1.VoidType.get("SELECT_todo7");
     }
@@ -27624,6 +27680,11 @@ class Source {
                             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                             return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                         }
+                        else if (foundType === null || foundType === void 0 ? void 0 : foundType.isGeneric()) {
+                            const message = "Cannot CONV to generic type";
+                            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                            return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
+                        }
                         this.addIfInferred(node, input, foundType);
                         return foundType;
                     }
@@ -30058,6 +30119,7 @@ const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@a
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 class ClassLocalFriends {
     runSyntax(node, input) {
+        var _a;
         const classNames = node.findAllExpressions(Expressions.ClassName);
         const found = classNames[0];
         if (found) {
@@ -30088,6 +30150,12 @@ class ClassLocalFriends {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
             }
+            else {
+                const befriendedName = (_a = classNames[0]) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr();
+                if (befriendedName) {
+                    input.scope.addLocalFriend(befriendedName, className);
+                }
+            }
         }
     }
 }
@@ -30569,6 +30637,7 @@ const types_1 = __webpack_require__(/*! ../../types */ "./node_modules/@abaplint
 const _object_oriented_1 = __webpack_require__(/*! ../_object_oriented */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_object_oriented.js");
 const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_type_utils.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");
 class CreateObject {
     runSyntax(node, input) {
         var _a;
@@ -30665,8 +30734,46 @@ class CreateObject {
             ooName = found.getVoided();
         }
         input.scope.addReference(t === null || t === void 0 ? void 0 : t.getFirstToken(), cdef, _reference_1.ReferenceType.ConstructorReference, input.filename, { ooName: ooName });
+        if (cdef !== undefined) {
+            const err = CreateObject.checkInstantiationAllowed(cdef, input);
+            if (err) {
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), err));
+                return;
+            }
+        }
         this.validateParameters(cdef, node, input);
     }
+    static checkInstantiationAllowed(cdef, input) {
+        var _a, _b;
+        const createVis = cdef.getCreateVisibility();
+        if (createVis === visibility_1.Visibility.Public) {
+            return undefined;
+        }
+        const enclosingClass = input.scope.getEnclosingClassName();
+        if (enclosingClass === undefined) {
+            return cdef.getName() + " cannot be instantiated, class is defined as " +
+                (createVis === visibility_1.Visibility.Private ? "private" : "protected");
+        }
+        if (enclosingClass.toUpperCase() === cdef.getName().toUpperCase()) {
+            return undefined;
+        }
+        if (cdef.getFriends().some(f => f.toUpperCase() === enclosingClass.toUpperCase()) ||
+            input.scope.isLocalFriend(cdef.getName(), enclosingClass)) {
+            return undefined;
+        }
+        if (createVis === visibility_1.Visibility.Protected) {
+            // subclasses are also allowed
+            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 = (_b = input.scope.findClassDefinition(sup)) === null || _b === void 0 ? void 0 : _b.getSuperClass();
+            }
+        }
+        return cdef.getName() + " cannot be instantiated, class is defined as " +
+            (createVis === visibility_1.Visibility.Private ? "private" : "protected");
+    }
     validateParameters(cdef, node, input) {
         var _a, _b, _c, _d;
         if (cdef === undefined) {
@@ -37817,6 +37924,7 @@ __exportStar(__webpack_require__(/*! ./date_type */ "./node_modules/@abaplint/co
 __exportStar(__webpack_require__(/*! ./decfloat_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/decfloat_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./decfloat16_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/decfloat16_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./decfloat34_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/decfloat34_type.js"), exports);
+__exportStar(__webpack_require__(/*! ./enum_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/enum_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./float_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/float_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./floating_point_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/floating_point_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./generic_object_reference_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/generic_object_reference_type.js"), exports);
@@ -38675,6 +38783,7 @@ class Attributes {
         this.instance = [];
         this.constants = [];
         this.aliases = [];
+        this.declaredInterfaces = [];
         this.tlist = [];
         this.filename = input.filename;
         this.parse(node, input);
@@ -38750,8 +38859,15 @@ class Attributes {
     }
     /////////////////////////////
     parse(node, input) {
+        var _a, _b;
         const cdef = node.findDirectStructure(Structures.ClassDefinition);
         if (cdef) {
+            for (const i of cdef.findAllStatements(Statements.InterfaceDef)) {
+                const name = (_a = i.findFirstExpression(Expressions.InterfaceName)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr();
+                if (name) {
+                    this.declaredInterfaces.push(name.toUpperCase());
+                }
+            }
             this.parseSection(cdef.findDirectStructure(Structures.PublicSection), visibility_1.Visibility.Public, input);
             this.parseSection(cdef.findDirectStructure(Structures.ProtectedSection), visibility_1.Visibility.Protected, input);
             this.parseSection(cdef.findDirectStructure(Structures.PrivateSection), visibility_1.Visibility.Private, input);
@@ -38759,6 +38875,12 @@ class Attributes {
         }
         const idef = node.findDirectStructure(Structures.Interface);
         if (idef) {
+            for (const i of idef.findAllStatements(Statements.InterfaceDef)) {
+                const name = (_b = i.findFirstExpression(Expressions.InterfaceName)) === null || _b === void 0 ? void 0 : _b.getFirstToken().getStr();
+                if (name) {
+                    this.declaredInterfaces.push(name.toUpperCase());
+                }
+            }
             this.parseSection(idef.findDirectStructure(Structures.SectionContents), visibility_1.Visibility.Public, input);
             return;
         }
@@ -38867,6 +38989,12 @@ class Attributes {
                     input.scope.addNamedIdentifier(aliasName.getStr(), foundAttribute);
                 }
             }
+            else if (this.declaredInterfaces.includes(name.toUpperCase()) || input.scope.getDDIC().inErrorNamespace(name) === false) {
+                input.scope.addReference(compToken, undefined, _reference_1.ReferenceType.ObjectOrientedVoidReference, input.filename);
+            }
+            else {
+                throw new Error("Interface " + name + " not found");
+            }
         }
     }
     parseAttribute(node, visibility, input) {
@@ -38940,6 +39068,7 @@ const event_definition_1 = __webpack_require__(/*! ./event_definition */ "./node
 const visibility_1 = __webpack_require__(/*! ../4_file_information/visibility */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/visibility.js");
 const _object_oriented_1 = __webpack_require__(/*! ../5_syntax/_object_oriented */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_object_oriented.js");
 const _reference_1 = __webpack_require__(/*! ../5_syntax/_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
+const _syntax_input_1 = __webpack_require__(/*! ../5_syntax/_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 class ClassDefinition extends _identifier_1.Identifier {
     constructor(node, input) {
         var _a;
@@ -38977,6 +39106,15 @@ class ClassDefinition extends _identifier_1.Identifier {
         this.testing = concat.includes(" FOR TESTING");
         this.sharedMemory = concat.includes(" SHARED MEMORY ENABLED");
         this.abstract = (def === null || def === void 0 ? void 0 : def.findDirectTokenByText("ABSTRACT")) !== undefined;
+        if (concat.includes(" CREATE PRIVATE")) {
+            this.createVisibilityValue = visibility_1.Visibility.Private;
+        }
+        else if (concat.includes(" CREATE PROTECTED")) {
+            this.createVisibilityValue = visibility_1.Visibility.Protected;
+        }
+        else {
+            this.createVisibilityValue = visibility_1.Visibility.Public;
+        }
         // perform checks after everything has been initialized
         this.checkMethodsFromSuperClasses(input.scope);
         this.checkMethodNameLength();
@@ -39020,6 +39158,9 @@ class ClassDefinition extends _identifier_1.Identifier {
     isSharedMemory() {
         return this.sharedMemory;
     }
+    getCreateVisibility() {
+        return this.createVisibilityValue;
+    }
     /*
       public getEvents() {
       }
@@ -39075,8 +39216,20 @@ class ClassDefinition extends _identifier_1.Identifier {
         const result = [];
         for (const n of ((_a = def === null || def === void 0 ? void 0 : def.findDirectExpression(Expressions.ClassFriends)) === null || _a === void 0 ? void 0 : _a.findDirectExpressions(Expressions.ClassName)) || []) {
             const token = n.getFirstToken();
-            this.addReference(token, input);
             const name = token.getStr();
+            const s = input.scope.findClassDefinition(name);
+            if (s) {
+                input.scope.addReference(token, s, _reference_1.ReferenceType.ObjectOrientedReference, input.filename, { ooName: name.toUpperCase(), ooType: "CLAS" });
+            }
+            else if (input.scope.existsObject(name) !== undefined) {
+                // DEFINITION DEFERRED friend
+            }
+            else if (input.scope.getDDIC().inErrorNamespace(name) === false) {
+                input.scope.addReference(token, undefined, _reference_1.ReferenceType.ObjectOrientedVoidReference, input.filename);
+            }
+            else {
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, token, name.toUpperCase() + " does not exist"));
+            }
             result.push(name);
         }
         return result;
@@ -54796,7 +54949,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.118.2";
+        return "2.118.7";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -56481,11 +56634,17 @@ MODIFY TABLE foo FROM bar.`,
                 match = this.tryMatch(statement, this.reg, Statements.DeleteDatabase);
             }
             else if (statement.get() instanceof Statements.ModifyInternal) {
-                match = this.tryMatch(statement, this.reg, Statements.ModifyDatabase);
+                match = this.tryMatch(statement, this.reg, Statements.ModifyDatabase) || this.isBareStatement(statement);
             }
             else if (statement.get() instanceof Statements.ModifyDatabase) {
                 match = this.tryMatch(statement, this.reg, Statements.ModifyInternal);
             }
+            else if (statement.get() instanceof Statements.InsertDatabase) {
+                match = this.isBareStatement(statement);
+            }
+            else if (statement.get() instanceof Statements.UpdateDatabase) {
+                match = this.isBareStatement(statement);
+            }
             if (match) {
                 const issue = issue_1.Issue.atStatement(file, statement, this.getMessage(), this.getMetadata().key, this.conf.severity);
                 issues.push(issue);
@@ -56493,6 +56652,11 @@ MODIFY TABLE foo FROM bar.`,
         }
         return issues;
     }
+    isBareStatement(st) {
+        const tokens = st.getTokens().slice(0);
+        tokens.pop();
+        return tokens.length === 2;
+    }
     tryMatch(st, reg, type1) {
         const ver = reg.getConfig().getVersion();
         const tokens = st.getTokens().slice(0);
@@ -56947,6 +57111,98 @@ exports.CallTransactionAuthorityCheck = CallTransactionAuthorityCheck;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/catch_and_raise.js"
+/*!************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/catch_and_raise.js ***!
+  \************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.CatchAndRaise = exports.CatchAndRaiseConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.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 _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const Structures = __webpack_require__(/*! ../abap/3_structures/structures */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.js");
+const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const Expressions = __webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+class CatchAndRaiseConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.CatchAndRaiseConf = CatchAndRaiseConf;
+class CatchAndRaise extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new CatchAndRaiseConf();
+    }
+    getMetadata() {
+        return {
+            key: "catch_and_raise",
+            title: "Catch and re-raise same exception",
+            shortDescription: `Reports CATCH blocks that only re-raise the caught exception without any handling`,
+            badExample: `TRY.\n  something( ).\nCATCH zcx_something INTO DATA(lv_exc).\n  RAISE EXCEPTION lv_exc.\nENDTRY.`,
+            goodExample: `TRY.\n  something( ).\nCATCH zcx_something.\n  " handle exception\nENDTRY.`,
+            tags: [_irule_1.RuleTag.SingleFile],
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        const stru = file.getStructure();
+        if (stru === undefined) {
+            return [];
+        }
+        for (const catchStruct of stru.findAllStructures(Structures.Catch)) {
+            const catchStatement = catchStruct.findDirectStatement(Statements.Catch);
+            if (catchStatement === undefined) {
+                continue;
+            }
+            const target = catchStatement.findFirstExpression(Expressions.Target);
+            if (target === undefined) {
+                continue;
+            }
+            const targetField = target.findFirstExpression(Expressions.TargetField);
+            if (targetField === undefined) {
+                continue;
+            }
+            const caughtVar = targetField.getFirstToken().getStr().toUpperCase();
+            const allStatements = catchStruct.findAllStatementNodes();
+            if (allStatements.length !== 2) {
+                continue;
+            }
+            if (!(allStatements[1].get() instanceof Statements.Raise)) {
+                continue;
+            }
+            const raiseStatement = allStatements[1];
+            const raiseConcat = raiseStatement.concatTokens().toUpperCase().replace(/\.$/, "").trim();
+            if (!raiseConcat.startsWith("RAISE EXCEPTION ")) {
+                continue;
+            }
+            const parts = raiseConcat.split(/\s+/);
+            if (parts.length !== 3) {
+                continue;
+            }
+            if (parts[2] === caughtVar) {
+                issues.push(issue_1.Issue.atStatement(file, catchStatement, this.getMessage(), this.getMetadata().key, this.conf.severity));
+            }
+        }
+        return issues;
+    }
+    getMessage() {
+        return "Caught exception is immediately re-raised, CATCH block has no effect";
+    }
+}
+exports.CatchAndRaise = CatchAndRaise;
+//# sourceMappingURL=catch_and_raise.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/cds_association_name.js"
 /*!*****************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/cds_association_name.js ***!
@@ -66715,6 +66971,7 @@ __exportStar(__webpack_require__(/*! ./avoid_use */ "./node_modules/@abaplint/co
 __exportStar(__webpack_require__(/*! ./begin_end_names */ "./node_modules/@abaplint/core/build/src/rules/begin_end_names.js"), exports);
 __exportStar(__webpack_require__(/*! ./begin_single_include */ "./node_modules/@abaplint/core/build/src/rules/begin_single_include.js"), exports);
 __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_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);
@@ -80112,9 +80369,11 @@ class PopulateTables {
                 else if (parameter.getMeta().includes("exporting" /* abaplint.IdentifierMeta.MethodExporting */)) {
                     pardecltyp = "1";
                 }
-                const paroptionl = optionalParameters.includes(parameter.getName()) ? "X" : " ";
+                const name = parameter.getName().toUpperCase();
+                const paroptionl = (optionalParameters.some(o => o.toUpperCase() === name) || method.getParameters().getParameterDefault(name) !== undefined) ? "X" : " ";
                 const type = parameter.getType().getQualifiedName()?.toUpperCase() || "";
-                ret.push(`INSERT INTO "seosubcodf" ("clsname", "cmpname", "sconame", "version", "editorder", "pardecltyp", "type", "paroptionl") VALUES ('${obj.getName()}', '${method.getName().toUpperCase()}', '${parameter.getName().toUpperCase()}', '1', ${editorder}, '${pardecltyp}', '${type}', '${paroptionl}');`);
+                const parvalue = method.getParameters().getParameterDefault(name)?.concatTokens() || "";
+                ret.push(`INSERT INTO "seosubcodf" ("clsname", "cmpname", "sconame", "version", "editorder", "pardecltyp", "type", "paroptionl", "parvalue") VALUES ('${obj.getName()}', '${method.getName().toUpperCase()}', '${name}', '1', ${editorder}, '${pardecltyp}', '${type}', '${paroptionl}', '${this.escape(parvalue)}');`);
             }
         }
         return ret;
@@ -80125,9 +80384,32 @@ class PopulateTables {
         if (def === undefined || !this.hasSEOSUBCOTX) {
             return [];
         }
+        const descriptions = [];
+        const xml = obj.getXML();
+        if (xml) {
+            const parsed = obj.parseRaw2();
+            const sub = parsed?.abapGit["asx:abap"]["asx:values"]?.DESCRIPTIONS?.SEOSUBCOTX;
+            if (sub) {
+                if (Array.isArray(sub)) {
+                    descriptions.push(...sub);
+                }
+                else {
+                    descriptions.push(sub);
+                }
+            }
+        }
         for (const method of def.getMethodDefinitions().getAll()) {
             for (const parameter of method.getParameters().getAll()) {
-                ret.push(`INSERT INTO "seosubcotx" ("clsname", "cmpname", "sconame", "langu", "descript") VALUES ('${obj.getName()}', '${method.getName().toUpperCase()}', '${parameter.getName().toUpperCase()}', 'E', 'todo');`);
+                const mName = method.getName().toUpperCase();
+                const pName = parameter.getName().toUpperCase();
+                let descript = "";
+                for (const d of descriptions) {
+                    if (d.CMPNAME?.toUpperCase() === mName && d.SCONAME?.toUpperCase() === pName) {
+                        descript = d.DESCRIPT || "";
+                        break;
+                    }
+                }
+                ret.push(`INSERT INTO "seosubcotx" ("clsname", "cmpname", "sconame", "langu", "descript") VALUES ('${obj.getName()}', '${mName}', '${pName}', 'E', '${this.escape(descript)}');`);
             }
         }
         return ret;
@@ -87715,6 +87997,13 @@ class DataTranspiler {
         else {
             value = DataTranspiler.buildValue(node, traversal_1.Traversal.prefixVariable(found.getName().toLowerCase()), traversal);
         }
+        // for enum types, initialize with the first enum value
+        if (found.getType() instanceof abaplint.BasicTypes.EnumType && value === "") {
+            const enumDefault = DataTranspiler.findEnumDefault(scope, found.getType());
+            if (enumDefault) {
+                value = "\n" + traversal_1.Traversal.prefixVariable(found.getName().toLowerCase()) + ".set(\"" + enumDefault + "\");";
+            }
+        }
         const ret = new chunk_1.Chunk()
             .appendString("let ")
             .appendString(traversal_1.Traversal.prefixVariable(traversal_1.Traversal.escapeNamespace(found.getName().toLowerCase())))
@@ -87723,6 +88012,26 @@ class DataTranspiler {
             .appendString(value);
         return ret;
     }
+    static findEnumDefault(scope, _enumType) {
+        let current = scope;
+        while (current) {
+            const vars = current.getData().vars;
+            for (const key of Object.keys(vars)) {
+                const v = vars[key];
+                if (v.getMeta().includes("enum" /* abaplint.IdentifierMeta.Enum */)) {
+                    const structType = v.getType();
+                    if (structType instanceof abaplint.BasicTypes.StructureType) {
+                        const components = structType.getComponents();
+                        if (components.length > 0) {
+                            return components[0].name.toUpperCase();
+                        }
+                    }
+                }
+            }
+            current = current.getParent();
+        }
+        return undefined;
+    }
     static buildValue(node, name, traversal) {
         let value = "";
         const val = node.findFirstExpression(abaplint.Expressions.Value);
@@ -93293,6 +93602,7 @@ __exportStar(__webpack_require__(/*! ./module */ "./node_modules/@abaplint/trans
 __exportStar(__webpack_require__(/*! ./select */ "./node_modules/@abaplint/transpiler/build/src/structures/select.js"), exports);
 __exportStar(__webpack_require__(/*! ./method */ "./node_modules/@abaplint/transpiler/build/src/structures/method.js"), exports);
 __exportStar(__webpack_require__(/*! ./try */ "./node_modules/@abaplint/transpiler/build/src/structures/try.js"), exports);
+__exportStar(__webpack_require__(/*! ./type_enum */ "./node_modules/@abaplint/transpiler/build/src/structures/type_enum.js"), exports);
 __exportStar(__webpack_require__(/*! ./types */ "./node_modules/@abaplint/transpiler/build/src/structures/types.js"), exports);
 __exportStar(__webpack_require__(/*! ./when */ "./node_modules/@abaplint/transpiler/build/src/structures/when.js"), exports);
 __exportStar(__webpack_require__(/*! ./while */ "./node_modules/@abaplint/transpiler/build/src/structures/while.js"), exports);
@@ -93706,6 +94016,51 @@ exports.TryTranspiler = TryTranspiler;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/transpiler/build/src/structures/type_enum.js"
+/*!*****************************************************************************!*\
+  !*** ./node_modules/@abaplint/transpiler/build/src/structures/type_enum.js ***!
+  \*****************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.TypeEnumTranspiler = void 0;
+const abaplint = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js");
+const traversal_1 = __webpack_require__(/*! ../traversal */ "./node_modules/@abaplint/transpiler/build/src/traversal.js");
+const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
+class TypeEnumTranspiler {
+    transpile(node, _traversal) {
+        const begin = node.findDirectStatement(abaplint.Statements.TypeEnumBegin);
+        if (begin === undefined) {
+            return new chunk_1.Chunk("");
+        }
+        const nsNames = begin.findAllExpressions(abaplint.Expressions.NamespaceSimpleName);
+        if (nsNames.length < 2) {
+            return new chunk_1.Chunk("");
+        }
+        const structureName = nsNames[1].concatTokens().toLowerCase();
+        const values = [];
+        for (const t of node.findDirectStatements(abaplint.Statements.Type)) {
+            const name = t.findFirstExpression(abaplint.Expressions.NamespaceSimpleName);
+            if (name) {
+                values.push(name.concatTokens().toLowerCase());
+            }
+        }
+        const fields = values.map(v => `"${v}": new abap.types.String()`).join(",\n");
+        let ret = `let ${traversal_1.Traversal.prefixVariable(structureName)} = new abap.types.Structure({
+${fields}});\n`;
+        for (const v of values) {
+            ret += `${traversal_1.Traversal.prefixVariable(structureName)}.get().${v}.set("${v.toUpperCase()}");\n`;
+        }
+        return new chunk_1.Chunk(ret);
+    }
+}
+exports.TypeEnumTranspiler = TypeEnumTranspiler;
+//# sourceMappingURL=type_enum.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/transpiler/build/src/structures/types.js"
 /*!*************************************************************************!*\
   !*** ./node_modules/@abaplint/transpiler/build/src/structures/types.js ***!
@@ -93976,9 +94331,15 @@ class TranspileTypes {
         }
         else if (type instanceof abaplint.BasicTypes.HexType) {
             resolved = featureHexUInt8 ? "HexUInt8" : "Hex";
-            if (type.getLength() !== 1) {
+            if (type.getLength() !== 1 && type.getQualifiedName() !== undefined) {
+                extra = "{length: " + type.getLength() + ", qualifiedName: \"" + type.getQualifiedName()?.toUpperCase() + "\"}";
+            }
+            else if (type.getLength() !== 1) {
                 extra = "{length: " + type.getLength() + "}";
             }
+            else if (type.getQualifiedName() !== undefined) {
+                extra = "{qualifiedName: \"" + type.getQualifiedName()?.toUpperCase() + "\"}";
+            }
         }
         else if (type instanceof abaplint.BasicTypes.FloatType) {
             resolved = "Float";
@@ -93995,6 +94356,12 @@ class TranspileTypes {
         else if (type instanceof abaplint.BasicTypes.DecFloat34Type) {
             resolved = "DecFloat34";
         }
+        else if (type instanceof abaplint.BasicTypes.EnumType) {
+            resolved = "String";
+            if (type.getQualifiedName() !== undefined) {
+                extra = "{qualifiedName: \"" + type.getQualifiedName()?.toUpperCase() + "\"}";
+            }
+        }
         else if (type instanceof abaplint.BasicTypes.UnknownType) {
             return `(() => { throw new Error("Unknown type: ${type.getError()}") })()`;
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.13.1",
+  "version": "2.13.3",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,8 +27,8 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.118.2",
-    "@abaplint/transpiler": "^2.13.1",
+    "@abaplint/core": "^2.118.7",
+    "@abaplint/transpiler": "^2.13.3",
     "@types/glob": "^8.1.0",
     "@types/node": "^24.12.0",
     "@types/progress": "^2.0.7",