@abaplint/core 2.85.17 → 2.85.21

package/build/abaplint.d.ts

@@ -3824,11 +3824,19 @@ declare class Normal implements IStructure {
 }
 
 declare class NumberRange extends AbstractObject {
+    private parsedXML;
     getType(): string;
     getAllowedNaming(): {
         maxLength: number;
         allowNamespace: boolean;
     };
+    getDomain(): string | undefined;
+    getPercentage(): number | undefined;
+    setDirty(): void;
+    parse(): {
+        updated: boolean;
+        runtime: number;
+    };
     getDescription(): string | undefined;
 }
 

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

@@ -5,7 +5,7 @@ const combi_1 = require("../combi");
 const _1 = require(".");
 class CondBody extends combi_1.Expression {
     getRunnable() {
-        const when = (0, combi_1.seq)("WHEN", _1.Cond, "THEN", (0, combi_1.alt)(_1.Source, _1.Throw));
+        const when = (0, combi_1.seq)("WHEN", (0, combi_1.alt)(_1.Cond, _1.Source), "THEN", (0, combi_1.alt)(_1.Source, _1.Throw));
         const elsee = (0, combi_1.seq)("ELSE", (0, combi_1.alt)(_1.Source, _1.Throw));
         return (0, combi_1.seq)((0, combi_1.opt)(_1.Let), (0, combi_1.plus)(when), (0, combi_1.opt)(elsee));
     }

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

@@ -3,9 +3,10 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.ReduceNext = void 0;
 const combi_1 = require("../combi");
 const _1 = require(".");
+const tokens_1 = require("../../1_lexer/tokens");
 class ReduceNext extends combi_1.Expression {
     getRunnable() {
-        const fields = (0, combi_1.seq)(_1.Field, "=", _1.Source);
+        const fields = (0, combi_1.seq)(_1.Field, (0, combi_1.altPrio)("=", (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WPlus), "=")), _1.Source);
         return (0, combi_1.seq)("NEXT", (0, combi_1.plus)(fields));
     }
 }

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

@@ -19,7 +19,7 @@ class Select extends combi_1.Expression {
         const fields = (0, combi_1.seq)("FIELDS", _1.SQLFieldList);
         const perm = (0, combi_1.per)(_1.SQLFrom, into, _1.SQLForAllEntries, where, _1.SQLOrderBy, up, offset, client, _1.SQLHaving, bypass, sql_group_by_1.SQLGroupBy, fields, _1.DatabaseConnection);
         const paren = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WParenLeftW), sql_field_name_1.SQLFieldName, (0, combi_1.tok)(tokens_1.WParenRightW));
-        const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)("DISTINCT", (0, combi_1.optPrio)((0, combi_1.seq)("SINGLE", (0, combi_1.optPrio)("FOR UPDATE")))), (0, combi_1.optPrio)((0, combi_1.altPrio)(_1.SQLFieldList, paren)), perm);
+        const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)("DISTINCT", (0, combi_1.optPrio)((0, combi_1.seq)("SINGLE", (0, combi_1.optPrio)("FOR UPDATE")))), (0, combi_1.optPrio)((0, combi_1.altPrio)(_1.SQLFieldList, paren)), perm, (0, combi_1.optPrio)(_1.SQLHints));
         return ret;
     }
 }

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

@@ -13,7 +13,7 @@ class TypeTable extends combi_1.Expression {
         const likeType = (0, combi_1.seq)((0, combi_1.opt)((0, combi_1.alt)("STANDARD", "HASHED", "INDEX", "SORTED", "ANY")), "TABLE OF", (0, combi_1.optPrio)("REF TO"), (0, combi_1.opt)(field_chain_1.FieldChain), (0, combi_1.opt)((0, combi_1.per)(header, initial, (0, combi_1.plusPrio)(type_table_key_1.TypeTableKey))));
         const rangeType = (0, combi_1.seq)("RANGE OF", _1.TypeName, (0, combi_1.opt)(header), (0, combi_1.opt)(initial));
         const rangeLike = (0, combi_1.seq)("RANGE OF", _1.SimpleFieldChain, (0, combi_1.opt)(header), (0, combi_1.opt)(initial));
-        const typetable = (0, combi_1.seq)(normal1, (0, combi_1.opt)((0, combi_1.per)(header, initial, (0, combi_1.plusPrio)(type_table_key_1.TypeTableKey))));
+        const typetable = (0, combi_1.seq)(normal1, (0, combi_1.alt)((0, combi_1.opt)((0, combi_1.per)(header, initial, (0, combi_1.plusPrio)(type_table_key_1.TypeTableKey))), (0, combi_1.seq)((0, combi_1.plus)(type_table_key_1.TypeTableKey), (0, combi_1.optPrio)(initial))));
         const occurs = (0, combi_1.seq)("OCCURS", _1.Integer);
         const oldType = (0, combi_1.seq)((0, combi_1.opt)("REF TO"), _1.TypeName, (0, combi_1.alt)((0, combi_1.seq)(occurs, (0, combi_1.opt)(header)), header));
         const oldLike = (0, combi_1.seq)((0, combi_1.opt)("REF TO"), field_chain_1.FieldChain, (0, combi_1.alt)((0, combi_1.seq)(occurs, (0, combi_1.opt)(header)), header));

package/build/src/abap/2_statements/statements/select.js

@@ -7,7 +7,7 @@ const version_1 = require("../../../version");
 class Select {
     getMatcher() {
         const union = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)("UNION", (0, combi_1.optPrio)((0, combi_1.altPrio)("DISTINCT", "ALL")), expressions_1.Select));
-        return (0, combi_1.seq)(expressions_1.Select, (0, combi_1.starPrio)(union), (0, combi_1.optPrio)(expressions_1.SQLHints));
+        return (0, combi_1.seq)(expressions_1.Select, (0, combi_1.starPrio)(union));
     }
 }
 exports.Select = Select;

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

