@abaplint/core 2.113.124 → 2.113.126

package/build/abaplint.d.ts

@@ -1506,6 +1506,7 @@ export declare class CurrentScope {
     getType(): ScopeType;
     push(stype: ScopeType, sname: string, start: Position, filename: string): void;
     isOO(): boolean;
+    isAnyOO(): boolean;
     isGlobalOO(): boolean;
     isTypePool(): boolean;
     setAllowHeaderUse(name: string): void;

package/build/src/abap/2_statements/expressions/compare.js

@@ -12,6 +12,8 @@ class Compare extends combi_1.Expression {
         const inn = (0, combi_1.seq)((0, combi_1.optPrio)("NOT"), "IN", (0, combi_1.altPrio)(_1.Source, list));
         const sopt = (0, combi_1.seq)("IS", (0, combi_1.optPrio)("NOT"), (0, combi_1.altPrio)("SUPPLIED", "BOUND", (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)("INSTANCE OF", _1.ClassName), version_1.Version.OpenABAP), "REQUESTED", "INITIAL"));
         const between = (0, combi_1.seq)((0, combi_1.optPrio)("NOT"), "BETWEEN", _1.Source, "AND", _1.Source);
+        // https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abennews-740_sp08-expressions.htm
+        // but also seems to work in v740sp05, blah
         const predicate = (0, combi_1.ver)(version_1.Version.v740sp08, _1.MethodCallChain);
         const rett = (0, combi_1.seq)(_1.Source, (0, combi_1.altPrio)((0, combi_1.seq)(_1.CompareOperator, _1.Source), inn, between, sopt));
         const fsassign = (0, combi_1.seq)(_1.SourceFieldSymbolChain, "IS", (0, combi_1.optPrio)("NOT"), "ASSIGNED");

package/build/src/abap/2_statements/expressions/sql_alias_field.js

@@ -1,10 +1,11 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SQLAliasField = void 0;
+const tokens_1 = require("../../1_lexer/tokens");
 const combi_1 = require("../combi");
 class SQLAliasField extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.regex)(/^(\/\w+\/)?\w+~(\/\w+\/)?\w+$/);
+        return (0, combi_1.seq)((0, combi_1.regex)(/^(\/\w+\/)?\w+~(\/\w+\/)?\w+$/), (0, combi_1.starPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.Dash), (0, combi_1.regex)(/^\w+$/))));
     }
 }
 exports.SQLAliasField = SQLAliasField;

package/build/src/abap/2_statements/expressions/sql_source.js

@@ -9,7 +9,7 @@ class SQLSource extends combi_1.Expression {
     getRunnable() {
         const paren = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), _1.Source, (0, combi_1.tok)(tokens_1.WParenRightW));
         const at = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), (0, combi_1.altPrio)(_1.SimpleSource3, paren)));
-        return (0, combi_1.alt)(_1.SQLAliasField, _1.SimpleSource3, at);
+        return (0, combi_1.altPrio)(_1.SQLAliasField, _1.SimpleSource3, at);
     }
 }
 exports.SQLSource = SQLSource;

package/build/src/abap/3_structures/structures/dynpro_loop.js

@@ -3,9 +3,10 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.DynproLoop = void 0;
 const Statements = require("../../2_statements/statements");
 const _combi_1 = require("./_combi");
+const chain_1 = require("./chain");
 class DynproLoop {
     getMatcher() {
-        return (0, _combi_1.beginEnd)((0, _combi_1.sta)(Statements.DynproLoop), (0, _combi_1.star)((0, _combi_1.sta)(Statements.Module)), (0, _combi_1.sta)(Statements.EndLoop));
+        return (0, _combi_1.beginEnd)((0, _combi_1.sta)(Statements.DynproLoop), (0, _combi_1.star)((0, _combi_1.alt)((0, _combi_1.sta)(Statements.Module), (0, _combi_1.sub)(chain_1.Chain))), (0, _combi_1.sta)(Statements.EndLoop));
     }
 }
 exports.DynproLoop = DynproLoop;

package/build/src/abap/3_structures/structures/process_after_input.js

