abaplint-sci-client icon indicating copy to clipboard operation
abaplint-sci-client copied to clipboard

Impact of changing abapGit login to HTML form

Open mbtools opened this issue 5 years ago • 2 comments

The password popup is referenced in two places:

  • ZABAPLINT_CONFIG
  • ZABAPLINT_DEPENDENCIES

It seems the first can be removed completely:

REPORT zabaplint_config.

*TABLES sscrfields.
*INCLUDE zabapgit_password_dialog.
*INCLUDE zabapgit_forms.
*INITIALIZATION.
*  lcl_password_dialog=>on_screen_init( ).
*AT SELECTION-SCREEN OUTPUT.
*  IF sy-dynnr = lcl_password_dialog=>c_dynnr.
*    lcl_password_dialog=>on_screen_output( ).
*  ENDIF.
*AT SELECTION-SCREEN.
*  IF sy-dynnr = lcl_password_dialog=>c_dynnr.
*    lcl_password_dialog=>on_screen_event( sscrfields-ucomm ).
*  ENDIF.

START-OF-SELECTION.
  CALL FUNCTION 'Z_ABAPLINT_CONFIGURATION'.

However, the second case is tricky. In ZABAPLINT_DEPENDENCIES there is no HTML GUI involved and therefore the raise_login exception won't be redirected to the HTML form. I think this requires a bit of a rewrite to capture the login details (user/password) upfront, set them in the login manager, and then run the dependency check (zcl_abaplint_deps_git->run). Hmm...

mbtools avatar Oct 22 '20 14:10 mbtools

The current approach doesn't seem to work either for me.

When clicking on "Update from abapGit":

image

sy-cprog is SAPLS_CODE_INSPECTOR in this case....

fabianlupa avatar Dec 22 '20 07:12 fabianlupa

nope, it doesnt work right now

to solve this the login must be decoupled from the UI in abapGit

larshp avatar Dec 22 '20 07:12 larshp