@abaplint/cli 2.110.8 → 2.111.1

package/build/cli.js

@@ -21432,6 +21432,23 @@ class CurrentScope {
         }
         this.current.getData().vars[upper] = identifier;
     }
+    addNamedIdentifierToParent(name, identifier) {
+        if (this.current === undefined) {
+            return;
+        }
+        const parent = this.current.getParent();
+        if (parent === undefined) {
+            return;
+        }
+        const upper = name.toUpperCase();
+        if (parent.getData().vars[upper] !== undefined) {
+            throw new Error(`Variable name "${name}" already defined`);
+        }
+        else if (this.isOO() && parent.getData().types[upper] !== undefined) {
+            throw new Error(`"${name}" already defined`);
+        }
+        parent.getData().vars[upper] = identifier;
+    }
     addIdentifier(identifier) {
         if (identifier === undefined) {
             return;
@@ -23047,17 +23064,18 @@ class BasicTypes {
                     }
                 }
                 else { // field name
-                    let sub = undefined;
                     if (type instanceof Types.TableType) {
                         type = type.getRowType();
                     }
                     if (type instanceof Types.StructureType) {
-                        sub = type.getComponentByName(child.getFirstToken().getStr());
+                        type = type.getComponentByName(child.getFirstToken().getStr());
+                        if (type === undefined) {
+                            return new Types.UnknownType("Type error, field not part of structure " + fullName);
+                        }
                     }
-                    if (sub === undefined) {
+                    else if (!(type instanceof Types.VoidType)) {
                         return new Types.UnknownType("Type error, field not part of structure " + fullName);
                     }
-                    type = sub;
                 }
             }
             if (type instanceof Types.VoidType) {
@@ -24221,7 +24239,7 @@ class ComponentCompareSimple {
                     }
                 }
                 else {
-                    throw "ComponentCompareSimple, unexpected node";
+                    throw new Error("ComponentCompareSimple, unexpected node");
                 }
             }
         }
