@abaplint/core 2.91.20 → 2.91.23

package/build/abaplint.d.ts

@@ -1705,6 +1705,15 @@ declare class EnhancementSpot extends AbstractObject {
     private parseXML;
 }
 
+declare class EventBinding extends AbstractObject {
+    getType(): string;
+    getAllowedNaming(): {
+        maxLength: number;
+        allowNamespace: boolean;
+    };
+    getDescription(): string | undefined;
+}
+
 declare class EventHandler extends Expression {
     getRunnable(): IStatementRunnable;
 }
@@ -3957,6 +3966,7 @@ declare namespace Objects {
         EnhancementImplementation,
         IBadiDefinition,
         EnhancementSpot,
+        EventBinding,
         ExtensionIndex,
         FieldCatalog,
         FormObjectForm,

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

@@ -2,10 +2,10 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.PerformChanging = void 0;
 const combi_1 = require("../combi");
-const source_1 = require("./source");
+const target_1 = require("./target");
 class PerformChanging extends combi_1.Expression {
     getRunnable() {
-        const changing = (0, combi_1.seq)("CHANGING", (0, combi_1.plus)(source_1.Source));
+        const changing = (0, combi_1.seq)("CHANGING", (0, combi_1.plus)(target_1.Target));
         return changing;
     }
 }

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

@@ -193,7 +193,7 @@ class BasicTypes {
         }
         return undefined;
     }
