search

waldo1001 / crs-al-language-extension

Make working with the (Dynamics NAV / 365) AL Language easier and more efficient.
MIT License
34 stars 42 forks source link

Conditional snippets excluding DataClassification / ApplicationArea ? #267

Open fvet opened 1 year ago

fvet commented 1 year ago

@waldo1001 Not sure if issues are actively monitored here, but wanted to get your opinion on the below ;)

Since AL runtime 10 some properties are no longer mandatory.

Some of the page field / table field related snippets contain default code for setting the DataClassification / ApplicationArea properties.

It would be great if - based on the AL runtime version - these settings could be removed / no longer suggested as part of the snippet.

One of the complexities is that these properties are only redundant on table / page objects and NOT on table extension / page extension objects.

PS: We're using following rules to discover / cleanup redundant properties.

https://github.com/StefanMaron/BusinessCentral.LinterCop/wiki/LC0019 https://github.com/StefanMaron/BusinessCentral.LinterCop/wiki/LC0020

waldo1001 commented 1 year ago

@fvet ,

Valid! I could do this, but I can' t do this based on runtime - it's "just" a snippet.

So - doing it "now" - would that make sense?