@abaplint/cli 2.110.8 → 2.111.0

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) {
@@ -26560,10 +26578,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 +26612,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 +32780,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 +32796,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 +51820,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.110.8";
+        return "2.111.0";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -62658,6 +62690,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 +64074,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 +71228,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.0",
     "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.0",
         "@types/chai": "^4.3.16",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",