-    simpleType(node) {
+    simpleType(node, noQualifiedName) {
         let nameExpr = node.findFirstExpression(Expressions.NamespaceSimpleName);
         if (nameExpr === undefined) {
             nameExpr = node.findFirstExpression(Expressions.DefinitionName);
@@ -218,6 +218,9 @@ class BasicTypes {
                 }
             }
         }
+        if (noQualifiedName === true) {
+            qualifiedName = undefined;
+        }
         const found = this.parseType(node, qualifiedName);
         if (found) {
             return new _typed_identifier_1.TypedIdentifier(name, this.filename, found);

package/build/src/abap/5_syntax/statements/perform.js

@@ -5,6 +5,7 @@ const Expressions = require("../../2_statements/expressions");
 const Statements = require("../../2_statements/statements");
 const _reference_1 = require("../_reference");
 const source_1 = require("../expressions/source");
+const target_1 = require("../expressions/target");
 class Perform {
     runSyntax(node, scope, filename) {
         if (!(node.get() instanceof Statements.Perform)) {
@@ -13,8 +14,8 @@ class Perform {
         ////////////////////////////
         // check parameters are defined
         for (const c of node.findDirectExpressions(Expressions.PerformChanging)) {
-            for (const s of c.findDirectExpressions(Expressions.Source)) {
-                new source_1.Source().runSyntax(s, scope, filename);
+            for (const s of c.findDirectExpressions(Expressions.Target)) {
+                new target_1.Target().runSyntax(s, scope, filename);
             }
         }
         for (const t of node.findDirectExpressions(Expressions.PerformTables)) {

package/build/src/abap/5_syntax/statements/type.js

@@ -7,12 +7,12 @@ const basic_1 = require("../../types/basic");
 const Expressions = require("../../2_statements/expressions");
 const type_table_1 = require("../expressions/type_table");
 class Type {
-    runSyntax(node, scope, filename) {
+    runSyntax(node, scope, filename, noQualifiedName) {
         const tt = node.findFirstExpression(Expressions.TypeTable);
         if (tt) {
             return new type_table_1.TypeTable().runSyntax(node, scope, filename);
         }
-        const found = new basic_types_1.BasicTypes(filename, scope).simpleType(node);
+        const found = new basic_types_1.BasicTypes(filename, scope).simpleType(node, noQualifiedName);
         if (found) {
             return found;
         }

package/build/src/abap/5_syntax/structures/types.js

@@ -20,7 +20,7 @@ class Types {
             const ctyp = c.get();
             if (c instanceof nodes_1.StatementNode) {
                 if (ctyp instanceof Statements.Type) {
-                    const found = new type_1.Type().runSyntax(c, scope, filename);
+                    const found = new type_1.Type().runSyntax(c, scope, filename, true);
                     if (found) {
                         components.push({ name: found.getName(), type: found.getType() });
                     }

package/build/src/objects/event_binding.js

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.EventBinding = void 0;
+const _abstract_object_1 = require("./_abstract_object");
+class EventBinding extends _abstract_object_1.AbstractObject {
+    getType() {
+        return "EVTB";
+    }
+    getAllowedNaming() {
+        return {
+            maxLength: 40,
+            allowNamespace: true,
+        };
+    }
+    getDescription() {
+        // todo
+        return undefined;
+    }
+}
+exports.EventBinding = EventBinding;
+//# sourceMappingURL=event_binding.js.map

package/build/src/objects/index.js

@@ -51,6 +51,7 @@ __exportStar(require("./ecatt_test_data_container"), exports);
 __exportStar(require("./ecatt_test_script"), exports);
 __exportStar(require("./enhancement_implementation"), exports);
 __exportStar(require("./enhancement_spot"), exports);
+__exportStar(require("./event_binding"), exports);
 __exportStar(require("./extension_index"), exports);
 __exportStar(require("./field_catalog"), exports);
 __exportStar(require("./form_object_form"), exports);

package/build/src/registry.js

@@ -68,7 +68,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.91.20";
+        return "2.91.23";
     }
     getDDICReferences() {
         return this.references;

package/build/src/rules/downport.js

@@ -340,8 +340,9 @@ Only one transformation is applied to a statement at a time, so multiple steps m
         const candidates = [high.findAllExpressionsRecursive(Expressions.SQLTarget),
             high.findAllExpressionsRecursive(Expressions.SQLSource),
             high.findAllExpressionsRecursive(Expressions.SQLSourceSimple)].flat();
-        for (const c of candidates) {
-            if (c.getFirstToken() instanceof tokens_1.WAt) {
+        for (const c of candidates.reverse()) {
+            if (c.getFirstToken() instanceof tokens_1.WAt
+                || c.getFirstToken() instanceof tokens_1.At) {
                 addFix(c.getFirstToken());
             }
         }
@@ -353,7 +354,7 @@ Only one transformation is applied to a statement at a time, so multiple steps m
             }
         }
         if (fix !== undefined) {
-            return issue_1.Issue.atToken(lowFile, low.getFirstToken(), "SQL, remove \" and ,", this.getMetadata().key, this.conf.severity, fix);
+            return issue_1.Issue.atToken(lowFile, low.getFirstToken(), "SQL, remove @ and ,", this.getMetadata().key, this.conf.severity, fix);
         }
         for (const c of high.findAllExpressionsRecursive(Expressions.SQLIn)) {
             const children = c.getChildren();

package/build/src/rules/sy_modification.js

@@ -19,7 +19,9 @@ class SyModification extends _abap_rule_1.ABAPRule {
             key: "sy_modification",
             title: "Modification of SY fields",
             shortDescription: `Finds modification of sy fields`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abensystem_fields.htm`,
+            extendedInformation: `https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abensystem_fields.htm
+
+Changes to SY-TVAR* fields are not reported`,
             tags: [_irule_1.RuleTag.SingleFile],
             badExample: `
 sy-uname = 2.
@@ -42,6 +44,9 @@ sy = sy.`,
             const firstChild = t.getChildren()[0];
             if (firstChild.get() instanceof Expressions.TargetField
                 && firstChild.getFirstToken().getStr().toUpperCase() === "SY") {
+                if (t.concatTokens().toUpperCase().startsWith("SY-TVAR")) {
+                    continue;
+                }
                 const message = "Modification of SY field";
                 const issue = issue_1.Issue.atToken(file, firstChild.getFirstToken(), message, this.getMetadata().key, this.conf.severity);
                 issues.push(issue);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.91.20",
+    "version": "2.91.23",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -45,10 +45,10 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@microsoft/api-extractor": "^7.28.7",
+        "@microsoft/api-extractor": "^7.29.0",
         "@types/chai": "^4.3.1",
         "@types/mocha": "^9.1.1",
-        "@types/node": "^18.6.3",
+        "@types/node": "^18.6.4",
         "chai": "^4.3.6",
         "eslint": "^8.21.0",
         "mocha": "^10.0.0",