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

Flag to disable rules incompatible with SAP Gui #262

Open
bnichell opened this issue Mar 5, 2024 · 2 comments
Open

Flag to disable rules incompatible with SAP Gui #262

bnichell opened this issue Mar 5, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@bnichell
Copy link

bnichell commented Mar 5, 2024

When an object is changed in the SAP Gui, it can lead to unrelated formatting changes in the coding of that object.
For instance, if a method signature is being adjusted in the Form Editor, the ! are being added to the parameter and the casing is changed.

For this reason, I propose to add a flag, with disables all rules that are affected by this incompatibility with editing ABAP in the SAP Gui.

In addition, for some rules it is not possible to only disable formatting that could be automatically changed by the SAP Gui.
E.g. the rule Convert upper and lower case has the settings Keywords/Identifiers in all other places, which also includes the class name or method names in the implementation part and these parts could be automatically adjusted by the SAP Gui.
Thus, it could make sense, to allow to only partially disable formatting, which could be automatically adjusted again by the SAP Gui.

@jmgrassau jmgrassau added the enhancement New feature or request label May 30, 2024
@jmgrassau
Copy link
Member

Hi Benedikt,

just to be sure, you would mean for such a flag to be part of the main window's "Cleanup Settings", i.e. on a similar level as "Restrict rules to syntax of: …", right? So, cleanup rules that (fully or partially or in case of certain configuration) conflict with PrettyPrinter could be active in a profile, but would still be suppressed if the flag is set?

That's conceivable but might get rather tricky, given all the combinations of options that you could set. But yes, I've heard of teams that are still in the process of convincing some colleagues of the advantages of ADT…

Kind regards,
Jörg-Michael

@bnichell
Copy link
Author

Hi Jörg-Michael,

yes, your understanding is (mostly) correct.

So, cleanup rules that (fully or partially or in case of certain configuration) conflict with PrettyPrinter could be active in a profile, but would still be suppressed if the flag is set?

I think some incompatibilities also come from the backend editor without using pretty printer, e.g. after editing the declaration part in the form editor the, e.g. the casing is adjusted automatically.
We also faced some undesired formatting changes when using gCTS, though these might be resolved in the future by the gCTS colleagues.

Cheers,
Benedikt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants