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 80 | 1x 1x 1x 1x 1x 1x 1x 1x 1x 1x 1x 1x 1x 1x 10784x 10784x 10784x 10784x 10784x 32183x 32183x 32183x 32183x 32183x 32183x 32183x 32183x 32183x 32183x 32183x 32183x 32183x 32183x 10784x 10784x 10267x 10267x 10784x 10784x 240x 240x 10784x 10784x 247x 247x 247x 247x 330x 252x 252x 330x 247x 247x 247x 10784x 10784x 321x 321x 321x 10x 10x 10x 10x 3x 3x 3x 3x 10x 1x 1x 1x 1x 1x 10x 10x 321x 321x 321x 10784x 10784x | import {Issue} from "../issue";
import {IRule, IRuleMetadata, RuleTag} from "./_irule";
import {IObject} from "../objects/_iobject";
import {IRegistry} from "../_iregistry";
import {BasicRuleConfig} from "./_basic_rule_config";
import {IMSAGReferences} from "../_imsag_references";
import {Position} from "../position";
import {MessageClass} from "../objects";
import {SyntaxLogic} from "../abap/5_syntax/syntax";
import {ABAPObject} from "../objects/_abap_object";
export class EasyToFindMessagesConf extends BasicRuleConfig {
}
export class EasyToFindMessages implements IRule {
private conf = new EasyToFindMessagesConf();
private msagReferences: IMSAGReferences;
public getMetadata(): IRuleMetadata {
return {
key: "easy_to_find_messages",
title: "Easy to find messages",
shortDescription: `Make messages easy to find`,
extendedInformation: `All messages must be statically referenced exactly once
Only MESSAGE and RAISE statments are counted as static references
Also see rule "message_exists"
https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#make-messages-easy-to-find`,
tags: [RuleTag.Styleguide],
};
}
public getConfig() {
return this.conf;
}
public setConfig(conf: EasyToFindMessagesConf) {
this.conf = conf;
}
public initialize(reg: IRegistry): IRule {
this.msagReferences = reg.getMSAGReferences();
// the SyntaxLogic builds the references
for (const obj of reg.getObjects()) {
if (obj instanceof ABAPObject) {
new SyntaxLogic(reg, obj).run();
}
}
return this;
}
public run(object: IObject): Issue[] {
const issues: Issue[] = [];
if (object.getType() === "MSAG") {
const msag = object as MessageClass;
for (const message of msag.getMessages()) {
const where = this.msagReferences.listByMessage(msag.getName().toUpperCase(), message.getNumber());
if (where.length === 0) {
const text = `Message ${message.getNumber()} not statically referenced`;
const position = new Position(1, 1);
const issue = Issue.atPosition(object.getFiles()[0], position, text, this.getMetadata().key, this.conf.severity);
issues.push(issue);
} else if (where.length >= 2) {
const text = `Message ${message.getNumber()} referenced more than once`;
const position = new Position(1, 1);
const issue = Issue.atPosition(object.getFiles()[0], position, text, this.getMetadata().key, this.conf.severity);
issues.push(issue);
}
}
}
return issues;
}
} |