@abaplint/cli 2.113.225 → 2.113.227

package/build/cli.js

@@ -31853,6 +31853,38 @@ exports.FetchNextCursor = FetchNextCursor;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/field_group.js":
+/*!***************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/field_group.js ***!
+  \***************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.FieldGroup = 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 basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+class FieldGroup {
+    runSyntax(node, input) {
+        const nameExpression = node.findFirstExpression(Expressions.Field);
+        if (nameExpression === undefined) {
+            return;
+        }
+        const name = nameExpression === null || nameExpression === void 0 ? void 0 : nameExpression.concatTokens();
+        if (name === undefined) {
+            return;
+        }
+        const id = new _typed_identifier_1.TypedIdentifier(nameExpression === null || nameExpression === void 0 ? void 0 : nameExpression.getFirstToken(), input.filename, basic_1.VoidType.get("FIELD-GROUP"));
+        input.scope.addIdentifier(id);
+    }
+}
+exports.FieldGroup = FieldGroup;
+//# sourceMappingURL=field_group.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/fieldsymbol.js":
 /*!***************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/fieldsymbol.js ***!
@@ -32036,6 +32068,30 @@ exports.Format = Format;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/free.js":
+/*!********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/free.js ***!
+  \********************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.Free = void 0;
+const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js");
+class Free {
+    runSyntax(node, input) {
+        for (const t of node.findDirectExpressions(Expressions.Target)) {
+            target_1.Target.runSyntax(t, input);
+        }
+    }
+}
+exports.Free = Free;
+//# sourceMappingURL=free.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_badi.js":
 /*!************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_badi.js ***!
@@ -32408,7 +32464,7 @@ const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modu
 const assert_error_1 = __webpack_require__(/*! ../assert_error */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/assert_error.js");
 class IncludeType {
     runSyntax(node, input) {
-        var _a, _b, _c;
+        var _a, _b, _c, _d, _e;
         const components = [];
         const iname = node.findFirstExpression(Expressions.TypeName);
         if (iname === undefined) {
@@ -32423,7 +32479,10 @@ class IncludeType {
                 ityp = found;
             }
         }
-        const as = (_b = node.findExpressionAfterToken("AS")) === null || _b === void 0 ? void 0 : _b.concatTokens();
+        else if (ityp instanceof basic_1.UnknownType) {
+            ityp = (_c = (_b = input.scope.findVariable(name)) === null || _b === void 0 ? void 0 : _b.getType()) !== null && _c !== void 0 ? _c : ityp;
+        }
+        const as = (_d = node.findExpressionAfterToken("AS")) === null || _d === void 0 ? void 0 : _d.concatTokens();
         if (as && ityp instanceof basic_1.StructureType) {
             ityp = new basic_1.StructureType(ityp.getComponents().concat([{
                     name: as,
@@ -32434,7 +32493,10 @@ class IncludeType {
         else if (ityp instanceof basic_1.TableType && isStructure) {
             ityp = ityp.getRowType();
         }
-        else if (ityp instanceof basic_1.VoidType) {
+        if (ityp instanceof basic_1.VoidType) {
+            return ityp;
+        }
+        else if (ityp instanceof basic_1.UnknownType) {
             return ityp;
         }
         if (!(ityp instanceof basic_1.StructureType)) {
@@ -32442,7 +32504,7 @@ class IncludeType {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, iname.getFirstToken(), message));
             return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
         }
-        const suffix = (_c = node.findExpressionAfterToken("SUFFIX")) === null || _c === void 0 ? void 0 : _c.concatTokens();
+        const suffix = (_e = node.findExpressionAfterToken("SUFFIX")) === null || _e === void 0 ? void 0 : _e.concatTokens();
         if (suffix && ityp instanceof basic_1.StructureType) {
             const components = [];
             for (const c of ityp.getComponents()) {
@@ -34002,6 +34064,30 @@ exports.Receive = Receive;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/refresh.js":
+/*!***********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/refresh.js ***!
+  \***********************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.Refresh = void 0;
+const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js");
+class Refresh {
+    runSyntax(node, input) {
+        for (const t of node.findDirectExpressions(Expressions.Target)) {
+            target_1.Target.runSyntax(t, input);
+        }
+    }
+}
+exports.Refresh = Refresh;
+//# sourceMappingURL=refresh.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/replace.js":
 /*!***********************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/replace.js ***!
@@ -35480,46 +35566,6 @@ class Data {
             }
             else if (c instanceof nodes_1.StatementNode && ctyp instanceof Statements.IncludeType) {
                 // INCLUDES
-                /*
-                const typeToken = c.findFirstExpression(Expressions.TypeName)?.getFirstToken();
-                const typeName = typeToken?.getStr();
-        
-                let foundId = input.scope.findType(typeName);
-                if (foundId === undefined) {
-                  foundId = input.scope.findVariable(typeName);
-                }
-        
-                let found = foundId?.getType();
-                if (found === undefined) {
-                  const f = input.scope.getDDIC().lookupTableOrView(typeName).type;
-                  if (f instanceof TypedIdentifier) {
-                    found = f.getType();
-                  } else {
-                    found = f;
-                  }
-                } else {
-                  input.scope.addReference(typeToken, foundId, ReferenceType.TypeReference, input.filename);
-                }
-                if (found instanceof Basic.VoidType) {
-                  if (table === true) {
-                    const ttyp = new Basic.TableType(found, {withHeader: true, keyType: Basic.TableKeyType.default});
-                    return new TypedIdentifier(name, input.filename, ttyp);
-                  } else {
-                    return new TypedIdentifier(name, input.filename, found);
-                  }
-                }
-                if (found instanceof Basic.UnknownType) {
-                  return new TypedIdentifier(name, input.filename, new Basic.UnknownType("unknown type, " + typeName));
-                }
-                if (found instanceof Basic.TableType && found.isWithHeader()) {
-                  found = found.getRowType();
-                }
-                if (!(found instanceof Basic.StructureType)) {
-                  const message = "not structured, " + typeName;
-                  input.issues.push(syntaxIssue(input, typeToken!, message));
-                  return new TypedIdentifier(name, input.filename, Basic.VoidType.get(CheckSyntaxKey));
-                }
-        */
                 const found = new include_type_1.IncludeType().runSyntax(c, input);
                 if (found instanceof Basic.VoidType) {
                     if (table === true) {
@@ -35530,6 +35576,9 @@ class Data {
                         return new _typed_identifier_1.TypedIdentifier(name, input.filename, found);
                     }
                 }
+                else if (found instanceof Basic.UnknownType) {
+                    return new _typed_identifier_1.TypedIdentifier(name, input.filename, found);
+                }
                 for (const c of found) {
                     components.push(c);
                 }
@@ -35812,7 +35861,7 @@ class Types {
                 }
                 else if (ctyp instanceof Statements.IncludeType) {
                     const found = new include_type_1.IncludeType().runSyntax(c, input);
-                    if (found instanceof basic_1.VoidType) {
+                    if (found instanceof basic_1.VoidType || found instanceof Basic.UnknownType) {
                         voidd = found;
                     }
                     else {
@@ -35921,6 +35970,8 @@ const do_1 = __webpack_require__(/*! ./statements/do */ "./node_modules/@abaplin
 const concatenate_1 = __webpack_require__(/*! ./statements/concatenate */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/concatenate.js");
 const call_function_1 = __webpack_require__(/*! ./statements/call_function */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/call_function.js");
 const clear_1 = __webpack_require__(/*! ./statements/clear */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/clear.js");
+const refresh_1 = __webpack_require__(/*! ./statements/refresh */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/refresh.js");
+const free_1 = __webpack_require__(/*! ./statements/free */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/free.js");
 const replace_1 = __webpack_require__(/*! ./statements/replace */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/replace.js");
 const get_bit_1 = __webpack_require__(/*! ./statements/get_bit */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_bit.js");
 const raise_1 = __webpack_require__(/*! ./statements/raise */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/raise.js");
@@ -36003,6 +36054,7 @@ const modify_entities_1 = __webpack_require__(/*! ./statements/modify_entities *
 const commit_entities_1 = __webpack_require__(/*! ./statements/commit_entities */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/commit_entities.js");
 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 field_group_1 = __webpack_require__(/*! ./statements/field_group */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/field_group.js");
 // -----------------------------------
 const map = {};
 function addToMap(handler) {
@@ -36046,6 +36098,8 @@ if (Object.keys(map).length === 0) {
     addToMap(new delete_internal_1.DeleteInternal());
     addToMap(new delete_cluster_1.DeleteCluster());
     addToMap(new clear_1.Clear());
+    addToMap(new free_1.Free());
+    addToMap(new refresh_1.Refresh());
     addToMap(new receive_1.Receive());
     addToMap(new get_bit_1.GetBit());
     addToMap(new class_local_friends_1.ClassLocalFriends());
@@ -36133,6 +36187,7 @@ if (Object.keys(map).length === 0) {
     addToMap(new translate_1.Translate());
     addToMap(new modify_internal_1.ModifyInternal());
     addToMap(new read_textpool_1.ReadTextpool());
+    addToMap(new field_group_1.FieldGroup());
     addToMap(new editor_call_1.EditorCall());
     addToMap(new delete_memory_1.DeleteMemory());
 }
@@ -54606,7 +54661,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.225";
+        return "2.113.227";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -56585,6 +56640,10 @@ DATA END OF foo.`,
             if (t.getChildren().length !== 3) {
                 continue;
             }
+            const data = t.findFirstStatement(Statements.DataBegin);
+            if (data === null || data === void 0 ? void 0 : data.findDirectTokenByText("OCCURS")) {
+                continue;
+            }
             if (t.findFirstStatement(Statements.IncludeType)) {
                 const token = t.getFirstToken();
                 const message = "DATA BEGIN with single INCLUDE";
@@ -70116,6 +70175,12 @@ class ObsoleteStatementConf extends _basic_rule_config_1.BasicRuleConfig {
         this.formDefinition = true;
         /** Check for FORM IMPLEMENTATION */
         this.formImplementation = true;
+        /** Check for COMMON PART */
+        this.commonPart = true;
+        /** Check for FIELD-GROUPS */
+        this.fieldGroups = true;
+        /** Check for REPLACE INTO */
+        this.replaceInto = true;
     }
 }
 exports.ObsoleteStatementConf = ObsoleteStatementConf;
@@ -70227,10 +70292,22 @@ ENDIF.`,
                 const issue = issue_1.Issue.atStatement(file, staNode, "COMMUNICATION is obsolete", this.getMetadata().key, this.conf.severity);
                 issues.push(issue);
             }
+            if (this.conf.commonPart && sta instanceof Statements.DataBegin && staNode.findDirectTokenByText("COMMON")) {
+                const issue = issue_1.Issue.atStatement(file, staNode, "COMMON PART is obsolete", this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+            }
+            if (this.conf.replaceInto && sta instanceof Statements.Replace && staNode.findDirectTokenByText("INTO")) {
+                const issue = issue_1.Issue.atStatement(file, staNode, "REPLACE INTO is obsolete", this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+            }
             if (this.conf.pack && sta instanceof Statements.Pack) {
                 const issue = issue_1.Issue.atStatement(file, staNode, "PACK is obsolete", this.getMetadata().key, this.conf.severity);
                 issues.push(issue);
             }
+            if (this.conf.fieldGroups && sta instanceof Statements.FieldGroup) {
+                const issue = issue_1.Issue.atStatement(file, staNode, "FIELD-GROUPS is obsolete", this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+            }
             if (this.conf.parameter && sta instanceof Statements.Parameter) {
                 const token = staNode.getFirstToken();
                 if (token.getStr().toUpperCase() === "PARAMETER") {

package/package.json

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