@abaplint/cli 2.118.3 → 2.118.5

package/build/cli.js

@@ -16307,7 +16307,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));
     }
 }
@@ -21633,7 +21635,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(),
@@ -21931,7 +21933,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(),
     },
@@ -22376,6 +22378,7 @@ class CurrentScope {
         }
     }
     constructor(reg, obj) {
+        this.localFriends = new Map();
         this.current = undefined;
         this.parentObj = obj;
         this.reg = reg;
@@ -22799,6 +22802,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";
     }
@@ -23552,6 +23576,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
@@ -23628,7 +23654,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;
@@ -23978,6 +24005,13 @@ 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;
+                }
                 return this.isAssignableStrict(source.getRowType(), target.getRowType());
             }
             else if (target instanceof basic_1.UnknownType
@@ -24035,6 +24069,13 @@ class TypeUtils {
                         return false;
                     }
                 }
+                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;
+                }
                 return true;
             }
             return false;
@@ -24706,7 +24747,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);
@@ -27895,6 +27936,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;
@@ -27928,6 +27970,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;
@@ -27949,6 +27998,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;
             }
         }
@@ -28714,6 +28770,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;
                     }
@@ -31148,6 +31209,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) {
@@ -31178,6 +31240,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);
+                }
+            }
         }
     }
 }
@@ -31659,6 +31727,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;
@@ -31755,8 +31824,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) {
@@ -35133,7 +35240,7 @@ const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modu
 class Split {
     runSyntax(node, input) {
         const intoTable = node.findTokenSequencePosition("INTO", "TABLE") !== undefined;
-        const type = intoTable ? new basic_1.TableType(basic_1.StringType.get(), { withHeader: false, keyType: basic_1.TableKeyType.default }) : basic_1.StringType.get();
+        const type = intoTable ? new basic_1.TableType(basic_1.StringType.get(), { withHeader: false, keyType: basic_1.TableKeyType.user }) : basic_1.StringType.get();
         for (const target of node.findAllExpressions(Expressions.Target)) {
             const inline = target.findDirectExpression(Expressions.InlineData);
             if (inline) {
@@ -38907,6 +39014,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);
@@ -39765,6 +39873,7 @@ class Attributes {
         this.instance = [];
         this.constants = [];
         this.aliases = [];
+        this.declaredInterfaces = [];
         this.tlist = [];
         this.filename = input.filename;
         this.parse(node, input);
@@ -39840,8 +39949,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);
@@ -39849,6 +39965,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;
         }
@@ -39957,6 +40079,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) {
@@ -40030,6 +40158,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;
@@ -40067,6 +40196,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();
@@ -40110,6 +40248,9 @@ class ClassDefinition extends _identifier_1.Identifier {
     isSharedMemory() {
         return this.sharedMemory;
     }
+    getCreateVisibility() {
+        return this.createVisibilityValue;
+    }
     /*
       public getEvents() {
       }
@@ -40165,8 +40306,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;
@@ -55886,7 +56039,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.118.3";
+        return "2.118.5";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -58037,6 +58190,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 ***!
@@ -67805,6 +68050,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);

package/package.json

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