Check for: IDs Instead of Names

[aptMattKoe]

There is no good reason to use an ID as SourceTable or in variables, parameters instead of the Names.

[StefanMaron]

Would you mind to test this rule? 😎
Otherwise I would close this issue in a few days

[ChristianHovenbitzer]

The rule seems to trigger on every property with a number in it. This includes:

• Descriptions
• Tooltips
• Captions
• DecimalPlace
• Width

[aptMattKoe]

I've tested it and it works for me! :) thanks!

[aptMattKoe]

The rule seems to trigger on every property with a number in it. This includes:

• Descriptions
• Tooltips
• Captions
• DecimalPlace
• Width

Could you take a screenshot? My Tests are not doing this.

[StefanMaron]

I think I fixed it meanwhile 😜
Sorry I forgot to mention.

Anyways. there are still issues:

• numbers in object name should not trigger this warning
• direct id usage in like codeunit.Run(1) should trigger this warning

[StefanMaron]

Numbers in Object Names should not trigger the warning anymore.
direct usage of codeunit.Run(1) should be a separate rule I think See #16

[ChristianHovenbitzer]

Codeunit Permissions falsely trigger warnings

[StefanMaron]

Good catch, will look into this later 👍

[StefanMaron]

Should be fixed with version 0.7