LC0052 interfaces about businesscentral.lintercop

[Arthurvdv]

@rvanbekkum, Thanks! That validation was are hiding in plain sight somehow 😅

I've moved this validation a bit up in the chain to early exclude procedures that are part of the interface.

[fvet]

The internal method PreProcessIncomingMessage in Codeunit ESCN IC Message Order (Access = Public) is declared but never used.

[Arthurvdv]

You're right, this seems like a false positive.

When a codeunit implements an interface, then LC0052 (and LC0053) should not raise a diagnostic.

[Arthurvdv]

This should now be resolved in the (pre)release version of v0.30.15 of the LinterCop.

[rvanbekkum]

The changes that were made are now leading to false negatives. It should not completely skip procedures in codeunits that implement an interface, because if you do that it skips procedures that are not part of the interface.
I think instead the method that checks if a procedure implements an interface procedure could/should be adjusted slightly. 😉

[Arthurvdv]

@rvanbekkum , you're right, thank you for bringing this to our attention.

I need to verify if the procure is declared in the interface and then skip these.

[rvanbekkum]

That's already been accounted for in the current implementation, but maybe there's an issue with the scenario in which the access modifiers of the procedures are not matching? Would need to be checked though, that's just a guess.

[Arthurvdv]

Version v0.30.16 of the LinterCop is now released. Is it possible to verify of the issue is now resolved?