@@ -169,9 +169,9 @@ class BuiltIn {
         components.push({ name: "folen", type: new basic_1.IntegerType() });
         components.push({ name: "host", type: new basic_1.CharacterType(32, "sy-host") });
         components.push({ name: "index", type: new basic_1.IntegerType() });
+        components.push({ name: "input", type: new basic_1.CharacterType(1, "sy-input") });
         components.push({ name: "langu", type: new basic_1.CharacterType(1, "sy-langu") });
         components.push({ name: "ldbpg", type: new basic_1.CharacterType(40, "sy-ldbpg") });
-        components.push({ name: "pdest", type: new basic_1.CharacterType(4, "sy-pdest") });
         components.push({ name: "lilli", type: new basic_1.IntegerType() });
         components.push({ name: "linct", type: new basic_1.IntegerType() });
         components.push({ name: "linno", type: new basic_1.IntegerType() });
@@ -194,6 +194,7 @@ class BuiltIn {
         components.push({ name: "msgv4", type: new basic_1.CharacterType(50, "sy-msgv4") });
         components.push({ name: "opsys", type: new basic_1.CharacterType(10, "sy-opsys") });
         components.push({ name: "pagno", type: new basic_1.IntegerType() });
+        components.push({ name: "pdest", type: new basic_1.CharacterType(4, "sy-pdest") });
         components.push({ name: "pfkey", type: new basic_1.CharacterType(20, "sy-pfkey") });
         components.push({ name: "repid", type: new basic_1.CharacterType(40, "sy-repid") });
         components.push({ name: "saprl", type: new basic_1.CharacterType(4, "sy-saprl") });

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

@@ -166,11 +166,7 @@ class CurrentScope {
         if (name === undefined) {
             return { found: false };
         }
-        const def = (_a = this.current) === null || _a === void 0 ? void 0 : _a.findDeferred(name);
-        if (def !== undefined) {
-            return { found: true, id: def };
-        }
-        const findLocalClass = (_b = this.current) === null || _b === void 0 ? void 0 : _b.findClassDefinition(name);
+        const findLocalClass = (_a = this.current) === null || _a === void 0 ? void 0 : _a.findClassDefinition(name);
         if (findLocalClass) {
             return { found: true, id: findLocalClass, type: _reference_1.ReferenceType.ObjectOrientedReference, ooType: "CLAS" };
         }
@@ -178,7 +174,7 @@ class CurrentScope {
         if (globalClas) {
             return { found: true, id: globalClas.getIdentifier(), type: _reference_1.ReferenceType.ObjectOrientedReference, ooType: "CLAS" };
         }
-        const findLocalInterface = (_c = this.current) === null || _c === void 0 ? void 0 : _c.findInterfaceDefinition(name);
+        const findLocalInterface = (_b = this.current) === null || _b === void 0 ? void 0 : _b.findInterfaceDefinition(name);
         if (findLocalInterface) {
             return { found: true, id: findLocalInterface, type: _reference_1.ReferenceType.ObjectOrientedReference, ooType: "INTF" };
         }
@@ -186,6 +182,10 @@ class CurrentScope {
         if (globalIntf) {
             return { found: true, id: globalIntf.getIdentifier(), type: _reference_1.ReferenceType.ObjectOrientedReference, ooType: "INTF" };
         }
+        const def = (_c = this.current) === null || _c === void 0 ? void 0 : _c.findDeferred(name);
+        if (def !== undefined) {
+            return { found: true, id: def };
+        }
         return { found: false };
     }
     ///////////////////////////

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

@@ -1,9 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.TypeUtils = void 0;
+const types_1 = require("../types");
 const basic_1 = require("../types/basic");
 class TypeUtils {
-    static isCharLike(type) {
+    constructor(scope) {
+        this.scope = scope;
+    }
+    isCharLike(type) {
         if (type === undefined) {
             return false;
         }
@@ -40,7 +44,7 @@ class TypeUtils {
         }
         return false;
     }
-    static isHexLike(type) {
+    isHexLike(type) {
         if (type === undefined) {
             return false;
         }
@@ -61,7 +65,96 @@ class TypeUtils {
         }
         return false;
     }
-    static isAssignable(source, target) {
+    isOOAssignable(source, target) {
+        let sid = source.getIdentifier();
+        if (!(sid instanceof types_1.ClassDefinition || sid instanceof types_1.InterfaceDefinition)) {
+            const found = this.scope.findObjectDefinition(sid.getName());
+            if (found) {
+                sid = found;
+            }
+            else {
+                return false;
+            }
+        }
+        let tid = target.getIdentifier();
+        if (!(tid instanceof types_1.ClassDefinition || tid instanceof types_1.InterfaceDefinition)) {
+            const found = this.scope.findObjectDefinition(tid.getName());
+            if (found) {
+                tid = found;
+            }
+            else {
+                return false;
+            }
+        }
+        const tname = tid.getName().toUpperCase();
+        if (sid instanceof types_1.ClassDefinition && tid instanceof types_1.ClassDefinition) {
+            if (sid.getName().toUpperCase() === tname) {
+                return true;
+            }
+            const slist = this.listAllSupers(sid);
+            if (slist.indexOf(tname) >= 0) {
+                return true;
+            }
+        }
+        else if (sid instanceof types_1.ClassDefinition && tid instanceof types_1.InterfaceDefinition) {
+            if (sid.getImplementing().some(i => i.name === tname)) {
+                return true;
+            }
+            const slist = this.listAllInterfaces(sid);
+            if (slist.indexOf(tname) >= 0) {
+                return true;
+            }
+        }
+        else if (sid instanceof types_1.InterfaceDefinition && tid instanceof types_1.InterfaceDefinition) {
+            if (sid.getName().toUpperCase() === tname) {
+                return true;
+            }
+            if (sid.getImplementing().some(i => i.name === tname)) {
+                return true;
+            }
+            const slist = this.listAllInterfaces(sid);
+            if (slist.indexOf(tname) >= 0) {
+                return true;
+            }
+        }
+        return false;
+    }
+    listAllInterfaces(cdef) {
+        var _a;
+        const ret = new Set();
+        const stack = [];
+        // initialize
+        cdef.getImplementing().forEach(i => stack.push(i.name));
+        if (cdef instanceof types_1.ClassDefinition) {
+            const supers = this.listAllSupers(cdef);
+            for (const s of supers) {
+                (_a = this.scope.findClassDefinition(s)) === null || _a === void 0 ? void 0 : _a.getImplementing().forEach(i => stack.push(i.name));
+            }
+        }
+        // main loop
+        while (stack.length > 0) {
+            const intf = stack.pop().toUpperCase();
+            ret.add(intf);
+            const idef = this.scope.findInterfaceDefinition(intf);
+            idef === null || idef === void 0 ? void 0 : idef.getImplementing().forEach(i => stack.push(i.name));
+        }
+        return Array.from(ret.values());
+    }
+    listAllSupers(cdef) {
+        var _a, _b;
+        const ret = [];
+        let sup = cdef.getSuperClass();
+        while (sup !== undefined) {
+            ret.push(sup === null || sup === void 0 ? void 0 : sup.toUpperCase());
+            sup = (_b = (_a = this.scope.findClassDefinition(sup)) === null || _a === void 0 ? void 0 : _a.getSuperClass()) === null || _b === void 0 ? void 0 : _b.toUpperCase();
+        }
+        return ret;
+    }
+    isCastable(_source, _target) {
+        // todo
+        return true;
+    }
+    isAssignable(source, target) {
         /*
             console.dir(source);
             console.dir(target);
@@ -78,6 +171,9 @@ class TypeUtils {
             }
             return false;
         }
+        else if (target instanceof basic_1.ObjectReferenceType && source instanceof basic_1.ObjectReferenceType) {
+            return this.isOOAssignable(source, target);
+        }
         else if (target instanceof basic_1.ObjectReferenceType
             || target instanceof basic_1.GenericObjectReferenceType) {
             if (source instanceof basic_1.ObjectReferenceType

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

@@ -4,30 +4,42 @@ exports.Cast = void 0;
 const basic_1 = require("../../types/basic");
 const Expressions = require("../../2_statements/expressions");
 const source_1 = require("./source");
+const _type_utils_1 = require("../_type_utils");
 class Cast {
     runSyntax(node, scope, targetType, filename) {
         var _a;
-        for (const s of node.findAllExpressions(Expressions.Source)) {
-            new source_1.Source().runSyntax(s, scope, filename);
+        const sourceNode = node.findDirectExpression(Expressions.Source);
+        if (sourceNode === undefined) {
+            throw new Error("Cast, source node not found");
         }
+        const sourceType = new source_1.Source().runSyntax(sourceNode, scope, filename);
+        let tt = undefined;
         const typeName = (_a = node.findDirectExpression(Expressions.TypeNameOrInfer)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr();
         if (typeName === undefined) {
             throw new Error("Cast, child TypeNameOrInfer not found");
         }
         else if (typeName === "#" && targetType) {
-            return targetType;
+            tt = targetType;
         }
         else if (typeName === "#") {
             throw new Error("Cast, todo, infer type");
         }
-        const found = scope.findObjectDefinition(typeName);
-        if (found === undefined && scope.getDDIC().inErrorNamespace(typeName) === false) {
-            return new basic_1.VoidType(typeName);
+        if (tt === undefined) {
+            const found = scope.findObjectDefinition(typeName);
+            if (found === undefined && scope.getDDIC().inErrorNamespace(typeName) === false) {
+                tt = new basic_1.VoidType(typeName);
+            }
+            else if (found === undefined) {
+                throw new Error("Type \"" + typeName + "\" not found in scope, Cast");
+            }
+            else {
+                tt = new basic_1.ObjectReferenceType(found);
+            }
         }
-        else if (found === undefined) {
-            throw new Error("Type \"" + typeName + "\" not found in scope, Cast");
+        if (new _type_utils_1.TypeUtils(scope).isCastable(sourceType, tt) === false) {
+            throw new Error("Cast, incompatible types");
         }
-        return new basic_1.ObjectReferenceType(found);
+        return tt;
     }
 }
 exports.Cast = Cast;

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

@@ -50,7 +50,7 @@ class MethodCallParam {
             if (sourceType === undefined) {
                 throw new Error("No source type determined, method source");
             }
-            else if (_type_utils_1.TypeUtils.isAssignable(sourceType, targetType) === false) {
+            else if (new _type_utils_1.TypeUtils(scope).isAssignable(sourceType, targetType) === false) {
                 throw new Error("Method parameter type not compatible");
             }
         }

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

@@ -65,7 +65,7 @@ class MethodParameters {
         }
         else if (target) {
             const targetType = new target_1.Target().runSyntax(target, scope, filename);
-            if (targetType && _type_utils_1.TypeUtils.isAssignable(type, targetType) === false) {
+            if (targetType && new _type_utils_1.TypeUtils(scope).isAssignable(type, targetType) === false) {
                 throw new Error("Method returning value not type compatible");
             }
         }
@@ -90,7 +90,7 @@ class MethodParameters {
             else if (item.targetType === undefined) {
                 throw new Error("Could not determine target type");
             }
-            else if (item.targetType && _type_utils_1.TypeUtils.isAssignable(parameterType, item.targetType) === false) {
+            else if (item.targetType && new _type_utils_1.TypeUtils(scope).isAssignable(parameterType, item.targetType) === false) {
                 throw new Error("Method parameter type not compatible, " + item.name);
             }
         }
@@ -109,7 +109,7 @@ class MethodParameters {
                 }
                 parameterType = parameter.getType();
             }
-            if (item.targetType && _type_utils_1.TypeUtils.isAssignable(parameterType, item.targetType) === false) {
+            if (item.targetType && new _type_utils_1.TypeUtils(scope).isAssignable(parameterType, item.targetType) === false) {
                 throw new Error("Method parameter type not compatible, " + item.name);
             }
             (_a = this.requiredParameters) === null || _a === void 0 ? void 0 : _a.delete(item.name);
@@ -129,7 +129,7 @@ class MethodParameters {
             if (parameter === undefined) {
                 throw new Error("Method importing parameter \"" + item.name + "\" does not exist");
             }
-            else if (_type_utils_1.TypeUtils.isAssignable(parameter.getType(), item.sourceType) === false) {
+            else if (new _type_utils_1.TypeUtils(scope).isAssignable(item.sourceType, parameter.getType()) === false) {
                 throw new Error("Method parameter type not compatible, " + item.name);
             }
             this.requiredParameters.delete(item.name);

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

@@ -0,0 +1,14 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.RaiseWith = void 0;
+const Expressions = require("../../2_statements/expressions");
+const source_1 = require("./source");
+class RaiseWith {
+    runSyntax(node, scope, filename) {
+        for (const f of node.findDirectExpressions(Expressions.Source)) {
+            new source_1.Source().runSyntax(f, scope, filename);
+        }
+    }
+}
+exports.RaiseWith = RaiseWith;
+//# sourceMappingURL=raise_with.js.map

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

@@ -30,7 +30,12 @@ class Move {
             new inline_data_1.InlineData().runSyntax(inline, scope, filename, sourceType);
             targetType = sourceType;
         }
-        if (_type_utils_1.TypeUtils.isAssignable(sourceType, targetType) === false) {
+        if (node.findDirectTokenByText("?=")) {
+            if (new _type_utils_1.TypeUtils(scope).isCastable(sourceType, targetType) === false) {
+                throw new Error("Incompatible types");
+            }
+        }
+        else if (new _type_utils_1.TypeUtils(scope).isAssignable(sourceType, targetType) === false) {
             throw new Error("Incompatible types");
         }
     }

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

@@ -7,6 +7,7 @@ const source_1 = require("../expressions/source");
 const _reference_1 = require("../_reference");
 const basic_1 = require("../../types/basic");
 const message_source_1 = require("../expressions/message_source");
+const raise_with_1 = require("../expressions/raise_with");
 class Raise {
     runSyntax(node, scope, filename) {
         // todo
@@ -47,6 +48,9 @@ class Raise {
                 new source_1.Source().runSyntax(s, scope, filename);
             }
         }
+        for (const s of node.findDirectExpressions(Expressions.RaiseWith)) {
+            new raise_with_1.RaiseWith().runSyntax(s, scope, filename);
+        }
         for (const s of node.findDirectExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(s, scope, filename);
         }

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

@@ -35,14 +35,14 @@ class ReadTable {
         const indexSource = node.findExpressionAfterToken("INDEX");
         if (indexSource) {
             const indexType = new source_1.Source().runSyntax(indexSource, scope, filename);
-            if (_type_utils_1.TypeUtils.isAssignable(indexType, new basic_1.IntegerType()) === false) {
+            if (new _type_utils_1.TypeUtils(scope).isAssignable(indexType, new basic_1.IntegerType()) === false) {
                 throw new Error("READ TABLE, INDEX must be simple");
             }
         }
         const fromSource = node.findExpressionAfterToken("FROM");
         if (fromSource) {
             const fromType = new source_1.Source().runSyntax(fromSource, scope, filename);
-            if (_type_utils_1.TypeUtils.isAssignable(fromType, new basic_1.IntegerType()) === false) {
+            if (new _type_utils_1.TypeUtils(scope).isAssignable(fromType, new basic_1.IntegerType()) === false) {
                 throw new Error("READ TABLE, FROM must be simple");
             }
         }
@@ -77,7 +77,7 @@ class ReadTable {
             const t = target.findFirstExpression(Expressions.Target);
             if (t) {
                 const targetType = new target_1.Target().runSyntax(t, scope, filename);
-                if (_type_utils_1.TypeUtils.isAssignable(rowType, targetType) === false) {
+                if (new _type_utils_1.TypeUtils(scope).isAssignable(rowType, targetType) === false) {
                     throw new Error("Incompatible types");
                 }
                 return;

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

@@ -16,12 +16,12 @@ class Shift {
         }
         const targetType = new target_1.Target().runSyntax(target, scope, filename);
         if (node.concatTokens().toUpperCase().includes(" IN BYTE MODE")) {
-            if (_type_utils_1.TypeUtils.isHexLike(targetType) === false) {
+            if (new _type_utils_1.TypeUtils(scope).isHexLike(targetType) === false) {
                 throw new Error("Shift, Target not hex like");
             }
         }
         else {
-            if (_type_utils_1.TypeUtils.isCharLike(targetType) === false) {
+            if (new _type_utils_1.TypeUtils(scope).isCharLike(targetType) === false) {
                 throw new Error("Shift, Target not char like");
             }
         }

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

@@ -13,8 +13,8 @@ class Write {
         for (const s of node.findAllExpressions(Expressions.Source)) {
             const type = new source_1.Source().runSyntax(s, scope, filename);
             if (s === second
-                && _type_utils_1.TypeUtils.isCharLike(type) === false
-                && _type_utils_1.TypeUtils.isHexLike(type) === false) {
+                && new _type_utils_1.TypeUtils(scope).isCharLike(type) === false
+                && new _type_utils_1.TypeUtils(scope).isHexLike(type) === false) {
                 throw new Error("Source not character like");
             }
         }

package/build/src/objects/number_range.js

@@ -12,9 +12,43 @@ class NumberRange extends _abstract_object_1.AbstractObject {
             allowNamespace: true,
         };
     }
+    getDomain() {
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.domain;
+    }
+    getPercentage() {
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.percentage;
+    }
+    setDirty() {
+        this.parsedXML = undefined;
+        super.setDirty();
+    }
+    parse() {
+        var _a, _b, _c, _d, _e, _f;
+        if (this.parsedXML) {
+            return { updated: false, runtime: 0 };
+        }
+        const start = Date.now();
+        this.parsedXML = {};
+        const parsed = super.parseRaw2();
+        if (parsed === undefined) {
+            return { updated: false, runtime: 0 };
+        }
+        const text = (_c = (_b = (_a = parsed.abapGit) === null || _a === void 0 ? void 0 : _a["asx:abap"]) === null || _b === void 0 ? void 0 : _b["asx:values"]) === null || _c === void 0 ? void 0 : _c.TEXT;
+        this.parsedXML.description = text === null || text === void 0 ? void 0 : text.TXT;
+        const attributes = (_f = (_e = (_d = parsed.abapGit) === null || _d === void 0 ? void 0 : _d["asx:abap"]) === null || _e === void 0 ? void 0 : _e["asx:values"]) === null || _f === void 0 ? void 0 : _f.ATTRIBUTES;
+        this.parsedXML.domain = attributes === null || attributes === void 0 ? void 0 : attributes.DOMLEN;
+        this.parsedXML.percentage = parseFloat((attributes === null || attributes === void 0 ? void 0 : attributes.PERCENTAGE) || "");
+        const end = Date.now();
+        return { updated: true, runtime: end - start };
+    }
     getDescription() {
-        // todo
-        return undefined;
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.description;
     }
 }
 exports.NumberRange = NumberRange;

package/build/src/registry.js

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

package/build/src/rules/downport.js

@@ -56,6 +56,7 @@ Current rules:
 * PARTIALLY IMPLEMENTED removed, it can be quick fixed via rule implement_methods
 * RAISE EXCEPTION ... MESSAGE
 * APPEND expression is outlined
+* Moving with +=, -=, /=, *=, &&=
 
 Only one transformation is applied to a statement at a time, so multiple steps might be required to do the full downport.`,
             tags: [_irule_1.RuleTag.Experimental, _irule_1.RuleTag.Downport, _irule_1.RuleTag.Quickfix],
@@ -163,6 +164,10 @@ Only one transformation is applied to a statement at a time, so multiple steps m
         if (found) {
             return found;
         }
+        found = this.moveWithOperator(high, lowFile);
+        if (found) {
+            return found;
+        }
         found = this.downportSelectInline(low, high, lowFile, highSyntax);
         if (found) {
             return found;
@@ -560,6 +565,48 @@ ${indentation}RAISE EXCEPTION ${uniqueName2}.`;
         }
         return undefined;
     }
+    moveWithOperator(high, lowFile) {
+        var _a, _b, _c;
+        if (!(high.get() instanceof Statements.Move)) {
+            return undefined;
+        }
+        const children = high.getChildren();
+        const secondChild = children[1];
+        if (secondChild === undefined) {
+            return undefined;
+        }
+        const op = secondChild.getFirstToken();
+        let operator = "";
+        switch (op.getStr()) {
+            case "+":
+                operator = " + ";
+                break;
+            case "-":
+                operator = " - ";
+                break;
+            case "/=":
+                operator = " / ";
+                break;
+            case "*=":
+                operator = " * ";
+                break;
+            case "&&=":
+                operator = " && ";
+                break;
+            default:
+                return undefined;
+        }
+        const target = (_a = high.findDirectExpression(Expressions.Target)) === null || _a === void 0 ? void 0 : _a.concatTokens();
+        if (target === undefined) {
+            return;
+        }
+        const sourceStart = (_c = (_b = high.findDirectExpression(Expressions.Source)) === null || _b === void 0 ? void 0 : _b.getFirstChild()) === null || _c === void 0 ? void 0 : _c.getFirstToken().getStart();
+        if (sourceStart === undefined) {
+            return;
+        }
+        const fix = edit_helper_1.EditHelper.replaceRange(lowFile, op.getStart(), sourceStart, "= " + target + operator);
+        return issue_1.Issue.atToken(lowFile, high.getFirstToken(), "Expand operator", this.getMetadata().key, this.conf.severity, fix);
+    }
     // must be very simple string templates, like "|{ ls_line-no ALPHA = IN }|"
     stringTemplateAlpha(node, lowFile) {
         var _a, _b, _c;
@@ -659,8 +706,34 @@ ${indentation}    output = ${topTarget}.`;
         }
         return undefined;
     }
+    outlineFor(forLoop, indentation) {
+        var _a, _b, _c, _d, _e, _f;
+        let body = "";
+        let end = "";
+        const loopSource = (_a = forLoop.findFirstExpression(Expressions.Source)) === null || _a === void 0 ? void 0 : _a.concatTokens();
+        const loopTargetField = (_b = forLoop.findFirstExpression(Expressions.TargetField)) === null || _b === void 0 ? void 0 : _b.concatTokens();
+        if (forLoop.findDirectTokenByText("UNTIL")) {
+            const name = (_c = forLoop.findFirstExpression(Expressions.Field)) === null || _c === void 0 ? void 0 : _c.concatTokens();
+            body += indentation + "DATA " + name + " TYPE i.\n";
+            const cond = forLoop.findFirstExpression(Expressions.Cond);
+            body += indentation + `WHILE NOT ${cond === null || cond === void 0 ? void 0 : cond.concatTokens()}.\n`;
+            const field = (_e = (_d = forLoop.findDirectExpression(Expressions.InlineFieldDefinition)) === null || _d === void 0 ? void 0 : _d.findFirstExpression(Expressions.Field)) === null || _e === void 0 ? void 0 : _e.concatTokens();
+            body += indentation + `  ${field} = ${field} + 1.\n`;
+            end = "ENDWHILE";
+        }
+        else if (loopTargetField) {
+            body += indentation + `LOOP AT ${loopSource} INTO DATA(${loopTargetField}).\n`;
+            end = "ENDLOOP";
+        }
+        else if (loopTargetField === undefined) {
+            const loopTargetFieldSymbol = (_f = forLoop.findFirstExpression(Expressions.TargetFieldSymbol)) === null || _f === void 0 ? void 0 : _f.concatTokens();
+            body += indentation + `LOOP AT ${loopSource} ASSIGNING FIELD-SYMBOL(${loopTargetFieldSymbol}).\n`;
+            end = "ENDLOOP";
+        }
+        return { body, end };
+    }
     outlineReduce(node, lowFile, highSyntax) {
-        var _a, _b, _c, _d;
+        var _a;
         for (const i of node.findAllExpressionsRecursive(Expressions.Source)) {
             const firstToken = i.getFirstToken();
             if (firstToken.getStr().toUpperCase() !== "REDUCE") {
@@ -682,19 +755,12 @@ ${indentation}    output = ${topTarget}.`;
                 name = init.getFirstToken().getStr();
                 body += indentation + `DATA(${name}) = ${(_a = reduceBody.findFirstExpression(Expressions.Source)) === null || _a === void 0 ? void 0 : _a.concatTokens()}.\n`;
             }
-            const loop = reduceBody.findFirstExpression(Expressions.InlineLoopDefinition);
-            if (loop === undefined) {
+            const forLoop = reduceBody.findDirectExpression(Expressions.For);
+            if (forLoop === undefined) {
                 continue;
             }
-            const loopSource = (_b = loop.findFirstExpression(Expressions.Source)) === null || _b === void 0 ? void 0 : _b.concatTokens();
-            const loopTargetField = (_c = loop.findFirstExpression(Expressions.TargetField)) === null || _c === void 0 ? void 0 : _c.concatTokens();
-            if (loopTargetField) {
-                body += indentation + `LOOP AT ${loopSource} INTO DATA(${loopTargetField}).\n`;
-            }
-            if (loopTargetField === undefined) {
-                const loopTargetFieldSymbol = (_d = loop.findFirstExpression(Expressions.TargetFieldSymbol)) === null || _d === void 0 ? void 0 : _d.concatTokens();
-                body += indentation + `LOOP AT ${loopSource} ASSIGNING FIELD-SYMBOL(${loopTargetFieldSymbol}).\n`;
-            }
+            const outlineFor = this.outlineFor(forLoop, indentation);
+            body += outlineFor.body;
             const next = reduceBody.findDirectExpression(Expressions.ReduceNext);
             if (next === undefined) {
                 continue;
@@ -713,7 +779,7 @@ ${indentation}    output = ${topTarget}.`;
                     body += n.concatTokens() + ".\n";
                 }
             }
-            body += indentation + `ENDLOOP.\n`;
+            body += indentation + outlineFor.end + `.\n`;
             body += indentation + `${uniqueName} = ${name}.\n`;
             const abap = `DATA ${uniqueName} TYPE ${type}.\n` +
                 body +
@@ -726,7 +792,6 @@ ${indentation}    output = ${topTarget}.`;
         return undefined;
     }
     outlineValue(node, lowFile, highSyntax) {
-        var _a, _b;
         const allSources = node.findAllExpressionsRecursive(Expressions.Source);
         for (const i of allSources) {
             const firstToken = i.getFirstToken();
@@ -741,11 +806,11 @@ ${indentation}    output = ${topTarget}.`;
             const uniqueName = this.uniqueName(firstToken.getStart(), lowFile.getFilename(), highSyntax);
             let indentation = " ".repeat(node.getFirstToken().getStart().getCol() - 1);
             let body = "";
-            const loop = valueBody === null || valueBody === void 0 ? void 0 : valueBody.findFirstExpression(Expressions.InlineLoopDefinition);
-            if (loop) {
-                const loopSource = (_a = loop.findFirstExpression(Expressions.Source)) === null || _a === void 0 ? void 0 : _a.concatTokens();
-                const loopTargetFieldSymbol = (_b = loop.findFirstExpression(Expressions.TargetFieldSymbol)) === null || _b === void 0 ? void 0 : _b.concatTokens();
-                body += indentation + `LOOP AT ${loopSource} ASSIGNING FIELD-SYMBOL(${loopTargetFieldSymbol}).\n`;
+            const forLoop = valueBody === null || valueBody === void 0 ? void 0 : valueBody.findDirectExpression(Expressions.For);
+            let outlineFor = { body: "", end: "" };
+            if (forLoop !== undefined) {
+                outlineFor = this.outlineFor(forLoop, indentation);
+                body += outlineFor.body;
                 indentation += "  ";
             }
             let structureName = uniqueName;
@@ -773,9 +838,9 @@ ${indentation}    output = ${topTarget}.`;
                     body += indentation + `APPEND ${structureName} TO ${uniqueName}.\n`;
                 }
             }
-            if (loop) {
-                indentation = indentation.substr(2);
-                body += indentation + `ENDLOOP.\n`;
+            if (forLoop !== undefined) {
+                indentation = indentation.substring(2);
+                body += indentation + outlineFor.end + `.\n`;
             }
             const abap = `DATA ${uniqueName} TYPE ${type}.\n` +
                 body +

package/build/src/rules/function_module_recommendations.js

@@ -32,6 +32,8 @@ class FunctionModuleRecommendationsConf extends _basic_rule_config_1.BasicRuleCo
             { name: "REUSE_ALV_GRID_DISPLAY", replace: "use CL_SALV_TABLE=>FACTORY or CL_GUI_ALV_GRID" },
             { name: "CALCULATE_HASH_FOR_RAW", replace: "use CL_ABAP_HMAC" },
             { name: "FUNCTION_EXISTS", replace: "surround with try-catch CX_SY_DYN_CALL_ILLEGAL_METHOD instead" },
+            { name: "IGN_TIMESTAMP_DIFFERENCE", replace: "use CL_ABAP_TSTMP" },
+            { name: "IGN_TIMESTAMP_PLUSMINUS", replace: "use CL_ABAP_TSTMP" },
         ];
     }
 }
@@ -45,7 +47,7 @@ class FunctionModuleRecommendations extends _abap_rule_1.ABAPRule {
         return {
             key: "function_module_recommendations",
             title: "Function Module Recommendations",
-            shortDescription: `Function Module Recommendations`,
+            shortDescription: `Suggests replacements for various function modules`,
             extendedInformation: `https://docs.abapopenchecks.org/checks/53/`,
             tags: [_irule_1.RuleTag.SingleFile],
         };

package/build/src/version.js

@@ -18,7 +18,7 @@ var Version;
     Version["v756"] = "v756";
     Version["Cloud"] = "Cloud";
 })(Version = exports.Version || (exports.Version = {}));
-exports.defaultVersion = Version.v755;
+exports.defaultVersion = Version.v756;
 function getPreviousVersion(v) {
     if (v === Version.OpenABAP) {
         return Version.v702;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.85.17",
+    "version": "2.85.21",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -48,17 +48,17 @@
         "@microsoft/api-extractor": "^7.19.4",
         "@types/chai": "^4.3.0",
         "@types/mocha": "^9.1.0",
-        "@types/node": "^17.0.17",
+        "@types/node": "^17.0.19",
         "chai": "^4.3.6",
-        "eslint": "^8.8.0",
-        "mocha": "^9.2.0",
+        "eslint": "^8.9.0",
+        "mocha": "^9.2.1",
         "c8": "^7.11.0",
         "source-map-support": "^0.5.21",
-        "ts-json-schema-generator": "^0.97.0",
+        "ts-json-schema-generator": "^0.98.0",
         "typescript": "^4.5.5"
     },
     "dependencies": {
-        "fast-xml-parser": "^4.0.2",
+        "fast-xml-parser": "^4.0.3",
         "json5": "^2.2.0",
         "vscode-languageserver-protocol": "^3.16.0",
         "vscode-languageserver-types": "^3.16.0"