@abaplint/core 2.120.8 → 2.120.9

package/build/src/abap/2_statements/expressions/compare.js

@@ -12,7 +12,7 @@ class Compare extends combi_1.Expression {
         const inn = (0, combi_1.seq)((0, combi_1.optPrio)("NOT"), "IN", (0, combi_1.altPrio)(_1.Source, list));
         const sopt = (0, combi_1.seq)("IS", (0, combi_1.optPrio)("NOT"), (0, combi_1.altPrio)("SUPPLIED", "BOUND", (0, combi_1.ver)(version_1.Release.v750, (0, combi_1.seq)("INSTANCE OF", _1.ClassName), { also: combi_1.AlsoIn.OpenABAP }), "REQUESTED", "INITIAL"));
         const between = (0, combi_1.seq)((0, combi_1.optPrio)("NOT"), "BETWEEN", _1.Source, "AND", _1.Source);
-        // https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abennews-740_sp08-expressions.htm
+        // https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abennews-740_sp08-expressions.html
         // but also seems to work in v740sp05, blah
         const predicate = (0, combi_1.ver)(version_1.Release.v740sp08, _1.MethodCallChain, { also: combi_1.AlsoIn.OpenABAP });
         const rett = (0, combi_1.seq)(_1.Source, (0, combi_1.altPrio)((0, combi_1.seq)(_1.CompareOperator, _1.Source), inn, between, sopt));

package/build/src/abap/2_statements/expressions/string_template_formatting.js

@@ -7,7 +7,7 @@ const version_1 = require("../../../version");
 const dynamic_1 = require("./dynamic");
 class StringTemplateFormatting extends combi_1.Expression {
     getRunnable() {
-        // https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abapcompute_string_format_options.htm
+        // https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abapcompute_string_format_options.html
         const alphaOptions = (0, combi_1.altPrio)("OUT", "RAW", "IN", _1.Source);
         const alignOptions = (0, combi_1.altPrio)("LEFT", "RIGHT", "CENTER", _1.Source, dynamic_1.Dynamic);
         const dateTimeOptions = (0, combi_1.altPrio)("RAW", "ISO", "USER", "ENVIRONMENT", _1.Source, dynamic_1.Dynamic);

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

@@ -372,7 +372,7 @@ class BuiltIn {
         const sy = new _typed_identifier_1.TypedIdentifier(id1, BuiltIn.filename, type, ["read_only" /* IdentifierMeta.ReadOnly */, "built-in" /* IdentifierMeta.BuiltIn */]);
         const id2 = new tokens_1.Identifier(new position_1.Position(this.row++, 1), "syst");
         const syst = new _typed_identifier_1.TypedIdentifier(id2, BuiltIn.filename, type, ["read_only" /* IdentifierMeta.ReadOnly */, "built-in" /* IdentifierMeta.BuiltIn */]);
-        // https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abennews-610-system.htm
+        // https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abennews-610-system.html
         const id3 = new tokens_1.Identifier(new position_1.Position(this.row++, 1), "sy-repid");
         const syrepid = new _typed_identifier_1.TypedIdentifier(id3, BuiltIn.filename, new basic_1.CharacterType(40, { qualifiedName: "sy-repid" }), ["read_only" /* IdentifierMeta.ReadOnly */, "built-in" /* IdentifierMeta.BuiltIn */]);
         const id4 = new tokens_1.Identifier(new position_1.Position(this.row++, 1), "syst-repid");

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

@@ -434,7 +434,7 @@ class TypeUtils {
             }
         }
         else if (source instanceof basic_1.IntegerType) {
-            if (target instanceof basic_1.StringType) {
+            if (target instanceof basic_1.StringType || target instanceof basic_1.DateType) {
                 return false;
             }
             else if (target instanceof basic_1.Integer8Type || target instanceof basic_1.PackedType) {

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

@@ -38,10 +38,15 @@ const Expressions = __importStar(require("../../2_statements/expressions"));
 const _typed_identifier_1 = require("../../types/_typed_identifier");
 const basic_1 = require("../../types/basic");
 const basic_types_1 = require("../basic_types");
+const _syntax_input_1 = require("../_syntax_input");
 const assert_error_1 = require("../assert_error");
 class Ranges {
     runSyntax(node, input) {
         var _a;
+        if (input.scope.isAnyOO()) {
+            const message = "RANGES is not allowed within classes";
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+        }
         const nameToken = (_a = node.findFirstExpression(Expressions.DefinitionName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
         const typeExpression = node.findFirstExpression(Expressions.SimpleFieldChain2);
         if (typeExpression === undefined) {

package/build/src/abap/5_syntax/structures/type_mesh.js

@@ -0,0 +1,101 @@
+"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.TypeMesh = void 0;
+const Expressions = __importStar(require("../../2_statements/expressions"));
+const Statements = __importStar(require("../../2_statements/statements"));
+const nodes_1 = require("../../nodes");
+const _typed_identifier_1 = require("../../types/_typed_identifier");
+const Basic = __importStar(require("../../types/basic"));
+const basic_types_1 = require("../basic_types");
+const type_1 = require("../statements/type");
+const _scope_type_1 = require("../_scope_type");
+class TypeMesh {
+    runSyntax(node, input) {
+        var _a;
+        const begin = node.findDirectStatement(Statements.TypeMeshBegin);
+        if (begin === undefined) {
+            return undefined;
+        }
+        const name = (_a = begin.findFirstExpression(Expressions.NamespaceSimpleName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+        if (name === undefined) {
+            return undefined;
+        }
+        const components = [];
+        for (const c of node.getChildren()) {
+            if (!(c instanceof nodes_1.StatementNode)) {
+                continue;
+            }
+            const ctyp = c.get();
+            if (ctyp instanceof Statements.Type) {
+                const found = new type_1.Type().runSyntax(c, input, name.getStr() + "-");
+                if (found) {
+                    components.push({ name: found.getName(), type: found.getType() });
+                }
+            }
+            else if (ctyp instanceof Statements.TypeMesh) {
+                const found = this.runMeshNode(c, input);
+                if (found) {
+                    components.push({ name: found.getName(), type: found.getType() });
+                }
+            }
+        }
+        let qualifiedName = name.getStr();
+        if (input.scope.getType() === _scope_type_1.ScopeType.ClassDefinition
+            || input.scope.getType() === _scope_type_1.ScopeType.Interface) {
+            qualifiedName = input.scope.getName() + "=>" + qualifiedName;
+        }
+        return new _typed_identifier_1.TypedIdentifier(name, input.filename, new Basic.StructureType(components, qualifiedName));
+    }
+    ////////////////////
+    runMeshNode(node, input) {
+        var _a;
+        const nameToken = (_a = node.findFirstExpression(Expressions.NamespaceSimpleName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+        if (nameToken === undefined) {
+            return undefined;
+        }
+        const typeName = node.findFirstExpression(Expressions.TypeName);
+        let type = new basic_types_1.BasicTypes(input).resolveTypeName(typeName);
+        if (type === undefined) {
+            type = new Basic.UnknownType("Mesh node, unknown type " + (typeName === null || typeName === void 0 ? void 0 : typeName.concatTokens()));
+        }
+        else if (node.concatTokens().toUpperCase().includes(" TYPE REF TO ")) {
+            type = new Basic.DataReference(type);
+        }
+        return new _typed_identifier_1.TypedIdentifier(nameToken, input.filename, type);
+    }
+}
+exports.TypeMesh = TypeMesh;
+//# sourceMappingURL=type_mesh.js.map

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

@@ -46,6 +46,7 @@ const objects_1 = require("../../objects");
 const position_1 = require("../../position");
 const data_1 = require("./structures/data");
 const type_enum_1 = require("./structures/type_enum");
+const type_mesh_1 = require("./structures/type_mesh");
 const types_1 = require("./structures/types");
 const statics_1 = require("./structures/statics");
 const constants_1 = require("./structures/constants");
@@ -480,6 +481,13 @@ class SyntaxLogic {
             this.scope.addIdentifier(new statics_1.Statics().runSyntax(node, input));
             return true;
         }
+        else if (stru instanceof Structures.TypeMesh) {
+            const found = new type_mesh_1.TypeMesh().runSyntax(node, input);
+            if (found) {
+                this.scope.addType(found);
+            }
+            return true;
+        }
         else if (stru instanceof Structures.TypeEnum) {
             const values = new type_enum_1.TypeEnum().runSyntax(node, input).values;
             this.scope.addList(values);

package/build/src/abap/types/class_attributes.js

@@ -47,6 +47,7 @@ const data_1 = require("../5_syntax/statements/data");
 const constant_1 = require("../5_syntax/statements/constant");
 const data_2 = require("../5_syntax/structures/data");
 const type_enum_1 = require("../5_syntax/structures/type_enum");
+const type_mesh_1 = require("../5_syntax/structures/type_mesh");
 const constants_1 = require("../5_syntax/structures/constants");
 const type_definitions_1 = require("./type_definitions");
 const types_1 = require("../5_syntax/structures/types");
@@ -207,6 +208,13 @@ class Attributes {
                         //            scope.addIdentifier(attr);
                     }
                 }
+                else if (ctyp instanceof Structures.TypeMesh) {
+                    const res = new type_mesh_1.TypeMesh().runSyntax(c, input);
+                    if (res) {
+                        input.scope.addType(res);
+                        this.tlist.push({ type: res, visibility });
+                    }
+                }
                 else if (ctyp instanceof Structures.Types) {
                     const res = new types_1.Types().runSyntax(c, input);
                     if (res) {

package/build/src/registry.js

@@ -75,7 +75,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.8";
+        return "2.120.9";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/7bit_ascii.js

@@ -19,7 +19,7 @@ class SevenBitAscii {
             shortDescription: `Only allow characters from the 7bit ASCII set.`,
             extendedInformation: `https://docs.abapopenchecks.org/checks/05/
 
-https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abencharacter_set_guidl.htm
+https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abencharacter_set_guidl.html
 
 Checkes files with extensions ".abap" and ".asddls"`,
             tags: [_irule_1.RuleTag.SingleFile],

package/build/src/rules/avoid_use.js

@@ -79,7 +79,7 @@ class AvoidUse extends _abap_rule_1.ABAPRule {
             shortDescription: `Detects usage of certain statements.`,
             extendedInformation: `DEFAULT KEY: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-default-key
 
-Macros: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abenmacros_guidl.htm
+Macros: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenmacros_guidl.html
 
 STATICS: use CLASS-DATA instead
 

package/build/src/rules/cds_comment_style.js

@@ -23,7 +23,7 @@ class CDSCommentStyle {
 
 Comments starting with "--" are considered obsolete
 
-https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abencds_general_syntax_rules.htm`,
+https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abencds_general_syntax_rules.html`,
             tags: [_irule_1.RuleTag.SingleFile],
             badExample: "-- this is a comment",
             goodExample: "// this is a comment",

package/build/src/rules/chain_mainly_declarations.js

@@ -81,7 +81,7 @@ class ChainMainlyDeclarations extends _abap_rule_1.ABAPRule {
             extendedInformation: `
 https://docs.abapopenchecks.org/checks/23/
 
-https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenchained_statements_guidl.htm
+https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenchained_statements_guidl.html
 `,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
             badExample: `CALL METHOD: bar.`,

package/build/src/rules/clear_exporting_parameters.js

@@ -37,7 +37,7 @@ before reading it, so a leftover value is not accidentally used.
 
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#clear-or-overwrite-exporting-reference-parameters
 
-https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenref_transf_output_param_guidl.htm
+https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenref_transf_output_param_guidl.html
 
 Note: EXPORTING parameters passed by VALUE are always initialized and are therefore not reported.
 Reading and writing the parameter in the same statement (e.g. "ev_result = ev_result + 1") is reported,

package/build/src/rules/constructor_visibility_public.js

@@ -22,7 +22,7 @@ class ConstructorVisibilityPublic {
 This only applies to global classes.
 
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#if-your-global-class-is-create-private-leave-the-constructor-public
-https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abeninstance_constructor_guidl.htm`,
+https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abeninstance_constructor_guidl.html`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
             badExample: `CLASS zcl_foo DEFINITION PUBLIC CREATE PRIVATE.
   PRIVATE SECTION.

package/build/src/rules/exit_or_check.js

@@ -59,8 +59,8 @@ class ExitOrCheck extends _abap_rule_1.ABAPRule {
             title: "Find EXIT or CHECK outside loops",
             shortDescription: `Detects usages of EXIT or CHECK statements outside of loops.
 Use RETURN to leave procesing blocks instead.`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenleave_processing_blocks.htm
-https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abapcheck_processing_blocks.htm
+            extendedInformation: `https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenleave_processing_blocks.html
+https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abapcheck_processing_blocks.html
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#check-vs-return`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
             badExample: `CHECK is_valid = abap_true.

package/build/src/rules/expand_macros.js

@@ -21,7 +21,7 @@ class ExpandMacros extends _abap_rule_1.ABAPRule {
             key: "expand_macros",
             title: "Expand Macros",
             shortDescription: `Allows expanding macro calls with quick fixes`,
-            extendedInformation: `Macros: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abenmacros_guidl.htm
+            extendedInformation: `Macros: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenmacros_guidl.html
 
 Note that macros/DEFINE cannot be used in the ABAP Cloud programming model`,
             badExample: `DEFINE _hello.

package/build/src/rules/fm_global_parameters_obsolete.js

@@ -50,7 +50,7 @@ class FMGlobalParametersObsolete {
             key: "fm_global_parameters_obsolete",
             title: "FM Global Parameters Obsolete",
             shortDescription: `Check for function modules with global parameters`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abenglobal_parameters_obsolete.htm`,
+            extendedInformation: `https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenglobal_parameters_obsolete.html`,
             tags: [],
         };
     }

package/build/src/rules/form_tables_obsolete.js

@@ -53,7 +53,7 @@ class FormTablesObsolete extends _abap_rule_1.ABAPRule {
             key: "form_tables_obsolete",
             title: "TABLES parameters are obsolete",
             shortDescription: `Checks for TABLES parameters in forms.`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapform_tables.htm`,
+            extendedInformation: `https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abapform_tables.html`,
             tags: [_irule_1.RuleTag.SingleFile],
             badExample: `FORM update_items TABLES items.
 ENDFORM.`,

package/build/src/rules/implicit_start_of_selection.js

@@ -57,7 +57,7 @@ class ImplicitStartOfSelection extends _abap_rule_1.ABAPRule {
             shortDescription: `Add explicit selection screen event handling`,
             extendedInformation: `Only runs for executable programs
 
-https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapstart-of-selection.htm`,
+https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abapstart-of-selection.html`,
             tags: [_irule_1.RuleTag.SingleFile],
             badExample: `REPORT zfoo.
 WRITE 'hello'.`,

package/build/src/rules/method_overwrites_builtin.js

@@ -19,7 +19,7 @@ class MethodOverwritesBuiltIn extends _abap_rule_1.ABAPRule {
             key: "method_overwrites_builtin",
             title: "Method name overwrites builtin function",
             shortDescription: `Checks Method names that overwrite builtin SAP functions`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abenbuilt_in_functions_overview.htm
+            extendedInformation: `https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abenbuilt_in_functions_overview.html
 
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-obscuring-built-in-functions
 

package/build/src/rules/mix_returning.js

@@ -56,7 +56,7 @@ class MixReturning extends _abap_rule_1.ABAPRule {
             shortDescription: `Checks that methods don't have a mixture of returning and exporting/changing parameters`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-either-returning-or-exporting-or-changing-but-not-a-combination
 
-This syntax is not allowed on versions earlier than 740sp02, https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abennews-740-abap_objects.htm#!ABAP_MODIFICATION_1@1@`,
+This syntax is not allowed on versions earlier than 740sp02, https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abennews-740-abap_objects.html#!ABAP_MODIFICATION_1@1@`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Syntax],
             badExample: `CLASS lcl DEFINITION.
   PUBLIC SECTION.

package/build/src/rules/no_macros.js

@@ -54,7 +54,7 @@ class NoMacros extends _abap_rule_1.ABAPRule {
             shortDescription: `Checks that macros are not used`,
             extendedInformation: `Macros reduce readability and are difficult to debug, use methods or form routines instead.
 
-https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenmacros_guidl.htm`,
+https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenmacros_guidl.html`,
             tags: [_irule_1.RuleTag.SingleFile],
             badExample: `DEFINE _macro.
   WRITE 'hello'.

package/build/src/rules/obsolete_statement.js

@@ -131,44 +131,44 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-func
 
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-obsolete-language-elements
 
-SET EXTENDED CHECK: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapset_extended_check.htm
+SET EXTENDED CHECK: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abapset_extended_check.html
 
-IS REQUESTED: https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abenlogexp_requested.htm
+IS REQUESTED: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenlogexp_requested.html
 
-WITH HEADER LINE: https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abapdata_header_line.htm
+WITH HEADER LINE: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abapdata_header_line.html
 
-FIELD-SYMBOLS STRUCTURE: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapfield-symbols_obsolete_typing.htm
+FIELD-SYMBOLS STRUCTURE: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abapfield-symbols_obsolete_typing.html
 
-TYPE-POOLS: from 702, https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abennews-71-program_load.htm
+TYPE-POOLS: from 702, https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abennews-71-program_load.html
 
-LOAD addition: from 702, https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abennews-71-program_load.htm
+LOAD addition: from 702, https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abennews-71-program_load.html
 
-COMMUICATION: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapcommunication.htm
+COMMUICATION: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abapcommunication.html
 
-OCCURS: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapdata_occurs.htm
+OCCURS: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abapdata_occurs.html
 
-PARAMETER: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapparameter.htm
+PARAMETER: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abapparameter.html
 
-RANGES: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abapranges.htm
+RANGES: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abapranges.html
 
-PACK: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abappack.htm
+PACK: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abappack.html
 
-MOVE: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapmove_obs.htm
+MOVE: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abapmove_obs.html
 
-SELECT without INTO: https://help.sap.com/doc/abapdocu_731_index_htm/7.31/en-US/abapselect_obsolete.htm
+SELECT without INTO: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abapselect_obsolete.html
 SELECT COUNT(*) is considered okay
 
-FREE MEMORY: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abapfree_mem_id_obsolete.htm
+FREE MEMORY: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abapfree_mem_id_obsolete.html
 
-SORT BY FS: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abapsort_itab_obsolete.htm
+SORT BY FS: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abapsort_itab_obsolete.html
 
-CALL TRANSFORMATION OBJECTS: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abapcall_transformation_objects.htm
+CALL TRANSFORMATION OBJECTS: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abapcall_transformation_objects.html
 
-POSIX REGEX: https://help.sap.com/doc/abapdocu_755_index_htm/7.55/en-US/index.htm
+POSIX REGEX: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/ABENREGEX_MIGRATING_POSIX.html
 
 OCCURENCES: check for OCCURENCES vs OCCURRENCES
 
-CLIENT SPECIFIED, from 754: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/index.htm?file=abapselect_client_obsolete.htm`,
+CLIENT SPECIFIED, from 754: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abapselect_client_obsolete.html`,
             badExample: `REFRESH itab.
 
 COMPUTE foo = 2 + 2.

package/build/src/rules/pragma_style.js

@@ -24,7 +24,7 @@ class PragmaStyle extends _abap_rule_1.ABAPRule {
             title: "Pragma Style",
             shortDescription: `Check pragmas placement and case`,
             tags: [_irule_1.RuleTag.SingleFile],
-            extendedInformation: `https://help.sap.com/doc/abapdocu_cp_index_htm/CLOUD/en-US/abenpragma.htm`,
+            extendedInformation: `https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/ABENPRAGMA.html`,
             badExample: `DATA field ##NO_TEXT TYPE i.`,
             goodExample: `DATA field TYPE i ##NO_TEXT.`,
         };

package/build/src/rules/rfc_error_handling.js

@@ -54,7 +54,7 @@ class RFCErrorHandling extends _abap_rule_1.ABAPRule {
             title: "RFC error handling",
             tags: [_irule_1.RuleTag.SingleFile],
             shortDescription: `Checks that exceptions 'system_failure' and 'communication_failure' are handled in RFC calls`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abenrfc_exception.htm`,
+            extendedInformation: `https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenrfc_exception.html`,
             badExample: `CALL FUNCTION 'ZRFC'
   DESTINATION lv_rfc.`,
             goodExample: `CALL FUNCTION 'ZRFC'

package/build/src/rules/strict_sql.js

@@ -55,9 +55,9 @@ class StrictSQL extends _abap_rule_1.ABAPRule {
             key: "strict_sql",
             title: "Strict SQL",
             shortDescription: `Strict SQL`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapinto_clause.htm
+            extendedInformation: `https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abapinto_clause.html
 
-https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abenopensql_strict_mode_750.htm
+https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/ABENABAP_SQL_STRICT_MODES.html
 
 Also see separate rule sql_escape_host_variables
 

package/build/src/rules/sy_modification.js

@@ -52,7 +52,7 @@ class SyModification extends _abap_rule_1.ABAPRule {
             key: "sy_modification",
             title: "Modification of SY fields",
             shortDescription: `Finds modification of sy fields`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abensystem_fields.htm
+            extendedInformation: `https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abensystem_fields.html
 
 Changes to SY-TVAR* fields are not reported
 

package/build/src/rules/tables_declared_locally.js

@@ -53,7 +53,7 @@ class TablesDeclaredLocally extends _abap_rule_1.ABAPRule {
             key: "tables_declared_locally",
             title: "Check for locally declared TABLES",
             shortDescription: `TABLES are always global, so declare them globally`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abaptables.htm`,
+            extendedInformation: `https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-us/abaptables.html`,
             tags: [_irule_1.RuleTag.SingleFile],
             badExample: `FORM foo.
   TABLES t100.

package/package.json

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