Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Impact of changing abapGit login to HTML form #221

Open
mbtools opened this issue Oct 22, 2020 · 2 comments
Open

Impact of changing abapGit login to HTML form #221

mbtools opened this issue Oct 22, 2020 · 2 comments

Comments

@mbtools
Copy link
Collaborator

mbtools commented Oct 22, 2020

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...

@fabianlupa
Copy link
Contributor

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....

@larshp
Copy link
Member

larshp commented Dec 22, 2020

nope, it doesnt work right now

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants