"Convert upper/lower" case rule does not consider first statement in global class

[suynwa]

"Convert upper/lower" case rule does not consider first statement in global class

[jmgrassau]

Hi Suhas,

so now with

the implementation section depends on the first "METHOD …" line, therefore you'd get:

Kind regards,
Jörg-Michael

[jmgrassau]

Hi Suhas,

with "Derive from first statement", ABAP cleaner indeed separately determines the casing from

• the first statement in a class definition (i.e. the CLASS ... DEFINITION line itself)
• the first statement in a class implementation (i.e. the CLASS ... IMPLEMENTATION line itself)

The idea behind this is that in class definitions, SE80 always seems to use lower case keywords and upper case identifiers (regardless of the settings in menu "Utilities / Settings" -> ABAP Editor -> Pretty Printer), while for class implementations, SE80 does observe these settings, so SE80 could produce different formatting for the implementations (e.g. upper case keywords and lower case identifiers).

Therefore, if "derive from first statement" would now only derive the settings from the CLASS … DEFINITION line and someone in the team still uses SE80, you would always end up to have everything in lower case keywords and upper case identifiers – and that might not be intended, even by the SE80 users.

But you're right: SE80 does seem to put CLASS <NAME> IMPLEMENTATION to upper case! That makes this line rather useless to derive the settings from, of course. I think the best way to solve this would be to derive the settings for the implementation part from the first "METHOD method_name" line.

If that's still not satisfying, we could of course consider to add another option to the "Auto-determine upper/lower case" selection, so everyone could decide for themselves:

• "derive from first statement in code document" (as ABAP Formatter does)
• "derive from first statement of a class definition / method implementation"

Kind regards,
Jörg-Michael

[suynwa]

"derive from first statement of a class definition / method implementation"
This is promising 🥇

[jmgrassau]

Hi Suhas,

thanks again for this hint! The renamed option and adjusted behavior was now released with version 1.5.3, hope it helps!

Kind regards,
Jörg-Michael