Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggestion: Error for Using xRec on OnInsert or OnDelete table trigger #778

Open
DanielGoehler opened this issue Oct 15, 2017 · 2 comments
Labels
code-analysis-framework suggestion This is either a suggestion of a completely new feature or a fairly big change request.

Comments

@DanielGoehler
Copy link
Contributor

It would be nice if an error occurs for using xRec on OnInsert or OnDelete table trigger.

@esbenk esbenk added suggestion This is either a suggestion of a completely new feature or a fairly big change request. and removed bug Product bug labels Oct 31, 2017
@esbenk
Copy link
Contributor

esbenk commented Oct 31, 2017

The xRec property is available in all triggers and methods in the table object. If you call another method from the OnInsert/OnDelete trigger it can still use the xRec. The same method can be called from both OnInsert and OnModify and we won't be able to detect this using static analysis.

We will (when ready) release a Analyzer framework that will allow developers to write their own rules. It will be possible to create a rule like that using that framework.

@DanielGoehler
Copy link
Contributor Author

Hi @esbenk, can I write my own rule yet?

@MarcHansenMicrosoft MarcHansenMicrosoft added CodeCop This is a specific static-code-analysis group (AA) code-analysis-framework and removed CodeCop This is a specific static-code-analysis group (AA) static-code-analysis labels Jan 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code-analysis-framework suggestion This is either a suggestion of a completely new feature or a fairly big change request.
Projects
None yet
Development

No branches or pull requests

5 participants