Press n or j to go to the next uncovered block, b, p or k for the previous block.
| 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 | 1x 1x 1x 1x 1x 1x 1x 1x 1x 1x 1x 1x 1x 1x 15473x 15473x 15473x 15473x 61199x 61199x 61199x 61199x 61199x 61199x 61199x 15473x 15473x 14908x 14908x 15473x 15473x 264x 264x 15473x 15473x 295x 295x 295x 1651x 1648x 1648x 3x 3x 3x 1651x 1x 1x 10x 10x 1x 1x 2x 1651x 27x 10x 10x 10x 27x 2x 295x 295x 295x 15473x 15473x 3x 3x 3x 15473x 15473x 3x 3x 3x 3x 3x 15473x 15473x | import * as Statements from "../abap/2_statements/statements";
import * as Expressions from "../abap/2_statements/expressions";
import {Issue} from "../issue";
import {ABAPRule} from "./_abap_rule";
import {BasicRuleConfig} from "./_basic_rule_config";
import {RuleTag, IRuleMetadata} from "./_irule";
import {ABAPFile} from "../abap/abap_file";
import {ExpressionNode} from "../abap/nodes/expression_node";
// todo: this rule needs refactoring
export class ParserBadExceptionsConf extends BasicRuleConfig {
}
export class ParserBadExceptions extends ABAPRule {
private conf = new ParserBadExceptionsConf();
public getMetadata(): IRuleMetadata {
return {
key: "parser_bad_exceptions",
title: "Parser Error, bad EXCEPTIONS in CALL FUNCTION",
shortDescription: `Checks for syntax not recognized by abaplint, related to EXCEPTIONS in CALL FUNCTION.`,
tags: [RuleTag.Syntax, RuleTag.SingleFile],
};
}
public getConfig() {
return this.conf;
}
public setConfig(conf: ParserBadExceptionsConf) {
this.conf = conf;
}
public runParsed(file: ABAPFile) {
const issues: Issue[] = [];
for (const statement of file.getStatements()) {
if (!(statement.get() instanceof Statements.CallFunction)) {
continue;
}
const exceptionTokens = this.findExceptionTokens(statement);
if (exceptionTokens !== undefined
&& this.containsMixedExceptionSyntax(statement.findAllExpressions(Expressions.ParameterException))) {
const message = "Bad EXCEPTIONS syntax in CALL FUNCTION";
for (const token of exceptionTokens) {
issues.push(Issue.atToken(file, token, message, this.getMetadata().key, this.conf.severity));
}
continue;
}
for (const e of statement.findAllExpressions(Expressions.ParameterException)) {
if (e.findDirectTokenByText("=") === undefined) {
const message = "Bad EXCEPTIONS syntax in CALL FUNCTION";
issues.push(Issue.atToken(file, e.getFirstToken(), message, this.getMetadata().key, this.conf.severity));
}
}
}
return issues;
}
private containsMixedExceptionSyntax(exceptions: readonly ExpressionNode[]) {
return exceptions.some(e => e.findDirectTokenByText("=") === undefined)
&& exceptions.some(e => e.findDirectTokenByText("=") !== undefined);
}
private findExceptionTokens(statement: ReturnType<ABAPFile["getStatements"]>[number]) {
const tokens = statement.getTokens();
const index = tokens.findIndex(t => t.getStr().toUpperCase() === "EXCEPTIONS");
if (index === -1) {
return undefined;
}
return tokens.slice(index).filter(t => t.getStr() !== ".");
}
}
|