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

Removing trailing spaces in field names #287

Open josefonsecatnp opened 1 year ago

josefonsecatnp commented 1 year ago

Hi and thanks for your attention in advance.

In a situation where a field was created with a trailing space (I know this is incorrect but happened by mistake), the tool is removing the trailing space.

To avoid the problem, the developer needs to disable the tool, obsolete the field, create a new one with a better name, and migrate the data across.

In my honest opinion the tool should not change field names in any way.

Thank you.

Phil-Sawyer commented 1 year ago

Hi,

I would like to second this.

I understand from a new field or object point of view this can be useful tool to 'tidy up' unintentional mistakes, however it's dangerous when modifying existing deployed code where the result will be that a live data structure will potentially be modified.

Many thanks.