@abaplint/core 2.85.7 → 2.85.11

package/README.md

@@ -1,7 +1,7 @@
-# @abaplint/core
-
-[abaplint](https://abaplint.org/) core library
-
-Exposes functionallity like the parser and rules, which can be used in other projects.
-
+# @abaplint/core
+
+[abaplint](https://abaplint.org/) core library
+
+Exposes functionallity like the parser and rules, which can be used in other projects.
+
 For more information see https://github.com/abaplint/abaplint

package/build/src/abap/2_statements/combi.js

@@ -313,13 +313,19 @@ class StarPrioroity {
             res = this.sta.run(res);
             if (res.length === 0) {
                 if (prev !== undefined) {
-                    result = prev;
+                    //          console.log("star length: " + prev.length);
+                    let best = Number.MAX_SAFE_INTEGER;
+                    for (const p of prev) {
+                        if (p.remainingLength() < best) {
+                            result = [p];
+                            best = p.remainingLength();
+                        }
+                    }
                 }
                 break;
             }
             prev = res;
         }
-        //    console.dir(result);
         return result;
     }
     railroad() {
@@ -401,13 +407,17 @@ class Sequence {
         const result = [];
         for (const input of r) {
             let temp = [input];
+            let match = true;
             for (const sequence of this.list) {
                 temp = sequence.run(temp);
                 if (temp.length === 0) {
+                    match = false;
                     break;
                 }
             }
-            result.push(...temp);
+            if (match === true) {
+                result.push(...temp);
+            }
         }
         return result;
     }
@@ -702,7 +712,13 @@ class Combi {
         this.ver = version;
         const input = new result_1.Result(tokens, 0);
         const result = runnable.run([input]);
-        //    console.log("res: " + result.length);
+        /*
+            console.log("res: " + result.length);
+            for (const res of result) {
+              console.dir(res.getNodes().map(n => n.get().constructor.name));
+              console.dir(res.getNodes().map(n => n.concatTokens()));
+            }
+        */
         for (const res of result) {
             if (res.remainingLength() === 0) {
                 return res.getNodes();

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

@@ -5,7 +5,7 @@ const combi_1 = require("../combi");
 const _1 = require(".");
 class SimpleSource3 extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.alt)(_1.Constant, _1.TextElement, _1.FieldChain);
+        return (0, combi_1.altPrio)(_1.Constant, _1.TextElement, _1.FieldChain);
     }
 }
 exports.SimpleSource3 = SimpleSource3;

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

@@ -8,7 +8,7 @@ const field_chain_1 = require("./field_chain");
 class SQLCDSParameters extends combi_1.Expression {
     getRunnable() {
         const param = (0, combi_1.seq)(_1.Field, "=", (0, combi_1.alt)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), field_chain_1.FieldChain), _1.Constant));
-        return (0, combi_1.seq)("(", param, (0, combi_1.star)((0, combi_1.seq)(",", param)), ")");
+        return (0, combi_1.seq)("(", param, (0, combi_1.starPrio)((0, combi_1.seq)(",", param)), ")");
     }
 }
 exports.SQLCDSParameters = SQLCDSParameters;

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

@@ -10,15 +10,15 @@ class SQLCompare extends combi_1.Expression {
         const val = new _1.SQLSource();
         const list = (0, combi_1.seq)((0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.WParenLeft), (0, combi_1.tok)(tokens_1.WParenLeftW)), val, (0, combi_1.starPrio)((0, combi_1.seq)(",", val)), ")");
         const subSelect = (0, combi_1.seq)("(", _1.Select, ")");
