RFC enabled Code Inspector checks

[se38]

Hello Lars,
currently I'm implementing a central ATC landscape (central ATC server with 7.51 and satellite DEV systems).
To use Code Inspector checks in such a landscape, the checks have to be RFC-enabled (see https://blogs.sap.com/2017/02/27/remote-code-analysis-in-atc-for-developers/ ).

Do you know already how to RFC-enable the abapOpenChecks? Or should we ask Thomas Fiedler?
If you publish a How-To on the wiki page, I'd help you to RFC-enable the checks one-by-one.

Cheers, Uwe (aka @se38)

[larshp]

Hi,
I looked into it a while ago, see #336

Somewhere there is a restriction that only static checks are executed remotely, however I'm not sure what it means, and dont remember much about it.

[se38]

Don't have any idea neither. If you look in a 7.51 system (ie the developer edition) you can identify the RFC-enabled check by the flash.

I asked alredy on SCN and will report back, as soon as I get an answer:. https://blogs.sap.com/2017/02/27/remote-code-analysis-in-atc-for-developers/#comment-391832

[larshp]

setting REMOTE_RFC_ENABLED = ABAP_TRUE in the constructor, and the icon shows up. However I get a strange dump in CL_ABAP_COMP_PROCS_E, but I'm also trying to setup central checks using just 1 system. UPDATE: after running program RS_ABAP_INIT_ANALYSIS I now get a proper dump in the abapOpenChecks code.

Note that check 04 is very simple, so it might be the easiest to get working first, plus it only uses scan information.

btw, the magic function module that runs the remote checks is SCI_REMOTE_RUN

[larshp]

try this: #423

[se38]

Ha, you made it (again). This fix works. Here's the (RFC!) result for Check #11 :

Just one Q:
What's the reason you want to ASSIGN the variable in the constructor? "remote_rfc_enabled = abap_true." simply works also.

[larshp]

I have merged the branch.

ASSIGN: I assume the field does not exist in version 702, abapOpenChecks is targeted 702 and higher.

Feel free to open a PR with additional fixes, note that some of the checks uses more than just the code scan information, if in doubt add as a separate PR and I will have a look. Also, I would like to move the code for setting RFC enabled to a method in ZCL_AOC_SUPER

ASSIGN ('REMOTE_RFC_ENABLED') TO <lv_rfc>.
IF sy-subrc = 0.
  <lv_rfc> = abap_true.
ENDIF.

[larshp]

Hi,
I have enabled more checks now, these are the ones I think will work via RFC, but I have not tested them

[se38]

I've checked some of the new RFC-enabled checks -> perfect, gracias

[larshp]

now also shown in the documentation, http://docs.abapopenchecks.org/checks/