abaplint icon indicating copy to clipboard operation
abaplint copied to clipboard

Check writes to importing parameters

Open FreHu opened this issue 4 years ago • 2 comments

REPORT zfoobar.

CLASS lcl_test DEFINITION CREATE PRIVATE.

  PUBLIC SECTION.
    METHODS parse_something
      IMPORTING text TYPE string.
  PROTECTED SECTION.
  PRIVATE SECTION.

ENDCLASS.

CLASS lcl_test IMPLEMENTATION.

  METHOD parse_something.
    text = 5.
  ENDMETHOD.

ENDCLASS.

image

  • No error if the parameter is IMPORTING VALUE
  • All write references should be checked, not just assignment (e.g. SORT importing_itab is also an error) image

FreHu avatar Aug 18 '21 12:08 FreHu

also

CONSTANTS bar TYPE i VALUE 2.
bar = 1.

larshp avatar Sep 01 '21 05:09 larshp

note that importing parameters(which are not pass by value), has recently gotten the "read only" metadata, which CONSTANTS also have

image

larshp avatar Apr 18 '22 11:04 larshp