-        const inn = (0, combi_1.seq)((0, combi_1.optPrio)("NOT"), "IN", (0, combi_1.altPrio)(_1.SQLSource, list, subSelect));
-        const between = (0, combi_1.seq)((0, combi_1.optPrio)("NOT"), "BETWEEN", _1.SQLSource, "AND", _1.SQLSource);
-        const like = (0, combi_1.seq)((0, combi_1.optPrio)("NOT"), "LIKE", _1.SQLSource, (0, combi_1.optPrio)((0, combi_1.seq)("ESCAPE", _1.SQLSource)));
-        const nul = (0, combi_1.seq)("IS", (0, combi_1.optPrio)("NOT"), (0, combi_1.alt)("NULL", (0, combi_1.ver)(version_1.Version.v753, "INITIAL")));
+        const inn = (0, combi_1.seq)("IN", (0, combi_1.altPrio)(_1.SQLSource, list, subSelect));
+        const between = (0, combi_1.seq)("BETWEEN", _1.SQLSource, "AND", _1.SQLSource);
+        const like = (0, combi_1.seq)("LIKE", _1.SQLSource, (0, combi_1.optPrio)((0, combi_1.seq)("ESCAPE", _1.SQLSource)));
+        const nul = (0, combi_1.seq)("IS", (0, combi_1.optPrio)("NOT"), (0, combi_1.altPrio)("NULL", (0, combi_1.ver)(version_1.Version.v753, "INITIAL")));
         const source = new _1.SQLSource();
         const sub = (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.altPrio)("ALL", "ANY", "SOME")), subSelect);
-        const builtin = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)((0, combi_1.alt)("lower", "upper"), (0, combi_1.tok)(tokens_1.ParenLeftW), _1.SQLFieldName, (0, combi_1.tok)(tokens_1.WParenRightW)));
-        const arith = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)(_1.SQLFieldName, (0, combi_1.plusPrio)((0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/"), _1.SQLFieldName))));
-        const rett = (0, combi_1.seq)((0, combi_1.altPrio)(builtin, arith, _1.SQLFieldName), (0, combi_1.altPrio)((0, combi_1.seq)(_1.SQLCompareOperator, (0, combi_1.altPrio)(sub, source)), inn, like, between, nul));
+        const builtin = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)((0, combi_1.altPrio)("lower", "upper"), (0, combi_1.tok)(tokens_1.ParenLeftW), _1.SQLFieldName, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const arith = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.plusPrio)((0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/"), _1.SQLFieldName)));
+        const rett = (0, combi_1.seq)((0, combi_1.altPrio)(builtin, (0, combi_1.seq)(_1.SQLFieldName, (0, combi_1.optPrio)(arith))), (0, combi_1.altPrio)((0, combi_1.seq)(_1.SQLCompareOperator, (0, combi_1.altPrio)(sub, source)), (0, combi_1.seq)((0, combi_1.optPrio)("NOT"), (0, combi_1.altPrio)(inn, like, between)), nul));
         const exists = (0, combi_1.seq)("EXISTS", subSelect);
         return (0, combi_1.altPrio)(exists, _1.Dynamic, rett);
     }

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

@@ -11,7 +11,7 @@ class SQLFromSource extends combi_1.Expression {
         // https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abennews-752-open_sql.htm#!ABAP_MODIFICATION_1@1@
         const tab = (0, combi_1.ver)(version_1.Version.v752, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), _1.FieldChain));
         const aas = (0, combi_1.seq)("AS", _1.SQLAsName);
-        return (0, combi_1.seq)((0, combi_1.altPrio)(_1.Dynamic, with_name_1.WithName, (0, combi_1.seq)(_1.DatabaseTable, (0, combi_1.optPrio)(_1.SQLCDSParameters)), tab), (0, combi_1.optPrio)((0, combi_1.ver)(version_1.Version.v752, "WITH PRIVILEGED ACCESS")), (0, combi_1.optPrio)(aas));
+        return (0, combi_1.seq)((0, combi_1.altPrio)(with_name_1.WithName, (0, combi_1.seq)(_1.DatabaseTable, (0, combi_1.optPrio)(_1.SQLCDSParameters)), tab), (0, combi_1.optPrio)((0, combi_1.ver)(version_1.Version.v752, "WITH PRIVILEGED ACCESS")), (0, combi_1.optPrio)(aas));
     }
 }
 exports.SQLFromSource = SQLFromSource;

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

@@ -8,9 +8,8 @@ const _1 = require(".");
 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));
-        // todo, this Source must be a simple field?
-        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.Source, paren)));
-        return (0, combi_1.alt)(_1.SQLAliasField, _1.Source, at);
+        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);
     }
 }
 exports.SQLSource = SQLSource;

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

@@ -5,12 +5,12 @@ const combi_1 = require("../combi");
 const version_1 = require("../../../version");
 const tokens_1 = require("../../1_lexer/tokens");
 const _1 = require(".");
-const constant_string_1 = require("./constant_string");
+// todo, delete this expression, its the same as SQLSource?
 class SQLSourceSimple 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.alt)(_1.FieldChain, paren)));
-        return (0, combi_1.alt)(_1.FieldChain, at, constant_string_1.ConstantString);
+        const at = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), (0, combi_1.alt)(_1.SimpleSource3, paren)));
+        return (0, combi_1.alt)(_1.SimpleSource3, at);
     }
 }
 exports.SQLSourceSimple = SQLSourceSimple;

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

@@ -9,7 +9,7 @@ class TypeTableKey extends combi_1.Expression {
         const uniqueness = (0, combi_1.alt)("NON-UNIQUE", "UNIQUE");
         const defaultKey = "DEFAULT KEY";
         const emptyKey = (0, combi_1.ver)(version_1.Version.v740sp02, "EMPTY KEY");
-        const key = (0, combi_1.seq)("WITH", (0, combi_1.opt)(uniqueness), (0, combi_1.altPrio)(defaultKey, emptyKey, (0, combi_1.seq)((0, combi_1.opt)((0, combi_1.alt)("SORTED", "HASHED")), "KEY", (0, combi_1.alt)((0, combi_1.seq)(_1.Field, "COMPONENTS", (0, combi_1.plus)(_1.FieldSub)), (0, combi_1.plus)(_1.FieldSub)))));
+        const key = (0, combi_1.seq)("WITH", (0, combi_1.opt)(uniqueness), (0, combi_1.altPrio)(defaultKey, emptyKey, (0, combi_1.seq)((0, combi_1.opt)((0, combi_1.alt)("SORTED", "HASHED")), "KEY", (0, combi_1.alt)((0, combi_1.seq)(_1.Field, "COMPONENTS", (0, combi_1.plus)(_1.FieldSub)), (0, combi_1.plus)(_1.FieldSub)))), (0, combi_1.optPrio)("READ-ONLY"));
         return key;
     }
 }

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

