You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I just noticed this great feature now.
Here are some examples from my code where I would have wanted the rule to act.
Best regards,
Edo
TYPES tp_line TYPE zvo3_c_sdopenorderschedline. " SD is the template also for MM
TYPES ttp_lines TYPE STANDARD TABLE OF tp_line WITH KEY zDocumentCategory zdocument zitem scheduleline.
ABAPCleaner should change zdocument etc. in the second line to zDocument etc, because it is a CDS field name (as is also implied from the previous line). (The first line is adapted by ABAPCleaner as expected)
TYPES:
BEGIN OF tp_key_fields,
zDocumentCategory TYPE zv_zvoov_type,
zdocument TYPE zv_document,
zitem TYPE zv_item,
scheduleline TYPE etenr,
END OF tp_key_fields.
Here, it is not evident from context that zdocument is also a CDS field name. Still, for the coding style that I tried in this program, I would like the CDS field name "camelization" to be applied.
I suppose you have intentionally not provided this logic so far.
It would be nice to have it, at least as a non-default, "one-time" option "replace ALL name matches".
TYPES ty_range_incomplete TYPE RANGE OF zvo3_c_sdopenorderschedline-zorderincomplete.
Another example where it is clear from context that it is a CDS field.
ASSERT ID zlog FIELDS c_line->zDocumentCategory c_line->zdocument c_line->zitem CONDITION c_line->zTotalRequiredQuantityForMat IS INITIAL
AND c_line->zunconfirmedptpstoquantity IS INITIAL.
One last example. It should be clear from the type of c_line that these are CDS field names. But I do not know how ABAPCleaner analyzes the code internally. If it is not technically possible for ABAPCleaner to recognize the context correctly, I would still love to have the "replace ALL name matches" option, mentioned above.
The text was updated successfully, but these errors were encountered:
Hi,
I just noticed this great feature now.
Here are some examples from my code where I would have wanted the rule to act.
Best regards,
Edo
ABAPCleaner should change zdocument etc. in the second line to zDocument etc, because it is a CDS field name (as is also implied from the previous line). (The first line is adapted by ABAPCleaner as expected)
Here, it is not evident from context that zdocument is also a CDS field name. Still, for the coding style that I tried in this program, I would like the CDS field name "camelization" to be applied.
I suppose you have intentionally not provided this logic so far.
It would be nice to have it, at least as a non-default, "one-time" option "replace ALL name matches".
TYPES ty_range_incomplete TYPE RANGE OF zvo3_c_sdopenorderschedline-zorderincomplete.
Another example where it is clear from context that it is a CDS field.
One last example. It should be clear from the type of c_line that these are CDS field names. But I do not know how ABAPCleaner analyzes the code internally. If it is not technically possible for ABAPCleaner to recognize the context correctly, I would still love to have the "replace ALL name matches" option, mentioned above.
The text was updated successfully, but these errors were encountered: