Do not use object ids in code like Codeunit.Run()

[StefanMaron]

This goes for

• Codeunit.Run()
• Report.Run()
• Report.Execute()
• Report.Save...()
• XMLPort.Export/Import/Run()
• Page.Run/RunModal()
• RecRef.Open()

[fvet]

Please also include the same for EventSubscribers:

[StefanMaron]

None of these should compile. But they do, and we only get told at runtime that someone goofed.

I get that the base Page object takes generic var Record: Table arguments, so at the base level those can't be validated - but it is hopefully possible for specific Pages, which have a specific SourceTable set, to error out if they are passed a Table that is not of the same type.

Please can you consider doing this? It would catch common thinkos and save much time.

I might've missed some methods taking Record: Table arguments from my example, in which case, please include all such methods in this fix.

microsoft/AL#6438
microsoft/AL#6525 (closed, but see my follow-up comment)

https://experience.dynamics.com/ideas/idea/?ideaid=df8e9c09-3986-eb11-8ced-0003ff45a07d

Posted by https://github.com/dnpb

[StefanMaron]

I tried to get this to work as well but it seems its not that easy.

[StefanMaron]

Boom!

[fvet]

Boom!

Very nice! Are you considering adding EventSubscribers as well?

[StefanMaron]

Ah, I forgot that one :D I will look into it!

[rvanbekkum]

Nice! 😊