@@ -20,6 +20,8 @@ class TypeUtils {
             || type instanceof basic_1.AnyType
             || type instanceof basic_1.UnknownType
             || type instanceof basic_1.NumericType
+            || type instanceof basic_1.IntegerType
+            || type instanceof basic_1.NumericGenericType
             || type instanceof basic_1.CSequenceType
             || type instanceof basic_1.DateType
             || type instanceof basic_1.CLikeType
@@ -53,8 +55,8 @@ class TypeUtils {
     }
     static isAssignable(source, target) {
         /*
-        console.dir(source);
-        console.dir(target);
+            console.dir(source);
+            console.dir(target);
         */
         if (target instanceof basic_1.TableType) {
             if (target.isWithHeader()) {
@@ -97,8 +99,13 @@ class TypeUtils {
                 || source instanceof basic_1.UnknownType) {
                 return true;
             }
-            else if (this.isCharLike(target)
-                && (this.isCharLike(source) || source instanceof basic_1.IntegerType)) {
+            else if (target.containsVoid() === true) {
+                return true;
+            }
+            else if (source instanceof basic_1.IntegerType) {
+                return false;
+            }
+            else if (this.isCharLike(target) && this.isCharLike(source)) {
                 return true;
             }
             return false;

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

@@ -18,7 +18,7 @@ class AttributeChain {
         if (!(first.get() instanceof expressions_1.AttributeName)) {
             throw new Error("AttributeChain, unexpected first child");
         }
-        const def = scope.findClassDefinition(inputContext.getIdentifierName());
+        const def = scope.findObjectDefinition(inputContext.getIdentifierName());
         if (def === undefined) {
             throw new Error("Definition for \"" + inputContext.getIdentifierName() + "\" not found in scope(AttributeChain)");
         }

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

@@ -8,6 +8,9 @@ class Constant {
         if (node.findDirectExpression(expressions_1.Integer)) {
             return new basic_1.IntegerType("I");
         }
+        else if (node.getFirstToken().getStr().startsWith("'")) {
+            return new basic_1.CharacterType(10);
+        }
         else {
             return new basic_1.StringType("STRING");
         }

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

@@ -70,6 +70,9 @@ class FieldChain {
                 context = new dereference_1.Dereference().runSyntax(context);
             }
             else if (current.get() instanceof Expressions.ComponentName) {
+                if (context instanceof basic_1.TableType && context.isWithHeader()) {
+                    context = context.getRowType();
+                }
                 context = new component_name_1.ComponentName().runSyntax(context, current);
             }
             else if (current instanceof nodes_1.ExpressionNode

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

@@ -45,6 +45,9 @@ class Select {
         for (const s of node.findAllExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(s, scope, filename);
         }
+        for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
+            new source_1.Source().runSyntax(s, scope, filename);
+        }
         if (scope.getType() === _scope_type_1.ScopeType.OpenSQL) {
             scope.pop(node.getLastToken().getEnd());
         }

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

@@ -6,7 +6,10 @@ const source_1 = require("./source");
 const basic_1 = require("../../types/basic");
 class SQLForAllEntries {
     runSyntax(node, scope, filename) {
-        const s = node.findFirstExpression(Expressions.Source);
+        let s = node.findFirstExpression(Expressions.Source);
+        if (s === undefined) {
+            s = node.findFirstExpression(Expressions.SimpleSource3);
+        }
         if (s) {
             const type = new source_1.Source().runSyntax(s, scope, filename);
             if (type instanceof basic_1.VoidType) {

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

@@ -5,6 +5,7 @@ const Expressions = require("../../2_statements/expressions");
 const target_1 = require("../expressions/target");
 const source_1 = require("../expressions/source");
 const dynamic_1 = require("../expressions/dynamic");
+const basic_types_1 = require("../basic_types");
 class CreateData {
     runSyntax(node, scope, filename) {
         for (const s of node.findDirectExpressions(Expressions.Source)) {
@@ -16,6 +17,10 @@ class CreateData {
         for (const t of node.findDirectExpressions(Expressions.Dynamic)) {
             new dynamic_1.Dynamic().runSyntax(t, scope, filename);
         }
+        const type = node.findDirectExpression(Expressions.TypeName);
+        if (type) {
+            new basic_types_1.BasicTypes(filename, scope).resolveTypeName(type);
+        }
     }
 }
 exports.CreateData = CreateData;

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

@@ -10,6 +10,9 @@ class DeleteDatabase {
         for (const s of node.findAllExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(s, scope, filename);
         }
+        for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
+            new source_1.Source().runSyntax(s, scope, filename);
+        }
         for (const d of node.findAllExpressions(Expressions.Dynamic)) {
             new dynamic_1.Dynamic().runSyntax(d, scope, filename);
         }

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

@@ -10,6 +10,9 @@ class InsertDatabase {
         for (const s of node.findAllExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(s, scope, filename);
         }
+        for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
+            new source_1.Source().runSyntax(s, scope, filename);
+        }
         for (const d of node.findAllExpressions(Expressions.Dynamic)) {
             new dynamic_1.Dynamic().runSyntax(d, scope, filename);
         }

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

@@ -4,6 +4,7 @@ exports.ModifyDatabase = void 0;
 const Expressions = require("../../2_statements/expressions");
 const dynamic_1 = require("../expressions/dynamic");
 const database_table_1 = require("../expressions/database_table");
+const source_1 = require("../expressions/source");
 class ModifyDatabase {
     runSyntax(node, scope, filename) {
         for (const d of node.findAllExpressions(Expressions.Dynamic)) {
@@ -13,6 +14,12 @@ class ModifyDatabase {
         if (dbtab !== undefined) {
             new database_table_1.DatabaseTable().runSyntax(dbtab, scope, filename);
         }
+        for (const s of node.findAllExpressions(Expressions.Source)) {
+            new source_1.Source().runSyntax(s, scope, filename);
+        }
+        for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
+            new source_1.Source().runSyntax(s, scope, filename);
+        }
     }
 }
 exports.ModifyDatabase = ModifyDatabase;

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

@@ -32,6 +32,9 @@ class UpdateDatabase {
         for (const s of node.findAllExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(s, scope, filename);
         }
+        for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
+            new source_1.Source().runSyntax(s, scope, filename);
+        }
         for (const d of node.findAllExpressions(Expressions.Dynamic)) {
             new dynamic_1.Dynamic().runSyntax(d, scope, filename);
         }

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

@@ -5,11 +5,18 @@ const Expressions = require("../../2_statements/expressions");
 const source_1 = require("../expressions/source");
 const target_1 = require("../expressions/target");
 const dynamic_1 = require("../expressions/dynamic");
+const _type_utils_1 = require("../_type_utils");
 class Write {
     runSyntax(node, scope, filename) {
         // todo, more
+        const second = node.getChildren()[1];
         for (const s of node.findAllExpressions(Expressions.Source)) {
-            new source_1.Source().runSyntax(s, scope, filename);
+            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) {
+                throw new Error("Source not character like");
+            }
         }
         for (const s of node.findAllExpressions(Expressions.Dynamic)) {
             new dynamic_1.Dynamic().runSyntax(s, scope, filename);

package/build/src/abap/flow/flow_graph.js

@@ -72,13 +72,13 @@ class FlowGraph {
         this.label = label;
     }
     toDigraph() {
-        return `digraph G {
-labelloc="t";
-label="${this.label}";
-graph [fontname = "helvetica"];
-node [fontname = "helvetica", shape="box"];
-edge [fontname = "helvetica"];
-${this.toTextEdges()}
+        return `digraph G {
+labelloc="t";
+label="${this.label}";
+graph [fontname = "helvetica"];
+node [fontname = "helvetica", shape="box"];
+edge [fontname = "helvetica"];
+${this.toTextEdges()}
 }`;
     }
     listSources(node) {

package/build/src/abap/types/basic/structure_type.js

@@ -43,7 +43,7 @@ class StructureType extends _abstract_type_1.AbstractType {
         return "StructureTypetoABAPtodo";
     }
     containsVoid() {
-        return this.getComponents().some(c => { c.type.containsVoid(); });
+        return this.getComponents().some(c => { return c.type.containsVoid(); });
     }
     toCDS() {
         return "abap.TODO_STRUCTURE";

package/build/src/lsp/help.js

@@ -19,13 +19,13 @@ class Help {
     /////////////////////////////////////////////////
     static dumpABAP(file, reg, textDocument, position) {
         let content = "";
-        content = `
-    <a href="#_tokens" rel="no-refresh">Tokens</a> |
-    <a href="#_statements" rel="no-refresh">Statements</a> |
-    <a href="#_structure" rel="no-refresh">Structure</a> |
-    <a href="#_files" rel="no-refresh">Files</a> |
-    <a href="#_info" rel="no-refresh">Info Dump</a>
-    <hr>
+        content = `
+    <a href="#_tokens" rel="no-refresh">Tokens</a> |
+    <a href="#_statements" rel="no-refresh">Statements</a> |
+    <a href="#_structure" rel="no-refresh">Structure</a> |
+    <a href="#_files" rel="no-refresh">Files</a> |
+    <a href="#_info" rel="no-refresh">Info Dump</a>
+    <hr>
     ` +
             "<tt>" + textDocument.uri + " (" +
             (position.line + 1) + ", " +

package/build/src/registry.js

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

package/build/src/rules/7bit_ascii.js

@@ -17,8 +17,8 @@ class SevenBitAscii {
             key: "7bit_ascii",
             title: "Check for 7bit ascii",
             shortDescription: `Only allow characters from the 7bit ASCII set.`,
-            extendedInformation: `https://docs.abapopenchecks.org/checks/05/
-
+            extendedInformation: `https://docs.abapopenchecks.org/checks/05/
+
 Checkes files with extension ".abap" and ".asddls"`,
             tags: [_irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/abapdoc.js

@@ -23,7 +23,7 @@ class Abapdoc extends _abap_rule_1.ABAPRule {
         return {
             key: "abapdoc",
             title: "Check abapdoc",
-            shortDescription: `Various checks regarding abapdoc.
+            shortDescription: `Various checks regarding abapdoc.
 Base rule checks for existence of abapdoc for public class methods and all interface methods.`,
             tags: [_irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/align_parameters.js

@@ -20,42 +20,42 @@ class AlignParameters extends _abap_rule_1.ABAPRule {
             key: "align_parameters",
             title: "Align Parameters",
             shortDescription: `Checks for vertially aligned parameters`,
-            extendedInformation: `Checks:
-* function module calls
-* method calls
-* VALUE constructors
-* NEW constructors
-* RAISE EXCEPTION statements
-* CREATE OBJECT statements
-* RAISE EVENT statements
-
-https://github.com/SAP/styleguides/blob/master/clean-abap/CleanABAP.md#align-parameters
-
-Does not take effect on non functional method calls, use https://rules.abaplint.org/functional_writing/
-
+            extendedInformation: `Checks:
+* function module calls
+* method calls
+* VALUE constructors
+* NEW constructors
+* RAISE EXCEPTION statements
+* CREATE OBJECT statements
+* RAISE EVENT statements
+
+https://github.com/SAP/styleguides/blob/master/clean-abap/CleanABAP.md#align-parameters
+
+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],
-            badExample: `CALL FUNCTION 'FOOBAR'
-  EXPORTING
-    foo = 2
-    parameter = 3.
-
-foobar( moo = 1
-  param = 1 ).
-
-foo = VALUE #(
-    foo = bar
+            badExample: `CALL FUNCTION 'FOOBAR'
+  EXPORTING
+    foo = 2
+    parameter = 3.
+
+foobar( moo = 1
+  param = 1 ).
+
+foo = VALUE #(
+    foo = bar
         moo = 2 ).`,
-            goodExample: `CALL FUNCTION 'FOOBAR'
-  EXPORTING
-    foo       = 2
-    parameter = 3.
-
-foobar( moo   = 1
-        param = 1 ).
-
-foo = VALUE #(
-    foo = bar
+            goodExample: `CALL FUNCTION 'FOOBAR'
+  EXPORTING
+    foo       = 2
+    parameter = 3.
+
+foobar( moo   = 1
+        param = 1 ).
+
+foo = VALUE #(
+    foo = bar
     moo = 2 ).`,
         };
     }

package/build/src/rules/ambiguous_statement.js

@@ -20,14 +20,14 @@ class AmbiguousStatement extends _abap_rule_1.ABAPRule {
         return {
             key: "ambiguous_statement",
             title: "Check for ambigious statements",
-            shortDescription: `Checks for ambiguity between deleting or modifying from internal and database table
-Add "TABLE" keyword or "@" for escaping SQL variables
-
+            shortDescription: `Checks for ambiguity between deleting or modifying from internal and database table
+Add "TABLE" keyword or "@" for escaping SQL variables
+
 Only works if the target version is 740sp05 or above`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `DELETE foo FROM bar.
+            badExample: `DELETE foo FROM bar.
 MODIFY foo FROM bar.`,
-            goodExample: `DELETE foo FROM @bar.
+            goodExample: `DELETE foo FROM @bar.
 MODIFY TABLE foo FROM bar.`,
         };
     }

package/build/src/rules/avoid_use.js

@@ -35,12 +35,12 @@ class AvoidUse extends _abap_rule_1.ABAPRule {
             key: "avoid_use",
             title: "Avoid use of certain statements",
             shortDescription: `Detects usage of certain statements.`,
-            extendedInformation: `DEFAULT KEY: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-default-key
-
-Macros: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abenmacros_guidl.htm
-
-STATICS: use CLASS-DATA instead
-
+            extendedInformation: `DEFAULT KEY: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-default-key
+
+Macros: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abenmacros_guidl.htm
+
+STATICS: use CLASS-DATA instead
+
 DESCRIBE TABLE LINES: use lines() instead (quickfix exists)`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/begin_end_names.js

@@ -23,11 +23,11 @@ class BeginEndNames extends _abap_rule_1.ABAPRule {
             title: "Check BEGIN END names",
             shortDescription: `Check BEGIN OF and END OF names match, plus there must be statements between BEGIN and END`,
             tags: [_irule_1.RuleTag.Syntax, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
-            badExample: `DATA: BEGIN OF stru,
-      field TYPE i,
+            badExample: `DATA: BEGIN OF stru,
+      field TYPE i,
     END OF structure_not_the_same.`,
-            goodExample: `DATA: BEGIN OF stru,
-      field TYPE i,
+            goodExample: `DATA: BEGIN OF stru,
+      field TYPE i,
     END OF stru.`,
         };
     }

package/build/src/rules/begin_single_include.js

@@ -21,19 +21,19 @@ class BeginSingleInclude extends _abap_rule_1.ABAPRule {
             title: "BEGIN contains single INCLUDE",
             shortDescription: `Finds TYPE BEGIN with just one INCLUDE TYPE, and DATA with single INCLUDE STRUCTURE`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `TYPES: BEGIN OF dummy1.
-  INCLUDE TYPE dselc.
-TYPES: END OF dummy1.
-
-DATA BEGIN OF foo.
-INCLUDE STRUCTURE syst.
-DATA END OF foo.
-
-STATICS BEGIN OF bar.
-INCLUDE STRUCTURE syst.
+            badExample: `TYPES: BEGIN OF dummy1.
+  INCLUDE TYPE dselc.
+TYPES: END OF dummy1.
+
+DATA BEGIN OF foo.
+INCLUDE STRUCTURE syst.
+DATA END OF foo.
+
+STATICS BEGIN OF bar.
+INCLUDE STRUCTURE syst.
 STATICS END OF bar.`,
-            goodExample: `DATA BEGIN OF foo.
-INCLUDE STRUCTURE dselc.
+            goodExample: `DATA BEGIN OF foo.
+INCLUDE STRUCTURE dselc.
 DATA END OF foo.`,
         };
     }

package/build/src/rules/call_transaction_authority_check.js

@@ -24,9 +24,9 @@ class CallTransactionAuthorityCheck extends _abap_rule_1.ABAPRule {
             extendedInformation: `https://docs.abapopenchecks.org/checks/54/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Security],
             badExample: `CALL TRANSACTION 'FOO'.`,
-            goodExample: `TRY.
-    CALL TRANSACTION 'FOO' WITH AUTHORITY-CHECK.
-  CATCH cx_sy_authorization_error.
+            goodExample: `TRY.
+    CALL TRANSACTION 'FOO' WITH AUTHORITY-CHECK.
+  CATCH cx_sy_authorization_error.
 ENDTRY.`,
         };
     }