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

ATC Check 69: RFC enabled? #1078

Open
cavabap opened this issue May 24, 2022 · 3 comments
Open

ATC Check 69: RFC enabled? #1078

cavabap opened this issue May 24, 2022 · 3 comments

Comments

@cavabap
Copy link
Contributor

cavabap commented May 24, 2022

Hello,

would it be possible to make Check 69 remote enabled?
Or is that too difficult. I don't mind submitting the PR, but it'd be great to get an idea what I'd need to change in advance.

What I could find that clearly isn't remote enabled was this.
SELECT SINGLE subc FROM reposrc INTO lv_subc
WHERE progname = object_name AND r3state = 'A'.

Could this be replaced with RFC_READ_TABLE? What else would need to be exchanged?

Thanks in advance

Martin

@larshp
Copy link
Owner

larshp commented May 24, 2022

well, the central system does not know the types, which makes it difficult

@cavabap
Copy link
Contributor Author

cavabap commented May 24, 2022

you mean like idoc function module, etc. But how is the check determining it? and could this be done via READ TABLE?
Would you accept a pull request for that?
Otherwise we might consider enhancing the coding to enable it but it be great to get a few pointers about what parts of the class do I need to replace...

@larshp
Copy link
Owner

larshp commented May 24, 2022

my recommendation is to not spend time on it, central ATC is more or less broken by design

however, it is possible to implement in https://github.com/abaplint/abaplint

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

No branches or pull requests

2 participants