@@ -24605,12 +24623,18 @@ exports.Default = void 0;
 const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const field_chain_1 = __webpack_require__(/*! ./field_chain */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/field_chain.js");
 const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
+const constant_1 = __webpack_require__(/*! ./constant */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/constant.js");
 class Default {
     runSyntax(node, scope, filename) {
         const chain = node.findDirectExpression(Expressions.FieldChain);
         if (chain) {
-            new field_chain_1.FieldChain().runSyntax(chain, scope, filename, _reference_1.ReferenceType.DataReadReference);
+            return new field_chain_1.FieldChain().runSyntax(chain, scope, filename, _reference_1.ReferenceType.DataReadReference);
+        }
+        const constant = node.findDirectExpression(Expressions.Constant);
+        if (constant) {
+            return new constant_1.Constant().runSyntax(constant);
         }
+        return undefined;
     }
 }
 exports.Default = Default;
@@ -24749,29 +24773,25 @@ const source_field_symbol_1 = __webpack_require__(/*! ./source_field_symbol */ "
 const source_field_1 = __webpack_require__(/*! ./source_field */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source_field.js");
 class FieldChain {
     runSyntax(node, scope, filename, refType) {
-        var _a, _b, _c, _d, _e;
-        const children = node.getChildren();
-        let context = undefined;
-        try {
-            context = this.findTop(children[0], scope, filename, refType);
-        }
-        catch (error) {
+        var _a, _b, _c;
+        if (((_a = node.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.SourceField
+            && node.findDirectExpression(Expressions.ComponentName)) {
+            // workaround for names with dashes, eg. "sy-repid"
             const concat = node.concatTokens();
-            if (concat.includes("-") && ((_a = node.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.SourceField) {
-                // workaround for names with dashes, eg. "sy-repid"
-                const offset = ((_b = node.findDirectExpression(Expressions.FieldOffset)) === null || _b === void 0 ? void 0 : _b.concatTokens()) || "";
-                const length = ((_c = node.findDirectExpression(Expressions.FieldLength)) === null || _c === void 0 ? void 0 : _c.concatTokens()) || "";
-                const found = scope.findVariable(concat.replace(offset, "").replace(length, ""));
-                if (found) {
-                    if (refType) {
-                        scope.addReference(node.getFirstToken(), found, refType, filename);
-                    }
-                    // this is not completely correct, but will work, dashes in names is a mess anyhow
-                    return found.getType();
+            const offset = ((_b = node.findDirectExpression(Expressions.FieldOffset)) === null || _b === void 0 ? void 0 : _b.concatTokens()) || "";
+            const length = ((_c = node.findDirectExpression(Expressions.FieldLength)) === null || _c === void 0 ? void 0 : _c.concatTokens()) || "";
+            const found = scope.findVariable(concat.replace(offset, "").replace(length, ""));
+            if (found) {
+                if (refType) {
+                    scope.addReference(node.getFirstToken(), found, refType, filename);
                 }
+                // this is not completely correct, but will work, dashes in names is a mess anyhow
+                return found.getType();
             }
-            throw error;
         }
+        let context = undefined;
+        const children = node.getChildren();
+        context = this.findTop(children[0], scope, filename, refType);
         for (let i = 1; i < children.length; i++) {
             const current = children[i];
             if (current === undefined) {
@@ -24822,30 +24842,7 @@ class FieldChain {
                 if (context instanceof basic_1.TableType && context.isWithHeader()) {
                     context = context.getRowType();
                 }
-                try {
-                    context = new component_name_1.ComponentName().runSyntax(context, current);
-                }
-                catch (error) {
-                    const concat = node.concatTokens();
-                    if (concat.includes("-")) {
-                        // workaround for names with dashes, eg. "sy-repid"
-                        const offset = ((_d = node.findDirectExpression(Expressions.FieldOffset)) === null || _d === void 0 ? void 0 : _d.concatTokens()) || "";
-                        const length = ((_e = node.findDirectExpression(Expressions.FieldLength)) === null || _e === void 0 ? void 0 : _e.concatTokens()) || "";
-                        const found = scope.findVariable(concat.replace(offset, "").replace(length, ""));
-                        if (found) {
-                            if (refType) {
-                                scope.addReference(node.getFirstToken(), found, refType, filename);
-                            }
-                            context = found.getType();
-                        }
-                        else {
-                            throw error;
-                        }
-                    }
-                    else {
-                        throw error;
-                    }
-                }
+                context = new component_name_1.ComponentName().runSyntax(context, current);
             }
             else if (current instanceof nodes_1.ExpressionNode
                 && current.get() instanceof Expressions.TableExpression) {
@@ -26560,10 +26557,11 @@ const sql_source_1 = __webpack_require__(/*! ./sql_source */ "./node_modules/@ab
 const sql_compare_1 = __webpack_require__(/*! ./sql_compare */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_compare.js");
 const sql_order_by_1 = __webpack_require__(/*! ./sql_order_by */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_order_by.js");
 const dynamic_1 = __webpack_require__(/*! ./dynamic */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/dynamic.js");
+const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const isSimple = /^\w+$/;
 class Select {
     runSyntax(node, scope, filename, skipImplicitInto = false) {
-        var _a, _b;
+        var _a;
         const token = node.getFirstToken();
         const from = node.findDirectExpression(Expressions.SQLFrom);
         const dbSources = from ? new sql_from_1.SQLFrom().runSyntax(from, scope, filename) : [];
@@ -26593,9 +26591,15 @@ class Select {
             const fields = (_a = node.findFirstExpression(Expressions.SQLAggregation)) === null || _a === void 0 ? void 0 : _a.concatTokens();
             const c = new RegExp(/^count\(\s*\*\s*\)$/, "i");
             if (fields === undefined || c.test(fields) === false) {
-                const name = (_b = from === null || from === void 0 ? void 0 : from.findDirectExpression(Expressions.SQLFromSource)) === null || _b === void 0 ? void 0 : _b.concatTokens();
-                if (name && scope.findVariable(name) === undefined) {
-                    throw new Error(`Target variable ${name} not found in scope`);
+                const nameToken = from === null || from === void 0 ? void 0 : from.findDirectExpression(Expressions.SQLFromSource);
+                if (nameToken) {
+                    const found = scope.findVariable(nameToken.concatTokens());
+                    if (found) {
+                        scope.addReference(nameToken.getFirstToken(), found, _reference_1.ReferenceType.DataWriteReference, filename);
+                    }
+                    else {
+                        throw new Error(`Target variable ${nameToken.concatTokens()} not found in scope`);
+                    }
                 }
             }
         }
@@ -32755,6 +32759,7 @@ exports.Tables = void 0;
 const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const _typed_identifier_1 = __webpack_require__(/*! ../../types/_typed_identifier */ "./node_modules/@abaplint/core/build/src/abap/types/_typed_identifier.js");
 const unknown_type_1 = __webpack_require__(/*! ../../types/basic/unknown_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/unknown_type.js");
+const _scope_type_1 = __webpack_require__(/*! ../_scope_type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_scope_type.js");
 class Tables {
     runSyntax(node, scope, filename) {
         var _a, _b;
@@ -32770,7 +32775,13 @@ class Tables {
         const found = (_b = scope.getDDIC()) === null || _b === void 0 ? void 0 : _b.lookupTableOrView(name);
         if (found) {
             scope.getDDICReferences().addUsing(scope.getParentObj(), { object: found.object, filename: filename, token: nameToken });
-            scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(nameToken, filename, found.type));
+            if (scope.getType() === _scope_type_1.ScopeType.Form || scope.getType() === _scope_type_1.ScopeType.FunctionModule) {
+                // hoist TABLES definitions to global scope
+                scope.addNamedIdentifierToParent(nameToken.getStr(), new _typed_identifier_1.TypedIdentifier(nameToken, filename, found.type));
+            }
+            else {
+                scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(nameToken, filename, found.type));
+            }
             return;
         }
         // this should never happen,
@@ -51788,7 +51799,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.110.8";
+        return "2.111.1";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -62658,6 +62669,7 @@ __exportStar(__webpack_require__(/*! ./superclass_final */ "./node_modules/@abap
 __exportStar(__webpack_require__(/*! ./superfluous_value */ "./node_modules/@abaplint/core/build/src/rules/superfluous_value.js"), exports);
 __exportStar(__webpack_require__(/*! ./sy_modification */ "./node_modules/@abaplint/core/build/src/rules/sy_modification.js"), exports);
 __exportStar(__webpack_require__(/*! ./tabl_enhancement_category */ "./node_modules/@abaplint/core/build/src/rules/tabl_enhancement_category.js"), exports);
+__exportStar(__webpack_require__(/*! ./tables_declared_locally */ "./node_modules/@abaplint/core/build/src/rules/tables_declared_locally.js"), exports);
 __exportStar(__webpack_require__(/*! ./try_without_catch */ "./node_modules/@abaplint/core/build/src/rules/try_without_catch.js"), exports);
 __exportStar(__webpack_require__(/*! ./type_form_parameters */ "./node_modules/@abaplint/core/build/src/rules/type_form_parameters.js"), exports);
 __exportStar(__webpack_require__(/*! ./types_naming */ "./node_modules/@abaplint/core/build/src/rules/types_naming.js"), exports);
@@ -64041,7 +64053,7 @@ class MacroNaming extends _abap_rule_1.ABAPRule {
             key: "macro_naming",
             title: "Macro naming conventions",
             shortDescription: `Allows you to enforce a pattern for macro definitions`,
-            extendedInformation: `Use rule "avoid_use" to avoid macros alotogether.`,
+            extendedInformation: `Use rule "avoid_use" to avoid macros altogether.`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
             badExample: `DEFINE something.
 END-OF-DEFINITION.`,
@@ -71195,6 +71207,72 @@ exports.TABLEnhancementCategory = TABLEnhancementCategory;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/tables_declared_locally.js":
+/*!********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/tables_declared_locally.js ***!
+  \********************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.TablesDeclaredLocally = exports.TablesDeclaredLocallyConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const Structures = __webpack_require__(/*! ../abap/3_structures/structures */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.js");
+const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+class TablesDeclaredLocallyConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.TablesDeclaredLocallyConf = TablesDeclaredLocallyConf;
+class TablesDeclaredLocally extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new TablesDeclaredLocallyConf();
+    }
+    getMetadata() {
+        return {
+            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`,
+            tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `FORM foo.
+  TABLES t100.
+ENDFORM.`,
+            goodExample: `TABLES t000.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        const structure = file.getStructure();
+        if (structure === undefined) {
+            return issues;
+        }
+        const procedures = structure.findAllStructuresMulti([Structures.Form, Structures.FunctionModule]);
+        for (const p of procedures) {
+            const tablesStatement = p.findFirstStatement(Statements.Tables);
+            if (tablesStatement) {
+                const message = "Declare TABLES globaly";
+                const issue = issue_1.Issue.atStatement(file, tablesStatement, message, this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+            }
+        }
+        return issues;
+    }
+}
+exports.TablesDeclaredLocally = TablesDeclaredLocally;
+//# sourceMappingURL=tables_declared_locally.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/try_without_catch.js":
 /*!**************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/try_without_catch.js ***!

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.110.8",
+    "version": "2.111.1",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.110.8",
+        "@abaplint/core": "^2.111.1",
         "@types/chai": "^4.3.16",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",