@abaplint/core 2.119.35 → 2.119.36

package/build/src/objects/function_group.js

@@ -169,9 +169,10 @@ class FunctionGroup extends _abap_object_1.ABAPObject {
         }
     }
     findTextFile() {
-        const search = this.getName() + ".fugr.sapl" + this.getName() + ".xml";
+        const name = this.getName().replace(/\//g, "#");
+        const regex = new RegExp(name + "\\.fugr\\.(#\\w+#)?sapl" + name.replace(/^#\w+#/, "") + "\\.xml", "i");
         for (const f of this.getFiles()) {
-            if (f.getFilename().includes(search.toLowerCase())) {
+            if (regex.test(f.getFilename())) {
                 return f;
             }
         }

package/build/src/registry.js

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

package/build/src/rules/prefer_inline.js

@@ -41,6 +41,7 @@ const _irule_1 = require("./_irule");
 const version_1 = require("../version");
 const _abap_object_1 = require("../objects/_abap_object");
 const syntax_1 = require("../abap/5_syntax/syntax");
+const _typed_identifier_1 = require("../abap/types/_typed_identifier");
 const _scope_type_1 = require("../abap/5_syntax/_scope_type");
 const _reference_1 = require("../abap/5_syntax/_reference");
 const edit_helper_1 = require("../edit_helper");
@@ -141,10 +142,16 @@ DATA(percentage) = CONV decfloat34( comment_number / abs_statement_number ) * 10
             }
             const file = obj.getABAPFileByName(identifier.getFilename());
             const writeStatement = edit_helper_1.EditHelper.findStatement(next, file);
+            if (writeStatement === undefined) {
+                continue;
+            }
             const statementType = writeStatement === null || writeStatement === void 0 ? void 0 : writeStatement.get();
             if (statementType === undefined) {
                 continue;
             }
+            else if (this.statementHasGenericRead(node, writeStatement)) {
+                continue;
+            }
             // for now only allow some specific target statements, todo refactor
             if (!(statementType instanceof Statements.Move
                 || statementType instanceof Statements.Catch
@@ -232,6 +239,21 @@ DATA(percentage) = CONV decfloat34( comment_number / abs_statement_number ) * 10
         }
         return undefined;
     }
+    statementHasGenericRead(node, statement) {
+        for (const ref of node.getData().references) {
+            if (ref.referenceType !== _reference_1.ReferenceType.DataReadReference
+                || ref.resolved === undefined
+                || statement.includesToken(ref.position.getToken()) === false
+                || !(ref.resolved instanceof _typed_identifier_1.TypedIdentifier)) {
+                continue;
+            }
+            else if (ref.resolved.getType().isGeneric() === true
+                || ref.resolved.getType().containsVoid() === true) {
+                return true;
+            }
+        }
+        return false;
+    }
     isLocalDefinition(node, identifier) {
         const { start, end } = node.calcCoverage();
         if (identifier.getStart().isAfter(start) && identifier.getStart().isBefore(end)) {

package/package.json

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