@@ -5,9 +5,10 @@ const Statements = require("../../2_statements/statements");
 const _combi_1 = require("./_combi");
 const chain_1 = require("./chain");
 const loop_1 = require("./loop");
+const dynpro_loop_1 = require("./dynpro_loop");
 class ProcessAfterInput {
     getMatcher() {
-        const pai = (0, _combi_1.star)((0, _combi_1.alt)((0, _combi_1.sta)(Statements.Module), (0, _combi_1.sta)(Statements.Field), (0, _combi_1.sta)(Statements.CallSubscreen), (0, _combi_1.sub)(chain_1.Chain), (0, _combi_1.sub)(loop_1.Loop)));
+        const pai = (0, _combi_1.star)((0, _combi_1.alt)((0, _combi_1.sta)(Statements.Module), (0, _combi_1.sta)(Statements.Field), (0, _combi_1.sta)(Statements.CallSubscreen), (0, _combi_1.sub)(chain_1.Chain), (0, _combi_1.sub)(dynpro_loop_1.DynproLoop), (0, _combi_1.sub)(loop_1.Loop)));
         return (0, _combi_1.seq)((0, _combi_1.sta)(Statements.ProcessAfterInput), pai);
     }
 }

package/build/src/abap/5_syntax/_current_scope.js

@@ -429,6 +429,19 @@ class CurrentScope {
         }
         return false;
     }
+    isAnyOO() {
+        let curr = this.current;
+        while (curr !== undefined) {
+            const stype = curr.getIdentifier().stype;
+            if (stype === _scope_type_1.ScopeType.ClassDefinition
+                || stype === _scope_type_1.ScopeType.ClassImplementation
+                || stype === _scope_type_1.ScopeType.Interface) {
+                return true;
+            }
+            curr = curr.getParent();
+        }
+        return false;
+    }
     isGlobalOO() {
         return this.parentObj.getType() === "INTF" || this.parentObj.getType() === "CLAS";
     }

package/build/src/abap/5_syntax/basic_types.js

@@ -139,7 +139,7 @@ class BasicTypes {
             if (type === undefined) {
                 type = (_a = this.input.scope.getDDIC().lookupNoVoid(name)) === null || _a === void 0 ? void 0 : _a.type;
             }
-            if (type === undefined && this.input.scope.isOO() === false && this.input.scope.getDDIC().inErrorNamespace(name) === false) {
+            if (type === undefined && this.input.scope.isAnyOO() === false && this.input.scope.getDDIC().inErrorNamespace(name) === false) {
                 this.input.scope.addReference(chain.getChildren()[0].getFirstToken(), undefined, _reference_1.ReferenceType.VoidType, this.input.filename);
                 return new Types.VoidType(name);
             }

package/build/src/abap/5_syntax/expressions/data_definition.js

@@ -32,7 +32,6 @@ class DataDefinition {
             }
         }
         if (name) {
-            console.dir("undef");
             return new _typed_identifier_1.TypedIdentifier(name.getFirstToken(), input.filename, new basic_1.UnknownType("DataDefinition, fallback"));
         }
         return undefined;

package/build/src/registry.js

@@ -67,7 +67,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.124";
+        return "2.113.126";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/xml_consistency.js

@@ -60,7 +60,7 @@ class XMLConsistency {
                 issues.push(issue_1.Issue.atRow(file, 1, "Class matching XML name not found in ABAP file", this.getMetadata().key, this.conf.severity));
             }
         }
-        if (obj instanceof Objects.Interface) {
+        else if (obj instanceof Objects.Interface) {
             const name = obj.getNameFromXML();
             if (name === undefined) {
                 issues.push(issue_1.Issue.atRow(file, 1, "Name undefined in XML", this.getMetadata().key, this.conf.severity));

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.113.124",
+    "version": "2.113.126",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -53,9 +53,9 @@
         "@microsoft/api-extractor": "^7.52.8",
         "@types/chai": "^4.3.20",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^22.15.21",
+        "@types/node": "^22.15.29",
         "chai": "^4.5.0",
-        "eslint": "^9.27.0",
+        "eslint": "^9.28.0",
         "mocha": "^11.5.0",
         "c8": "^10.1.3",
         "source-map-support": "^0.5.21",