@abaplint/cli 2.101.15 → 2.101.17

package/build/cli.js

@@ -20557,12 +20557,19 @@ class CurrentScope {
         }
     }
     addReference(usage, referencing, type, filename, extra) {
-        var _a;
+        var _a, _b;
         if (usage === undefined || type === undefined) {
             return;
         }
         const position = new _identifier_1.Identifier(usage, filename);
-        (_a = this.current) === null || _a === void 0 ? void 0 : _a.getData().references.push({ position, resolved: referencing, referenceType: type, extra });
+        if (Array.isArray(type)) {
+            for (const t of type) {
+                (_a = this.current) === null || _a === void 0 ? void 0 : _a.getData().references.push({ position, resolved: referencing, referenceType: t, extra });
+            }
+        }
+        else {
+            (_b = this.current) === null || _b === void 0 ? void 0 : _b.getData().references.push({ position, resolved: referencing, referenceType: type, extra });
+        }
     }
     addSQLConversion(fieldName, message, token) {
         var _a;
@@ -21508,6 +21515,7 @@ class TypeUtils {
         else if (type instanceof basic_1.XStringType
             || type instanceof basic_1.HexType
             || type instanceof basic_1.VoidType
+            || type instanceof basic_1.XGenericType
             || type instanceof basic_1.XSequenceType
             || type instanceof basic_1.AnyType
             || type instanceof basic_1.UnknownType) {
@@ -21624,12 +21632,15 @@ class TypeUtils {
         }
         return false;
     }
-    isAssignableStrict(source, target) {
+    isAssignableStrict(source, target, containsMethodCall = false) {
         var _a, _b, _c, _d, _e, _f;
         /*
             console.dir(source);
             console.dir(target);
         */
+        if (containsMethodCall) {
+            return this.isAssignable(source, target);
+        }
         if (source instanceof basic_1.CharacterType) {
             if (target instanceof basic_1.CharacterType) {
                 if (((_a = source.getAbstractTypeData()) === null || _a === void 0 ? void 0 : _a.derivedFromConstant) === true) {
@@ -21850,7 +21861,7 @@ class BasicTypes {
     }
     lookupQualifiedName(name) {
         var _a;
-        // argh, todo, rewrite this entire method, more argh
+        // argh, todo, rewrite this entire method, more argh, again argh
         if (name === undefined) {
             return undefined;
         }
@@ -21871,7 +21882,10 @@ class BasicTypes {
                     const stru = oo.getTypeDefinitions().getByName(subTypeName);
                     const struType = stru === null || stru === void 0 ? void 0 : stru.getType();
                     if (stru && struType instanceof basic_1.StructureType) {
-                        const f = struType.getComponentByName(fieldName);
+                        let f = struType.getComponentByName(fieldName);
+                        if (split[2] && f instanceof basic_1.StructureType) {
+                            f = f.getComponentByName(split[2]);
+                        }
                         if (f) {
                             return new _typed_identifier_1.TypedIdentifier(stru.getToken(), stru.getFilename(), f);
                         }
@@ -21893,7 +21907,10 @@ class BasicTypes {
             if (type) {
                 const stru = type.getType();
                 if (stru instanceof basic_1.StructureType) {
-                    const f = stru.getComponentByName(fieldName);
+                    let f = stru.getComponentByName(fieldName);
+                    if (split[2] && f instanceof basic_1.StructureType) {
+                        f = f.getComponentByName(split[2]);
+                    }
                     if (f) {
                         return new _typed_identifier_1.TypedIdentifier(type.getToken(), type.getFilename(), f);
                     }
@@ -22794,7 +22811,9 @@ class AttributeChain {
         if (context === undefined) {
             throw new Error("Attribute or constant \"" + name + "\" not found in \"" + def.getName() + "\"");
         }
-        scope.addReference(nameToken, context, type, filename);
+        for (const t of type) {
+            scope.addReference(nameToken, context, t, filename);
+        }
         // todo, loop, handle ArrowOrDash, ComponentName, TableExpression
         return context.getType();
     }
@@ -24598,7 +24617,7 @@ class MethodCallParam {
             if (sourceType === undefined) {
                 throw new Error("No source type determined, method source");
             }
-            else if (new _type_utils_1.TypeUtils(scope).isAssignableStrict(sourceType, targetType) === false) {
+            else if (new _type_utils_1.TypeUtils(scope).isAssignableStrict(sourceType, targetType, child.findFirstExpression(Expressions.MethodCallChain) !== undefined) === false) {
                 throw new Error("Method parameter type not compatible");
             }
         }
@@ -24699,6 +24718,9 @@ class MethodParam {
         if (concat === "TYPE C" || concat.startsWith("TYPE C ")) {
             return new _typed_identifier_1.TypedIdentifier(name.getFirstToken(), filename, new cgeneric_type_1.CGenericType(), meta);
         }
+        else if (concat === "TYPE X" || concat.startsWith("TYPE X ")) {
+            return new _typed_identifier_1.TypedIdentifier(name.getFirstToken(), filename, new basic_1.XGenericType(), meta);
+        }
         const found = new basic_types_1.BasicTypes(filename, scope).parseType(type);
         if (found) {
             return new _typed_identifier_1.TypedIdentifier(name.getFirstToken(), filename, found, meta);
@@ -25500,7 +25522,7 @@ const _typed_identifier_1 = __webpack_require__(/*! ../../types/_typed_identifie
 * DATA(bar) = VALUE #( ... ).     give error, no type can be derived
 */
 class Source {
-    runSyntax(node, scope, filename, targetType) {
+    runSyntax(node, scope, filename, targetType, writeReference = false) {
         if (node === undefined) {
             return undefined;
         }
@@ -25627,12 +25649,16 @@ class Source {
             return undefined;
         }
         let context = new unknown_type_1.UnknownType("todo, Source type");
+        const type = [_reference_1.ReferenceType.DataReadReference];
+        if (writeReference) {
+            type.push(_reference_1.ReferenceType.DataWriteReference);
+        }
         while (children.length >= 0) {
             if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.MethodCallChain) {
                 context = new method_call_chain_1.MethodCallChain().runSyntax(first, scope, filename, targetType);
             }
             else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.FieldChain) {
-                context = new field_chain_1.FieldChain().runSyntax(first, scope, filename, _reference_1.ReferenceType.DataReadReference);
+                context = new field_chain_1.FieldChain().runSyntax(first, scope, filename, type);
             }
             else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.StringTemplate) {
                 context = new string_template_1.StringTemplate().runSyntax(first, scope, filename);
@@ -25645,15 +25671,14 @@ class Source {
             }
             else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.Dereference) {
                 context = new dereference_1.Dereference().runSyntax(context);
-            }
-            else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.ArrowOrDash) {
+                //      } else if (first instanceof ExpressionNode && first.get() instanceof Expressions.ArrowOrDash) {
                 //        console.dir("dash");
             }
             else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.ComponentChain) {
                 context = new component_chain_1.ComponentChain().runSyntax(context, first, scope, filename);
             }
             else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.AttributeChain) {
-                context = new attribute_chain_1.AttributeChain().runSyntax(context, first, scope, filename, _reference_1.ReferenceType.DataReadReference);
+                context = new attribute_chain_1.AttributeChain().runSyntax(context, first, scope, filename, type);
             }
             first = children.shift();
             if (first === undefined) {
@@ -29285,12 +29310,13 @@ class Loop {
         if (target === undefined) {
             target = node.findDirectExpression(Expressions.FSTarget);
         }
+        const write = (loopTarget === null || loopTarget === void 0 ? void 0 : loopTarget.findDirectTokenByText("ASSIGNING")) !== undefined;
         const sources = node.findDirectExpressions(Expressions.Source);
         let firstSource = node.findDirectExpression(Expressions.SimpleSource2);
         if (firstSource === undefined) {
             firstSource = sources[0];
         }
-        let sourceType = firstSource ? new source_1.Source().runSyntax(firstSource, scope, filename, targetType) : undefined;
+        let sourceType = firstSource ? new source_1.Source().runSyntax(firstSource, scope, filename, targetType, write) : undefined;
         let rowType = undefined;
         const concat = node.concatTokens().toUpperCase();
         if (sourceType === undefined) {
@@ -34708,7 +34734,7 @@ class HexType extends _abstract_type_1.AbstractType {
     constructor(length, qualifiedName) {
         super({ qualifiedName: qualifiedName });
         if (length <= 0) {
-            throw new Error("Bad LENGTHm, Hex");
+            throw new Error("Bad LENGTH, Hex");
         }
         this.length = length;
     }
@@ -34787,6 +34813,7 @@ __exportStar(__webpack_require__(/*! ./time_type */ "./node_modules/@abaplint/co
 __exportStar(__webpack_require__(/*! ./unknown_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/unknown_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./utc_long_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/utc_long_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./void_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/void_type.js"), exports);
+__exportStar(__webpack_require__(/*! ./xgeneric_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/xgeneric_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./xsequence_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/xsequence_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./xstring_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/xstring_type.js"), exports);
 //# sourceMappingURL=index.js.map
@@ -35427,6 +35454,39 @@ exports.VoidType = VoidType;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/types/basic/xgeneric_type.js":
+/*!*********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/types/basic/xgeneric_type.js ***!
+  \*********************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.XGenericType = void 0;
+const _abstract_type_1 = __webpack_require__(/*! ./_abstract_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/_abstract_type.js");
+class XGenericType extends _abstract_type_1.AbstractType {
+    toText() {
+        return "```x```";
+    }
+    isGeneric() {
+        return true;
+    }
+    toABAP() {
+        throw new Error("x, generic");
+    }
+    containsVoid() {
+        return false;
+    }
+    toCDS() {
+        return "abap.TODO_CGENERIC";
+    }
+}
+exports.XGenericType = XGenericType;
+//# sourceMappingURL=xgeneric_type.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/types/basic/xsequence_type.js":
 /*!**********************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/types/basic/xsequence_type.js ***!
@@ -48352,7 +48412,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.101.15";
+        return "2.101.17";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -48993,7 +49053,9 @@ const Expressions = __webpack_require__(/*! ../abap/2_statements/expressions */
 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 position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
 const __1 = __webpack_require__(/*! .. */ "./node_modules/@abaplint/core/build/src/index.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 class AlignParametersConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.AlignParametersConf = AlignParametersConf;
@@ -49020,8 +49082,9 @@ https://github.com/SAP/styleguides/blob/master/clean-abap/CleanABAP.md#align-par
 
 Does not take effect on non functional method calls, use https://rules.abaplint.org/functional_writing/
 
-Also https://rules.abaplint.org/max_one_method_parameter_per_line/ can help aligning parameter syntax`,
-            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Styleguide],
+If parameters are on the same row, no issues are reported, see
+https://rules.abaplint.org/max_one_method_parameter_per_line/ for splitting parameters to lines`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix],
             badExample: `CALL FUNCTION 'FOOBAR'
   EXPORTING
     foo = 2
@@ -49083,16 +49146,28 @@ DATA(sdf) = VALUE type(
             return undefined;
         }
         let expectedEqualsColumn = 0;
+        let row = 0;
         for (const p of candidate.parameters) {
             const currentCol = p.left.getLastToken().getCol() + p.left.getLastToken().getStr().length + 1;
+            if (p.eq.getRow() === row) {
+                return undefined;
+            }
+            row = p.eq.getRow();
             if (currentCol > expectedEqualsColumn) {
                 expectedEqualsColumn = currentCol;
             }
         }
         for (const p of candidate.parameters) {
             if (p.eq.getCol() !== expectedEqualsColumn) {
+                let fix;
+                if (p.eq.getCol() < expectedEqualsColumn) {
+                    fix = edit_helper_1.EditHelper.insertAt(file, p.eq, " ".repeat(expectedEqualsColumn - p.eq.getCol()));
+                }
+                else {
+                    fix = edit_helper_1.EditHelper.deleteRange(file, new position_1.Position(p.eq.getRow(), expectedEqualsColumn), p.eq);
+                }
                 const message = "Align parameters to column " + expectedEqualsColumn;
-                return issue_1.Issue.atPosition(file, p.eq, message, this.getMetadata().key, this.getConfig().severity);
+                return issue_1.Issue.atPosition(file, p.eq, message, this.getMetadata().key, this.getConfig().severity, fix);
             }
         }
         return undefined;
@@ -56885,11 +56960,11 @@ DATA lt_bar TYPE STANDARD TABLE OF ty.`,
             const concat = tt.concatTokens().toUpperCase();
             if (concat.includes("TYPE TABLE OF")) {
                 const message = "Specify table type";
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                issues.push(issue_1.Issue.atPosition(file, tt.getFirstToken().getStart(), message, this.getMetadata().key, this.conf.severity));
             }
             else if (concat.includes(" WITH ") === false && concat.includes(" RANGE OF ") === false) {
                 const message = "Specify table key";
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                issues.push(issue_1.Issue.atPosition(file, tt.getFirstToken().getStart(), message, this.getMetadata().key, this.conf.severity));
             }
         }
         return issues;
@@ -64669,7 +64744,6 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.SelectSingleFullKey = exports.SelectSingleFullKeyConf = void 0;
 const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.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 __1 = __webpack_require__(/*! .. */ "./node_modules/@abaplint/core/build/src/index.js");
 class SelectSingleFullKeyConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
@@ -64689,7 +64763,7 @@ class SelectSingleFullKey {
             shortDescription: `Detect SELECT SINGLE which are possibily not unique`,
             extendedInformation: `Table definitions must be known, ie. inside the errorNamespace`,
             pseudoComment: "EC CI_NOORDER",
-            tags: [_irule_1.RuleTag.Experimental],
+            tags: [],
         };
     }
     initialize(reg) {
@@ -66982,6 +67056,12 @@ const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./
 const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 class UnnecessaryPragmaConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        /** Allow NO_TEXT in global CLAS and INTF definitions,
+            its added automatically by SE24 in some cases where it should not */
+        this.allowNoTextGlobal = false;
+    }
 }
 exports.UnnecessaryPragmaConf = UnnecessaryPragmaConf;
 class UnnecessaryPragma extends _abap_rule_1.ABAPRule {
@@ -67031,6 +67111,7 @@ ENDIF.`,
     runParsed(file) {
         const issues = [];
         let noHandler = false;
+        let globalDefinition = false;
         const statements = file.getStatements();
         for (let i = 0; i < statements.length; i++) {
             const statement = statements[i];
@@ -67038,6 +67119,16 @@ ENDIF.`,
             if (statement.get() instanceof Statements.EndTry) {
                 noHandler = false;
             }
+            else if (statement.get() instanceof Statements.ClassDefinition
+                || statement.get() instanceof Statements.Interface) {
+                if (statement.findDirectExpression(Expressions.ClassGlobal)) {
+                    globalDefinition = true;
+                }
+            }
+            else if (statement.get() instanceof Statements.EndClass
+                || statement.get() instanceof Statements.EndInterface) {
+                globalDefinition = false;
+            }
             else if (statement.get() instanceof _statement_1.Comment) {
                 continue;
             }
@@ -67050,9 +67141,16 @@ ENDIF.`,
             else {
                 noHandler = this.containsNoHandler(statement, statements[i + 1]);
             }
-            issues.push(...this.checkText(statement, file));
+            if (this.getConfig().allowNoTextGlobal === true && globalDefinition === true) {
+                // skip
+            }
+            else {
+                issues.push(...this.checkText(statement, file));
+            }
             issues.push(...this.checkNeeded(statement, file));
-            issues.push(...this.checkSubrc(statement, nextStatement, file));
+            if (globalDefinition === false) {
+                issues.push(...this.checkSubrc(statement, nextStatement, file));
+            }
         }
         return issues;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.101.15",
+    "version": "2.101.17",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,25 +38,25 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.101.15",
+        "@abaplint/core": "^2.101.17",
         "@types/chai": "^4.3.5",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.2",
         "@types/mocha": "^10.0.1",
-        "@types/node": "^20.2.5",
+        "@types/node": "^20.2.6",
         "@types/progress": "^2.0.5",
         "chai": "^4.3.7",
         "chalk": "^5.2.0",
         "eslint": "^8.42.0",
         "glob": "^7.2.3",
         "json5": "^2.2.3",
-        "memfs": "^3.5.2",
+        "memfs": "^3.5.3",
         "minimist": "^1.2.8",
         "mocha": "^10.2.0",
         "progress": "^2.0.3",
         "typescript": "^5.1.3",
-        "webpack": "^5.85.1",
-        "webpack-cli": "^5.1.3",
+        "webpack": "^5.86.0",
+        "webpack-cli": "^5.1.4",
         "xml-js": "^1.6.11"
     },
     "dependencies": {}