Lars Hvam

better handling of duplicate filenames

1

should give an error ie. if the file is defined in deps and local repo

dont use `cl_ujt_utility=>string2xstring(`

prefer_corresponding is clean abap relevant

1

somewhere in https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md it says to use new constructs when possible

IF in LOOP, reduces to loop

4

```abap LOOP AT ref_scan_manager->get_tokens( ) ASSIGNING FIELD-SYMBOL() FROM statement-from TO statement-to WHERE type = 'I'. IF ( -str CS 'CL_GUI_' ). has_redirection = abap_true. ENDIF. ENDLOOP. ``` to ```abap LOOP...

syntax error not reported, types class def

1

ty_edge is referenced before definition, ```abap CLASS zcl_oapi_graph DEFINITION PUBLIC. PUBLIC SECTION. METHODS add_vertex IMPORTING iv_vertex TYPE string. METHODS add_edge IMPORTING is_edge TYPE ty_edge. PRIVATE SECTION. DATA mt_vertices TYPE STANDARD...

should give error for ```abap DATA lt_tadir TYPE STANDARD TABLE OF tadir WITH DEFAULT KEY. SELECT * FROM tadir INTO TABLE lt_tadir ORDER BY obj_name UP TO 100 ROWS. ```...

hmm, refactor method `check`

new rule check that the report for the transaction code exists `TRAN`

quite a large change, but would make some things easier