Loop At screen - Simple syntax but not working

[vallamuthu_madheswaran2]

Hi ABAPERs,

In my program, I am using 2 radio buttons. Based on that I wnated to hide the parameter. I am writing the validation in the event "AT SELECTION-SCREEN ON RADIOBUTTON GROUP g1." Kindly find the below syntax it's not working.

  LOOP AT SCREEN.
    IF rb_upd = abap_true.
      IF screen-name = 'PA_FILE' OR screen-name = '%_PA_FILE_%_APP_%-TEXT'.
        screen-active = 0.
        screen-invisible = '1'.
        MODIFY SCREEN.
      ENDIF.
    ENDIF.
  ENDLOOP.

Thanks with regards,

Vallamuthu M.

[RaymondGiuseppi]

The LOOP AT SCREEN can be used to change fields attribues at PBO (before output)

Your code is executed in PAI (after input) so has no effect as overriden in the next PBO

1. Move your code to the AT SELECTION-SCREEN OUTPUT
2. Don't use hardcoded field names, better use option MODIF ID to attach a group id to some parameters or select-options, and check this value in SCREEN-GROUP1 during LOOP AT SCREEN process

[Sandra_Rossi]

ABAP Keyword Documentation - MODIFY SCREEN

"makes sense only during PBO processing"