Problem automatic added suffix for Page Controls

[RaKuNbg]

Hi Waldo,
I guess that my problem is related to your al-language-extension..

I have these settings in my settings.json file:

Now I had to do a little change in one of my page extensions.
When I saved the file, I got theses changes:

Questions:

• When was this feature introduced that adds the suffix to the controls (when I last changed the object in January, these changes were not applied - I'm pretty sure, I had the same settings)
• Is it intentionally that these changes are applied when "crs.OnSaveAlFileAction": "Rename"
• If yes: Is there a setting to control this functionality to be executed
• Why are the controls in the lower part (addlast Item) not updated

[waldo1001]

THis is what the changelog says:
It was introduced on: [1.1.0] - 2019-03-26 because of this issue: #93

There is no setting, since this is quite mandatory.

First time I see that the controls in the lower part are not updated .. seems there is a bug :-/.

[RaKuNbg]

Thanks for the quick response
I just checked some more files in my repository and it seems that I have the suffix in almost every page extension except some single ones.

And I think the cause of the bug could be parenthesis in the field or functionname of the control:

Here are two examples where the suffix was not added after changing the object:


Please check if that is the problem

But that still does not explain why in first example had the suffix in none of the controls

But one of my questions of the initial post is still open:
Is it intentionally that these changes are applied when "crs.OnSaveAlFileAction": "Rename"
==> Shouldn't it be only if the option is "Reorganize" or what excactly is the difference between the two options?
==> Would a third option "Rename and Reorganize" make sense?

[pri-kise]

I think this issue is somehow similiar to my last issue #210:

The problematic line is probably the following:

crs-al-language-extension/src/NAVObject.ts

Line 592 in 8d1444c

return /.*(field\( *"?([ a-zA-Z0-9._/&-]+)"? *; *([" a-zA-Z0-9._/&-]+) *\))/g;

Maybe all of the .fieldRegEx functions should be updated.

@christianbraeunlich maybe you can check those regexes, since you fixed the other one perfectly.